Discussion :

[IronBibs]

Bonjour à tou(te)s,

Je suis curieux depuis quelques temps sur cette façon de faire, c'est assez basique mais j'aimerais votre avis tant sur la faisabilité que si c'est une "best practice" de faire l'une plutôt que l'autre.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
//showTransmisDuCheckBox : boolean
//selectTransmisDu : swt.Button
 
//Variante 1
if (showTransmisDuCheckBox && selectTransmisDu != null && selectTransmisDu.getSelection()) {
	//blabla
}
 
//Variante 2
if (showTransmisDuCheckBox && selectTransmisDu && selectTransmisDu.getSelection()) {
	//blabla
}

Merci pour vos réponses.

EDIT: Une fois l'application lancée, avec la solution 2, le compilateur hurle alors que l'IDE ne disait rien. Ma question tombe à l'eau dommage. à locker.

[joel.drigo]

Salut,

Tu es sûr que ton IDE ne signale rien (Quel IDE d'ailleurs ?). Même si j'écris dans Eclipse :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
Boolean b=null;
 
if ( b ) {
 
}

Je peux compiler, mais j'ai un warning qui me dit justement que je vais obtenir un NullPointerException lors du unboxing (on pourrait s'attendre à ce que ça ne compile pas, mais c'est un autre débat).
En revanche, si selectTransmisDu n'est pas boolean ou Boolean, et ce n'est visiblement pas le cas (Boolean n'a pas de méthode getSelection()), je vois mal comment ton IDE ne te signalerait pas que ça ne peut pas compiler.