- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
- 11
- 12
- 13
- 14
- 15
private int SortWObyAD(WorkOrderShort wo1, WorkOrderShort wo2)
{
if (wo1.AD == wo2.AD)
{
return 0;
}
else if (wo1.AD > wo2.AD)
{
return 1;
}
else
{
return -1;
}
}
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+2
private int SortWObyAD(WorkOrderShort wo1, WorkOrderShort wo2)
{
if (wo1.AD == wo2.AD)
{
return 0;
}
else if (wo1.AD > wo2.AD)
{
return 1;
}
else
{
return -1;
}
}
Сразу 2 достоинства:
1) абсолютно не понятно, что такое AD;
2) ах да, все это можно было бы заменить одной строкой "return wo1.AD - wo2.AD", т.к. все это будет использоваться в одной из стандартных функций сортировки
Steve_Brown 11.10.2018 18:08 # +1
SortWObyAD(-0x7FFFFFFF, 0x7FFFFFFF);
Moses 11.10.2018 18:36 # +1
Moses 11.10.2018 18:40 # +3
6arPoBblu_nemyx 05.04.2019 18:38 # 0