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

    • ava qsort 20.07.2012 10:36 # +4

      И где тут говнокод?
      Ответить

      • ava vistefan 20.07.2012 11:11 # +1

        Дак вон же, вон говнище... а нет, показалось.
        Ответить

      • ava 3.14159265 20.07.2012 14:19 # 0

        >throws IOException
        ну вот нахуя он там?

        И на самом деле мне непонятно почему Java-говно минусуют, а годом ранее идеологически схожее С++ говно - плюсовали: http://govnokod.ru/7019
        Ответить

        • ava bormand 20.07.2012 14:51 # +1

          То говно стоит плюсануть за мега-геттер getAllValues(). А это уныло.
          Ответить

    • ava guest 20.07.2012 11:17 # +5

      It's Enterprise, baby!
      Ответить

    • ava roman-kashitsyn 20.07.2012 12:12 # +1

      Mediator pattern: practice
      Ответить

    • ava guest 20.07.2012 13:27 # 0

      А почему бы не на полях @Inject'тить? Или класс состоит из одного конструктора?
      Ответить

      • ava roman-kashitsyn 20.07.2012 13:29 # 0

        а почему бы и нет? В мануале по Guice предлагают именно такой подход.
        Ответить

        • ava 3.14159265 20.07.2012 14:22 # 0

          >а почему бы и нет?
          Роман, а как же идеи карринга?
          Ответить

          • ava roman-kashitsyn 23.07.2012 11:02 # 0

            А при чём здесь карринг?
            Ответить

            • ava 3.14159265 23.07.2012 16:17 # 0

              Ну применение свойств к объекту по одному:
              new MainWindowController().setSession(sessio n)
              Ответить

              • ava roman-kashitsyn 23.07.2012 16:18 # 0

                Так мы ж не руками будем сеттить аттрибуты, за нас это сделает Guice...
                Ответить

                • ava 3.14159265 23.07.2012 16:23 # 0

                  Ну а вдруг где придется сущность руками напихивать. Или такой вариант полностью исключен?
                  Ответить

                  • ava roman-kashitsyn 23.07.2012 16:26 # 0

                    В тестах придётся руками моки запихивать, но вроде от fluent interface легче не сделается...
                    Ответить

    • ava 3.14159265 20.07.2012 14:17 # 0 

      protected MainWindowController(
...
		final SessionController session,
		final QueueController queueController,
		final MachineController machineController,
		final PrintController printController,

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

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