1. C# / Говнокод #24897

    +2

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 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 достоинства:
    1) абсолютно не понятно, что такое AD;
    2) ах да, все это можно было бы заменить одной строкой "return wo1.AD - wo2.AD", т.к. все это будет использоваться в одной из стандартных функций сортировки

    Запостил: Moses, 11 Октября 2018

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

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