1. Список говнокодов пользователя velosipedistorg

    Всего: 4

  2. PHP / Говнокод #16087

    +157

    1. 1
    2. 2
    3. 3
    if (strlen($arResult["ERROR_MESSAGE"]) <= 0){
    // любая ахинея
    }

    2014 год. Битрикс все так же терпеливо ожидает патент на строки отрицательной длины.
    Полагаю, такой икспрешн используется не только в компоненте корзины, учитывая страсть авторов к копипасте.

    velosipedistorg, 30 Мая 2014

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

    +154

    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
    if ($_SERVER["REQUEST_METHOD"] == "POST" && strlen($_REQUEST["backButton"]) > 0 && ($arParams["DELIVERY_NO_SESSION"] == "N" || check_bitrix_sessid()))
    {
    	if($arResult["POST"]["CurrentStep"] == 6 && $arResult["SKIP_FORTH_STEP"] == "Y")
    		$arResult["CurrentStepTmp"] = 3;
    
    	if($arResult["POST"]["CurrentStepTmp"] <= 5 && $arResult["SKIP_THIRD_STEP"] == "Y")
    		$arResult["CurrentStepTmp"] = 2;
    
    	if($arResult["POST"]["CurrentStepTmp"] <= 3 && $arResult["SKIP_SECOND_STEP"] == "Y")
    		$arResult["CurrentStepTmp"] = 1;
    
    	if(IntVal($arResult["CurrentStepTmp"])>0)
    		$arResult["CurrentStep"] = $arResult["CurrentStepTmp"];
    	else
    		$arResult["CurrentStep"] = $arResult["CurrentStep"] - 2;
    	$arResult["BACK"] = "Y";
    }
    if ($arResult["CurrentStep"] <= 0)
    	$arResult["CurrentStep"] = 1;
    $arResult["ERROR_MESSAGE"] = "";

    Компонент страницы заказа в магазине Битрикса, 14я версия.
    Тут вам и отсутствие констант для осмысленных шагов, и отсутствие конфигурирования этих шагов, как они i18n-ятся, вообще не представляю.
    КАК ЭТО ВСЁ РАБОТАЕТ??!!!

    ЗЫ мало того, тут же в компонент вкопипащен (или переговнокоден заново, не сравнивал) процесс залогинивания/регистрации нового юзера, елси он не залогинен. Не смог перекопипастить эту копипасту лишь потому, что она содержит 310 строк. Неслабый DRY.

    velosipedistorg, 30 Мая 2014

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

    +172

    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
    $var=trim($var);
    $var=str_replace("\t", ' ',$var);
    $var=str_replace("\r", ' ',$var);
    $var=str_replace("\n", ' ',$var);
    $var=str_replace('          ', ' ',$var);
    $var=str_replace('         ', ' ',$var);
    $var=str_replace('        ', ' ',$var);
    $var=str_replace('       ', ' ',$var);
    $var=str_replace('      ', ' ',$var);
    $var=str_replace('     ', ' ',$var);
    $var=str_replace('    ', ' ',$var);
    $var=str_replace('   ', ' ',$var);
    $var=str_replace('  ', ' ',$var);
    $var=str_replace('  ', ' ',$var);
    $var=str_replace('  ', ' ',$var);
    $var=str_replace('  ', ' ',$var);
    $var=str_replace('  ', ' ',$var);
    $var=str_replace('  ', ' ',$var);
    $var=str_replace('  ', ' ',$var);

    Надежно, чо.

    velosipedistorg, 21 Февраля 2012

    Комментарии (29)
  5. PHP / Говнокод #9286

    +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
    ##############################################
    # Bitrix: SiteManager                        #
    # Copyright (c) 2002-2006 Bitrix             #
    # http://www.bitrixsoft.com                  #
    # mailto:[email protected]                #
    ##############################################
    
    if (!class_exists("CCaptcha"))
    {
    	class CCaptcha
    	{
    		var $imageWidth = 180;
    		var $imageHeight = 40;
    // ...etc

    Это Битрикс. Опять. bitrix\modules\main\classes\general\capt cha.php
    Определение нативной капчи.
    В строке 8 создатели сего как бы задаются воспросом «А вдруг еще никто не писал до нас капчи?».
    Или перестраховываются — «а вдруг require() уже вызывался? И что такое require_once(), про который все так много говорят?»
    Добротный, защищенный на все сто, класс капчи. Невозможно сломать, уже просто потому, что невозможно понять...

    velosipedistorg, 01 Февраля 2012

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