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

    +160.1

    1. 1
    2. 2
    3. 3
    public function getLastThreeIdeas($limit=3) {
    return $this->DB->select('SELECT id, title, user_name FROM ideas ORDER BY date DESC LIMIT ?d', $limit);
}

    Отличный пример несоответствия названия метода и того, что он делает. По названию метода можно предположить, что он возвращает последние 3 записи. На деле можно получить больше записей блягодаря параметру $limit.

    Запостил: vavaka, 08 Декабря 2009

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

    • ava ivmaroz 08.12.2009 11:29 # +4

      Как будто Вам не приходилось расширять функционал.

      Раньше возвращало три записи. Потом понадобилось больше (или меньше), а метод уже использовался во многих местах. Поэтому дописали не обязательный параметр $limit=3, чтобы не править по всему коду. А где нужно уже подставляли и 5, и 10.

      P.S. Набирайтесь опыта :)
      Ответить

      • ava Oleg_quadro 08.12.2009 18:56 # +3

        Да не надо количество пихать в название функции.
        Ответить

        • ava bildja 11.12.2009 00:28 # 0

          Могла быть фишка именно такая.. Привязанная к количеству и для неё именно вот легко узнаваемое было б такое название.. Типа такого..
          Ответить

    • ava ivmaroz 08.12.2009 11:34 # +6

      Правда, чтобы соответствовало можно было сделать так
      public function getLastIdeas($limit) {
    return $this->DB->select('SELECT id, title, user_name FROM ideas ORDER BY date DESC LIMIT ?d', $limit);
}
public function getLastThreeIdeas() {
    return $this->getLastIdeas(3);
}

      Но это лишняя функция
      Ответить

      • ava guest 08.12.2009 12:41 # 0

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

        • ava ivmaroz 08.12.2009 13:03 # +1

          Я из личного опыта пояснил, откуда могло взяться $limit=3. Под одни нужды пишется функция. Потом оказывается, что нужно немного расширить функционал. Тогда в параметр добавляется необязательный аргумент, чтобы в остальных местах местах не менять код и можно дальше использовать эту же функцию, но уже передавать свои аргументы
          Ответить

      • ava gecko 08.12.2009 12:52 # 0

        А может не следовало вообще зашивать количество в имени функции?
        Ответить

        • ava ivmaroz 08.12.2009 13:04 # 0

          Лично я не называю функции, чтобы там было количество
          Ответить

      • ava guest 08.12.2009 14:07 # +1

        Лишняя функция уберётся при рефакторинге и получится симпатичный код.
        Ответить

    • ava guest 10.12.2009 14:57 # 0

      Скопировать под нужным именем, да и отметить как Deprecated. Делов-то.
      Ответить

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