Discussion :

[kamikazbe]

Salutations à tous,

je vais tenter d'expliquer brièvement ce que je voudrais faire avec ma boucle while.
Je collecter des informations dans ma table "commandes" et "personnel" qui ont notamment les champs suivants : pour "commandes"(MATRAGENT, CODEART,TOTART,...) pour "personnel"(MATRAGENT, SOLDDISPO).
En fonction du code d'article je dois sélectionner le MATRAGENT et le TOTART de ma table "commandes" pour additionner TOTART à SOLDDISPO de ma table personnel en fonction du MATRAGENT correspondant.
La récupération des informations se fait correctement, mais la méthode boucle plusieurs fois sur le même MATRAGENT et récupère le SOLDDISPO du matricule précédent dans la table.
Voici le code source de la méthode qui devrait faire cette opération:

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
 
private String sqlQuerySupCom,rechArtSup,rechAgtSD,rechAgtSDMaj,totArtSup,soldDispoArtSup,mAAS,mAASRet;
  private int soldDisposupp,totArtSu,matrAgent;
  private RequestDB selArtSup,selAgtArtSup,majSDAgt;
  private void SupToutCmdes(java.awt.event.ActionEvent evt)                              
  {                                  
    // TODO add your handling code here:
    majSDAgt = new RequestDB();
    selArtSup = new RequestDB();
    selAgtArtSup = new RequestDB();
    soldDisposupp = totArtSu=0;
    rechArtSup = "SELECT c.MATRAGENT , c.TOTART , p.SOLDDISPO FROM commandes AS c, personnel AS p LEFT JOIN personnel ON c.MATRAGENT = p.MATRAGENT WHERE CODEART ='"+codArtSuppress.getText()+"'";
    selArtSup.openMyDataBaseConnection();
 
    if (selArtSup.byStatementQuery(rechArtSup) != null)
    {JOptionPane.showMessageDialog(null,"CODEART : "+codArtSuppress.getText());
      while (selArtSup.myGoNext())
      {
        mAAS=selArtSup.myGetString("c.MATRAGENT");
        mAASRet = mAAS.toString();
        totArtSu=Integer.parseInt(selArtSup.myGetString("c.TOTART"));
        soldDispoArtSup=selArtSup.myGetString("p.SOLDDISPO");
        JOptionPane.showMessageDialog(null,"Matricule à updater 1 : "+
                mAASRet+" total de l'article à supprimer : "+totArtSu+" Solde Disponible : "+soldDispoArtSup);
 
        JOptionPane.showMessageDialog(null,"SoldDispoArtSup : "+soldDispoArtSup);
 
        soldDisposupp=(Integer.parseInt(soldDispoArtSup))+totArtSu;
        JOptionPane.showMessageDialog(null, "après addition : "+soldDisposupp+" matricule à updater : "+mAASRet);
 
        rechAgtSDMaj = "UPDATE personnel SET SOLDDISPO='"+soldDisposupp+"' WHERE MATRAGENT ='"+ mAAS +"'";
 
        majSDAgt.openMyDataBaseConnection();
 
        majSDAgt.byStatementUpdate(rechAgtSDMaj);
 
 
        majSDAgt.closeMyDataBaseConnection();
 
        selArtSup.myGoNext();
 
      }
 
      selArtSup.myGoNext();
    }
    else
    {
      JOptionPane.showMessageDialog(null, "Aucune commande 96");
    }
    majSDAgt.closeMyDataBaseConnection();
    selArtSup.closeMyDataBaseConnection();
 
  }

Merci de votre aide car je tourne en rond dessus

Kamikazbe

[in]

bon ... j'ai pas lu et je pense que c'est le cas de tout le monde

Utilise la balise code (#) pour la mise en forme et n'affiche que ce qui est nécessaire ...

[EDIT] +1 avec le message ci-dessous. N'affiche que le bout de code nécessaire et explique clairement ce qui ne marche pas. Là ça donne pas envie de décortiquer le code pour comprendre ... surtout le matin

[_Mac_]

Je comprends rien à ton pb : elle est ou ta boucle while, c'est quoi les trucs que tu veux ajouter, etc. En gros, donne plus de détails clairs

[Roy Miro]

Bonjour,

A la fin du bloc d'instructions de ta boucle while, il faut ajouter cette instruction pour faire avancer l'itération de la boucle.

Code :

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

selArtSup.myGoNext()

[in]

Moi je dirais plutot que ça serait ça (en même temps je ne suis pas sur d'avoir capté) :

il faudrait que tu fasses ça en une seule requete, genre (à vérifier) :

Code :

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

1
2
3
SELECT MATRAGENT, TOTART, nvl(SOLDDISPO,0)
FROM commandes c LEFT JOIN personnel p ON c.MATRAGENT = p.NMATR
WHERE CODEART ='"+codArtSuppress.getText()+"'

en même temps, si cette requete est effectuée plusieurs fois, tu devrais passer par un PreparedStatement, ça serait mieux ...