- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
- 11
int checkSpecialPrice(ItemId _item, TransDate _fromDate, TransDate _toDate)
{
if (specialPrice == true)
{
return 1;
}
else
{
return 0;
}
}
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+46.2
int checkSpecialPrice(ItemId _item, TransDate _fromDate, TransDate _toDate)
{
if (specialPrice == true)
{
return 1;
}
else
{
return 0;
}
}
Такой вот метод класса. И весь класс такой же.
Язык, на самом деле, был X++, но сути это не меняет =)
guest 18.01.2009 12:40 # +1
Похоже на заглушку для будущего функционала, или может быть непродуманная иерархия классов/интерфейсов?
guest 19.01.2009 15:57 # 0
Скорее, просто не доупрощали =))
Ну, то есть, там раньше был, наверное, какой-то хитрый алгоритм от трёх параметров, а потом оказалось, что можно тупо использовать глобальную переменную =)
guest 27.01.2009 21:37 # 0
Не, реально заглушка. Иначе параметры тоже хромали, а так логично все.
guest 28.01.2009 12:31 # 0
Не, не заглушка - я этот метод убил вообще, когда класс переписывал. Точнее, я, практически, вообще весь класс убил..
А сделан этот класс был под 1 конкретную задачу. Больше всего было похоже как раз на "Так, хз, как это определить - вынесу в отдельный метод, потом подумаю", потому как из таких методов класс состоял чуть менее, чем полностью - за исключением 1 "главного" метода размером около трёхсот строк.
guest 10.04.2009 21:32 # −4
!!! Жесть какая... Сделать int функцию в которой возращается, по сути, double... Да, это не Си++.
guest 20.08.2009 16:44 # 0
xDDD
guest 27.08.2009 18:19 # 0
guest8 24.09.2019 22:33 # −999