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

    +81

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    public void getApplicationContext(ApplicationContext applicationContext ) {
    applicationContext.setUserId(currentUserService.getCurrentUser().getName());
    applicationContext.setTransactionType(transactionType);
    applicationContext.setApplication(applicationName);
    applicationContext.setSubSystem(subSystemName);
    applicationContext.setAddress(ipAddress);
    applicationContext.setCorrelationId(correlationIDGenerator.getRandomGUID());
}

    *facepalm*

    Запостил: roman-kashitsyn, 12 Января 2012

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

    • ava kyzi007 12.01.2012 13:07 # −1

      Фишка в сете в гете?
      Ответить

      • ava roman-kashitsyn 12.01.2012 13:11 # +1

        В названии метода. Это и не сеттер и не геттер.
        Ответить

        • ava TarasB 12.01.2012 13:21 # 0

          Я сам иногда путаюсь в названии сеттеров и геттеров.
          Например
          SetCurPos(var x,y: integer);
          Правильно, конечно, гет, т.к. она делает гет возиции мышки. Но иногда руки пишут сет, потому что она сетит икс и игрек, т.е. метод читается как "записать позицию мышки в переменные", записать=сет
          Ответить

          • ava roman-kashitsyn 12.01.2012 13:30 # +4

            С моей точки зрения, функция, изменяющая свои аргументы (sic!), исли уж она имеет место быть, должна называться fill<Something> или как-то в этом роде.
            Функциональщики негодуют
            Ответить

        • ava Lure Of Chaos 12.01.2012 13:36 # 0

          выставляет значения полей?
          Ответить

    • ava tir 12.01.2012 14:12 # 0

      название действительно неудачное. пожалуй, поставлю плюс
      Ответить

    • ava Eugene 12.01.2012 16:09 # 0

      Как многа букав, чтобы выразить такую простую мысль.
      Ответить

    • ava guest8 09.04.2019 12:30 # −999

      показать все, что скрытоvanished
      Ответить

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