Си диез / Говнокод #19561 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  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 kontora, (Updated )

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

  • Вспоминается PHP и его незабвенный T_PAAMAYIM_NEKUDOTAYIM.
    Ответить
      • Двойное двоеточие на иврите. В пыхе ошибка о нём выводится именно так.
        Ответить
        • И чего до сих пор не пофиксили? Или это прикол разработчиков такой?
          Ответить
            • Так оставить и то, и то. А в сообщениях об ошибках использовать новую константу.

              Старые говнокоды не поломаются, а новые будут использовать новую константу.
              Ответить
          • Так все уже привыкли (даже те кто на пхп не пишет).
            Если уберут, то все начнут массово жаловаться: "где мой T_PAAMAYIM_NEKUDOTAYIM", "вирните мне ево", "запилите как выпилили".
            Ответить
            • Ну и нахуй их посылать. Никуда они не денутся, перепишут свое говно.
              Ответить
              • Гость, нахуй посылать можно, но только на говнокодике. В реальности любые серьезные изменения с несовместимостями вызывают дикий вой, ненависть и непопулярность. Луркай переход на третий питон.
                Ответить
                • При переходе на php 7 и по хуже несовместимости есть. Не думаю что T_PAAMAYIM_NEKUDOTAYIM такая популярная херотень, что её нельзя выпилить.
                  Ответить

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

Где здесь C++, guest?!

    А не использовать ли нам bbcode?


    8