Non penso che l'articolo implichi che il metodo ogni dovrebbe essere registrato in quel modo, sta solo dicendo quando stai loggando assicurati di catturare il contesto in cui si è verificato il registro.
Ad esempio, se si dispone di un registro che dice solo "Utente non può essere trovato", se si sta effettivamente cercando di capire lo scenario che porta a quel registro, probabilmente è necessario conoscere almeno l'ID utente. In genere gli argomenti delle funzioni hanno l'effetto più significativo sull'output di una funzione, quindi questo è il contesto più importante da catturare, ma le variabili di classe o globali potrebbero essere altrettanto importanti a seconda della situazione.
Nel caso di un'applicazione web Spring, sicuramente non inserirò il log in ogni singola funzione, ma registrerei dove sono probabili i fallimenti. Ad esempio, se si effettua una chiamata a un servizio esterno.
L'articolo stesso afferma:
Of course, you must be reasonable but every method that: accesses
external system (including database), blocks, waits, etc. should be
considered.
Sebbene alcuni di questi log potrebbero essere già forniti dal tuo ORM o RDBMS, quindi non aggiungerei la registrazione ad ogni singola chiamata di database.
L'articolo afferma inoltre:
You should consider DEBUG or TRACE levels as best suited for these
types of logs.
Che tipicamente non sono registrati in produzione, quindi potresti considerarlo un'alternativa alle istruzioni temporanee di stampa / eco durante lo sviluppo. Di nuovo, solo dove appropriato.