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

    +1

    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
    public void notificationSuccess(String operation) {
        Logger.getLogger(this.getClass().getName()).log(Level.INFO, "Operation " + operation + " success");
        FacesMessage msg = null;
        msg = new FacesMessage(FacesMessage.SEVERITY_INFO, "Notification", "Success");
        FacesContext.getCurrentInstance().addMessage(null, msg);
    }

    public void notificationError(Exception e, String operation) {
        Logger.getLogger(this.getClass().getName()).log(Level.ERROR, "Operation " + operation + " Error ", e);
        FacesMessage msg = null;
        msg = new FacesMessage(FacesMessage.SEVERITY_INFO, "Notification", "Error");
        FacesContext.getCurrentInstance().addMessage(null, msg);
}

    А не созавать ли мне новый экземпляр логгера при каждом уведомлении...

    Запостил: proselyte, 03 Июля 2016

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

    • ava alopen 03.07.2016 10:47 # +1

      Например здесь
      https://github.com/jcabi/jcabi-log/blob/master/src/main/java/com/jcabi/log/Logger.java
      думают, что "создавать каждый раз новый экземпляр" гораздо правильней чем держать в классе поле типа logger. Тем более, getLogger должен кешировать свои результаты.
      Ответить

    • ava bormand 03.07.2016 11:27 # +1

      Да хрен с ним, с логгером. Этот getLogger() поди ничтожен на фоне записи на диск и/или сеть. Меня вот больше беспокоит гора бойлерплейта:
      Logger.getLogger(this.getClass().getName()).log(
      в каждом месте, где надо что-то залогировать... Нечитабельно же.
      Ответить

    • ava guest 05.07.2016 22:45 # +1

      Это разве не синголтон?
      Ответить

    • ava snowtester 06.07.2016 17:30 # +1

      Мусоргщик все уберет...
      Ответить

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