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

    +68

    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
    public function makeCategoryArray() {
        $a = array();
        for($i = 1; $i <=3 ; $i++) {
            $j = $i != 1 ? $i : '';
            $Method = "getCategoryid" . $j;
            $categoryid = $this->$Method();
            if($this->$Method()) {
                try{
                    $category = BC::Get()->getCompanyService()->getCategoryByID($categoryid);
                    $a[] = array(
                        'name' => $category->getName(),
                        'url' => $category->makeURL()
                    );
                } catch(Exception $e) {
                    return $a;
                }
            }
        }
        return $a;
    }

    Эпичный вызов getCategoryid1(), getCategoryid2(), getCategoryid3() когда есть getField(fieldname)...

    Запостил: max_wp, 13 Июля 2012

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

    • ava dev6alexander 16.07.2012 21:57 # +1

      Да автор нагородил забор, было бы куда изящнее юзать __call
      http://www.php.net/manual/ru/language.oop5.overloading.php#object.cal l
      Ответить

      • ava max_wp 16.07.2012 22:12 # 0

        Куда изящнее было бы убрать try-catch ибо он не нужен там,
        затем вызвать getCategory(INDEX) :)
        Ответить

        • ava dev6alexander 17.07.2012 08:42 # 0

          Возможно эти методы - getCategoryid1(), getCategoryid2(), getCategoryid3()....
          реализуют разную функциональность, об этом история умалчивает. Поэтому он вынес реализацию в разные методы а не в один как вы предлагаете - getCategory(INDEX)
          Ответить

          • ava roman-kashitsyn 17.07.2012 09:37 # 0

            По мне так обычные геттеры. Тем более, топикстартеру виднее.
            Ответить

            • ava dev6alexander 17.07.2012 09:40 # 0

              сорри не заметил, что он создатель топика :)
              Гетеры то гетеры но так как он их вызывает...
              Ответить

              • ava bormand 17.07.2012 12:10 # +2

                Гетеры развлекали, утешали и образовывали мужчин. Гетеры не обязательно торговали телом, а скорее щедро обогащали знаниями.
                Ответить

                • ava dev6alexander 17.07.2012 12:11 # 0

                  :)
                  Ответить

                • ava defecate-plusplus 17.07.2012 12:15 # +2


                  - Василий Иваныч, а что такое оргия?
                  - Ну попойка такая.
                  - А кто такие гетеры?
                  - Да проститутки, Петька.
                  - А патриции?..
                  - Чего ты там такое читаешь, скажи полностью?
                  - "Патриции часто устраивали оргии с гетерами."
                  - Так там описка, не патриции, а партийцы.
                  Ответить

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