1. Лучший говнокод

    В номинации:
    За время:

  2. Java / Говнокод #16831

    +84

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    private static final int INT_5 = 5;
private static final int INT_3 = 3;
private static final int INT_4 = 4;
private static final int INT_6 = 6;
private static final int INT_7 = 7;
private static final int INT_8 = 8;

    xaoc, 09 Октября 2014

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

  3. ActionScript / Говнокод #16727

    −84

    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
    public function isContainsHiddenId(hiddenId: String): Boolean
 		{
-			for (var index: int = 0; index < hiddenIds.length; index++)
+			/*for (var index: int = 0; index < hiddenIds.length; index++)
 			{
 				if (hiddenIds[index] == hiddenId)
 				{
 					return true;
 				}
 			}
-			return false;
+			return false;*/
+			return hiddenIds.indexOf(hiddenId) == -1 ? false : true;

    От автора MathAndGeometryUtils, после того, как ему посоветовали использовать indexOf.

    wvxvw, 19 Сентября 2014

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

  4. C++ / Говнокод #16659

    +54

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    Exchange::Params pars = rawParams;
for(Exchange::Params::const_iterator i = rawParams.constBegin(); i!= rawParams.constEnd(); i++){
    LOGN() << "Work with " << i.key() << "=" << i.value();
    if(m_specific.contains(i.key())){
        pars[i.key()] =
            (this->*m_specific.value(i.key())) (i.value());     //черная магия :)
    }
}

    Наследие из большого рабочего проекта.
    Чтобы сохранить ясность ума на весь день решил не пытаться понять что оно делает.

    FlySnake, 08 Сентября 2014

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

  5. PHP / Говнокод #16617

    +159

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    if (!isset($_SESSION[$_SESSION['tab_name']]['FILE_ATTRIBUTES'][$request['itemId']][$request['itemFieldId']])) {
      $_SESSION[$_SESSION['tab_name']]['FILE_ATTRIBUTES'][$request['itemId']][$request['itemFieldId']] = $request['itemFieldId'];
} else {
      unset($_SESSION[$_SESSION['tab_name']]['FILE_ATTRIBUTES'][$request['itemId']][$request['itemFieldId']]);
}

    И это часный проект где программисту платять большие деньги.
    А еще у автора более 5 лет опыта
    А сам сок, такого дерьма в проекте более 1000 обращений.

    vv3d0x, 29 Августа 2014

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

  6. Java / Говнокод #16283

    +69

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    public static boolean isEmpty(CharSequence str) {
    if (str == null || str.length() == 0)
        return true;
    else
        return false;
}

    Я считаю это странным
    Нашёл в android.text.TextUtils

    jonasas, 07 Июля 2014

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

  7. VisualBasic / Говнокод #16218

    −130

    1. 1
    2. 2
    3. 3
    4. 4
    Привет всем, помогите решить проблему.
reached limit: cannot create any more controls for this from
Как я понял, число контроллеров не должно превышать 255-256 в одной форме. Как этого избежать?
Можно ли вообще создавать формы с общими переменными? Спасибо

    http://vbbook.ru/visual-basic/vvedenie-visual-basic/

    brutushafens, 24 Июня 2014

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

  8. C# / Говнокод #16197

    +142

    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
    public Config(string[] str)
        {
            InitializeComponent();
            try
            {
                textBox5.Text = str[0];
                textBox6.Text = str[1];
                textBox1.Text = str[2];
                textBox2.Text = str[3];
                textBox3.Text = str[4];
                textBox4.Text = str[5];
                textBox7.Text = str[6];
                raz = str[7];
            }
            catch (Exception)
            {
                return;
            }
        }

    Классика. Говнокод эпохи барокко. Строк мало, но пованивает.

    GreatMASTERcpp, 19 Июня 2014

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

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

    +159

    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
    97. 97
    98. 98
    99. 99
    <?
function flevel($exp)
{
include_once "config/mysql.php";

	$querylevel = "SELECT maxexp FROM level";
	$levelquery = mysql_query($querylevel);
	while($rowslvl = mysql_fetch_row($levelquery))
	{
	$levelarr[] = $rowslvl[0];
	}
	
	
	
	switch(TRUE)
    {
		
    case ($exp <= $levelarr[0]):
    return $level = "1";
    break;

    case ($exp <= $levelarr[1]):
    return $level = "2";
    break;

    case ($exp <= $levelarr[2]):
    return $level = "3";
    break;

    case ($exp <= $levelarr[3]):
    return $level = "4";
    break;
	
	case ($exp <= $levelarr[4]):
    return $level = "5";
    break;
	
	case ($exp <= $levelarr[5]):
    return $level = "6";
    break;
	
	case ($exp <= $levelarr[6]):
    return $level = "7";
    break;
	
	case ($exp <= $levelarr[7]):
    return $level = "8";
    break;
	
	case ($exp <= $levelarr[8]):
    return $level = "9";
    break;
	
	case ($exp <= $levelarr[9]):
    return $level = "10";
    break;
	
	case ($exp <= $levelarr[10]):
    return $level = "11";
    break;
	
	case ($exp <= $levelarr[11]):
    return $level = "12";
    break;
	/* ... */
	case ($exp <= $levelarr[42]):
    return $level = "43";
    break;
	
	case ($exp <= $levelarr[43]):
    return $level = "44";
    break;
	
	case ($exp <= $levelarr[44]):
    return $level = "45";
    break;
	
	case ($exp <= $levelarr[45]):
    return $level = "46";
    break;
	
	case ($exp <= $levelarr[46]):
    return $level = "47";
    break;
	
	case ($exp <= $levelarr[47]):
    return $level = "48";
    break;
	
	case ($exp <= $levelarr[48]):
    return $level = "49";
    break;
	
	case ($exp > $levelarr[49]):
    return $level = "50";
    break;
    }
}
?>

    Пришел в проект по созданию веб игры, смотрю организацию проекта... Жопа там, что папки, что код, что база одно и тоже, каша. Вот нашел одну забавную функцию. Она возвращает уровень персонажа судя по кол-во XP. Вместо того что бы писать 'SELECT `level` FROM `level` WHERE `minexp` <= ' . $exp . ' AND `maxexp` > ' . $exp надо было switch .. case писать. Нету слов.

    volter9, 11 Июня 2014

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

  10. PHP / Говнокод #16126

    +166

    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
    $response = LINQ::from($products->as_array('id'))
                    ->join($images)
                    ->on(function ($from_key, $from_value, $join_key, $join_value){
                        return $from_key === $join_key;
                    })
                    ->select(function($from_key, $from_value, $join_key, $join_value)use($searchStr){
                        $replaceStr = '<span class="badge badge-success">'.$searchStr.'</span>';
                        return array(
                           'title' => str_ireplace($searchStr, $replaceStr, $from_value->title),
                           'href' => '/product/'.$from_value->alias.'.html',
                           'thumb' => '/public/'.$join_value->thumb
                        );
                    })
                    ->result();

    Так бывает когда приверженца C# заставляют писать на PHP. Он обязательно напишет свой аналог LINQ с театром и балеринами, а потом будет ныть о том, что в похапе нет нормальных лямбд.

    Strannik1941, 08 Июня 2014

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

  11. C# / Говнокод #16097

    +133

    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
    //checks if the string is a hex stream e.g. "31 32 33 6A F8"
        private bool _IsHexStream(string sValue)
        {
            sValue = sValue.Trim();

            
            if (sValue.Length < 2)
            {
                return false;
            }

            for (int i = 0; i < sValue.Length; i++)
            {
                if(_IsHexChar(Convert.ToChar(sValue.Substring(i,1))) == false)
                {
                    return false;
                }
            }

            //every third char must be a space, only possible in case of two bytes
            if (sValue.Length > 3)
            {
                for (int i = 2; i < sValue.Length; i += 3)
                {
                    string sBuffer = sValue.Substring(i, 1);

                    if (sBuffer.Equals(" ") == false)
                    {
                        return false;
                    }
                }
            }

            //string is a hex stream 
            return true;
        }

    blackhearted, 02 Июня 2014

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