Scrittura elegante codice

[franksisca]

c'è qualcosa di più elegante di questo???

			calendarId = request.getParameter(SoggettiConstant.SUBJECT_CALENDAR_ID) == null ? "all" : request.getParameter(SoggettiConstant.SUBJECT_CALENDAR_ID);

[Alex'87]

Originariamente inviato da franksisca
c'è qualcosa di più elegante di questo???

			calendarId = request.getParameter(SoggettiConstant.SUBJECT_CALENDAR_ID) == null ? "all" : request.getParameter(SoggettiConstant.SUBJECT_CALENDAR_ID);

Beh, potresti salvarti il risultato di getParameter, evitando di scrivere due volte quella sbrodolata

String param = request.getParameter(SoggettiConstant.SUBJECT_CALENDAR_ID);
calendarId = param  == null ? "all" : param;

[andbin]

Originariamente inviato da franksisca
c'è qualcosa di più elegante di questo???

			calendarId = request.getParameter(SoggettiConstant.SUBJECT_CALENDAR_ID) == null ? "all" : request.getParameter(SoggettiConstant.SUBJECT_CALENDAR_ID);

Se questo è l'unico punto in cui fai una cosa del genere, non mi sembra una cosa così grave o brutta .... al massimo non farlo proprio su 1 riga unica (che è lungo) ma spezzalo, ad esempio andando a capo prima del '?'.

Se invece una logica del genere la fai più volte, con lo stesso parametro o con diversi, allora potresti pensare ad una piccola classettina di "utilità" del tipo (abbozzo solo):

public class RequestUtils {
    private RequestUtils() {}

    public static String getParameter(ServletRequest request, String name, String defaultValue) {
        // .....
    }
}

Verifica però se tra i framework/librerie che eventualmente usi non ci sia già una cosa del genere .... sono necessità abbastanza frequenti (gestire un default, convertire in primitivi, ecc...).

[franksisca]

grazie ad entrambi.

Allora leggendo e discutendo su altri forum sono arrivato a trovare la "soluzione", ovvero l'elvis operator, peccato che sia implementato solo da java 7.

al momento applico già la soluzione del metodo utilità (non una classe intera ma un metodo), ma cercavo qualcosa di meno prolisso proprio a livello di codice.

grazie comunque

[andbin]

Originariamente inviato da franksisca
sono arrivato a trovare la "soluzione", ovvero l'elvis operator, peccato che sia implementato solo da java 7.

Se non sbaglio, l'elvis operator era stato proposto per Java 7 ma poi è stato rifiutato.

[franksisca]

Originariamente inviato da andbin
Se non sbaglio, l'elvis operator era stato proposto per Java 7 ma poi è stato rifiutato.

giusto!

avevo letto a metà e mi ero fermato in quanto comunque non posso usare java7