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

    +968

    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 TList<ProductType> ProductTypeListTable
        {
            get
            {
                ProductTypeService productTypeService = new ProductTypeService();

                TList<ProductType> productTypes = productTypeService.GetAll();
                foreach (ProductType pt in productTypes)
                {
                    pt.Name = Server.HtmlDecode(pt.Name);
                }
                
                return productTypes;
            }
        }

    Некая дорогущая АСП.НЕТ платформа. Ну и че там, будем читать базу, пока не надоест...

    Запостил: MagisterCrazy, 17 Октября 2011

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

    • ava SmackMyBitchUp 17.10.2011 20:21 # +1

      >пока не надоест...
      базе
      Ответить

    • ava roman-kashitsyn 17.10.2011 20:29 # +1

      Прекрасный образец IoC/DI и ленивой загрузки данных
      Ответить

    • ava ohlol 17.10.2011 20:37 # −3 

      http://multator.ru/toon/5itkz3f9bxrj
      Ответить

    • ava Lennis 17.10.2011 21:09 # −1

      кроме того что нет кеширования, еще и попытка изменения коллекции в foreach
      Ответить

      • ava roman-kashitsyn 17.10.2011 21:13 # 0

        А можно поподробнее, где тут изменение коллекции?
        Ответить

        • ava Lennis 17.10.2011 21:49 # 0

          pt.Name = Server.HtmlDecode(pt.Name);
          Ответить

          • ava roman-kashitsyn 17.10.2011 21:50 # +1

            А я думал, тут изменяются элементы коллекции, а не сама коллекция.
            Ответить

            • ava Lennis 17.10.2011 21:52 # −1

              и я об этом-же в своем первом сообщении. просто немного некорректно выразился
              Ответить

              • ava absolut 17.10.2011 21:57 # 0

                и что плохого в таких изменениях?
                Ответить

                • ava ohlol 17.10.2011 22:22 # −2

                  изменения коллекции во время проходом форича - приводит к выкидыванию исключения
                  Ответить

                  • ava roman-kashitsyn 17.10.2011 22:31 # 0

                    как мы только что выяснили, коллекция тут не изменяется
                    Ответить

                  • ava absolut 17.10.2011 22:32 # +1

                    В коллекции лежат объекты-ссылки типа ProductType. Они не меняются, меняются данные, на которые они ссылаются. А если там окажутся объекты-значения, то просто изменятся их копии.
                    Ответить

                  • ava abatishchev 18.10.2011 10:12 # 0

                    учи мат-часть :Р
                    Ответить

    • ava piocsic 17.10.2011 23:05 # +1

      Вообще-то из приведенного кода совершенно не понятен сарказм автора. Ну обрабатываем коллекцию, ну и что? Server.HtmlDecode всего лишь предобразовывает текст, а не обращается к базе.
      Ответить

      • ava roman-kashitsyn 17.10.2011 23:08 # 0

        а ProductTypeService список из воздуха, наверное, порождает
        Ответить

        • ava piocsic 17.10.2011 23:10 # 0

          А почему вы считаете, что ProductTypeService этот список уже не породил? Там ProductTypeService.GetAll() вызывается всего единожды, а не в цикле.

          Кроме того, откуда такая уверенность, что ProductTypeService сам не кеширует этот список (если он например из базы берется)?
          Ответить

          • ava absolut 17.10.2011 23:16 # +1

            Отсюда мораль: внешне код - говно, но не зная тип ProductTypeService нельзя сказать: работоспособное это говно или нет.
            Ответить

          • ava roman-kashitsyn 17.10.2011 23:17 # 0

            > А почему вы считаете, что ProductTypeService этот список уже не породил?
            > new ProductTypeService();
            Ну не знаю, что там может заранее породить только-что созданный объект

            > ProductTypeService.GetAll() вызывается всего единожды
            Он вызывается при каждом обращении к свойству

            Кроме того, цикл с HtmlDecode как бэ намекает, что айтемы в списке всегда свеженькие
            Ответить

            • ava piocsic 17.10.2011 23:21 # 0

              Я прокомментировал сарказм автора по поводу обращения к базе пока не надоест. В приведенном коде этого нет. Если бы автор указал где и как эта проперти вызывается, тогда да, можно было бы и посудачить о криворуких программистах.
              Ответить

            • ava mefcorvi 29.10.2011 14:43 # 0

              Только, что созданный объект может читать данные из static переменной, или из Cache объекта. При этом, это может быть правильно реализовано с поддержкой многопоточного окружения. Говнокод тут в основном в создании сервиса внутри геттера, и в неправильном использовании этого геттера, т.к. я почти уверен в том, что тут подразумевался метод GetProductTypeListTable.
              Ответить

    • ava guest8 09.04.2019 11:45 # −999

      показать все, что скрытоvanished
      Ответить

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