Discussion :

[jean-simon]

Bonjour à tous,
je n'arrive pas à combiner la lecture de la saisie clavier avec le changement de valeur d'un booleen:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
import java.util.Scanner;
 
public class Programme {
 
			public static void main(String[] args) {
 
				boolean choux=false;
 
				Scanner sc=new Scanner(System.in);
 
				System.out.println("Savez-vous planter les choux ? (lettre o minuscule pour oui)");
				String str=sc.nextLine();
				if(str=="o"){
						choux=true;
					}
 
				System.out.println(choux);
 
	}
 
}

me donne en sortie console:

Savez-vous planter les choux ? (lettre o minuscule pour oui)
o
false

ce doit être normal... Mais pourquoi ? et comment faire pour inverser la valeur ?
merci de vos éclairages...

[thelvin]

Hello,

le problème n'est pas le booléen, le problème est ce que tu as mis dans ton if.

Les String sont des objets et en Java on ne compare pas les objets avec ==
Il faut utiliser equals()

Code :

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

str.equals("o")

[jean-simon]

Merci !

[yildiz-online]

Hello,

le problème n'est pas le booléen, le problème est ce que tu as mis dans ton if.

Les String sont des objets et en Java on ne compare pas les objets avec ==
Il faut utiliser equals()

Code :

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

str.equals("o")

Il est préférable d'inverser le test d'égalité pour gérer correctement les cas null:

"o".equals(str);

[thelvin]

Je te souhaite bien du courage pour obtenir un null en appelant Scanner.nextLine().

Mais si tu es de l'école qui préfère ne pas réfléchir pour savoir si une situation de null est possible ou pas et considérer que tout est toujours possible, dans ce cas la solution correcte est :

Code :

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

Objects.equals(str, "o")

plutôt qu'un vieux hack qu'on utilisait pour gagner du temps à l'époque où la classe utilitaire Objects n'existait pas.

[yildiz-online]

Quitte à donner une réponse, autant qu'elle soit valide dans tous les cas, tu ne crois pas? Tu remarqueras donc que j'ai répondu à "str.equals("o")" et non à "sc.nextline..."

Il n'est pas question de réfléchir ou non, mais d'avoir une cohérence dans les pratiques en minimisants les possibilités d'erreurs humaine et en réduisant le temps de développement.

Et moins on réfléchi sur les cas généralisables, plus on peut se concentrer sur les problèmes qui méritent qu'on s'y attardent.

"".equals(string) n'a rien d'un hack, c'est la seule solution valide pré java 7 (et ici la version de java n'est pas spécifiée il me semble). Et n'est en rien moins efficace en terme d'utilisation, ni de lisibilité, ni de respect de standards.

Merci donc d'éviter les attaques ad hominem sur les capacités à réfléchir et sur les affirmation gratuite (LA solution valide) sans apporter d'argumentation valide.