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

    • ava guest 18.01.2009 12:40 # +1

      J:
      Похоже на заглушку для будущего функционала, или может быть непродуманная иерархия классов/интерфейсов?
      Ответить

    • ava guest 19.01.2009 15:57 # 0

      Chamie:
      Скорее, просто не доупрощали =))
      Ну, то есть, там раньше был, наверное, какой-то хитрый алгоритм от трёх параметров, а потом оказалось, что можно тупо использовать глобальную переменную =)
      Ответить

    • ava guest 27.01.2009 21:37 # 0

      mikluxo:
      Не, реально заглушка. Иначе параметры тоже хромали, а так логично все.
      Ответить

    • ava guest 28.01.2009 12:31 # 0

      ОП:
      Не, не заглушка - я этот метод убил вообще, когда класс переписывал. Точнее, я, практически, вообще весь класс убил..
      А сделан этот класс был под 1 конкретную задачу. Больше всего было похоже как раз на "Так, хз, как это определить - вынесу в отдельный метод, потом подумаю", потому как из таких методов класс состоял чуть менее, чем полностью - за исключением 1 "главного" метода размером около трёхсот строк.
      Ответить

    • ava guest 10.04.2009 21:32 # −4

      Звездочёт:
      !!! Жесть какая... Сделать int функцию в которой возращается, по сути, double... Да, это не Си++.
      Ответить

    • ava guest 20.08.2009 16:44 # 0

      1 и 0 = double? 0_0

      xDDD
      Ответить

    • ava guest 27.08.2009 18:19 # 0

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

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