Discussion :

[Hind4Dev]

je veux faire des statistiques sur ma BDD en java et avoir le résultat dans un JTable, alors j'ai créé la méthode suivante:

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
 
public String getStat(String table, String as, String champ,String condition,String group){
 
		try {
			String nbr = "";
			String query = "SELECT COUNT(*) FROM "+table+" AS "+as+" WHERE "+champ+"='"+condition+"' GROUP BY "+group;
			stat = cnx.prepareStatement(query);
			rslt = stat.executeQuery();
			if(rslt.next()){
				nbr = rslt.getString(as);
			}
			return nbr;
		} catch (Exception e) {
			e.printStackTrace();
			return e.getMessage();
		}
	}

pour visualiser le résultat j'ai dessiné un tableau tableFil, mais je veux que mon tableau affiche pour chaque filière le nombre des filles et garçons

Code :

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

1
2
3
4
5
6
7
8
 
JButton btnStat = new JButton("Générer Statistiques");
		btnStat.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent e) {
				sql.getStat("etudiant", "Garçons", "genre", "M", "code_f");
				tableFil.setModel(DbUtils.resultSetToTableModel(rslt));
			}
		});

mais mon code génère plusieurs erreurs, comme: Column 'genre' not found.
je ne sais comment régler celà ??

[joel.drigo]

Salut,

Le message d'erreur dit que tu n'as pas de colonne appelée genre dans ta table etudiant. Il y'en a une ou pas ?

Par ailleurs

1. Je doute que count(*) soit un String ! Ce serait plutôt un int, non ?
2. Dans l'actionListener, tu ne récupères pas le résultat de l'appel de getStat(). Mais peut-être cet appel ne sert-il qu'à affecter le rslt utilisé en argument dans DbUtils.resultSetToTableModel(rslt) ? Pourquoi ne pas faire plus simplement et proprement que la méthode retourne le TableModel ?
3. Tu ne fermes ni statement, ni resultset ! En plus, les variables semblent être déclarées en variable de classe : à chaque appel, on perd les précédentes instances qui ne seront jamais fermées sauf à la fermeture de la connexion. Un bon moyen d'avoir de la consommation mémoire inutile.
4. ça sert à quoi d'utiliser un preparedStatement pour n'utiliser que des paramètres en dur ? Utilise les paramètres !
5. A quoi sert l'alias dans ta requête ?

[Hind4Dev]

le champ genre existe dans ma table etudiant, avant j'ai mis le type int mais ça donne trop d'erreurs

[joel.drigo]

Et c'est quoi les autres erreurs ?

Par ailleurs, je n'avais pas vu, mais nbr = rslt.getString(as);, ça ne risque pas de fonctionner : la variable as c'est l'alias de table, alors que c'est un nom de champ qu'il faut comme argument. Mais tu voulais peut-être en faire un alias pour count(*), comme ça :

Code :

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

String query = "SELECT COUNT(*) AS " + as + " FROM "+table + " WHERE "+champ+"='"+condition+"' GROUP BY "+group;

Par ailleurs, cet alias est inutile : tu peux aussi récupérer la valeur par nbr = rslt.getString( 1 );

[Hind4Dev]

en faite je reconnais que ma fonction getStat() est fausse car là le résultat n'est pas un entier mais un tableau car il va m'afficher selon chaque group by le nombre qui lui correspond alors je ne sais comment procéder

[joel.drigo]

Bah, tout simplement comme je te l'ai indiqué :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
public TableModel getStat(String table, String as, String champ,String condition,String group){
 
		try { 
			String query = "SELECT COUNT(*) FROM "+table+" AS "+as+" WHERE "+champ+"='"+condition+"' GROUP BY "+group;
			stat = cnx.prepareStatement(query);
			rslt = stat.executeQuery();
                        return DbUtils.resultSetToTableModel(rslt); 
		} catch (Exception e) {
			e.printStackTrace();
			return new DefaultTableModel();
		}
	}

Code :

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

1
2
3
4
5
6
JButton btnStat = new JButton("Générer Statistiques");
		btnStat.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent e) {
				tableFil.setModel(getStat("etudiant", "Garçons", "genre", "M", "code_f"));
			}
		});

Bon, ça c'est avec le minimum de modifications de ton code, mais il faudrait au moins fermer le Statement :

Code :

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

1
2
3
4
5
6
7
8
9
10
public TableModel getStat(String table, String as, String champ,String condition,String group){
 
		try (PreparedStatement statement = cnx.prepareStatement( "SELECT COUNT(*) FROM "+table+" WHERE "+champ+"=? GROUP BY "+group) ) { 
			statement.setString(1, condition); 
                        return DbUtils.resultSetToTableModel(stat.executeQuery()); 
		} catch (Exception e) {
			e.printStackTrace();
			return new DefaultTableModel();
		}
	}

[Hind4Dev]

merci ceci fonctionne mais pas comme je l'ai prévu , il m'affiche juste le résultat sans indication il me faut plus de détail, n'auriez vous pas un tuto à me passer svp???

[joel.drigo]

merci ceci fonctionne mais pas comme je l'ai prévu , il m'affiche juste le résultat sans indication il me faut plus de détail, n'auriez vous pas un tuto à me passer svp???

Il faut adapter ta requête SQL évidemment, pour récupérer les informations qui t'intéressent. Pour les tutoriels SQL, je suppose qu'en allant dans le forum SGBD/SQL, rubrique Tutoriels, on doit pouvoir en trouver.

[Hind4Dev]

j'ai besoin d'afficher la filière,nombre des garçons et nombre des filles ;
j'ai fait cette instruction

Code :

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

1
2
 
String query = "SELECT filiere, genre, COUNT(*) FROM etudiant INNER JOIN filiere ON etudiant.code_f=filiere.code_f  WHERE genre='M' GROUP BY code_f";

ça me génère cette erreur : Column 'code_f' in group statement is ambiguous


est-il possible d'avoir le nombre des filles et des garçons dans la même instruction sql pour que je les affiche dans le même tableau?

[joel.drigo]

ça me génère cette erreur : Column 'code_f' in group statement is ambiguous

Tu as ce message parce que tu as deux champs code_f, et que SQL ne sait pas lequel prendre : etudiant.code_f=filiere.code_f. Il suffit que tu précises lequel (ils sont égaux par la jointure, donc tu peux indiquer n'importe lequel des deux).

[Hind4Dev]

même comme ça il génère la même erreur

Code :

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

INNER JOIN filiere ON code_f

[joel.drigo]

ça me génère cette erreur : Column 'code_f' in group statement is ambiguous
?

C'est dans le group by (on te l'indique dans le message !) qu'il y ambiguïté.

Code :

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

String query = "SELECT filiere, genre, COUNT(*) FROM etudiant INNER JOIN filiere ON etudiant.code_f=filiere.code_f  WHERE genre='M' GROUP BY etudiant.code_f";

[Hind4Dev]

maintenant il m'affiche le genre et la filiere avec le nombre mais comme ça:


ce dont j'ai besoin est d'avoir si possible en java les filières à gauche et en haut les colonnes filles garçons??? comme ceci

[joel.drigo]

C'est plutôt une question de pur SQL à poser dans le forum SQL. En Java, on pourrait traiter le résultat de la requête à plat, pour faire un regroupement (avec des Map, et éventuellement un Collectors.groupingBy()).