Discussion :

[info3licen]

Salut

j'utilise ArrayListe et je rencontre un probleme :

Dans mon interface l'utilisateur saisie une date, et dans mon programme je controle cette date si elle existe dans ma table Mois dans la BDD alors je modifie uniquement les information d'une autre table si elle existe pas je l'insère dans ma table Mois.

Voila on bout de 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
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
 
 
...
                                      String dad = jTextField.getText();
                                      ArrayList<String> lis = new ArrayList();
						int ok=0;
 
						try{
 
							ResultSet res =stat.executeQuery("Select Mois From Mois");
							while(res.next()){
 
								lis.add(res.getString("Mois"));
 
							}
 
							for(int i=0; i<lis.size();i++)
							{
								if(lis.contains(dad))
									{   ok =1;
 
									}
								else{
									ok = -1;
 
								}
							}
 
						if(ok==1){
 
							String requete = "UPDATE Objectif SET Mois='"+dad+"', ObjTmvTrx='"+ObjTmvTrx+"', ObjTmc='"+ObjTmc+"', ObjTauxInc='"+ObjTauxInc+"', ObjFrm='"+ObjFrm+"', ObjTec='"+ObjTec+"', ObjdelaiAc='"+ObjdelaiAc+"', ObjdelaiCv='"+ObjdelaiCv+"', ObjdelaiPe='"+ObjdelaiPe+"' WHERE Nom_dd='"+dd+"' " ;  
							stat.executeUpdate(requete);
 
						}
						else if(ok ==-1){
 
							String insDat = "Insert into Mois (Mois) VALUES ('"+dad+"')";
							stat.executeUpdate(insDat);
							String req = "UPDATE Objectif SET Mois='"+dad+"', ObjTmvTrx='"+ObjTmvTrx+"', ObjTmc='"+ObjTmc+"', ObjTauxInc='"+ObjTauxInc+"', ObjFrm='"+ObjFrm+"', ObjTec='"+ObjTec+"', ObjdelaiAc='"+ObjdelaiAc+"', ObjdelaiCv='"+ObjdelaiCv+"', ObjdelaiPe='"+ObjdelaiPe+"' WHERE Nom_dd='"+dd+"' " ;  
							stat.executeUpdate(req);
 
						}
}catch(Exception ex){
 ex.printStackTrace();
}

Mon probleme c'est qu'il trouve toujours ok=-1 Même si je rentre une date qui existe déjà dans la table Mois!

Pourriez vous me dire c'est quoi ce probleme ?!

Estce que le probleme vient de :

Code :

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

1
2
 
if(lis.contains(dad))

?!

y a t'il une autre méthode dans les listes pour comparé une variable si elle est dans la liste ?!

Merci.

[danimo]

Salut,

Par exemple:

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
   import java.util.ArrayList;
 
   public class ArrayListDemo3
   {
      public static void main(String[] args)
      {
         ArrayList<Object> arrL = new ArrayList<Object>();
 
         ArrayList<String> t3=new ArrayList<String>();
 
 
 
         t3.add ("Bonjour");
         t3.add ("Salut");
         t3.add ("xyz");
         t3.add ("salut");
         t3.add ("Salut");
         System.out.println("\nContenu de t3: " + t3);
 
      // Obtenir l'indice d'un item de l'ArrayList arrL
 
         boolean trouve = false;
         Object obj;
 
         obj = "Salut";
         int i = 0;
         System.out.println("On recherche l'objet " + obj + ":");
         for(i=0;i<t3.size();i++)
         {
            if(t3.get(i).equals(obj))
            {
               System.out.println("item " + i + ": " + t3.get(i) + ".");
               trouve = true;
            }
         }
         System.out.println("");
         if(!trouve)
            System.out.println("L'objet " + obj + " n'existe pas...\n");
      }
   }

Cordialement,

Dan

[babycart]

Bonjour,

Tu peux aussi faire plus simplement :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
        List<String> mois = new ArrayList<String>();
        mois.add("Janvier");
        mois.add("Février");
        mois.add("Mars");
        //...
 
        String unMois = "Février";
 
        System.out.println("dans mois ? : " + mois.contains(unMois)); // renvoie true ou false
        System.out.println("dans mois ? : " + mois.indexOf(unMois)); // renvoie l'index ou -1 s'il n'est pas dans la liste

Voire dans ton cas, un HashSet suffit.
Pour ton erreur, es-tu sur que ta liste contienne bien le mois entré ? Fait un println de "dad" et "lis" pour vérifier.

[tchize_]

une raison particulière de faire un table "mois"?

Mis à part ça, quelle nécessité de
1) récupérer l'intégralité d'un table pour
2) vérifier si un entrée s'y trouve et
3) si c'est le cas l'utiliser dans un update ?

C'est un peu contraire à la plupart des bonne pratique. De même que la construction de votre requete plutot que d'utiliser un PreparedStatement.


Donc 1 créer un preparedStatement avec select * from Mois where Mois = ?
Si 1 resultat au moins -> Le mois existe (pas besoin de pomper toute la table)

et pour l'insertion, idem, un preparedStatement:

Code :

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

UPDATE Objectif SET Mois=? , ObjTmvTrx=?, ObjTmc=?, ObjTauxInc=?, ObjFrm=?, ObjTec=?, ObjdelaiAc=? , ObjdelaiCv=?, ObjdelaiPe=? WHERE Nom_dd=? " ;

C'est plus lisible et évite les problème d'injection sql

[info3licen]

Merci danimo pour votre réponse

Pour :

Code :

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

 if(t3.get(i).equals(obj))

Je l'avais essai au début et elle ma donnée le même résulta qu'avec Contains

Même si la date est dans le arrayliste mais le probleme passe directement a else !!!

Aidez moi svp.

[info3licen]

Dsl tchize je viens de voir votre réponse, Merci a vous aussi

une raison particulière de faire un table "mois"?

Mis à part ça, quelle nécessité de
1) récupérer l'intégralité d'un table pour
2) vérifier si un entrée s'y trouve et
3) si c'est le cas l'utiliser dans un update ?

la table moi est lié a une autre table entre ces deux table ce trouve une classe associative Objectif donc lors du passege aux modeles relationnelles j'aurais une table :
Objectif(Mois,dd,obj)

Et dans mon application l'utilisateur peux faire certaine modification sur les objectif, donc si il modifier la date et cette date elle ne serait pas dèja presente dans la table mois, la il y aura une erreur avec Sql, c'est pour cela je serais obligé de vérifier si la date existe déja si elle existe pas je l'insere. Voila

Donc 1 créer un preparedStatement avec select * from Mois where Mois = ?
Si 1 resultat au moins -> Le mois existe (pas besoin de pomper toute la table)

Je comprends pas bien le rôle des préparedStatement mais je vais éssai de voir, peut être je vais sortir avec


Merci

[info3licen]

Ouf Tant mieux j'ai résolue le probleme j'avais un probleme au niveau des accolades.

Merci encore une fois danimo, votre morceau de code ma bien aidé Thank's

Et Merci a vous tous pour toutes vos réponses

[Tora21]

Je comprends pas bien le rôle des préparedStatement mais je vais éssai de voir, peut être je vais sortir avec

La réponse de tchize_:

évite les problème d'injection sql

tous simplement une question de sécurité.

[tchize_]

tous simplement une question de sécurité.

et de lisibilité du code, et de facilité de programmation

[Tora21]

voilà tous est dit !!!

[danimo]

Merci danimo pour votre réponse

Pour :

Code :

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

 if(t3.get(i).equals(obj))

Je l'avais essai au début et elle ma donnée le même résulta qu'avec Contains

Même si la date est dans le arrayliste mais le probleme passe directement a else !!!

Aidez moi svp.

Modifier

Code :

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

1
2
3
4
5
6
7
8
for(int i=0; i<lis.size();i++) {
 if(lis.contains(dad)) { 
ok =1; 
}
 else{ 
ok = -1; 
}
}

comme ci-dessous:

Code :

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

1
2
3
4
5
6
7
8
9
for(int i=0; i<lis.size();i++) {
 if(lis.contains(dad)) { 
ok =1; 
} 
else{ 
if(ok = 0)
ok = -1; 
}
}