Кресты / Говнокод #391 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  10. 10
  11. 11
int checkSpecialPrice(ItemId _item, TransDate _fromDate, TransDate _toDate)
{
  if (specialPrice == true)
  {
   return 1;
  }
  else
  {
   return 0;
  }
}

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

Запостил: guest guest, (Updated )

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

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

      https://www.zbfarm.info/
      ________
      Ответить

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

Помни, guest, за тобой могут следить!

    А не использовать ли нам bbcode?


    8