- 1
- 2
- 3
- 4
- 5
- 6
- 7
- 8
- 9
- 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;
}
}
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
0
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", т.к. все это будет использоваться в одной из стандартных функций сортировки
SortWObyAD(-0x7FFFFFFF, 0x7FFFFFFF);