1. C++ / Говнокод #391

    +46.2

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    int checkSpecialPrice(ItemId _item, TransDate _fromDate, TransDate _toDate)
    {
      if (specialPrice == true)
      {
       return 1;
      }
      else
      {
       return 0;
      }
    }

    Такой вот метод класса. И весь класс такой же.
    Язык, на самом деле, был X++, но сути это не меняет =)

    Запостил: guest, 14 Января 2009

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

    • J:
      Похоже на заглушку для будущего функционала, или может быть непродуманная иерархия классов/интерфейсов?
      Ответить
    • Chamie:
      Скорее, просто не доупрощали =))
      Ну, то есть, там раньше был, наверное, какой-то хитрый алгоритм от трёх параметров, а потом оказалось, что можно тупо использовать глобальную переменную =)
      Ответить
    • mikluxo:
      Не, реально заглушка. Иначе параметры тоже хромали, а так логично все.
      Ответить
    • ОП:
      Не, не заглушка - я этот метод убил вообще, когда класс переписывал. Точнее, я, практически, вообще весь класс убил..
      А сделан этот класс был под 1 конкретную задачу. Больше всего было похоже как раз на "Так, хз, как это определить - вынесу в отдельный метод, потом подумаю", потому как из таких методов класс состоял чуть менее, чем полностью - за исключением 1 "главного" метода размером около трёхсот строк.
      Ответить
    • Звездочёт:
      !!! Жесть какая... Сделать int функцию в которой возращается, по сути, double... Да, это не Си++.
      Ответить
    • 1 и 0 = double? 0_0

      xDDD
      Ответить
    • Скорее не double а bool...
      Ответить

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