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

    +143

    1. 1
    2. 2
    if (medicalRecords == null)
    new List<MedicalRecordDispatchingInfoViewModel>();

    Просто сделай мне лист

    Запостил: Vadimyan, 26 Февраля 2014

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

    • ava Konardo 26.02.2014 14:59 # −11

      показать все, что скрытоЛакни мои яйца.
      Ответить

    • ava Lure Of Chaos 26.02.2014 21:37 # +3

      а может, просто опечатка?
      Ответить

      • ava inkanus-gray 26.02.2014 22:06 # +7

        А вдруг автор чего-то ждёт от побочного эффекта конструкторов? Может быть, там побочноэффектно-ориентированное программирование?
        Ответить

        • ava guest 26.02.2014 22:11 # +10

          Побочно-ориентированное программирование
          Ответить

      • ava Vadimyan 27.02.2014 08:35 # +2

        Там весь проект чудесный. Создать ViewModel и сразу вызвать Dispose перед использованием, потому что в диспозе происходит отписка от каких-то событий, которые не нужны - это норма.
        Ответить

        • ava guest 02.03.2014 03:50 # 0

          А какие побочные эффекты могут быть от вызова dispose? Алсо, в сишарпе же так чудесно работает рефакторинг.
          Ответить

          • ava cyperh 03.03.2014 23:48 # 0

            http://msdn.microsoft.com/ru-ru/library/system.idisposable.dispose(v=vs.110).asp x
            При вызове Dispose освобождаются ресурсы, занятые объектом, а при определенном стечении обстоятельств сборщик мусора может отнести такой объект к подлежащиму финализации и отправить в очередь финализации. А при этом, по заявлениям автора, с таким объектом потом пытаются работать, а это может привести к exception. И при чем тут рефакторинг вообще?
            Ответить

            • ava guest 08.03.2014 05:02 # 0

              При том, что ШВУ может чуть ли не в 2 клика выделить код в отдельный метод.
              Ответить

    • ava guest 18.04.2014 16:12 # +1

      Попадался код, где вьюмодель создавала окно в своём конструкторе, там вполне можно было такое увидеть (правда не List, а одиночный экземпляр создавался и сам себя отображал.) Смущали такие куски
      Ответить

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