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

    +153

    1. 1
    2. 2
    3. 3
    if (5 < strlen($method) && '_init' === substr($method, 0, 5)){
    $this->_classResources[strtolower(substr($method, 5))] = $method;
}

    ZendFramework (c)

    Запостил: Bydlo, 21 Декабря 2010

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

    • ava Lure Of Chaos 21.12.2010 11:46 # 0

      шо, и #5000 зафейлили?
      Ответить

    • ava AndrewStephanoff 21.12.2010 12:01 # +1

      А в чём заключён говнокод?
      Ответить

    • ava KomprendaTerrorSquad 21.12.2010 12:21 # −13

      показать все, что скрытоBydlo, иди еби собак
      Ответить

    • ava istem 21.12.2010 14:10 # +4

      "Метод - как строка, и строка как метод", - искренне Ваш PHP
      Ответить

      • ava Мистер Хэнки 21.12.2010 16:44 # 0

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

        • ava istem 21.12.2010 17:53 # 0

          '_init' === substr($method, 0, 5)
          вот это ещё понравилось...
          Ответить

          • ava Int 21.12.2010 20:28 # 0

            На _init провериться хочешь ты.
            Ответить

            • ava AndrewStephanoff 22.12.2010 19:12 # +1

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

              • ava istem 23.12.2010 03:57 # 0

                здесь чуть-чуть другое.
                Коментом ниже правильно замечено, что вместо === хватило бы и ==
                так как:
                во-первых '_init' стоит вначале, что намекает на молчаливом преобразовании сравниваемого значения к строковому типу;
                во-вторых substr так и так возвращает строку
                Ответить

                • ava AndrewStephanoff 23.12.2010 11:05 # 0

                  Коммент ниже мой :) Int имел ввиду, что слева в сравнении стоит строковой литерал, что на первый взгляд может показаться нелогичным, а я объяснил из каких соображений это делают. На самом деле, в приведённом примере это неважно, так как сравнивается с функцией и ошибка присваивания в любом случае будет видна (если она будет). Скорее всего, автор поместил строковой литерал слева чтобы поддерживать целостность своего кода, так как видимо он во всех сравнениях помещает литералы слева. Ну а строгое сравнение с проверкой типа здесь, конечно же, не нужно.
                  Ответить

                  • ava Int 24.12.2010 10:04 # 0

                    Лично мне такие условия сложно в голове преобразовывать, особенно if (2 < $a), лично я не сразу в голове преобразовываю в "а больше двух", приходится лишнее действие делать
                    Ответить

                    • ava AndrewStephanoff 24.12.2010 12:50 # 0

                      поначалу мне тоже было сложно воспринимать, а сейчас нормально читаю, это лишь вопрос практики.
                      Ответить

                • ava AndrewStephanoff 23.12.2010 11:06 # 0

                  И, собственно, я удивлён, что из всего говнокода ZF был выбран только этот кусочек, в котором говнокода практически и нет :-D
                  Ответить

              • ava bugmenot 24.12.2010 10:07 # 0

                присваивания или кошмарного оператора => :)
                Ответить

          • ava AndrewStephanoff 22.12.2010 19:13 # 0

            здесь достаточно было бы обычного сравнения (==) без проверки типа
            Ответить

        • ava Lure Of Chaos 21.12.2010 18:56 # +1

          вот единственное, что мне нравится в пыхе, так это возможность написать
          $object->$method()
          а в той же жабке пришлось бы рефлекшн юзать
          Ответить

    • ava qweqweqwe 25.08.2021 10:20 # 0

      vanished
      Ответить

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