1. Java / Говнокод #17099

    +86

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    public interface ArchitectureUnit<T> {
		
		public T release(Object... params); 
		
	}

    максимальная гибкость

    _a_o_O, 12 Ноября 2014

    Комментарии (8)

  2. Python / Говнокод #17098

    −101

    1. 1
    self.exclude = list(set(list(self.exclude or []) + ['str1', 'str2']))

    american_idiot, 12 Ноября 2014

    Комментарии (14)

  3. JavaScript / Говнокод #17097

    +160

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    $scope.close_menu = function(resize){
        if(window.innerWidth<=1023) var width=280;
        else var width=390;
        var duration=(resize)? 0:300;
        $('.menu').removeClass('active').animate({right:-width+'px'},duration);
        $('.wrapper').animate({right:'0px'},duration);
        $('body').css({'overflow':'auto'});
        $('.menu_btn').animate({marginRight:'0px'},duration,function(){});
        $('.videocontainer').animate({right:'0px'},duration);
        $('.blog_share.active > a,.blog_share.active > ul').animate({right:'0px'},duration);
        $('.backgorund_transparent').hide();

        if(typeof isMoving != 'undefined') isMoving=false;
    }

    Просто AngularJs...

    keitoaino, 12 Ноября 2014

    Комментарии (26)

  4. Си / Говнокод #17096

    +135

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    9. 9
    static enum rc (*request_functions[])(void) = {
    ko,
    koko,
    kokoko,
    illegal_request
};
static inline enum rc illegal_request(void) { return ILLEGAL_REQUEST; }

reply.rc = request_functions[cmd.opcode < NKEYS(request_functions) ? cmd.opcode : ILLEGAL_REQUEST]();

    Вызываем функцию по опкоду с абортом в случае index_out_of_bounds.

    codemonkey, 12 Ноября 2014

    Комментарии (38)

  5. Ruby / Говнокод #17095

    −82

    1. 1
    2. 2
    3. 3
    4. 4
    @old_article_purchase.old_article.warehouse_placement = '102A1'
if @old_article_purchase.old_article.valid?
   @old_article_purchase.old_article.save   
 end

    andrejlr, 12 Ноября 2014

    Комментарии (2)

  6. Objective C / Говнокод #17094

    −396

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    33. 33
    34. 34
    35. 35
    36. 36
    37. 37
    38. 38
    39. 39
    40. 40
    41. 41
    42. 42
    43. 43
    44. 44
    45. 45
    @implementation SlideMenuNavigationBar

- (void)layoutSubviews
{
    [super layoutSubviews];

    for (UIView *aView in self.subviews) {
        
        // Correcting menu toggle button position
        if ([[aView.class description] isEqualToString:@"UIButton"] && aView.frame.origin.x < self.frame.size.width/2) {
            CGRect frame = aView.frame;
            frame.origin.x = 12; // 8 for correlation
            frame.origin.y = -1; // 1 for correlation
            aView.frame = frame;
        }
        
        if (aView.frame.origin.x > self.frame.size.width/2 && !isIPad && ![[aView.class description] isEqualToString:@"ColoredView"]) {
            CGRect frame = aView.frame;
            frame.origin.x = 260; // 8 for correlation
            frame.origin.y = -1;  // 1 for correlation
            aView.frame = frame;
        }
        
        // Correcting back button and right button positions
        if ([[aView.class description] isEqualToString:@"_UINavigationBarBackIndicatorView"]) {
            CGRect frame = aView.frame;
            frame.origin.y = [self.class navigationBarHeight] - kDefaultNavigationBarHeight + 5; // 5 for correlation
            aView.frame = frame;
        }
        if ([[aView.class description] isEqualToString:@"UINavigationButton"]) {
            CGRect frame = aView.frame;
            frame.origin.y = [self.class navigationBarHeight] - kDefaultNavigationBarHeight + 2; // 2 for correlation
            aView.frame = frame;
        }
        
        if ([aView isKindOfClass:[NavigationBarButton class]]) {
            CGRect frame = aView.frame;
            frame.origin.x = 278; // 8 for correlation
            frame.origin.y = 6;   // 1 for correlation
            aView.frame = frame;
        }
    }
}

@end

    фиг знает что думали :D

    l0gg3r, 11 Ноября 2014

    Комментарии (6)

  7. C++ / Говнокод #17093

    +50

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    #include <iostream>
using namespace std;

void f(int i)
{
	cout << "----" << endl;
	void *array[] = { &&foo, &&bar, &&baz };
	goto *array[i];
	foo:
	cout << "Foo" << endl;
	bar:
	cout << "Bar" << endl;
	baz:
	cout << "Baz" << endl;
}

int main() {
	// your code goes here
	f(0);
	f(1);
	f(2);
	return 0;
}

    http://ideone.com/63tRls
    gcc extension

    DlangGovno, 10 Ноября 2014

    Комментарии (108)

  8. PHP / Говнокод #17091

    +156

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    // Real itemid is "contained" in public itemid in a bit tricky way. The formula is:

public_itemid = itemid * 256 + anum

// Quite easy, isn't it? Except where to get that anum. anum is some number assigned when you create a new post in LJ.

// But in fact, you don't need it. Because it is always less than 256, we can avoid searching for it, and just use simple Math to find itemid.

$anum = $public_itemid - floor($public_itemid / 256) * 256;
$itemid = ($public_itemid - $anum) / 256;

    Прекрасный подход, то место, где вордпресс и ЖЖ сошлись воедино.

    http://drumrock.skipitnow.org/livejournal-tools/itemid/

    Fike, 10 Ноября 2014

    Комментарии (1)

  9. PHP / Говнокод #17090

    +159

    1. 1
    $image_type = substr($image_data,strpos($image_data, '/')+1,strpos($image_data, ';')-(strpos($image_data, '/')+1));

    Выдирание типа из заголовка base64-закодированного изображения (data:image/jpeg;base64,......................)

    Anxizzz, 10 Ноября 2014

    Комментарии (13)

  10. C# / Говнокод #17089

    +134

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    33. 33
    34. 34
    35. 35
    36. 36
    37. 37
    38. 38
    39. 39
    40. 40
    41. 41
    42. 42
    43. 43
    44. 44
    45. 45
    46. 46
    47. 47
    48. 48
    49. 49
    50. 50
    51. 51
    52. 52
    53. 53
    54. 54
    55. 55
    56. 56
    57. 57
    58. 58
    59. 59
    60. 60
    61. 61
    62. 62
    63. 63
    64. 64
    65. 65
    66. 66
    67. 67
    68. 68
    69. 69
    70. 70
    71. 71
    72. 72
    73. 73
    74. 74
    75. 75
    76. 76
    77. 77
    78. 78
    79. 79
    80. 80
    81. 81
    82. 82
    83. 83
    84. 84
    85. 85
    86. 86
    87. 87
    88. 88
    89. 89
    90. 90
    91. 91
    92. 92
    93. 93
    94. 94
    95. 95
    96. 96
    class ParametersArray
{
        ...
        /// <summary>
        /// Копирующей конструктор (создает копию коллекции)
        /// </summary>
        /// <param name="prs">Коллекция на основе, которой будет создаваться копия</param>
        public ParametersArray(ParametersArray prs)
        {
            paParameters = new ArrayList();
            for (int i = 0; i < prs.Count; i++)
            {
                Parameter p = new Parameter(prs[i]);
                paParameters.Add(p);
            }
        }
        ...
        #region Внутренние состояния

        /// <summary>
        /// Внутреннее поле - массив параметров
        /// </summary>
        private ArrayList paParameters;

        /// <summary>
        /// Внутреннее поле – идентификатор операции, к которой относятся параметры
        /// </summary>
        private string paOperationId = "";

        #endregion
        /// <summary>
        /// Удаляет параметр из коллекции
        /// </summary>
        /// <param name="parameterId">Идентификатор параметра</param>
        public void Delete(string parameterId)
        {
            ArrayList list = new ArrayList();

            foreach (Parameter prt in paParameters)
            {
                if (prt.Id.Trim() != parameterId.Trim())
                {
                    list.Add(prt);
                }
            }
            paParameters = list;
        }
    }
}

...

  /// <summary>
  /// Класс для поиска и обработки шаблонов в параметрах операций
  /// </summary>
  public class Parser
  {
    ...
    #region Конструкторы

    /// <summary>
    /// Конструктор инициализирующей класс
    /// </summary>
    /// <param name="requestUserId">ID пользователя, который используется для обработки шаблона ##USER()</param>
    public Parser(string requestUserId)
    {
      pParameters = null;
      pUserId = requestUserId;
    }

    #endregion

    #region Внешние методы

    /// <summary>
    /// Метод перебирает все параметры операции, ищет там шаблоны и заменяет их значениями
    /// </summary>
    /// <param name="opr">Операция</param>
    /// <returns>Операция с обработанными шаблонами</returns>
    public Operation Parse(Operation opr)
    {
      pParameters = new ParametersArray(opr.Parameters);

      for (int i = 0; i < pParameters.Count; i++)
        if (FindTemplate(pParameters[i]))
        {
          ReplaceTemplate(pParameters[i]);

          //Удаляет параметр с ошибкой
          if (pParameters[i].Value == "parse err")
            pParameters.Delete(pParameters[i].Id);
        }

      opr.Parameters = new ParametersArray(pParameters);
      return opr;
    }

    Внимание привлекает код в строках 81-95

    В строке 81-82 мы создаём временную копию параметров операции opr во внутреннем поле класса Parser. Прекрасно. Ещё раз - в публичном в методе инициализируем внутреннее поле (Классный side effect - поле используется в других методах).

    В строке 91 удаляет элемент из внутренней коллекции, пересоздавая коллекцию целиком, то есть требования к памяти временно удваиваются на выходе из метода Delete (до очистки памяти). В методе Delete. Да. Супер.
    потом инициализируем вновь созданной коллекцией публичное поле Parameters во входящем параметре opr метода Delete.

    P.S.: Диагноз - хроническая форма неизлечимой Java. Во первых, автор этих строк искренне думает, что в природе ничего, кроме коллекции ArrayList, не существует. Во вторых, для удаления элемента коллекции мы пересоздаём всю коллекцию целиком, то есть видимо искренне полагая, что коллекция ArrayList - immutable, как и все поля класса Parser (типа ParametersArray) равно как и поле класса ParametersArray (типа ArrayList). Занавес.

    Говнокодище.

    hack2root, 10 Ноября 2014

    Комментарии (7)