1. PHP / Говнокод #16610

    +155

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    global $strError, $MESS, $HTTP_GET_VARS, $arrFORM_FILTER;
	global $find_date_create_1, $find_date_create_2, $lAdmin;
	$str = "";
	CheckFilterDates($find_date_create_1, $find_date_create_2, $date1_wrong, $date2_wrong, $date2_less);
	if ($date1_wrong=="Y") $str.= GetMessage("FORM_WRONG_DATE_CREATE_FROM")."<br>";
	if ($date2_wrong=="Y") $str.= GetMessage("FORM_WRONG_DATE_CREATE_TO")."<br>";
	if ($date2_less=="Y") $str.= GetMessage("FORM_FROM_TILL_DATE_CREATE")."<br>";

    bitrix

    - использование HTTP_GET_VARS уже давно deprecated
    - магические переменные find_date_create_1, find_date_create_2, да и lAdmin тоже не понятно что
    - зачем-то используются символы Y и N вместо true и false или 1 и 0

    Запостил: memclutter, 28 Августа 2014

    Комментарии (19) RSS

    • ava andr435 28.08.2014 10:12 # 0

      Админ наверно может вносить любые данные.
      А если не исползовать true/false, то не вижу принципиалной разницы межды 1 и "Y".
      Другое дело, исползовать global - вот ето уже не феншуй.
      Ответить

      • ava Lure Of Chaos 28.08.2014 17:53 # 0

        ну и к <br> можно придраться
        Ответить

      • ava guest 30.08.2014 10:57 # +2

        php, автоматически преобразует 1 в true, а 0 в false. Но вот мне кажется конструкция if ($boolValue) будет быстрее чем if ($stringValue == 'Y'), т.к. в втором случае помимо проверки условия идет сравнение строк, а это займет немного времени, конечно это не критично, но все же в программировании true и false лучше чем 'Y' 'N'.
        Ответить

        • ava kegdan 30.08.2014 12:17 # 0

          кстати, какой у пхп приоритет приведения? например если сравнивать число и строку он число к строке приведет или наоборот попытается?
          Ответить

          • ava inkanus-gray 30.08.2014 12:53 # 0

            ПХП попытается всё сделать наихудшим образом. Совсем недавно был пример, когда строка "0lol" при сравнении с числом приводилась к числу.
            Ответить

            • ava kegdan 30.08.2014 12:58 # 0

              "0lol" успешно приводилось к 0? Жесть
              Ответить

              • ava bormand 30.08.2014 13:13 # 0

                А еще в старых пыхах "312321368721836821638123" == "312321368721836821638124".
                Ответить

                • ava kegdan 30.08.2014 13:16 # 0

                  потому что он сначала кастует в инт, а все символы в инт не влезают?)
                  Ответить

                  • ava bormand 30.08.2014 13:16 # 0

                    Теперь вроде как нет.
                    Ответить

                    • ava inkanus-gray 30.08.2014 23:04 # +1

                      Исправили только в 5.4.

                      Во всё ещё популярном 5.3 до сих пор "312321368721836821638123" == "312321368721836821638124".
                      Ответить

                  • ava bormand 30.08.2014 13:40 # +1

                    Ну а вообще, кошерный способ сравнивать строки - это "===".
                    Ответить

              • ava bormand 30.08.2014 13:15 # +1

                > "0lol" успешно приводилось к 0
                Да оно и сейчас успешно приводится. Просто, емнип, == для джвух строк перестало пробовать кастоваться в инты.
                Ответить

                • ava kegdan 30.08.2014 13:18 # 0

                  >>Просто, емнип, == для джвух строк перестало пробовать кастоваться в инты.

                  кто вообще подумал, что кастование строк в инты при проверке равенства - хорошая идея?
                  Ответить

                  • ava bormand 30.08.2014 13:33 # 0

                    Чтобы ленивые макаки могли не кастовать параметры скрипта в числа, а юзать прям так, в виде строк.
                    Ответить

                    • ava kegdan 30.08.2014 13:34 # 0

                      Но это же... безумие!
                      Ответить

                      • ava bormand 30.08.2014 13:38 # 0

                        Вся суть оригинального PHP в безумии - магические кавычки, register globals и имена многих функций (например parse_str) чего стоят.

                        В пятой ветке хоть немного причесали.
                        Ответить

                      • ava eth0 30.08.2014 15:43 # +1

                        Или Спарта?
                        Ответить

                        • ava kegdan 30.08.2014 15:57 # +1

                          Это была настолько очевидная шутка, что она не нуждалась в озвучивании.
                          Ответить

    • ava guest 29.08.2014 20:34 # 0

      это же битрикс, там свои значения для boolean
      Ответить

    Добавить комментарий