Discussion :

[robert_trudel]

salut

j'ai redéfinie equals pour un objets

longVal étant de type long
stringVal étant de type String

j'obtient une java.lang.nullPointerException à l'endroit spécifié

il y a une comparaison de 1 et 21 et ça crash

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
 
public boolean equals(Object o) {
        if (o == this) {
            return true;
        }
 
        if (o instanceof User) {
 
            User other = (User) o;
 
            if (this.longVal != other.longVal) { //crash ici
                return false;
            }
 
            if (this.stringVal != null) {
                if (!this.stringVal.equals(other.stringVal)) {
                    return false;
                }
            }
 
            if (this.stringVal == null && other.stringVal!=null) {
                    return false;
            }
 
            return true;
        }
        return false;
    }

une idée?

[adiGuba]

Salut,

Tu es sûr du code ? Parce que théoriquement c'est impossible d'avoir une NPE sur cette ligne :

Code :

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

if (this.longVal != other.longVal) { //crash ici

Seul other pourrait être null, or dans ce cas là le instanceof renverrait false...


a++

[divxdede]

La seule autre explication serait
other.longVal soit un Long
this.longVal soit un long

Le NullPointerException frapperait lors de l'unboxing.
sauf que other et this semble être de même type, donc c'est à priori impossible que la déclaration de longVal soit differente.

[robert_trudel]

une image


et le null me renvoie bien à la ligne spécifié...

[robert_trudel]

dans le code j'ai bien

long longVal;

[divxdede]

Ton débugger à réussi à évaluer other.longVal donc il n'y a pas eu de nullpointer de ce coté là. this.longVal ne peut pas lever cette exception non plus...
Le débugger ne serait pas arretté a cette ligne uniquement parceque tu as mis un break-point (un point d'arret) ?

Tu peu nous montrer la "stack trace" de l'exception ?

[robert_trudel]

Ton débugger à réussi à évaluer other.longVal donc il n'y a pas eu de nullpointer de ce coté là. this.longVal ne peut pas lever cette exception non plus...
Le débugger ne serait pas arretté a cette ligne uniquement parceque tu as mis un break-point (un point d'arret) ?

Tu peu nous montrer la "stack trace" de l'exception ?

la valeur est inséré de cette façon

Code :

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

1
2
 
oldRoot = (Long) resultMap.get("idConditionRoot");

la méthode est appelé de cette façon

Code :

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

1
2
 
if (!idRoot.equals(oldRoot))

comment ont fait pour obtenir cette stack trace?

[adiGuba]

la valeur est inséré de cette façon

Code :

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

1
2
 
oldRoot = (Long) resultMap.get("idConditionRoot");

Donc ce n'est pas un objet User et on ne devrait pas rentrer dans le if du instanceof

comment ont fait pour obtenir cette stack trace?

Le stacktrace est affiché sous le message de l'exception.

a++

[DarkMolo]

Salut,

D'aprés ce que tu dis, on peut affirmer que other est nul !

Si tu pouvais nous montrer comment t'as appellé la méthode et avec quels paramétres.

[CheryBen]

D'aprés ce que tu dis, on peut affirmer que other est nul !

Non puisque o instanceof User est vrai, c'est un cas cornellien