1. Список говнокодов пользователя kontora

    Всего: 3

  2. C# / Говнокод #20058

    +6

    1. 1
    2. 2
    3. 3
    if (selectedGroup == null)
    return null;
return selectedGroup;

    зачем if то?

    kontora, 23 Мая 2016

    Комментарии (24)

  3. C# / Говнокод #20016

    0

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    9. 9
    foreach (var r in rezList)
{
    int newId = rnd.Next();
    rez.Add(new FileItem()
    {
        Id = newId,
        /* ..... */
     });
}

    Новый способ генерирования ID...

    kontora, 16 Мая 2016

    Комментарии (69)

  4. C# / Говнокод #19561

    −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
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    [HttpPost]
        [Route("api/Pateikimai/SaveList")]
        public OperacijosRezultatas SaveList(List<PateikimasEditItem> models)
        {
            var atsakymas = new OperacijosRezultatas();
            int failCounter = 0;
            foreach (var model in models) {
                try {
                    PateikimaiBll.SaveOrUpdate(model);
                }
                catch (Exception ex) {
                    failCounter++;
                }
            }
            if (failCounter > 0)
            {
                atsakymas.SekmingaOperacija = false;
                atsakymas.PridetiKlaida("Ne visi įrašai buvo išsaugoti");
            }
            else {
                atsakymas.PridetiSekme("Pateikimų įrašas sėkmingai išsaugotas");
            }
            

            return atsakymas;
        }

    Смесь литовского с английским. Try/catch, который не записывает в лог и юзается для того, чтобы в front-end'e не запариватся с масивами (повторяющееся элементы). Уже не говорю об отсутсвии транзакций и foreach'e в контроллере.

    kontora, 02 Марта 2016

    Комментарии (14)