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

• хуита 10 лет ago # 0

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

  Ответить

  • bormand 10 лет ago # 0 ⇈

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

    Ответить

    • guest 10 лет ago # 0 ⇈

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

      Ответить

      • roman-kashitsyn 10 лет ago # 0 ⇈

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

        Ответить

• Fike 10 лет ago # 0

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

  Ответить

• huesto 10 лет ago # 0

  Че не так?

  Ответить

  • bayan 10 лет ago # 0 ⇈

    ниасилил DI

    Ответить

    • huesto 10 лет ago # 0 ⇈

      Это я и сам прочитал.

      Ответить

• bormand 10 лет ago # 0

  Асилил за тебя DI, проверь:

  controller = new Controller();
  controller.network = new Network(controller);

  Ответить

  • bayan 10 лет ago # 0 ⇈

    Network netowork = di.getService(Network.class);

    Ответить

    • bormand 10 лет ago # 0 ⇈

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

      Ответить

      • bayan 10 лет ago # 0 ⇈

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

        Ответить

        • bormand 10 лет ago # 0 ⇈

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

          Ответить

          • barop 10 лет ago # 0 ⇈

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

            Ответить

            • bormand 10 лет ago # 0 ⇈

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

              Ответить

              • roman-kashitsyn 10 лет ago # 0 ⇈

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

                Ответить

                • bormand 10 лет ago # 0 ⇈

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

                  Ответить

                • bayan 10 лет ago # 0 ⇈

                  >>Thread.currentThread().getStackTrace ()
                  Фу, нет.
                  
                  надо кинуть эксепшен, сразу жего поймать, и посмотреть стек трейс

                  Ответить

                  • roman-kashitsyn 10 лет ago # 0 ⇈

                    Мой самый первый говнокод http://govnokod.ru/6946

                    Ответить

                    • ProctologistForYou 10 лет ago # 0 ⇈

                      👬🏳️‍🌈

                      Ответить

  • хуита 10 лет ago # 0 ⇈

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

    Ответить

• Her 10 лет ago # 0

  вот правильный код на Java
  

  factory = NetworkFactory.getInstance();
  connector= factory.getNeworkConnector();
  this.network = connector.connect();

  Ответить

  • inkanus-gray 10 лет ago # 0 ⇈

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

    Ответить

    • bormand 10 лет ago # 0 ⇈

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

      Ответить

      • CHayT 10 лет ago # 0 ⇈

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

        Ответить