Problème avec JList

Version imprimable

Problème avec JList

salut,

j'ai un problème concernant la remplissage de jList

j'ai un jComboBox qui est rempli automatiquement de la base de donne

et ca marche

j'ajoute un bouton et je fait un test .

selon mon choix il rempli le jList et ca marche avec sucée voila le code
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 private void validerActionPerformed(java.awt.event.ActionEvent evt) { String choix = jComboBox1.getSelectedItem().toString(); v = new Vector<Enseignant>(); Iterator<Enseignant> l = eDB.findAll().iterator(); while (l.hasNext()) { Enseignant e = l.next(); if(e.getPU().equalsIgnoreCase(choix)) { v.add(e); jList1.setListData(v); } } }
avec biensur la déclaration
Code:

1 2 Vector<Enseignant> v; EnseignantDB eDB = new EnseignantDB();
et pour la classe EnseigantDB j'ai une méthode findAll() qui retourne une liste d'enseignant.

mon problème qui quant je clique pour la première fois sur le bouton il rempli jList avec l'enseignant qui vérifier la condition

ca marcha avec sucé mais quant je clique pour la deuxième fois il m'affiche c vrai le résultat juste mais 2 fois

de même quant je clique pour la troisième fois il m'affiche 3 fois

j'essaye avec cette manière : au debut de code pour le bouton j'ajoute
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 private void validerActionPerformed(java.awt.event.ActionEvent evt) { jList1.removeAll(); String choix = jComboBox1.getSelectedItem().toString(); v = new Vector<Enseignant>(); Iterator<Enseignant> l = eDB.findAll().iterator(); while (l.hasNext()) { Enseignant e = l.next(); if(e.getPU().equalsIgnoreCase(choix)) { v.add(e); jList1.setListData(v); } } }
mais le problème reste le même

merci d'avance

20/12/2010, 18h30
Jose.N70
Bonjour, tu peux éventuellement utiliser un modèle sur la Jlist et faire une manip dans ce genre :
Code:

1 2 3 DefaultListModel dm = new DefaultListModel(); dm.addElement(""); jList1.setModel(dm);
C'est une simple idée ...

j'essaye avec ce code mais le même problème reste encore
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 private void validerActionPerformed(java.awt.event.ActionEvent evt) { DefaultListModel dm = new DefaultListModel(); dm.clear(); // pour rafraichir si on clique une deuxième fois sur le bouton ici le problème que je dois le corriger String choix = jComboBox1.getSelectedItem().toString(); v = new Vector<Enseignant>(); Iterator<Enseignant> l = eDB.findAll().iterator(); while (l.hasNext()) { Enseignant e = l.next(); if(e.getPU().equalsIgnoreCase(choix)) { v.add(e); dm.addElement(v); jList1.setModel(dm); } } }
est ce que ce possible sans utiliser "DefaultListModel " avec directement une methode de jlist (j'essaye avec jList1.repaint(); et jList1.clear()) mais le probleme rest toujours (juste affichage juste pour la premiere fois)

l'erreur de "DefaultListModel" est si j'ai deux enseignant en premier lieux qui vérifie la condition (je parle pour le premier clique sur je bouton de test) il m'affiche le résultat 2 fois

J'utilise également une JList qui doit agir plus ou moins comme toi, voici ma façon de faire ( peut être pas la meilleur )
Code:

1 2 3 4 5 6 7 public DefaultListModel listModel; public JList list; ... ... listModel = new DefaultListModel(); list = new JList(listModel);
Puis pour vider et avant de recharger ma liste j'ai une methode comme celle ci :
Code:

1 2 3 4 5 6 public void clear(){ list.removeAll(); listModel = null; listModel = new DefaultListModel(); list.setModel(listModel); }

le code de la class EnseignantDB est :

Code:

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
import java.sql.PreparedStatement;
import java.sql.SQLException;
import java.sql.*;
import java.util.ArrayList;
import java.util.List;
 
 
 
public class EnseignantDB {
   public  EnseignantDB(){
 
    }
    List enseignants = new ArrayList<Enseignant>();
 
  public  List  findAll() {
 
         connection b = new connection();
 
          try {
 
                b.prepare = (PreparedStatement) b.conn.prepareStatement("select * from enseignant");
                b.result=b.prepare.executeQuery();
                while(b.result.next())
              {
                    Enseignant e=new Enseignant();
 
                                e.setCin(b.result.getString("cin"));
                                e.setNom(b.result.getString("nom"));
                                e.setPrenom(b.result.getString("prenom"));
                                e.setTel(b.result.getString("tel"));
                                e.setEmail(b.result.getString("email"));
                                e.setLogin(b.result.getString("login"));
                                e.setPassword(b.result.getString("password"));
                                e.setPu(b.result.getString("pu"));
 
 
                                enseignants.add(e);
 
 
              }
        }catch(SQLException e){
            System.err.println(e);
        }
        return  enseignants;
      }
}

le code de la class connection.

Code:

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
import java.sql.Connection;
import java.sql.DriverManager;
import java.sql.PreparedStatement;
import java.sql.ResultSet;
 
 
public class connection {
public static Connection conn =null;
	public PreparedStatement prepare=null;
	public ResultSet result=null;
	public connection()
	{
 
		try {
			Class.forName("com.mysql.jdbc.Driver");
 
 
			String url = "jdbc:mysql://localhost/mabase";
			String user = "root";
			String passwd = "root";
 
			 conn = (Connection) DriverManager.getConnection(url, user, passwd);
 
 
		} catch (Exception e) {
			e.printStackTrace();
		}
 
 
        }
}

25/12/2010, 22h17
tchize_

a chaque appel de findAll vous ajoutez des enseignant à la List qui se trouve dans DAO. Sois vous ne remplissez qu'une seule fois la liste, soit vous la videz avant de la remplir!
25/12/2010, 23h42
mdh12

merci pour votre intérêt .

le problème est comment vider ma jlist (avec jList1.clear(), jList1.repaint(),jList1.removeAll() je n'arrive pas a vider la jlist)
26/12/2010, 01h58
sinok

Il te suffit de créer un nouveau DefaultListModel à partir de tes données puis de l'affecter à ta JList via un simple setModel.
26/12/2010, 08h59
tchize_

ce n'est pas ta jlist qu'il faut vider, mais celle utilisée dans findAll !

tu as raaison.

je dois vider celle utilisée dans findAll

j'essaye avec

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
  String choix = jComboBox1.getSelectedItem().toString();
          v = new Vector<Enseignant>();
 
           List l=null;
           l=eDB.findAll();
 
 
 
 
          for(int i=0;i<l.size();i++)
          {
              if(((Enseignant)l.get(i)).getPU().equalsIgnoreCase(choix))
              {
 
                  v.add((Enseignant) l.get(i));
 
 
 
 
 
          }
          }
            jList1.setListData(v);

mais toujours meme probleme.

j'essaye aussi avec un changement au niveau de la methode findAll

mais meme probleme .

Code:

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
 public  List  findAll() {
 
         connection b = new connection();
 
          try {
 
                b.prepare = (PreparedStatement) b.conn.prepareStatement("select distinct (pu) from enseignant");
                b.result=b.prepare.executeQuery();
                while(b.result.next())
              {
                    Enseignant e=new Enseignant();
 
                               // e.setCin(b.result.getString("cin"));
                                //e.setNom(b.result.getString("nom"));
                                //e.setPrenom(b.result.getString("prenom"));
                               // e.setTel(b.result.getString("tel"));
                                //e.setEmail(b.result.getString("email"));
                                //e.setLogin(b.result.getString("login"));
                                //e.setPassword(b.result.getString("password"));
                                e.setPu(b.result.getString("pu"));
 
 
                                enseignants.add(e);
 
 
              }
        }catch(SQLException e){
            System.err.println(e);
        }
        return  enseignants;
      }

26/12/2010, 09h38
tchize_

C'est la liste utilisée dans findAll qu'il faut vider

merci pour vos reponses.

vraiment je n'arrive pas a corriger mon code .

je vide la liste avec

Code:

1
2
List l=null;
           l=eDB.findAll();

et vous me dis que je dois vider la list dans findAll

tu veux dire que je dois faire comme celle ci

Code:

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
public  List  findAll() {
 
         connection b = new connection();
 
          try {
                b.result=null; // cette ligne que je pense qu'elle dois vide la list 
                b.prepare = (PreparedStatement) b.conn.prepareStatement("select * from enseignant");
                b.result=b.prepare.executeQuery();
                while(b.result.next())
              {
                    Enseignant e=new Enseignant();
 
                               e.setCin(b.result.getString("cin"));
                                e.setNom(b.result.getString("nom"));
                                e.setPrenom(b.result.getString("prenom"));
                                e.setTel(b.result.getString("tel"));
                                e.setEmail(b.result.getString("email"));
                                e.setLogin(b.result.getString("login"));
                                e.setPassword(b.result.getString("password"));
                                e.setPu(b.result.getString("pu"));
 
 
                                enseignants.add(e);
 
 
              }
        }catch(SQLException e){
            System.err.println(e);
        }
        return  enseignants;
      }

26/12/2010, 12h58
mdh12

cette ligne que j'ai ajouter

Code:

b.result=null

ne sert pas a vider la list dans la fonction findAll

mais comme vous dite la solution est au tour de cette fonction

mais je n'arrive pas à la résoudre
26/12/2010, 13h49
DarkMolo

Salut,

Comme déjà dit:

Citation:

tchize_ dit:
C'est la liste utilisée dans findAll qu'il faut vider

Ta méthode :

Code:

public List findAll() {

Retourne une liste, forcément dans son code, il devrait y avoir un return d'une liste, c'est cette liste qu'il faut vider.
26/12/2010, 14h08
mdh12

je suis d'accord avec vous

tous vous parler de vider la liste qui est retourné par

la fonction "findAll()"

mais comment c ça le problème

j'espère cette fois que vous puisez me répondre avec un code car moi je fait a mon avis les changements possibles mais le problème reste le même

merci d'avance