Discussion :

[kcirtap.tim]

Salut!

j ai un petit souci dans mon projet de réservation de chambre d’hôtel.
j essaye de récupérer les champs date d une requete pour les mettre dans une méthode qui doit me trouver les chambre
dispos a cette date
voici ma méthode

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
public Collection<Chambre> getChambreLibre(Date dateDebut, Date dateFin) {
 
        List<Chambre> lc = new ArrayList<>();
        String req = "select c from Chambre c  join c.reservation on chambre.numChambre=reservation.chambreNumChambre"
                + "and '" + dateDebut + "' BETWEEN reservation.dateDebut AND reservation.dateFin"
                + "OR '" + dateFin + "'BETWEEN reservation.dateDebut AND reservation.dateFin"
                + "OR reservation.dateDebut BETWEEN '" + dateDebut + "' AND '" + dateFin + "'"
                + "OR reservation.dateFin BEETWEEN '" + dateDebut + "'AND '" + dateFin + "'"
                + "WHERE reservation.numChambre IS NULL ";
        Query qr = em.createNamedQuery(req);
        lc = qr.getResultList();
        return lc;
 
seulement dans mon controleur je suis embété quand j essaye de recuperer ces dates dans a travers la methode du bean
 if (session.getAttribute("ch") == null) {
            session.setAttribute("ch", beanReservation.getChambreLibre(null,null));

mon probleme c 'est comment recuperer les valeurs des dates de la requete pour les mettres a la place des champs null, null

J'ai bien essayé beanReservation.getChambreLibre(request.getParameter"dateDebut"..."dateFin")
le probleme c est que ma methode attend des dates pas un string!!!!
y a t il un petit genie dans les parrages?

[freddou17]

Slt,

Code :

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

1
2
3
4
 
SimpleDateFormat dateFormat = new SimpleDateFormat("dd/MM/yyyy");
String dateToString= "dd/mm/aaaa";
Date date = formatter.parse(dateToString);

bon courage
++

[OButterlin]

Ce n'est pas une requête JPA conforme, tu devrais avoir

Code :

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

1
2
String req = "select c from Chambre c  join c.reservation r WHERE r.numChambre IS NULL ";
Query qr = em.createQuery(req);

Ensuite, pour les tests, ce serait quelque chose comme

Code :

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

1
2
3
4
5
6
7
8
9
 
...
req += " and r.dateDebut <= :dateDebut and r.dateFin >= :dateFin";
Query qr = em.createQuery(req);
qr.setParameter("dateDebut", leParametreDateDebut); <--- Objet Date
qr.setParameter("dateFin", leParametreDateFin);
 
List<Chambre> list = qr.getResultList();
...

Une bonne habitude à prendre : ne pas injecter les valeurs passées en paramètre dans une requête SQL, tu risques l'injection de dépendance avec des paramètres String...
Sans JPA, utiliser un PreparedStatement plutôt qu'un Statement

[kcirtap.tim]

salut OButterlin
merci de a réponse mais j ai vraiment du mal avec le jpql et ta requete en 2 parties ne m'éclaire pas plus
en fait le but de cette requete serait de recuperer les chambres libres a une date donnée donc , il faudrait que la requete prenne en compte les intervalles de réservation
ex: si une chambre est réserveé du 25 au 29 qu un client qui veut reserver la meme chambre du 24 au 30 puisse le faire
car les dates de debut et de fin sont differents sauf que dans l intervalle la chambre est réservée!
Bon je pense que t'as compris ce que je voulais faire mais simplement ta requete me laisse perplexe!
peut tu mieux me l expliquer et pourquoi est elle en 2 parties?

[tchize_]

salut OButterlin
merci de a réponse mais j ai vraiment du mal avec le jpql et ta requete en 2 parties ne m'éclaire pas plus

En gros, jamais de "+ uneVariable" dans une requête SQL ou JPQL, toujours utiliser des paramètres. J'en profite pour corriger tes espaces manquant entre tes +, par utiliser la bonne méthode (createQuery pas createNamedQuery) et par écrire correctement le join (pas de join...on en jpql, le on est calculé à partir de la structure de l'entity bean)

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
public Collection<Chambre> getChambreLibre(Date dateDebut, Date dateFin) {
 
        String req = "select c from Chambre c  join c.reservation reservation "
                + "where (:debut BETWEEN reservation.dateDebut AND reservation.dateFin "
                       + "OR :fin 'BETWEEN reservation.dateDebut AND reservation.dateFin "
                       + "OR reservation.dateDebut BETWEEN :debut and :fin "
                       + "OR reservation.dateFin BEETWEEN :debut and :fin) "
                + "AND reservation.numChambre IS NULL ";
        Query qr = em.createQuery(req);
        return em.createQuery(req)
                .setParameter("debut",dateDebut)
                .setParameter("fin",dateFin)
                .getResultList();

A noter aussi que ton test est supercompliqué pour faire un truc simple qui me semble donner le même résultat, a supposer que ta table est saine (debut<=fin) et que dateDebut <= dateFin

Code :

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

1
2
3
4
5
6
7
8
9
10
11
public Collection<Chambre> getChambreLibre(Date dateDebut, Date dateFin) {
 
        String req = "select c from Chambre c  join c.reservation reservation "
                + "WHERE :debut <= reservation.dateFin "
                + "AND :fin >= reservation.dateDebut "
                + "AND reservation.numChambre IS NULL ";
        Query qr = em.createQuery(req);
        return em.createQuery(req)
                .setParameter("debut",dateDebut)
                .setParameter("fin",dateFin)
                .getResultList();

C'est pas plus lisible?

[kcirtap.tim]

J'ai bien testé ton code. Le problème est que ca n'a pas fonctionné. J'ai eu plein d'erreurs :

JPQL: The state field path cannot be resolved to a valid type!

[kcirtap.tim]

salut !
j ai tésté tes codes jpql les 2 me renvoient des exceptions du style

61, 281] The expression is not a valid conditional expression.
[281, 318] The query contains a malformed ending.]]
j 'ai bien essayé de régler ca mais aucun resultat

[kcirtap.tim]

salut freddou17

je comprends bien qu'il faut formater la date dans la classe...sauf que pour recuperer le champs du request.getParameter()
ne vaut il pas mieux d'abord que je declare ma methode des strings en argument de ma methode?
quitte a formater ce string en date dans ma classe par la suite?

[freddou17]

Pourquoi ne pas faire le parse dans ton controler? tu peux aussi le faire dans ta classe... à toi de voir


Pour la requête OButterlin t'as décrit la bonne méthode avec le bind de paramètres

Code :

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

1
2
 
:dateFin

:dateFin = normalisation du paramètre (il pourrait s'appeler :toto) qui va se voir attribuer une valeur par la mth

Code :

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

1
2
3
4
 
qr.setParameter("dateDebut", leParametreDateDebut);
//ou si on avait mis :toto
qr.setParameter("toto", leParametreDateDebut);

si tu ne vois toujours pas recherche sur le net le binding de parametres...

bon courage
++

[kcirtap.tim]

Salut,

J'ai bien formaté la date dans ma méthode mais j'ai une exception

Grave: java.text.ParseException: Unparseable date: "dd/mm/aaaa"

Voici mon code :

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
 
@Override
    public Collection<Chambre> getChambreLibre(String dateDebut, String dateFin) {
        try {
 
            List<Chambre> lc = new ArrayList();
            //String req = "select c from Chambre c join c.reservation reservation where: debut<=reservation.dateFin and :fin >=reservation.dateFin AND reservation.numchambre is null";
            String req="select c from Chambre c join c.reservation r where c.numChambre is null and r.dateDebut<=:dateDebut and r.dateFin>=:dateFin";
            Query qr = em.createQuery(req);
            SimpleDateFormat dateFormat = new SimpleDateFormat("dd/MM/yyyy",Locale.FRANCE);
            dateDebut = "dd/mm/aaaa";
            dateFin = "dd/mm/aaaa";
            Date debut = formatter.parse(dateDebut);
            Date fin = formatter.parse(dateFin);
 
            qr.setParameter("dateDebut", debut);
            qr.setParameter("dateFin", fin);
            lc = qr.getResultList();
            return lc;
        } catch (NoSuchEJBException ff) {
 
            return null;
 
        } catch (ParseException ex) {
            Logger.getLogger(BeanReservation.class.getName()).log(Level.SEVERE, null, ex);
        }
        return null;
 
    }
}

merci