Quote Originariamente inviata da fermat Visualizza il messaggio
a parte il funzionamento (l'ho testato e funziona), può andar bene??
Grosso modo sì. Ma ci sono alcune cose dubbie.

Innanzitutto cosa fa ReadJson? Non vedo argomenti al suo costruttore e neanche al getAllAuthors. Quindi legge un JSON da cosa/dove e con quale API? Probabilmente è tutto "cablato" nel ReadJson.

Poi non mi piace molto quel ArrayList<? extends Object>. Dovresti cercare di parametrizzare in modo più accurato questa lista. Anche perché in un ArrayList parametrizzato così non puoi inserire nulla (a parte un null letterale). E questo quindi dipende da cosa fa getAllAuthors.