Il codice finale rispetto a quello iniziale è più comprensibile. Senza alcun rumore sintattico balza subito all’occhio qual è l’aspetto fondamentale dell’implementazione: “il titolo è valido se non è nullo e la sua lunghezza è maggiore di zero.”.

L’intelligenza visiva è infatti la più istintiva (è la prima che si sviluppa nei bambini), al contrario dell’intelligenza logica, che è quella che si sviluppa più tardi di tutte e che richiede il maggiore esercizio ed allenamento.
Avere un if-then-else crea (nell’esempio di sopra) un disturbo visivo da ciò che invece è fondamentale, annegando il cuore del concetto dentro dettagli ed orpelli sintattici che velano il reale significato dell’implementazione, anche solo visualmente. E’ un po’ quanto avviene con un solitario: per esaltare le caratteristiche del diamante, lo si fa svettare, da solo, su una striscia d’oro.

Da qui si può prendere spunto per parlare, su altra scala, del paradigma ad oggetti, che ha tra i suoi punti di forza proprio lo spostare la complessità logico/procedurale verso una complessità strutturale che permette una rappresentazione per diagrammi più maneggevole e su cui si ragiona con molta più facilità.

if (A) {
return true;
} else {
return false;
}

rispetto a

return A;

???

Sono esterrefatto. Ammetto che io ho passato molto tempo a ragionare su espressioni booleane. Ma non accetterei codice come quello. Mi dà semplicemente l’idea che l’autore non sa ragionare sulle espressioni booleane. Lo rifattorizzo senza neanche battere ciglio.

In generale, bisogna cercare di togliere i cicli e le decisioni dal codice. L’ideale è codice privo di if e while, una semplice sequenza di operazioni (semplici). Usare un if quando è sufficiente una semplice espressione è per me impensabile.

public boolean isTitleValid() {
boolean result = true;
if((title==null) || “”.equals(title)) {
result = false;
}
return result;
}