Discussion :

[cisco.nat]

bonsoir
je veux tester une requette à partie d'un numéro quelconque
par exemple j'ai le numéro 10, alors je veux tester numéro par numéro jusu'à 1
si le numéro existe dans la colonne de la table sql alors passer au suivant vl--
sinon afficher un message
voilà ce que j'ai fait

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
 
 
stmt = conn.createStatement();
ResultSet res=stmt.executeQuery( "SELECT num FROM table ");                         
int vl=10;
boolean found = false;
while(res.next()) {
    do{
        if(vl==Integer.parseInt(res.getString(1))){
 
        found = true;
        vl--;
        break;       
        }
 
    else if(!found){       
    System.out.print("NO");
    break;
 }       
}while(vl>1);       }
 res.close();

[Nico02]

Et quel est le problème ? Y'a t'il une erreur ?

Quel est le but de ce code ? Que cherches tu a tester ?

Code :

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

1
2
3
4
5
6
 
if(vl==Integer.parseInt(res.getString(1))){
 found = true;
 vl--;
 break;       
}

C'est pas très beau..

Écris plutôt quelque chose comme ça

Code :

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

1
2
3
4
5
6
7
8
 
while (!found && res.next())
{
 // ......
  found = (vl==Integer.parseInt(res.getString(1))
 
  if (!found) System.out.println("No");
}

[cisco.nat]

merci çà marche avec ce 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
 
for(int vl=20;vl>1;vl--) {
while(res.next())
                {
                    boolean found = false;
                   if(Integer.parseInt(res.getString(1))==vl)
                   {
                       System.out.print("exist"+vl);
                       found = true;
                       break;
                   }
                   else if (!found){
                         System.out.print("NO"+vl);
                       break;
                }
                   }

Ce que je veux c'est tester en commençant par 10 par exemple donc si 10 existe dans la base de données alors passer à 10-- sinon afficher un message NO ainsi de suite

[Nico02]

merci çà marche avec ce code:

J'ai pas dis que ça marchait pas, mais que c’était pas beau(pas bien). En programmation il y a des principes et des convention de codage a respecter pour une meilleure compréhension des codes source (et donc, entre autre, pour une meilleure maintenabilité des applications).

Quand on lit ton code on a pas vraiment envie de savoir comment il marche..

Code :

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

boolean found = false;

Tu vas créer autant de booleen que de tour de boucle alors qu'avec un seul tu fais la même chose.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
if(Integer.parseInt(res.getString(1))==vl)
{
  System.out.print("exist"+vl);
  found = true;
   break;
}
else if (!found){
  System.out.print("NO"+vl);
  break;
}

En gros :

Si on trouve :
On met trouve a true
On break
Sinon c'est qu'on a pas trouvé
On break aussi

Double break inutile, tu peux utiliser l'expression que je t'ai déjà exposer plus haut et t'en servir comme condition de boucle pour éviter ce genre de mauvaise pratique.

Code :

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

1
2
3
 
for(int vl=20;vl>1;vl--) {
while(res.next())

Tu testes si ta requête est un chiffre entre 2 et 20 (?)

Tu refais une requête a chaque tour de boucle ? Car la comme l'analyse de la requête est imbriquer dans ta boucle for il va falloir faire autant de requête que de tour. Pourquoi ne pas faire l'inverse plutôt ?

Sinon je réitère ma question précédente, y'a t'il un problème ou quelque chose que l'on puisse faire pour toi ?

Cdt.

[cisco.nat]

non pas du tout juste votre code ne contient pas aucune décrémentation pour le numéro 10 sachant que moi l'essentiel c'est que je dois tester numéro par numéro de 10 à 1 s'il existe dans la base ok décrémente et passe au suivant sinon faire des instructions.

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
 
            conn = DriverManager.getConnection("jdbc:mysql://localhost:3306/"+getdb(), "root", "123456");     
                stmt = conn.createStatement();
ResultSet res=stmt.executeQuery( "SELECT VLAN FROM tout ");                         
        for(vl=20;vl>1;vl--) {
while(res.next())
                {
                    boolean found = false;
                   if(Integer.parseInt(res.getString(1))==vl)
                   {
                       System.out.print("exist"+vl);
                       found = true;
                       break;
                   }
                   else if (!found){
                        ....[Faire des instructions]
 
                         break;
                }
                   }              
               }
 res.close();

[Nico02]

Il est pas complet c'est juste pour la structure..

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
conn = DriverManager.getConnection("jdbc:mysql://localhost:3306/"+getdb(), "root", "123456");     
stmt = conn.createStatement();
ResultSet res=stmt.executeQuery( "SELECT VLAN FROM tout "); 
boolean found = false;                        
for(vl=20;vl>1;vl--) {
     while res.next() && !found){
 
		found = Integer.parseInt(res.getString(1)) == vl;
 
		if(found)
			System.out.print("exist"+vl);
		else
			[Faire des instructions]          
    }
    found = false;
}
res.close();

je dois tester numéro par numéro de 10 à 1

Ici vous allez de 20 a 2.

[cisco.nat]

c'est pas grave juste exemple
pour ce code çà marche pas lorsque je fait un system.out après le else
il m'affiche plusieurs 19 le 20 existe dans la base alors que tous les autre nombre moins de 20 non .
Mais il affiche juste plein de 19 et n’arrête à cause de la boucle