Discussion :

[JackDannyL]

Salut
J'ai encore un probleme avec if(==)

voici ma portion de code... j'ai pourtant suivi tout ce que Hizin ma apris.
Jai 2 EditText, editPassword1 et editPassword2... je veux le comparer pour que les 2 password soit pareille. J'ai meme enlever le inputType="textPassword" en XML.

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
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
 
editPassword1 = (EditText)findViewById(id.editPassword);
editPassword1.addTextChangedListener(new TextWatcher(){
 
    @Override
    public void afterTextChanged(Editable s) {
        checkPass1 = String.valueOf(editPassword1.getText());
        checkPass2 = String.valueOf(editPassword2.getText());
 
        if (checkPass1 == checkPass2) {
            textPasswordInfo.setText(R.string.textPasswordInfo_Ok);
	    passwordOk = true;
        }
 
        else {
            textPasswordInfo.setText(R.string.textPasswordInfo);
	    passwordOk = false;
	    Toast.makeText(getBaseContext(), "pas pareille " + checkPass1 + " " + checkPass2, Toast.LENGTH_SHORT).show();
        }
 
        if (playerNameOk && passwordOk && emailOk){
            buttonNext.setEnabled(true);
        }
 
    }
 
    @Override
    public void beforeTextChanged(CharSequence s, int start, int count, int after){
    }
 
    @Override
    public void onTextChanged(CharSequence s, int start, int before, int count) {
    }
});
 
editPassword2 = (EditText)findViewById(id.editPassword);
editPassword2.addTextChangedListener(new TextWatcher(){
 
    @Override
    public void afterTextChanged(Editable s) {
        checkPass1 = String.valueOf(editPassword1.getText());
        checkPass2 = String.valueOf(editPassword2.getText());
 
        if (checkPass1 == checkPass2) {
            textPasswordInfo.setText(R.string.textPasswordInfo_Ok);
	    passwordOk = true;
        }
 
        else {
            textPasswordInfo.setText(R.string.textPasswordInfo);
	    passwordOk = false;
	    Toast.makeText(getBaseContext(), "pas pareille " + checkPass1 + " " + checkPass2, Toast.LENGTH_SHORT).show();
        }
 
        if (playerNameOk && passwordOk && emailOk){
            buttonNext.setEnabled(true);
        }
 
    }
 
    @Override
    public void beforeTextChanged(CharSequence s, int start, int count, int after){
    }
 
    @Override
    public void onTextChanged(CharSequence s, int start, int before, int count) {
    }
});

Je rentre pourtant a123 dans les deux EditText.
Pourtant le Toast qui est dans la boucle else de mon if(checkPass1 == checkPass2) me donne comme retour : Pas pareille a123 a123???!!!!

Pourquoi, qu'est-ce que j'ai pas compris encore, si vous avez besoin de plus de code dites moi le mais je crois que le nécessaire est la.

J'ai meme essayer dans un nouveau projet

Code :

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

1
2
3
4
5
6
7
String test1 = "test";
String test2 = "test";
 
if (test1 == test2){
    Toast.makeText(getBaseContext(), "pareille", Toast.LENGTH_SHORT).show();}		
else {
    Toast.makeText(getBaseContext(), "pas pareille", Toast.LENGTH_SHORT).show();}

et ca marche, le Toast donne pareille

Merci encore

[joel.drigo]

On ne compare pas les String par == (sauf quand on maîtrise vraiment ce qu'on fait, et encore). Des String, ça se compare par equals() :

Code :

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

if (editPassword1.getText().equals(editPassword2.getText()) ){

Parce que == compare les références contenues dans les variables, et que 2 String différentes ont 2 références différentes.

Dans ton 2ième test, ça fonctionne parce que tu initialises les String avec des constantes. Il y a un "système" dans java qui fait que 2 chaines de même valeur ne sont stockée qu'une fois en mémoire, pour optimiser l'occupation. Quand tu utilises des constantes, tu es sûr que tes variables utilises ces valeurs "partagées". Dans le cas contraire, il est plus difficile de le savoir, surtout quand le code qui génère ces String n'est pas de toi, mais d'une lib.
Si en plus tu fais un String s2 = new String( s1 ) , tu peux être sûr que ce ne sera plus la même instance, même si c'est la même valeur, et les 2 String s1 et s2 seront de référence différentes.

En résumé, comme tu ne peux jamais être sûr que 2 String de même valeur sont la même instance de classe String, pour tester si elles sont égales, utilises toujours equals().

[JackDannyL]

merci j'essaie à l'instant

[JackDannyL]

merci joel.. donc si je comprend bien chaque fois que je veux comparer 2 string je devrais utiliser cette methode...

En tout cas Résolu por moi

[joel.drigo]

merci joel.. donc si je comprend bien chaque fois que je veux comparer 2 string je devrais utiliser cette methode...

Oui.

[thelvin]

Des String et n'importe quel objet, en fait. String, Integer, Long, Date, List, même combat. equals() et pas == sauf exception dont on comprend parfaitement la raison.