1. Java / Говнокод #21908

    −48

    1. 1
    this.network = new Network(this);

    Я ниасилил DI.

    Запостил: хуита, 28 Декабря 2016

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

    • ava хуита 28.12.2016 09:36 # 0

      Даже так:
      public Controller(){
      this.network = new Network(this);
      }
      Ответить

      • ava bormand 28.12.2016 17:23 # +4

        И Network получает недоконструированный Controller...
        Ответить

        • ava guest 27.01.2017 06:25 # −2

          нет, все норм будет.
          http://ideone.com/AuZBDE
          Ответить

          • ava roman-kashitsyn 27.01.2017 12:23 # 0

            > http://ideone.com/AuZBDE

            Код не компилируется. Да и не будет всё норм: объект в жавке не стоит публиковать до завершения конструктора. Вдруг network стартует в конструкторе тред, в котором вызывает методы Controller. Читай про безопасную публикацию объектов в жаве.http://vlkan.com/blog/post/2014/02/14/java-safe-publication/
            Ответить

    • ava Fike 28.12.2016 12:29 # 0

      боюсь даже предположить, чем занимается Network
      Ответить

    • ava huesto 28.12.2016 16:57 # 0

      Че не так?
      Ответить

    • ava bormand 28.12.2016 17:09 # +3

      Асилил за тебя DI, проверь:
      controller = new Controller();
controller.network = new Network(controller);
      Ответить

      • ava bayan 28.12.2016 17:20 # 0 

        Network netowork = di.getService(Network.class);
        Ответить

        • ava bormand 28.12.2016 17:21 # 0

          Это внутри Controller'а или снаружи? Если внутри - то это уже говнолокатор сервисов, а не DI получается.
          Ответить

          • ava bayan 28.12.2016 17:39 # 0

            между типизированным говнолокатором и DI разница не велика

            но лучше конечно автоинжект в конструктор
            Ответить

            • ava bormand 28.12.2016 17:42 # +1

              Велика. Говнолокатор не знает, кому он возвращает текущий объект интерфейс.
              Ответить

              • ava barop 29.12.2016 04:44 # 0

                DI тоже пофиг кому он что возвращает

                Важно что он собирает тебе всю деревяшку объектов
                Внутри Network может еще 140 инжектов быть, и 3 из них в конструктор
                Ответить

                • ava bormand 29.12.2016 18:22 # 0

                  > DI тоже пофиг кому он что возвращает
                  DI может одному объекту заинжектить SimpleNetwork, второму - AdvancedNetwork, а третьему - вообще NetworkMock. Локатор так не сможет, т.к. он не знает, куда пойдёт Network о котором его попросили.
                  Ответить

                  • ava roman-kashitsyn 29.12.2016 18:28 # +2

                    > Локатор так не сможет

                    Он может посмотреть в Thread.currentThread().getStackTrace() и решить
                    Ответить

                    • ava bormand 29.12.2016 18:49 # 0

                      А самое ужасное - конструктор Network'а в общем-то тоже может это сделать...
                      Ответить

                    • ava bayan 29.12.2016 19:34 # 0

                      >>Thread.currentThread().getStackTrace ()
                      Фу, нет.

                      надо кинуть эксепшен, сразу жего поймать, и посмотреть стек трейс
                      Ответить

      • ava хуита 28.12.2016 17:35 # +4

        Проверил, JDK распидорасило, пишу спитона.
        Ответить

    • ava Her 28.12.2016 20:10 # +1

      вот правильный код на Java
      factory = NetworkFactory.getInstance();
connector= factory.getNeworkConnector();
this.network = connector.connect();
      Ответить

      • ava inkanus-gray 28.12.2016 20:11 # +4

        Что-то не слишком ынтырпрайзно. Не все паттерны использованы.
        Ответить

        • ava bormand 28.12.2016 20:19 # +7

          > Не все паттерны использованы
          Надо в статический анализатор правило такое добавить.
          Ответить

          • ava CHayT 28.12.2016 20:40 # +7

            Продай эту идею разработчикам пивас-студии. Представляю их следующую статью про ядро...
            Ответить

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