Registrazione dai metodi di interfaccia predefiniti

Question

Saluto a tutti i guru Java!

Poiché Java8 possiamo avere implementazioni predefinite nelle interfacce (yay!). Tuttavia, il problema sorge quando si desidera effettuare il log dal metodo predefinito.

Ho la sensazione che non sia saggio chiamare .getLogger() ogni volta che voglio registrare qualcosa in un metodo predefinito.

Sì, è possibile definire una variabile statica in un'interfaccia, ma non è comunque una buona pratica per le interfacce + espone il registratore (deve essere pubblico).

soluzione che ho in questo momento:

interface WithTimeout<Action> { 

    default void onTimeout(Action timedOutAction) { 
     LogHolder.LOGGER.info("Action {} time out ignored.", timedOutAction); 
    } 

    static final class LogHolder { 
     private static final Logger LOGGER = getLogger(WithTimeout.class); 
    } 
} 

LogHolder è ancora visibile a tutti che in realtà non ha alcun senso in quanto non prevede alcun metodo e dovrebbe essere interno all'interfaccia.

Qualcuno di voi sa di una soluzione migliore? :)

EDIT: Io uso SLF4J sostenuta da Logback

Answer 1

Se non si vuole esporre la classe LogHolder al pubblico, non rendono una classe membro del interface. Non vi è alcun vantaggio nel farne una classe membro, non si risparmia nemmeno la digitazione in quanto è necessario qualificare l'accesso al campo con il nome della classe titolare, indipendentemente dal fatto che si tratti di una classe membro o di una classe all'interno dello stesso pacchetto :

public interface WithTimeout<Action> { 

    default void onTimeout(Action timedOutAction) { 
     LogHolder.LOGGER.info("Action {} time out ignored.", timedOutAction); 
    } 
} 
final class LogHolder { // not public 
    static final Logger LOGGER = getLogger(WithTimeout.class); 
} 

Answer 2

Ecco qui.

Il logger è privato all'interfaccia. Nessuno oltre a questa interfaccia e i suoi metodi predefiniti possono accedere a qualsiasi cosa all'interno di Test2. E nulla può estendere la classe Test2.

Nessuno ti sta suggerendo di farlo ... ma funziona! Ed è un ottimo modo per accedere a un logger di classe per l'interfaccia principale, e possibilmente un modo intelligente per fare qualcos'altro non del tutto pazzo.

Questo è esattamente lo stesso di LogHolder nella domanda OP, tranne che la classe Test2 è anche tutti i metodi privati ​​e privati ​​del costruttore e la classe non è contrassegnata come statica.

E come bonus aggiuntivo, mantiene lo stato, staticamente e per istanza. (non farlo in un programma vero e proprio, per favore!)

public class TestRunner { 
    public static void main(String[] args) { 
     Test test = new Test() { 
     }; 
     test.sayHello("Jane"); 
     System.out.println("Again"); 
     test.sayHello("Bob"); 
    } 
} 
public interface Test { 
    default void sayHello(String name) { 
     Logger log = Test2.log; 
     Test2 ref = Test2.getMine.apply(this); 
     int times = ref.getTimes(); 
     for (int i = 0; i < times; i++) { 
      System.out.println(i + ": Hello " + name); 
     } 
     log.info("just said hello {} times :)",times); 
    } 
    final class Test2 { 
     private static final Logger log = LoggerFactory.getLogger(Test.class); 
     private static final Map lookup = new WeakHashMap(); 
     private static final Function getMine = (obj) -> { 
      return lookup.computeIfAbsent(obj, (it) -> new Test2()); 
     }; 
     private int calls = 0; 
     private Test2() { 
     } 
     private void setCalls(int calls) { 
      this.calls = calls; 
     } 
     private int getCalls() {return calls;} 
     private int getTimes() {return ++calls;} 
    } 
}