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

    +1001

    1. 1
    2. 2
    3. 3
    if ($captcha_url != '' && 1 == 1) {
        //echo $html;
        require('antigate.php');

    Интересно, зачем потребовалось единицы сравнивать

    Запостил: OnlyFirster, 28 Июля 2015

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

    • ava inkanus-gray 28.07.2015 13:54 # 0

      Может быть, заплатка? Вдруг там вместо одной единицы была переменная или вызов функции, а потом убрали? Ну а вдруг?
      Ответить

      • ava kegdan 28.07.2015 14:02 # 0

        поддерживаю. Иначе там было бы
        && 1 == 1 && 1 == 1 && 1 == 1 && 1 == 1 && 1 == 1 && 1 == 1 && 1 == 1&& 1 == 1 ...
        Ответить

        • ava Elvenfighter 29.07.2015 01:15 # +2

          Я считаю, что
          1 == 1 && 2 == 2 && 3 == 3 && 4 == 4 && ...

          куда рациональней! А
          1 == 1 && 2 == 2 && 4 == 4 && 8 == 8 && 16 == 16 && ...
          еще и оптимальней
          Ответить

      • ava Asen 02.08.2015 20:55 # 0

        А вдруг вместо единицы там была другая цифра?
        Ответить

    • ava guest 28.07.2015 14:02 # +2

      В параллельной вселенной, где 1 != 1, antigate.php не нужен.
      Ответить

    • ava guest 28.07.2015 16:57 # +5

      Так делают некоторые кодеры во время отладки, когда возникает потребность не выполнять код после условия (могут ставить для этого условие 1 != 1), затем снова учитывать этот блок кода (1 == 1), потом опять не учитывать и т. д. Просто кто-то забыл подтереть это "тестировочное" условие после такого рода отладок.
      Ответить

      • ava guest 28.07.2015 17:43 # 0

        в пыхе (1) не true?
        Ответить

        • ava kegdan 28.07.2015 17:52 # +4

          пыха вообще не true
          Ответить

          • ava Vasiliy 28.07.2015 21:04 # 0

            Сударь Я вызываю Вас на дуэль....
            Ответить

            • ava kegdan 28.07.2015 21:09 # 0

              Я принимаю ваш вызов! Победит тот, кто расколет больше дров. Дрова я предоставлю
              Ответить

              • ava CMTV 29.07.2015 10:23 # 0

                Я могу быть секундантом Vasiliy!
                Ответить

                • ava kegdan 29.07.2015 10:30 # 0

                  ага, проверишь топор, расколешь чурок 10
                  Ответить

                  • ava CMTV 29.07.2015 10:33 # 0

                    Не только. Я буду следить за вами, сударь. Вдруг вы решите принести уже готовые, расколотые дрова. И будьте уверены, мсье, я буду смотреть внимательно!
                    Ответить

      • ava 3_14dar 29.07.2015 01:48 # +1

        Тему уже поднимали не раз. Они в блокноте кодят? Даже в npp есть комбинация на коммент выделенных строк.
        Ответить

        • ava inkanus-gray 30.07.2015 15:27 # 0

          Не всегда удобно выключать код комментированием.
          Ответить

          • ava 3_14dar 30.07.2015 18:51 # 0

            Я всегда так делал.
            Ответить

            • ava Vasiliy 30.07.2015 19:20 # +1

              тебе можно ты пидар.
              Ответить

              • ava 3_14dar 30.07.2015 19:56 # 0

                А ты, гетераст, небось о такой возможности и не знал?
                Ответить

                • ava Vasiliy 30.07.2015 20:05 # 0

                  Нормальные поцаны юзают assert
                  Ответить

                  • ava bormand 30.07.2015 20:16 # 0

                    Чего-чего? Ну-ка поясни баке-анимешнику, как для отключения куска кода воспользоваться assert()'ом?
                    Ответить

                    • ava kegdan 30.07.2015 20:22 # 0

                      как, как! никак..

                      Но пидор об этом может и не знать
                      Ответить

                    • ava Vasiliy 30.07.2015 21:32 # 0

                      // Тестовая функция
                      function test_assert($parameter)
                      {
                      assert(is_bool($parameter));
                      }

                      // настройки проверки
                      assert_options(ASSERT_ACTIVE, true);
                      assert_options(ASSERT_BAIL, true);
                      assert_options(ASSERT_WARNING, false);
                      assert_options(ASSERT_CALLBACK, 'assert_failure');

                      // заведомо ошибочное утверждение
                      test_assert(1);

                      // Этот код не будет выполняться, пока ASSERT_BAIL
                      // равен true
                      echo 'Никогда не будет выведено';
                      Ответить

                      • ava bormand 30.07.2015 21:37 # 0

                        Так весь код, который идёт ниже, не будет выполняться... А тут речь идёт о том, чтобы блок кода не исполнять.
                        Ответить

    • ava guest 28.07.2015 22:42 # 0

      потому что место где надо часто отлаживать. меняешь одну из 1 на 2 и получаешь стабильный фолс. туповато, но собственно... и такое бывает.
      Ответить

      • ava bormand 30.07.2015 21:19 # 0

        Не, ну я всё понимаю, PHP, все дела... Но неужели нельзя вытащить "место, где надо часто отлаживать" в функцию и написать к ней немножко юнит-тестов?
        Ответить

        • ava kegdan 30.07.2015 21:34 # 0

          можно. Но юнит-тесты знают только избранные, люк. Реально многие даже не в курсе что такое бывает
          Ответить

          • ava bormand 30.07.2015 21:41 # 0

            Забавно, что я "изобрёл" их сам, за много лет перед тем как прочитал и узнал о том, что это называется тестом :)

            Тупо написал пачку вызовов функции, сравнивая ответ с нужным, потому что лень было руками проверять 100500 вариантов...
            Ответить

            • ava kegdan 30.07.2015 21:53 # 0

              я тоже еще курсе на 2 или 3 какую то лабу тестил - написал функцию с кучей тестов и подумал - клево бы если это было встроенно в ide и плюшек сверху - погуглил - и действительно.

              В программировании без любознательности и смекалки никуда
              Ответить

              • ava 3_14dar 30.07.2015 22:56 # 0

                На "смекалке" (а точнее, гуглении на каждый чих) далеко не уедешь.
                Ответить

        • ava Vasiliy 30.07.2015 21:39 # 0

          У меня например есть такое место. Когда файлы забираются с почтовых ящиков, парсятся, заливаются на фтп и попутно чего то то там рассылаются на другие ящики + плюс в БД активно пишется и ищется там чего нить. Все это сделано в одном файле без разделения на функции. Вот как это говнище unit тестами просто не представляю.
          Ответить

          • ava bormand 30.07.2015 21:42 # +1

            > говнище
            Ну на то оно и говнище, чтобы быть нечитаемым, неподдерживаемым и нетестируемым ;)
            Ответить

          • ava 3_14dar 30.07.2015 22:57 # 0

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

            • ava Vasiliy 30.07.2015 23:23 # −3

              Зачем муднем достаточно быть Укропом. Я к тому что автор хохол.
              Ответить

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