Codice paranoico!

if((a != null && a.length > 5) && (a != null && a.indexOf(“hello”) != -1)){
…..

Non si sa mai che a diventi nullo tra un AND e l’altro!!

Comments

… o che improvvisamente la congiunzione logica cessi di essere associativa… (in altre parole si potevano togliere 2 coppie di parentesi)
Posted by matteov on 03/05/2006 11:16:45 AM

Si, ma se dovessi mettermi a considerare anche queste cose penso che riempirei il blog… Siamo ottimisti e supponiamo che le parentesi siano state inserite per raggruppare concettualmente i controlli (si prega di non ridere).
Posted by ratta on 03/05/2006 11:29:40 AM

Ah, e si prega anche di non suggerire un extract method per raggrupare i controlli.. quel codice era, come è evidente, anni luce da queste cose :) Meglio! Se no non avrei nulla da scrivere qui!
Posted by ratta on 03/05/2006 11:34:35 AM

Advertisements

Leave a Reply

Fill in your details below or click an icon to log in:

WordPress.com Logo

You are commenting using your WordPress.com account. Log Out / Change )

Twitter picture

You are commenting using your Twitter account. Log Out / Change )

Facebook photo

You are commenting using your Facebook account. Log Out / Change )

Google+ photo

You are commenting using your Google+ account. Log Out / Change )

Connecting to %s