Discussion :

[javacolas]

Bonjour à tous, je suis en train de développer une application de CRUD avec JPA hybernate. J'ai une erreur sur une de mes requête en particulier.
1ere question comment peut on tester les requêtes jpql ?
2eme question, voila la requête qui pose probleme. Si jamais quelqu'un voit l'erreur je suis preneur.

Code :

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

1
2
3
4
5
6
  public PointDePassage getByPosition(Long idPosition) throws Exception {
        Query query = em.createQuery("SELECT pdp FROM PointDePassage pdp WHERE pdp.position_id LIKE :"+idPosition);
        //query.setParameter("position_id", "%" + idPosition + "%");
        PointDePassage pdp = (PointDePassage) query.getSingleResult();
        return pdp;
    }

Pour préciser, l'entity PointDePassage possede un objet Position. La table PointDePassage possède un champ position_id pour relier les 2 entitys.
J'utilise netbeans 6.5.

[chepac]

Tu as demandé a Wawa? ^^

[Paci88]

Les ":" après ton like

Si tu cherches des exemples sur le net :

Code :

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

1
2
3
4
5
 
String ejbql = "SELECT i from Item i WHERE i.name LIKE :pattern ESCAPE
:esc";
Query query = em.createQuery(ejbql);
query.setParameter("pattern", "\\_%");

Les ":" signifient que c'est un paramètre.

Du coup, ton code se transforme en :

Code :

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

1
2
3
4
5
6
7
 
public PointDePassage getByPosition(Long idPosition) throws Exception {
Query query = em.createQuery("SELECT pdp FROM PointDePassage pdp WHERE pdp.position_id LIKE :idPosition");
query.setParameter("idPosition", "%" + idPosition + "%");
PointDePassage pdp = (PointDePassage) query.getSingleResult();
return pdp;
}

EDIT : Ceci dit, un Like sur un nombre...

[javacolas]

Voila l'erreur générée par le code que tu m'a donné.

java.lang.IllegalArgumentException: org.hibernate.QueryException: could not resolve property: position_id of: entreprise.PointDePassage [SELECT pdp FROM entreprise.PointDePassage pdp WHERE pdp.position_id LIKE :idPosition]

[javacolas]

Et voila l'erreur généré par mon code sans ta modification.

java.lang.IllegalArgumentException: org.hibernate.hql.ast.QuerySyntaxException: expecting IDENT, found '1' near line 1, column 75 [SELECT pdp FROM entreprise.PointDePassage pdp WHERE pdp.position_id LIKE :1]

[Paci88]

Oups en effet...

Dans la ligne "SELECT pdp FROM PointDePassage pdp WHERE pdp.position_id"

Le nom du champ position_id doit être celui du champ coté Java et non côté SQL.
Je supposerai donc que ça doit être positionId.

Soit :

Code :

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

1
2
3
4
5
6
7
 
public PointDePassage getByPosition(Long idPosition) throws Exception {
Query query = em.createQuery("SELECT pdp FROM PointDePassage pdp WHERE pdp.positionId LIKE :idPosition");
query.setParameter("idPosition", "%" + idPosition + "%");
PointDePassage pdp = (PointDePassage) query.getSingleResult();
return pdp;
}

[javacolas]

comment faire en sorte que idPosition soit pris en compte comme une variable dans la requete?
En sql il faut mettre +nomVarible; par exemple.

[Paci88]

Je ne comprends pas ta question.

[javacolas]

pour insérer une variable dans une requete Sql on fait :
"requettteeeeee"+variable+"requeettte";

comment faut il faire en jpql.
Je sais pas si c'est trés claire je vois pas comment le dire autrement.

[Paci88]

Ok, je vois ce que tu veux dire!

C'est justement ce que je t'explique plus haut!!

Alors reprenons. Tu as ta requête :

Code :

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

1
2
 
Query query = em.createQuery("SELECT pdp FROM PointDePassage pdp WHERE pdp.positionId LIKE :idPosition");

Le fait de mettre les ":" avant idPosition signifie justement que c'est un paramètre qui va être passé!

Ensuite, tu vas justement setter ton paramètre dans la requête, grâce à la ligne suivante :

Code :

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

1
2
 
query.setParameter("idPosition", "%" + idPosition + "%");

En gros, elle dit : "Là où tu trouveras le paramètre idPosition, tu mettras la valeurs "%" + idPosition + "%" "

Est-ce plus clair?

[javacolas]

Ok, c'est maintenant claire. Merci.
Par contre l'erreur persiste
java.lang.IllegalArgumentException: org.hibernate.QueryException: could not resolve property: positionId of: entreprise.PointDePassage [SELECT pdp FROM entreprise.PointDePassage
Je vais continuer à chercher...

[Paci88]

Ok, après tout dépend de comment tu as fait ton mapping.
Pourrais-tu mettre ici ta classe PointDePassage?

[javacolas]

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
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
/*
 * To change this template, choose Tools | Templates
 * and open the template in the editor.
 */
package entreprise;
 
import java.io.Serializable;
import javax.persistence.DiscriminatorColumn;
import javax.persistence.DiscriminatorType;
import javax.persistence.DiscriminatorValue;
import javax.persistence.Entity;
import javax.persistence.GeneratedValue;
import javax.persistence.GenerationType;
import javax.persistence.Id;
import javax.persistence.OneToOne;
 
/**
 *
 * @author nicolas
 */
@Entity
@DiscriminatorColumn(name = "TYPE", discriminatorType = DiscriminatorType.STRING, length = 1)
@DiscriminatorValue("P")
public class PointDePassage implements Serializable {
 
    private static final long serialVersionUID = 1L;
    @Id
    @GeneratedValue(strategy = GenerationType.AUTO)
    private Long id;
    private String nomRue;
    private int numeroRue;
    private String nom;
    @OneToOne
    protected  Position position;
 
    public PointDePassage() {
    }
 
    public PointDePassage(String nomRue, int numeroRue, Position position, String nom) {
        this.nomRue = nomRue;
        this.numeroRue = numeroRue;        
        this.position = position;
        this.nom = nom;
    }
 
    public Long getId() {
        return id;
    }
 
    public void setId(Long id) {
        this.id = id;
    }
 
    @Override
    public int hashCode() {
        int hash = 0;
        hash += (id != null ? id.hashCode() : 0);
        return hash;
    }
 
    @Override
    public boolean equals(Object object) {
        // TODO: Warning - this method won't work in the case the id fields are not set
        if (!(object instanceof PointDePassage)) {
            return false;
        }
        PointDePassage other = (PointDePassage) object;
        if ((this.id == null && other.id != null) || (this.id != null && !this.id.equals(other.id))) {
            return false;
        }
        return true;
    }
 
    @Override
    public String toString() {
        return "rue " + this.getNomRue() + " numero " + this.getNumeroRue();
    }
 
    /**
     * @return the nomRue
     */
    public String getNomRue() {
        return nomRue;
    }
 
    /**
     * @param nomRue the nomRue to set
     */
    public void setNomRue(String nomRue) {
        this.nomRue = nomRue;
    }
 
    /**
     * @return the numeroRue
     */
    public int getNumeroRue() {
        return numeroRue;
    }
 
    /**
     * @param numeroRue the numeroRue to set
     */
    public void setNumeroRue(int numeroRue) {
        this.numeroRue = numeroRue;
    }
 
    /**
     * @return the position
     */
    public Position getPosition() {
        return position;
    }
 
    /**
     * @param position the position to set
     */
    public void setPosition(Position position) {
        this.position = position;
    }
 
    /**
     * @return the nom
     */
    public String getNom() {
        return nom;
    }
 
    /**
     * @param nom the nom to set
     */
    public void setNom(String nom) {
        this.nom = nom;
    }
}

[Paci88]

Et la classe Position aussi

[javacolas]

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
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
/*
 * To change this template, choose Tools | Templates
 * and open the template in the editor.
 */
package entreprise;
 
import java.io.Serializable;
import javax.persistence.Entity;
import javax.persistence.GeneratedValue;
import javax.persistence.GenerationType;
import javax.persistence.Id;
 
/**
 *
 * @author nicolas
 */
@Entity
public class Position implements Serializable {
 
    private static final long serialVersionUID = 1L;
    @Id
    @GeneratedValue(strategy = GenerationType.AUTO)
    private Long id;
    private float latitude;
    private float longitude;
    private char ptCardiLati;
    private char ptCardiLongi;
 
    public Position(){
 
}
public Position(float latitude, float longitude, char ptCardiLati, char ptCardiLongi){
   // this.id=id;
    this.latitude=latitude;
    this.longitude=longitude;
    this.ptCardiLati=ptCardiLati;
    this.ptCardiLongi=ptCardiLongi;
}
 
    public Long getId() {
        return id;
    }
 
    public void setId(Long id) {
        this.id = id;
    }
 
    @Override
    public int hashCode() {
        int hash = 0;
        hash += (id != null ? id.hashCode() : 0);
        return hash;
    }
 
    @Override
    public boolean equals(Object object) {
        // TODO: Warning - this method won't work in the case the id fields are not set
        if (!(object instanceof Position)) {
            return false;
        }
        Position other = (Position) object;
        if ((this.id == null && other.id != null) || (this.id != null && !this.id.equals(other.id))) {
            return false;
        }
        return true;
    }
 
    @Override
    public String toString() {
        return String.valueOf(latitude) + ptCardiLati + String.valueOf(longitude) + getPtCardiLongi();
    }
 
    /**
     * @return the latitude
     */
    public float getLatitude() {
        return latitude;
    }
 
    /**
     * @param latitude the latitude to set
     */
    public void setLatitude(float latitude) {
        this.latitude = latitude;
    }
 
    /**
     * @return the longitude
     */
    public float getLongitude() {
        return longitude;
    }
 
    /**
     * @param longitude the longitude to set
     */
    public void setLongitude(float longitude) {
        this.longitude = longitude;
    }
 
    /**
     * @return the ptCardiLati
     */
    public char getPtCardiLati() {
        return ptCardiLati;
    }
 
    /**
     * @param ptCardiLati the ptCardiLati to set
     */
    public void setPtCardiLati(char ptCardiLati) {
        this.ptCardiLati = ptCardiLati;
    }
 
    /**
     * @return the ptCardiLongi
     */
    public char getPtCardiLongi() {
        return ptCardiLongi;
    }
 
    /**
     * @param ptCardiLongi the ptCardiLongi to set
     */
    public void setPtCardiLongi(char ptCardiLongi) {
        this.ptCardiLongi = ptCardiLongi;
    }
}

[Paci88]

Alors, là je pense que ça devrait être bon...

Code :

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

1
2
 
Query query = em.createQuery("SELECT pdp FROM PointDePassage pdp WHERE pdp.position.id LIKE :idPosition");

[javacolas]

Ah il y a du changement. L'erreur est maintenant sur la ligne

Code :

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

   PointDePassage pdp = (PointDePassage) query.getSingleResult();

elle n'est plus sur la ligne de la requête.

Voila l'erreur générée:
java.lang.IllegalArgumentException: org.hibernate.QueryException: Not all named parameters have been set: [idPosition] [SELECT pdp FROM PointDePassage pdp WHERE pdp.position.id LIKE :idPosition]

[Paci88]

Tu fais bien le :

Code :

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

1
2
 
query.setParameter("idPosition", "%" + idPosition + "%");

Sinon, y a un truc qui me turlupine, pourquoi fais-tu un "like" sur un id et non un "=" ??

[javacolas]

Un nouveau problème viens d'apparaître. Mon AddStation() ne fonctionne pas alors que le addPointDePassage() fonctionne.
Station hérite de pointDePassage.
Voici la classe Station :

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
/*
 * To change this template, choose Tools | Templates
 * and open the template in the editor.
 */
package entreprise;
 
import java.io.Serializable;
import javax.persistence.DiscriminatorValue;
import javax.persistence.Entity;
 
/**
 *
 * @author nicolas
 */
@Entity
@DiscriminatorValue("S")
public class Station extends PointDePassage implements Serializable {
 
 
    public Station() {
    }
 
    public Station(String nomRue, int numeroRue, Position position, String nom) {
        super(nomRue, numeroRue, position, nom);
    }
}

[javacolas]

je met un = c'est vrai que le like n'a pas d'intérêt sur un id.