{"comment":{"url":"https://api.github.com/repos/namero999/goeuro-dev-test/comments/17392709","html_url":"https://github.com/namero999/goeuro-dev-test/commit/749bf4c26e2bff0826f779661e51cc4ad16dac57#commitcomment-17392709","id":17392709,"user":{"login":"namero999","id":1874223,"avatar_url":"https://avatars.githubusercontent.com/u/1874223?v=3","gravatar_id":"","url":"https://api.github.com/users/namero999","html_url":"https://github.com/namero999","followers_url":"https://api.github.com/users/namero999/followers","following_url":"https://api.github.com/users/namero999/following{/other_user}","gists_url":"https://api.github.com/users/namero999/gists{/gist_id}","starred_url":"https://api.github.com/users/namero999/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/namero999/subscriptions","organizations_url":"https://api.github.com/users/namero999/orgs","repos_url":"https://api.github.com/users/namero999/repos","events_url":"https://api.github.com/users/namero999/events{/privacy}","received_events_url":"https://api.github.com/users/namero999/received_events","type":"User","site_admin":false},"position":76,"line":69,"path":"src/JSONAPIProcessor.java","commit_id":"749bf4c26e2bff0826f779661e51cc4ad16dac57","created_at":"2016-05-08T14:20:37Z","updated_at":"2016-05-08T14:20:37Z","body":"Di tutti i commenti questo e' l'unico \"serio\".\r\n\r\nQuesto metodo alza un po' di bandierine gialle e rosse. Ha una complessita' ciclomatica non banale e in caso di errore il writer non e' chiuso, quindi consuma risorse. Ora, in tutta onesta' mi dovrei andare a riguardare se il compilatore in questi casi, dato che writer ha solo visibilita' locale, riesce automaticamente ad ottimizzare e chiudere la risorsa, ma essere espliciti in questi casi e' cosa molto gradita oltre che probabilmente necessaria. Guardati il try-with-resources (java 7) o alla peggio metti un finally dove chiudi il writer.\r\n\r\nCerca anche di riscrivere questa catena di if e for con le lambda di java 8. Dovresti riuscire a ridurre la complessita' a 1.\r\n\r\nL'uso di system.out e' un po meh, un logger sarebbe meglio, ma e' un test e forse ci passerei sopra.\r\n\r\nAltra cosa, assolutamente scrivimi un test JUnit! Non usare un main, secondo me in sede di interview viene visto male."}}