Discussion :

[japhettchabao]

Bonsoir, il est 21h chez moi ; je suissur un logiciel que je developpe en java avec netbeans pour ma daronne ; le problème c'est que j'éssaie de parcourir un tableau et de recuperer des valeurs pour ensuite effectuer des mises à jour dans la bdd. et je crois que ce morceau de code pose problème. La table s'appelle facture. Coordialement

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
 
int i = facture.getColumnCount();
         for (int j = 0; j < i; j++) {
             int quantite_acheter = Integer.parseInt(facture.getValueAt(facture.getSelectedRow(),0).toString());
              String nom = facture.getValueAt(facture.getSelectedRow(), 1).toString();
              String marque = facture.getValueAt(facture.getSelectedRow(), 2).toString();
              String taille= facture.getValueAt(facture.getSelectedRow(), 3).toString();
                String upd = "select * from articles where nom='"+nom+"'taille = '"+taille+"' marque ='"+marque+"' ";
 
        try {
            rq.connexion();
 
            ResultSet res = rq.executer(upd); 
            int id =res.getInt("id");
            int quantite_actu =res.getInt("stock");
            int new_stock = quantite_actu - quantite_acheter;
            String enf = "update article set stock ='"+new_stock+"' where id ='"+id+"'";
            rq.modifier(enf);
 
        } catch (SQLException e) {
            JOptionPane.showMessageDialog(null, "");
        }
            }

[joel.drigo]

Salut,

Je ne sais pas de quel problème tu parles, puisque tu ne le décris pas.

Mais voilà ce que je peux voir rapidement :

1. 
   

   Code :

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

   1
   2
   int i = facture.getColumnCount();
            for (int j = 0; j < i; j++) {

   Tu fais une boucle sur les colonnes : donc le code dans le bloc du for est fait autant de fois qu'il y a de colonnes. Comme ce code ne dépend pas de j, tu exécute plusieurs fois la même chose, alors qu'une seule fois suffit (ça fait la même chose autant fois de qu'il y a de colonnes).
2. facture.getValueAt(facture.getSelectedRow()/*...*/Avec ce code, tu récupères la valeur de la ligne sélectionnée. Si ton but est de parcourir l'ensemble de la table, il faudrait boucler sur les lignes :
   

   Code :

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

   1
   2
   int i = facture.getRowCounr();
   for(int j = 0; j<i; j++) {

   Et faire facture.getValueAt(j/*...*/
3. int quantite_acheter = Integer.parseInt(facture.getValueAt(facture.getSelectedRow(),0).toString());Cette double conversion ne sert à rien. Il suffirait de caster pour récupérer la valeur.
4. Tu devrais regarder les PreparedStatement, au lieu des Statement : ils sont plus simple à utiliser avec les paramètres (pas besoin de faire des concaténations, de se soucier des quotes) et en plus empêche l'injection SQL.
5. Plutôt que de chercher à retrouver l'id par un select, tu devrais avoir à disposition cet id en mémoire (non seulement tu économiserais un SELECT, mais en plus, tu n'aurais pas de souci en cas de doublon). Pour ce faire, il faut faire une classe qui représente une ligne (un simple pojo avec getter et setter, servant de bean) et faire un modèle de table spécifique.

[japhettchabao]

merci de ton aide mais pour tout avouer, je suis pas fort en java. mais j'ai essayer de bricoler un truc parce que c'était urgent. je prendrai le temps pour approfondir mes connaissances mais pour l'instant le problème est que le tableau facture contient des donnees designation quantite ... le logiciel marche bien, sauf que j'arrive pas à diminuer la quantité acheter sur le stock actuel de la base de donnees. je voudrais avoir une boucle qui parcours le tableau facture et recupere la quantité achetée puis fasse un update du stock pour chaque article se trouvant dans ce tableau. En espérant avoir éclairci mon problème, Coordialement

[japhettchabao]

J'ai bricoler cousi cousa et j'ai trouvé. Je sais que c'est pas de la programmation pour être embauché chez microsoft mais sa marche. merci quand même