1. 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) RSS

    • ava someone 02.03.2016 19:54 # +2

      Вспоминается PHP и его незабвенный T_PAAMAYIM_NEKUDOTAYIM.
      Ответить

      • ava fajes_rown 02.03.2016 19:56 # −2

        Что-что, простите?
        Ответить

        • ava Propovednik_01 02.03.2016 19:58 # −4

          fajes, угостили бы девушку "солнцедаром"...
          Ответить

        • ava bormand 02.03.2016 20:22 # +2

          Двойное двоеточие на иврите. В пыхе ошибка о нём выводится именно так.
          Ответить

          • ava 3_dar 02.03.2016 21:05 # 0

            И чего до сих пор не пофиксили? Или это прикол разработчиков такой?
            Ответить

            • ava 3_14dar 02.03.2016 22:47 # +2

              Обратная совместимость, сэр.
              Ответить

              • ava 3_dar 04.03.2016 17:54 # 0

                Так оставить и то, и то. А в сообщениях об ошибках использовать новую константу.

                Старые говнокоды не поломаются, а новые будут использовать новую константу.
                Ответить

            • ava inkanus-gray 03.03.2016 13:20 # −1

              Mudakai, sir.
              Ответить

            • ava 3.14159265 03.03.2016 15:37 # +4

              Так все уже привыкли (даже те кто на пхп не пишет).
              Если уберут, то все начнут массово жаловаться: "где мой T_PAAMAYIM_NEKUDOTAYIM", "вирните мне ево", "запилите как выпилили".
              Ответить

              • ava 3_dar 04.03.2016 21:40 # 0

                Прочитал сначала как "верните мне евро"
                Ответить

              • ava guest 04.03.2016 22:32 # +1

                Ну и нахуй их посылать. Никуда они не денутся, перепишут свое говно.
                Ответить

                • ava 3_14dar 04.03.2016 23:13 # +2

                  Гость, нахуй посылать можно, но только на говнокодике. В реальности любые серьезные изменения с несовместимостями вызывают дикий вой, ненависть и непопулярность. Луркай переход на третий питон.
                  Ответить

                  • ava guest 04.03.2016 23:30 # +1

                    При переходе на php 7 и по хуже несовместимости есть. Не думаю что T_PAAMAYIM_NEKUDOTAYIM такая популярная херотень, что её нельзя выпилить.
                    Ответить

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