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

    В номинации:
    За время:
  2. PHP / Говнокод #18232

    +144

    1. 1
    $baner->disabled = !empty($input['disabled'])?!$input['disabled']:TRUE;

    огонь)

    mahnat, 25 Мая 2015

    Комментарии (2)
  3. PHP / Говнокод #18229

    +144

    1. 1
    2. 2
    3. 3
    4. 4
    if(count($proxy) === 0 ){
      echo 'SMTH went wrong or all proxy pososali vse' . PHP_EOL;
      exit;
    }

    Останавливаемся если закончились прокси

    romkoll, 22 Мая 2015

    Комментарии (2)
  4. C++ / Говнокод #18222

    +144

    1. 1
    sleh = UpCase(92); // просто слеш

    Просто слеш.

    bormand, 22 Мая 2015

    Комментарии (15)
  5. Java / Говнокод #18219

    +144

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    private String compId(final String name) {
    		final Pattern compiled = Pattern.compile("^on([_A-Za-z0-9]+)From([_A-Za-z0-9]+)$");
    		final Matcher matcher = compiled.matcher(name);
    		return matcher.matches() ? matcher.group(2) : null;
    	}
    
    	private String compEvent(final String name) {
    		final Pattern compiled = Pattern.compile("^on([_A-Za-z0-9]+)(From([_A-Za-z0-9]+))?$");
    		final Matcher matcher = compiled.matcher(name);
    		return matcher.matches() ? matcher.group(1) : null;
    	}

    Lure Of Chaos, 22 Мая 2015

    Комментарии (1)
  6. C++ / Говнокод #18209

    +144

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    ...
    GameItem* new_game_item = GetGameItem(id);
    bool flag = !new_game_item;
    if (flag)
    {
        new_game_item = new GameItem();
    }
    ...

    Меня больше всего поразило название локальной переменной new_game_item, которая чаще даже совсем не new. Но кого это волнует...

    pomahin, 20 Мая 2015

    Комментарии (1)
  7. PHP / Говнокод #18207

    +144

    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
    <!--Женская одежда Блузки-->
    				<?if($arResult["IBLOCK_SECTION_ID"] == 487 ):?> 
    				<table class="tproperty">
    	<tr><td class="tdproperty"><?=$arResult["PROPERTIES"]["SOSTAV"]["NAME"];?></td><td><?=$arResult["PROPERTIES"]["SOSTAV"]["VALUE"];?></td></tr>
    	<tr><td class="tdproperty" ><?=$arResult["PROPERTIES"]["TSVET"]["NAME"];?></td><td><?=$arResult["PROPERTIES"]["TSVET"]["VALUE"];?></td></tr>
    	<tr><td class="tdproperty" ><?=$arResult['PROPERTIES']['RAZMER']['NAME'];?></td><td><?=$arResult["PROPERTIES"]["RAZMER"]["VALUE"];?></td></tr>
    	<tr><td class="tdproperty" ><?=$arResult['PROPERTIES']['UPAKOVKA']['NAME'];?></td><td><?=$arResult["PROPERTIES"]["UPAKOVKA"]["VALUE"];?></td></tr>
    	<tr><td class="tdproperty"><?=$arResult['PROPERTIES']['POKROY']['NAME'];?></td><td><?=$arResult["PROPERTIES"]["POKROY"]["VALUE"];?></td></tr>
    	<tr><td class="tdproperty"><?=$arResult['PROPERTIES']['DLINNA_RUKAVA']['NAME'];?></td><td><?=$arResult["PROPERTIES"]["DLINNA_RUKAVA"]["VALUE"];?></td></tr>
    				</table>
    
    <!--Женская одежда Брючные костюмы-->
    				<?elseif($arResult["IBLOCK_SECTION_ID"] == 489 ):?> 
    <table class="tproperty">
    	<tr><td class="tdproperty"><?=$arResult["PROPERTIES"]["SOSTAV"]["NAME"];?></td><td><?=$arResult["PROPERTIES"]["SOSTAV"]["VALUE"];?></td></tr>
    	<tr><td class="tdproperty"><?=$arResult["PROPERTIES"]["TSVET"]["NAME"];?></td><td><?=$arResult["PROPERTIES"]["TSVET"]["VALUE"];?></td></tr>
    	<tr><td class="tdproperty"><?=$arResult['PROPERTIES']['RAZMER']['NAME'];?></td><td><?=$arResult["PROPERTIES"]["RAZMER"]["VALUE"];?></td></tr>
    	<tr><td class="tdproperty"><?=$arResult['PROPERTIES']['UPAKOVKA']['NAME'];?></td><td><?=$arResult["PROPERTIES"]["UPAKOVKA"]["VALUE"];?></td></tr>
    				</table>
    
    <!--Женская одежда Жакеты-->
    <?elseif($arResult["IBLOCK_SECTION_ID"] == 491 ):?> 
    <table class="tproperty" >
    	<tr><td class="tdproperty"><?=$arResult["PROPERTIES"]["SOSTAV"]["NAME"];?></td><td><?=$arResult["PROPERTIES"]["SOSTAV"]["VALUE"];?></td></tr>
    	<tr><td class="tdproperty"><?=$arResult["PROPERTIES"]["TSVET"]["NAME"];?></td><td><?=$arResult["PROPERTIES"]["TSVET"]["VALUE"];?></td></tr>
    	<tr><td class="tdproperty"><?=$arResult['PROPERTIES']['RAZMER']['NAME'];?></td><td><?=$arResult["PROPERTIES"]["RAZMER"]["VALUE"];?></td></tr>
    	<tr><td class="tdproperty"><?=$arResult['PROPERTIES']['UPAKOVKA']['NAME'];?></td><td><?=$arResult["PROPERTIES"]["UPAKOVKA"]["VALUE"];?></td></tr>
    	<tr><td class="tdproperty"><?=$arResult['PROPERTIES']['POKROY']['NAME'];?></td><td><?=$arResult["PROPERTIES"]["POKROY"]["VALUE"];?></td></tr>
    	<tr><td class="tdproperty"><?=$arResult['PROPERTIES']['DLINNA_RUKAVA']['NAME'];?></td><td><?=$arResult["PROPERTIES"]["DLINNA_RUKAVA"]["VALUE"];?></td></tr>
    </table>
    
    
    ........................................

    И так еще 900 строчек.... Как такие люди рождаются вообще? :)

    lscin, 20 Мая 2015

    Комментарии (9)
  8. C# / Говнокод #18202

    +144

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    string q = DateTime.Now.ToString().Substring(3, 3);
    string w = DateTime.Now.ToString().Substring(0, 2);
    string e = DateTime.Now.ToString().Substring(5, DateTime.Now.ToString().Length - 5);
    string dsasd = q + w + e;
    
    CrmDateTimeProperty _new_date_fitst_update_rstatus = new CrmDateTimeProperty();
    _new_date_fitst_update_rstatus.Name = "new_date_first_update_rstatus";
    _new_date_fitst_update_rstatus.Value = new CrmDateTime();
    _new_date_fitst_update_rstatus.Value.Value = dsasd;
    dynamicEntity.Properties.Add(_new_date_fitst_update_rstatus);

    Попался проект на фрилансе. Попросили исправить ошибки. Начал исправлять и вот такое.

    Shturman, 19 Мая 2015

    Комментарии (0)
  9. PHP / Говнокод #18191

    +144

    1. 1
    2. 2
    3. 3
    4. 4
    /* Convert IP address to unsigned long int. */
    function ip2ulong($ip) {
        return sprintf("%u", ip2long32($ip));
    }

    https://github.com/pfsense/pfsense/blob/master/etc/inc/util.inc#L414-L417

    Продолжаем осторожно идти дальше. Это нормально вообще? Да, я видел всякие смехуёчки с магическим приведением типов в PHP и JS, так что наверняка оно где-то правильно конвертится и все работает несмотря ни на что и вопреки. Может, это даже нормальная практика в мире PHP, но мне страшно.

    superhacker777, 17 Мая 2015

    Комментарии (19)
  10. PHP / Говнокод #18190

    +144

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    if ($ipaddr == $ip_reverse) {
        return true;
    } else {
        return false;
    }

    https://github.com/pfsense/pfsense/blob/master/etc/inc/util.inc#L625-L629

    Давно хотел спросить: это как-нибудь здравым смыслом оправдывается? Часто встречаю похожее, когда чужой код смотрю. Может, это профи делают для… читаемости какой-то, я не знаю. Или я ищу хорошее там, где его нет?

    superhacker777, 17 Мая 2015

    Комментарии (4)
  11. C++ / Говнокод #18189

    +144

    1. 1
    2. 2
    3. 3
    4. 4
    int spisok::kolvo()
    {
    	return spisok::N;
    }

    KGOH, 17 Мая 2015

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