Discussion :

[BenkessiratAmina]

Salut les développeurs,
J’ai un petit problème avec une requête SQL, je vais me casser les dents dessus.
J’ai une table Achat, contenant parmi autres deux colonnes (date et paiement). Quand je veux sommer les paiements entre une plage de date, si au moins l’une des deux dates des bords des plages existent dans ma table la somme est calculée normalement. Si les deux dates n’y figure pas une exception est générée. Voici ma petite voyelle de requête :

Code :

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

String query1 = "select  SUM (paiement) from Achat WHERE Achat.date between '"+( date )+"' and  '"+( date1 )+"'";

Et voice l’exception:
Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
at vues.Paiement.jButton1ActionPerformed(Paiement.java:234)
at vues.Paiement.access$300(Paiement.java:16)
………………
Ça pointe vers:

Code :

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

System.out.println("  "+rsltSet.getObject(1).toString());

Ou est ce que j’ai fauté??!

[Cincinnatus]

Code :

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

System.out.println("  "+rsltSet.getObject(1).toString());

ça peut être : (1) rsltSet.getObject(1) si rsltSet est null
ou (2) rsltSet.getObject(1).toString() si le getObject(1) renvoie null.

A vérifier, il faudra ajouter 2 lignes de log, tant qu'à faire, au lieu des System.out.println(...);
Et le .toString() est le plus souvent facultatif. S'en passer peut éviter le cas de plantage n.2.

[tchize_]

avoir du code un peu plus complet, histoire de voir ce que tu fais, et une stacktrace entière, ça aiderait à t'aider

[BenkessiratAmina]

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
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
        String s, s1;
        String J, M, A, A1;
        String date, date1;
        J = jj.getSelectedItem().toString();
        M = mm.getSelectedItem().toString();
        A = aa.getSelectedItem().toString();
        String t[] = A.split("");
        String aa;
        aa = "" + t[2] + "" + t[3];
        date = J + "/" + M + "/" + aa; //date debut
 
        J = jj1.getSelectedItem().toString();
        M = mm1.getSelectedItem().toString();
        A1 = aa1.getSelectedItem().toString();
 
        String t2[] = A1.split("");
        String aa1;
        aa1 = "" + t2[2] + "" + t2[3];
        date1 = J + "/" + M + "/" + aa1; //date fin
 
        String url = "jdbc:h2:./src/BD/BDH22";
        String login = "root";
        String password = "";
        Connection connection = null;
        Statement stmt = null;
        ResultSetMetaData rsltSetMeta = null;
 
        try {
            connection = DriverManager.getConnection(url, login, password);
        } catch (SQLException ex) {
            Logger.getLogger(Paiement.class.getName()).log(Level.SEVERE, null, ex);
        }
        System.out.println("c connecté pour ouvrir dossier");
        try {
            stmt = connection.createStatement();
 
 
        } catch (SQLException ex) {
            Logger.getLogger(Paiement.class.getName()).log(Level.SEVERE, null, ex);
        }
 
 
        String query1 = "select  SUM (paiement) from Achat WHERE Achat.date between '" + (date) + "' and  '" + (date1) + "'";
        ResultSet rsltSet = null;
        try {
            rsltSet = stmt.executeQuery(query1);
            rsltSetMeta = rsltSet.getMetaData();
 
        } catch (SQLException ex) {
            Logger.getLogger(Paiement.class.getName()).log(Level.SEVERE, null, ex);
        }
 
 
        try {
            while (rsltSet.next()) {
 
                System.out.println(" ffffffffff " + rsltSet.getMetaData().getColumnCount());
                for (int i = 1; i <= rsltSetMeta.getColumnCount(); i++) {
 
 
                    System.out.println("  " + rsltSet.getObject(1).toString());
                    this.c.setText(rsltSet.getObject(1).toString());  //ecriture dans mon interface
                }
            }
        } catch (SQLException ex) {
            Logger.getLogger(Paiement.class.getName()).log(Level.SEVERE, null, ex);
 
        }
        c.setVisible(true);
        c.setEditable(false);
        da.setVisible(true);

tchize_ voila le boutton de tout mon code
Cincinnatus je n'ai pas compris!

[tchize_]

Tu peux nous donner la méthode complète (de préférence en corrigeant ton indentation qu'on sache suivre) et la sortie complète de ta console. JE suspecte que tu as eu une exception juste avant le parcours du result set. Vu que tu continue quand même, tu te retrouve à parcourir un vieux ResultSet où la colonne 1 contient un null.

D'une manière générale, quand tu as une exception, traite là. Juste l'afficher et continuer, c'est cacher le problème sous le tapis.

[Népomucène]

Plusieurs choses qui polluent ton code :

1) Pourquoi utiliser un A.split("") qui est inefficace dans le code suivant :

Code :

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

1
2
3
4
5
 
        A=aa.getSelectedItem().toString();       
        String t[]=A.split("");
        String aa;
        aa=""+t[2]+""+t[3];

Cela fait que si dans aa.getSelectedItem().toString() a pour valeur "2017" alors aa = "17"
Dans ce cas, il vaut mieux écrire :

Code :

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

        aa = getSelectedItem().toString().substring(2);

2) En SQL c'est toujours mieux de nommer les colonnes :

Code :

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

String query1 = "select  SUM (paiement) AS mon_total from Achat WHERE Achat.date between '"+( date )+"' and  '"+( date1 )+"'";

3) Il n'est pas utile de recourir au ResultSetMetaData pour parcourir les colonnes du resultset vu qu'il n'y a qu'une seule colonne !
d'autant plus inutile puisque tu écris dans la boucle rsltSet.getObject(1) ce qui fait que même s'il y avait eu plusieurs colonnes, tu ne lis que la première.

Ensuite, cette requête SQL ne renvoie qu'une seule ligne (pas plus)
Donc tu peux allègrement supprimer le while qui sert à parcourir les enregistrements quand il y en a plusieurs.

Simplification du code :

Code :

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

1
2
3
4
5
6
7
8
9
        try {
            rsltSet.next()  // pour positionner sur le 1er et unique enregistrement
            System.out.println(rsltSet.getObject(1).toString());
            this.c.setText(rsltSet.getObject(1).toString());  //ecriture dans mon interface
 
        } catch (SQLException ex) {
            Logger.getLogger(Paiement.class.getName()).log(Level.SEVERE, null, ex);
 
        }