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

    +164

    1. 1
    $update = !empty($id) and $id > 0;

    Вопреки ожиданиям автора, выражение $id > 0 вообще никогда не принимается во внимание.
    /* Ознакомьтесь с приоритетом операций */
    Сначала отработает $update = !empty($id)
    потом значение из $update будет сравниваться с $id > 0 и результат сравнения никуда не попадёт.

    Рекомендация: используйте && вместо оператора "and".

    Запостил: zabuhailo, 09 Февраля 2011

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

    • ava Kevin 09.02.2011 16:21 # +2

      Достаточным было бы использовать скобочки для данной операции.

      $update = ( !empty($id) and $id > 0 );
      Ответить

      • ava zabuhailo 10.02.2011 08:53 # 0

        Верно. Использование скобок вернёт смыл этому коду, и в данном конкретном случае это будет даже наглядно показывать намерения программиста. Однако если скобок много в одном месте (и ещё они вложенные) и проч. код сразу перестаёт читаться. Всё же следуйте рекомендации - используйте && и не парьтесь.
        Ну уж если вы великий гуру и пишете только конкретный код только для себя - пожалуйста, можете даже использовать && и "and" в одном выражении.
        Ответить

    • ava unfunk 09.02.2011 17:00 # +2

      уроки пхп на гк.ру
      Ответить

    • ava Lure Of Chaos 09.02.2011 18:02 # −1

      какой дебильный язык.
      Ответить

      • ava Zho 09.02.2011 22:18 # 0

        не пишите на нем никогда =)
        Ответить

      • ava Int 09.02.2011 23:05 # 0

        Внезапно! И в перле тоже && приоритетнее =, который приоритетнее AND. Долой языки программирования, долой компьютеры, даёшь натуральное хозяйство!
        Ответить

      • ava Kevin 10.02.2011 13:54 # +1

        И в Ruby «=» имеет больший приоритет нежели «and». ;-)
        Ответить

        • ava Lure Of Chaos 10.02.2011 13:57 # 0

          эпидемия. Зачем так делают?
          Ответить

          • ava Kevin 10.02.2011 14:05 # 0

            Чтобы можно было сделать так:
            b = 10
d = 20
if a = b and d == 20 then
	puts a # а имеет значение 10
end

            Возможность есть, а стоит ей пользоваться — каждый решает сам.
            Ответить

            • ava Kevin 10.02.2011 14:11 # 0

              А если использовать &&, то:
              if a = b && d == 20 then # эквивалентно a = ( b && d == 20 )
	puts a # теперь это — true
end
              Ответить

            • ava Lure Of Chaos 10.02.2011 14:13 # 0

              во-первых, присваивание в большинстве случаев лучше выносить в отдельный оператор. Особенно учитывая, что довольно частая описка = вместо ==.
              во-вторых, лишняя пара скобок - не такая уж избыточность.
              Ответить

              • ava Kevin 10.02.2011 14:17 # 0

                Что куда лучше выносить — зависит от задачи. Язык дает возможности, а мы уже решаем, какие возможности подходят под задачи. ;-)

                А есть еще битовые операции: & и |, которые синтаксически напоминают логические. :-)
                Ответить

                • ava Lure Of Chaos 10.02.2011 14:19 # 0

                  может быть ) но в приоритетах можно и запутаться
                  а с битово-логическими путаницы нет )
                  Ответить

                  • ava Kevin 10.02.2011 14:22 # 0

                    Запутаться можно лишь по незнанию, а не по невнимательности. Ведь с тем, что у && приоритет выше, чем у || мало у кого возникают трудности. ;-)
                    Ответить

      • ava Spider 11.02.2011 14:46 # 0

        Вы не поверите, но в Руби && и and работают точно так же
        Тоже дебильный язык, да?
        Ответить

        • ava Lure Of Chaos 11.02.2011 15:52 # 0

          я им как раз начал интересоваться. Пока не дал оценку, но на первый взгляд, пугает )
          Ответить

          • ava Spider 15.02.2011 13:02 # 0

            На мой взгляд там много вкуснстей, но слишком много синтаксического сахара
            Ответить

        • ava zabuhailo 15.02.2011 09:24 # 0

          Язык не дебильный, он тут вообще не при чём.
          Говнокод пишет быдлокодер а не язык.
          Ответить

          • ava Spider 15.02.2011 13:01 # 0

            Вот и я о том же
            Ответить

            • ava bugmenot 15.02.2011 13:29 # +2

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

              • ava guest 15.02.2011 13:45 # 0

                Сможете упорядочить языки по атрибуту привлекательности для быдлокодеров?
                Ответить

                • ava bugmenot 15.02.2011 15:20 # 0

                  сходу вижу два три критерия:
                  * нижайший порог вхождения
                  * за работу на языке X платят хоть какие-то деньги
                  * не эзотерический (важно чтобы существовали источники копипасты)
                  Ответить

                  • ava Lure Of Chaos 15.02.2011 15:52 # +2

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

                    на данный момент, это
                    1. пхп
                    2. ява
                    3. дельфи (паскаль)
                    4. си-решетка
                    Ответить

                    • ava bugmenot 15.02.2011 16:14 # 0

                      > на данный момент
                      на данный момент есть эмпирические данные, теперь надо вывести теорию

                      будет теория - сделаем
                      0. Visual Bydlol
                      Ответить

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