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

    +107

    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
    @Inject
    protected MainWindowController(
    		final SessionController session,
    		final QueueController queueController,
    		final MachineController machineController,
    		final PrintController printController,
    		final MainWindow window,
    		final UIHandler uiHandler,
    		final ImageCache imageCache,
    		final EventBus bus,
    		final ActiveOrderHolder orderHolder,
    		final ApiClientAsync apiClient,
    		final ReferenceDataCache dataCache,
    		@SvnRevision final int svnRevision) throws IOException {

    Запостил: someone, 20 Июля 2012

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

    • И где тут говнокод?
      Ответить
      • Дак вон же, вон говнище... а нет, показалось.
        Ответить
      • >throws IOException
        ну вот нахуя он там?

        И на самом деле мне непонятно почему Java-говно минусуют, а годом ранее идеологически схожее С++ говно - плюсовали: http://govnokod.ru/7019
        Ответить
        • То говно стоит плюсануть за мега-геттер getAllValues(). А это уныло.
          Ответить
    • It's Enterprise, baby!
      Ответить
    • Mediator pattern: practice
      Ответить
    • А почему бы не на полях @Inject'тить? Или класс состоит из одного конструктора?
      Ответить
      • а почему бы и нет? В мануале по Guice предлагают именно такой подход.
        Ответить
        • >а почему бы и нет?
          Роман, а как же идеи карринга?
          Ответить
          • А при чём здесь карринг?
            Ответить
            • Ну применение свойств к объекту по одному:
              new MainWindowController().setSession(sessio n)
              Ответить
              • Так мы ж не руками будем сеттить аттрибуты, за нас это сделает Guice...
                Ответить
                • Ну а вдруг где придется сущность руками напихивать. Или такой вариант полностью исключен?
                  Ответить
                  • В тестах придётся руками моки запихивать, но вроде от fluent interface легче не сделается...
                    Ответить
    • protected MainWindowController(
      ...
      		final SessionController session,
      		final QueueController queueController,
      		final MachineController machineController,
      		final PrintController printController,

      Надо было юзать final ControlController controlController или фабрику контролеров контролеров.
      Ответить

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