Remplir une combobox en lien avec une autre combobox

Version imprimable

Bonjour,

Je travaille actuellement sur une petite application permettant l'ajout des operateurs dans une base de données pour ensuite être payer. Pour alimenter mes combo-box je fais des requêtes dans une base existante jusque la pas de soucis.
Je souhaiterais que lorsque je sélectionne le matricule de la personne , la combo-box "nom" se remplisse automatiquement par rapport au matricule ?

Une autre petite question est il possible d'afficher la SectionSalarie, et le Libelle dans : comboBox_1.addItem(rs.getString("SectionSalarie"));

"SELECT DISTINCT SectionSalarie,Libelle FROM SectionSalarie"

Table SectionSalarie :
id
SectionSalarie
Libelle

Table Salarie:
ID
matricule
NomPrenomSalarie
sectionsalarie

Voici le code de ma Jframe avec mes combo-box.

Merci beaucoup

Alexandre
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 47 48 49 50 51 52 53 54 55 56 57 58 JComboBox comboBox_3 = new JComboBox(new DefaultComboBoxModel()); comboBox_3.setBounds(279, 149, 160, 20); frmSaisieDesHeures.getContentPane().add(comboBox_3); PreparedStatement stmt = null; ResultSet rs = null; try { stmt = Connexion_BDD.get().prepareStatement("SELECT NomPrenomSalarie FROM SalariesPresents"); rs = stmt.executeQuery(); while (rs.next()) { comboBox_3.addItem(rs.getString("NomPrenomSalarie")); } } catch (SQLException e1) { // TODO Auto-generated catch block e1.printStackTrace(); } JComboBox comboBox_2 = new JComboBox(); comboBox_2.setBounds(81, 149, 113, 20); frmSaisieDesHeures.getContentPane().add(comboBox_2); try { stmt = Connexion_BDD.get().prepareStatement("SELECT Matricule FROM MOD_SalariesPresents"); rs = stmt.executeQuery(); while (rs.next()) { comboBox_2.addItem(rs.getString("Matricule")); } } catch (SQLException e1) { // TODO Auto-generated catch block e1.printStackTrace(); } JComboBox comboBox_1 = new JComboBox(); comboBox_1.setBounds(530, 149, 135, 20); frmSaisieDesHeures.getContentPane().add(comboBox_1); try { stmt = Connexion_BDD.get().prepareStatement("SELECT DISTINCT SectionSalarie,Libelle FROM MOD_SectionSalarie "); rs = stmt.executeQuery(); while (rs.next()) { comboBox_1.addItem(rs.getString("SectionSalarie")); } } catch (SQLException e1) { // TODO Auto-generated catch block e1.printStackTrace(); }

Salut,

Pour le lien des combos, tu peux regarder l'exemple dans ce post.

Pour l'affichage, utilise un ListCellRenderer. Exemple avec extension de DefaultListCellRenderer :

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
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
public class DemoListCellRenderer {
 
	public static void main(String[] args) {
 
		JFrame frame = new JFrame("Démo");
		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
 
		MyObject[] data = {
				new MyObject(1, "Marie", "Directeur de projet"),
				new MyObject(1, "Paul", "Chef de projet"),
				new MyObject(1, "Caroline", "Développeur"),
				new MyObject(1, "Marc", "Développeur"),
		};
 
		JComboBox<MyObject> combo = new JComboBox<>(data);
		combo.setRenderer(new DefaultListCellRenderer() {
 
			@Override
			public Component getListCellRendererComponent(JList<?> list, Object value, int index, boolean isSelected,
					boolean cellHasFocus) {
 
				if ( value instanceof MyObject ) {
					MyObject object = (MyObject) value;
					setText(object.name+" ("+object.type+")");
				}
 
				 return this;
			}
 
 
		});
 
		combo.addActionListener(e-> afficheSelection(combo.getSelectedItem()));
 
		frame.add(combo, BorderLayout.SOUTH);
		frame.setSize(300, 200);
		frame.setLocationRelativeTo(null);
		frame.setVisible(true);
 
	}
 
 
	private static void afficheSelection(Object selectedItem) {
		if ( selectedItem instanceof MyObject ) {
			System.out.println("Object sélectionné : " + ((MyObject)selectedItem).id + "-" + ((MyObject)selectedItem).name);
		}
	}
 
 
	public static class MyObject {
		public final int id;
		public final String name;
		public final String type;
		public MyObject(int id, String name, String type) {
			this.id=id;
			this.name=name;
			this.type=type;
		}
 
	}
 
}

27/09/2016, 11h53
Alex1407

merci pour votre réponse,

j'ai déjà regardé se post mais je n'arrive pas à le mettre en place dans mon code. Avec l'id on ne peut pas simplifier les choses ?

et pour l'affichage , vous mettez des valeurs en dur dans votre Object data , moi c'est ma requête qui va chercher les données donc comment mettre sa dans mon code ?

merci

Citation:

Envoyé par Alex1407

j'ai déjà regardé se post mais je n'arrive pas à le mettre en place dans mon code. Avec l'id on ne peut pas simplifier les choses ?

Le fait de créer une classe qui représente ce que tu récupères de ta base simplifie les choses : tu ne manipules plus des trucs qui se baladent séparement ou un tableau.
Il te faudrait effectivement l'id dans la requête, parce que ça te permettra de retrouver ton élément dans la table (avec un where sur l'id, et non un where sur un nom qui pourrait être dupliqué)

Citation:

Envoyé par Alex1407

et pour l'affichage , vous mettez des valeurs en dur dans votre Object data , moi c'est ma requête qui va chercher les données donc comment mettre sa dans mon code ?

C'est pareil, au lieu de passer une String constante, tu passes les résultats de ResultSet.getString("champ truc") en argument.

Avec ta requête :

Code:

stmt = Connexion_BDD.get().prepareStatement("SELECT DISTINCT id,SectionSalarie,Libelle FROM MOD_SectionSalarie ");
Code:

1 2 3 4 5 6 rs = stmt.executeQuery(); while (rs.next()) { comboBox_3.addItem(new SectionSalarie(rs.getString("id"), rs.getString("SectionSalarie"), rs.getString("Libelle")); }
Difficile de faire plus simple.
Au passage, tu ne fermes ni ton ResultSet, ni ton Statement ! Il faut absolument le faire. Avec un try-with-resource, ce simple(et il vaut mieux utiliser des variables locales au lieu de recycler) :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 try(PreparedStatement stmt = Connexion_BDD.get().prepareStatement("SELECT DISTINCT id,SectionSalarie,Libelle FROM MOD_SectionSalarie ")) { ResultSet rs = stmt.executeQuery(); while (rs.next()) { comboBox_3.addItem(new SectionSalarie(rs.getString("id"), rs.getString("SectionSalarie"), rs.getString("Libelle")); } } catch(SQLException e) { e.printStackTrace(); System.err.println(e.getSQLState()); }
Ensuite, tu peux récupérer ton l'id de l'item sélectionné dans l'ItemListener pour faire la requête qui va remplir l'autre combo, et afficher les 2 autres informations.

j'ai créer la classe SectionSalarie, mais mon code n'ai pas reconnu. Je doit mal m'y prendre, mais je suis un peut perdu la.

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
import java.beans.Statement;
import java.sql.ResultSet;
import java.sql.SQLException;
 
import javax.swing.JComboBox;
 
import DB.Connexion_BDD;
 
public class SectionSalarie {
 
	public SectionSalarie(String string, String string2) {
 
		JComboBox comboBox_1 = new JComboBox();
		comboBox_1.setBounds(530, 149, 135, 20);
		frmSaisieDesHeures.getContentPane().add(comboBox_1);
		ResultSet rs;
		Statement stmt;
 
		try {
			stmt = Connexion_BDD.get().prepareStatement("SELECT DISTINCT id,SectionSalarie,Libelle FROM MOD_SectionSalarie ");
 
			rs = stmt.executeQuery();
 
			while (rs.next()) {
 
				comboBox_3.addItem(new SectionSalarie(rs.getString("id"),rs.getString("SectionSalarie"), rs.getString("Libelle"));
			}     
 
 
		} catch (SQLException e1) {
			// TODO Auto-generated catch block
			e1.printStackTrace();
		}finally{
 
			rs.close();
		}
 
 
	}
 
}

Citation:

il vaut mieux utiliser des variables locales au lieu de recycler

c'est a dire utiliser des nouvelle variables à chaque fois pour mes requêtes ?

merci

27/09/2016, 14h36
joel.drigo

Citation:

Envoyé par Alex1407

j'ai créer la classe SectionSalarie, mais mon code n'ai pas reconnu. Je doit mal m'y prendre, mais je suis un peut perdu la.

La classe SectionSalarie c'est juste une classe pour représenter les enregistrements de ta base de données (il faudrait une classe différente par table manipulée au moins). Ce n'est pas cette classe qui doit gérer la combobox : la combobox c'est de l'UI et c'est dans une autre classe (d'UI). Dans mon exemple j'ai une classe DemoListCellRenderer pour l'UI et la classe MyObject pour représenter les objets. Elle est définie dans la classe DemoLsitCellRenderer, mais pourrait être définie dans son propre fichier (en non static) et ça reviendrait au même.

Citation:

Envoyé par Alex1407

c'est a dire utiliser des nouvelle variables à chaque fois pour mes requêtes ?

Oui. Sinon, c'est le meilleure moyen pour se mélanger les pinceaux, oublier de libérer une ressource, parce qu'elle a été remplacée par une autre, etc.

Quand je met le code suivant il ne m'accepte pas la ligne soulignée

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
JComboBox comboBox_3 = new JComboBox();
		comboBox_3.setBounds(530, 149, 135, 20);
		frmSaisieDesHeures.getContentPane().add(comboBox_3);
		try {
			stmt = Connexion_BDD.get().prepareStatement("SELECT DISTINCT SectionSalarie,Libelle FROM MOD_SalariesPresents_Vue01");

			rs = stmt.executeQuery();

			while (rs.next()) {

				comboBox_3.addItem(new SectionSalarie(rs.getString("id"), rs.getString("SectionSalarie"), rs.getString("Libelle"));
				
				rs.close();
			}     
		        

		} catch (SQLException e1) {
			// TODO Auto-generated catch block
			e1.printStackTrace();
		}

merci

27/09/2016, 16h29
joel.drigo

Montre-moi ta classe SectionSalarie.

Voici la classe SectionSalarie

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
package DB;
 
import java.sql.PreparedStatement;
import java.sql.ResultSet;
import java.sql.SQLException;
 
public class Sectionsalarie {
 
	public static void main(String[] args) {
		// TODO Auto-generated method stub
		PreparedStatement stmt = null;
		ResultSet rs = null;
 
		try(PreparedStatement stmt1 = Connexion_BDD.get().prepareStatement("SELECT DISTINCT id,SectionSalarie,Libelle FROM MOD_SectionSalarie ")) {
			   ResultSet rs1 = stmt1.executeQuery();
 
			   while (rs1.next()) {
			       comboBox_3.addItem(new Sectionsalarie(rs1.getString("id"), rs1.getString("SectionSalarie"), rs1.getString("Libelle"));
			   }
 
			}
			catch(SQLException e) {
			   e.printStackTrace();
			   System.err.println(e.getSQLState());
			}
 
 
	}
 
}

Je le répète une seconde fois : la classe SectionSalarie sert uniquement à représenter/manipuler quelque chose qui représente un enregistrement dans la table MOD_SectionSalarie (d'où le nom de la classe = SectionSalarie) : rien à voir avec de l'UI (combo, panel, ou je ne sais quoi). Il s'agit juste d'un objet avec les propriétés nécessaires, en l'occurence je suis parti de ta requête, ou j'ai ajouté un id (parce quand on manipule des données de base de données, on les manipule par leur id (recherche, update, etc.), donc c'est utile, voire indispensable).

Donc la classe SectionSalarie (dont j'ai dit qu'elle était le pendant de la classe MyObject de mon exemple initiale) c'est une truc comme ça (attention les types sont peut être à adapter, ne connaissant pas ton modèle de base, je ne peux que supputer) :
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 public class SectionSalarie { private final String id; private String sectionSalarie; private String libelle; public SectionSalarie(String id, String sectionSalarie, String libelle) { this.id=id; this.sectionSalarie=sectionSalarie; this.libelle=libelle; } public String getId() { return id; } public String getSectionSalarie() { return sectionSalarie; } public String getLibelle() { return sectionLibelle; } public void setLibelle(String libelle) { this.libelle=libelle; } public void setSectionSalarie(String selectionSalarie) { this.sectionSalarie=sectionSalarie; } }
Ou bien, si tu es sûr que tu n'auras jamais à modifier un libelle ou une sectionSalarie, qui sera un tout peu plus facile à manipuler (disons plutôt moins verbeux).
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 public class SectionSalarie { public final String id; public final String sectionSalarie; public final String libelle; public SectionSalarie(String id, String sectionSalarie, String libelle) { this.id=id; this.sectionSalarie=sectionSalarie; this.libelle=libelle; } }
Donc pour créer les instances de SectionSalarie depuis le résultat de la requête, afin de remplir la combo, tu peux parcourir le ResultSet :
Code:

1 2 3 4 5 while(( rs1.next() ) { comboBox_3.addItem(new Sectionsalarie(rs1.getString("id"), rs1.getString("SectionSalarie"), rs1.getString("Libelle")); }
La combo devra être créée comme ça :

Code:

JComboBox<SectionSalarie> comboBox_3 = new JComboBox<>();

Malheureusement, le typage ne permet pas de s'abstenir de caster, mais on peut récupérer un élément comme ça :

Code:

SectionSalarie sectionSalarie = (SectionSalarie)comboBox_3.getSelectedItem();

Ce qui fait qu'on récupère une instance d'une classe de l'objet métier qu'on manipule, et pas seulement une String, qui va pas nous aider à déterminer les propriétés de l'objet métie. Il faudrait faire des requêtes, ou utiliser des maps, ou encore parser peut-être, ou autres trucs plus ou moins tortueux pour récupérer les différentes informations, alors que là, on les a direct :
dans le premier cas de syntaxe présenté :

Code:

sectionSalarie.getLibelle(); // donne le libelle de la sectionSalarie

Dans le second ce serait :

Code:

sectionSalarie.libelle; // donne le libelle de la sectionSalarie

Le fait d'avoir créé la combo comme ça fait que dans le modèle de la combo, on a des instances de SectionSalarie au lieu de String plutôt limitée pour représenter autre chose que des chaînes de caractères. Et ces informations nous permettent justement par l'entremise du ListCellRender d'afficher ces informations comme on veut (sous forme d'une concaténation comme dans mon exemple, mais on pourait utiliser des couleurs, des styles de textes, des images ou n'importe quoi d'autres qu'une simple String).

1 pièce(s) jointe(s)

Bonjour,

Merci pour toutes ces explications, cela me permet de mieux comprendre ce que je fais.

je serais certainement amené à changer des sections et des libelles : voici donc ma classe SectionSalarie

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
package DB;
public class SectionSalarie {
 
    private final int id;
    private String sectionSalarie;
    private String libelle;
 
    public SectionSalarie(int id, String sectionSalarie, String libellesection) {
         this.id=id;
         this.sectionSalarie=sectionSalarie;
         this.libelle=libellesection;
    }
 
    public int getId() {
         return id;
    }
 
    public String getSectionSalarie() {
         return sectionSalarie;
    }
 
    public String getLibellesection() {
         return libelle;
    }
 
    public void setLibelle(String libelle) {
         this.libelle=libelle;
    }
 
    public void setSectionSalarie(String sectionSalarie) {
        this.sectionSalarie=sectionSalarie;
    }
 
}

ensuite le code de ma Jframe :

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
	JComboBox<SectionSalarie> comboBox_3 = new JComboBox<>();
		SectionSalarie sectionsalarie = (SectionSalarie)comboBox_3.getSelectedItem();
		comboBox_3.setBounds(530, 149, 135, 20);
		frmSaisieDesHeures.getContentPane().add(comboBox_3);
 
 
 
		try (PreparedStatement stmt1 = Connexion_BDD.get().prepareStatement("SELECT DISTINCT id,SectionSalarie,LibelleSection FROM MOD_SalariesPresents_Vue01 ")) {
			   ResultSet rs1 = stmt1.executeQuery();
 
			 while((rs1.next())) {
 
		          comboBox_3.addItem(new SectionSalarie(rs1.getInt("id"), rs1.getString("SectionSalarie"), rs1.getString("LibelleSection")));
 
		          sectionsalarie.getId();
			         sectionsalarie.getSectionSalarie();
			         sectionsalarie.getLibellesection();
			 }
			  rs1.close();
 
		} catch (SQLException e1) {
			// 
			e1.printStackTrace();
		}

Ma table :
Pièce jointe 221380

je récupère des caractères dans ma combo-box du style : DB.SECTIONSALARIE@

Merci beaucoup

Citation:

Envoyé par Alex1407

je récupère des caractères dans ma combo-box du style : DB.SECTIONSALARIE@

Il faut maintenant que tu ajoutes à ta combobox un ListCellRenderer pour afficher comme tu veux les SectionSalarie (il faut adapter la partie que je t'ai surlignée en jaune) :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 JComboBox<SectionSalarie> comboBox_3 = new JComboBox<>(); comboBox_3 .setRenderer(new DefaultListCellRenderer() { @Override public Component getListCellRendererComponent(JList<?> list, Object value, int index, boolean isSelected, boolean cellHasFocus) { if ( value instanceof SelectionSalarie ) { SelectionSalarie sectionSalarie = (SelectionSalarie ) value; setText(sectionSalarie.getLibellesection() + " ("+sectionSalarie.getSectionSalarie()+")"); // par exemple } return this; } });
Au passage, le SectionSalarie sectionsalarie = (SectionSalarie)comboBox_3.getSelectedItem(); que tu as mis juste après le JComboBox<SectionSalarie> comboBox_3 = new JComboBox<>(); ne sert à rien : la combo vient juste d'être créée (vide en plus), il ne risque pas d'y avoir quelque chose de sélectionné.

Super cela fonctionne , je me penche sur la partie lien entre mes combo-box maintenant.

Je voulais avoir s'il était possible de désactiver le "survol" de la souris sur des Jradiobuttons ?
Quand je survol le oui par exemple il me l'active alors qu'il n'est pas cocher.
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 rdbtnNon.setSelected(true); textArea.setEnabled(false); comboBox_9.setEnabled(false); rdbtnOui.addChangeListener(new ChangeListener() { public void stateChanged(ChangeEvent arg0) { rdbtnNon.setSelected(!rdbtnOui.isSelected()); textArea.setEnabled(true); comboBox_9.setEnabled(true); } }); rdbtnNon.addChangeListener(new ChangeListener() { public void stateChanged(ChangeEvent e) { rdbtnOui.setSelected(!rdbtnNon.isSelected()); textArea.setEnabled(false); comboBox_9.setEnabled(false); } });
merci

28/09/2016, 11h03
joel.drigo

Tu ne devrais pas utiliser cet écouteur (ou alors il faut gérer finement les différents états - il n'y a pas que "selectionné"). ActionListener permet de réagir directement au changement de sélection.
D'autre part, tu n'as pas gérer toi-même la sélection des autres radios : utilise un javax.swing.ButtonGroup qui le fera pour toi. Ainsi, dans l'ActionListener tu n'as plus cas gérer les états des autres composants (JTextArea et JComboBox).

Merci , je vais regarder sa.

J'essaie de faire le lien entre mes combo-box pour le moment.
Il faut que je crée une classe pour chaque combobox ?

Pour la combobox3 j'ai déjà la classe Sectionsalarie donc je peux la réutiliser ?
Dans ma jframe j'ai :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 JComboBox<Salarie> comboBox_1 = new JComboBox<>(); JComboBox<Salarie> comboBox_2 = new JComboBox<>(); JComboBox<SectionSalarie> comboBox_3 = new JComboBox<>(); Panel.add(comboBox_1); Panel.add(comboBox_2); Panel.add(comboBox_3); comboBox_1.addItemListener(e->selectCombo1(combo1,combo2)); comboBox_2.addItemListener(e->selectCombo2(combo2,combo3)); comboBox_3.addItemListener(e->selectCombo3(combo3,indiceField)); selectcomboBox_1(comboBox_1, comboBox_2); selectcomboBox_2(comboBox_2, comboBox_3);
Merci

28/09/2016, 13h47
joel.drigo

Citation:

Envoyé par Alex1407

Pour la combobox3 j'ai déjà la classe Sectionsalarie donc je peux la réutiliser ?

Je suppose que les tables qui nourrissent chaque combo sont différentes, n'est-ce-pas ? Donc, forcément, ce n'est pas la même classe. J'ai dit une classe par table, vu que la classe est là pour représenter les données d'une table. A moins éventuellement que les tables aient les mêmes données de même sens, mais ce serait mieux que la classe ait un nom sémantiquement correcte (faire une abstraction dans ce cas et étendre vers 2 classes avec chacune un nom sémantique).

j'utilise la même table pour mes 3 combo-box

classe salarie :

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
 
public class Matricule {
 
 
	private int id;
	private String matricule;
 
	public Matricule(int id,String matricule){
		this.id=id;
		this.matricule=matricule;
 
	}
 
	public int getId(){
	return id;
	}
 
	public String getmatricule() {
        return matricule;
   }
 
	public void setMatricule(String matricule){
		this.matricule=matricule;
	}
}

classe matricule

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
 
public class Matricule {
 
 
	private int id;
	private String matricule;
 
	public Matricule(int id,String matricule){
		this.id=id;
		this.matricule=matricule;
 
	}
 
	public int getId(){
	return id;
	}
 
	public String getmatricule() {
        return matricule;
   }
 
	public void setMatricule(String matricule){
		this.matricule=matricule;
	}
}

et ma jframe

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
	JComboBox<Salarie> comboBox_1 = new JComboBox<>();
		JComboBox<Matricule> comboBox_2 = new JComboBox<>();
		JComboBox<SectionSalarie> comboBox_3 = new JComboBox<>();
 
		Panel.add(comboBox_1);
		Panel.add(comboBox_2);
		Panel.add(comboBox_3);
 
		comboBox_1.addItemListener(e->selectcomboBox_1(comboBox_1,comboBox_3));
		comboBox_2.addItemListener(e->selectcomboBox_2(comboBox_2,comboBox_3));
		comboBox_3.addItemListener(e->selectcomboBox_3(comboBox_3,indiceField));
 
		selectcomboBox_1(comboBox_1, comboBox_2);
		selectcomboBox_2(comboBox_2, comboBox_3);

merci

28/09/2016, 14h13
joel.drigo
Citation:

Envoyé par Alex1407

j'utilise la même table pour mes 3 combo-box

Pour alors dans ton premier code on voit trois requêtes sur trois tables différentes :
- MOD_SalariesPresents
- MOD_SectionSalarie
- SalariesPresents
28/09/2016, 14h25
Alex1407

Ah oui désoler, je ne l'ai pas précisé, sa à changer entre deux.

j'utilise cette base de données, toutes les infos des salaries sont dans celle ci: MOD_SalariesPresents_Vue01.
28/09/2016, 14h29
joel.drigo

Ok. Une classe peut suffire à représenter la donnée dans ce cas, avec tous les champs. Mais on pourrait faire des classes différentes si on veut des "vues" différentes.

Comment simplifier mes 3 combobox , est il possible d'avoir qu'une seul requête pour les 3 ?

ma classe :

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
 
public class Salarie {
 
	private int id;
	private String matricule;
	private String nometprenom;
 
	 public Salarie(int id, String matricule, String nometprenom) {
         this.id=id;
         this.matricule=matricule;
         this.nometprenom=nometprenom;
	 }
 
	 public int getId() {
         return id;
    }
 
    public String getmatricule() {
         return matricule;
    }
 
    public void setLibelle(String nometprenom) {
         this.setNometprenom(nometprenom);
    }
 
	public String getNometprenom() {
		return nometprenom;
	}
 
	public void setNometprenom(String nometprenom) {
		this.nometprenom = nometprenom;
	}
 
}

ma Jframe

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
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
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
 
	JPanel panel = new JPanel();
 
		JComboBox<Salarie> comboBox_1 = new JComboBox<>();
		JComboBox<Salarie> comboBox_2 = new JComboBox<>();
		JComboBox<SectionSalarie> comboBox_3 = new JComboBox<>();
 
		panel.add(comboBox_1);
		panel.add(comboBox_2);
		panel.add(comboBox_3);
 
		frmSaisieDesHeures.add(panel);
		String selection = combobox_1.getSelectedItem();
		String selectionCombo2 = (String) comboBox_2.getSelectedItem();
		comboBox_2.removeAllItems();
 
		comboBox_2.setSelectedItem(selectionCombo2); 
 
		comboBox_1.addItemListener(e->selectcomboBox_1(comboBox_1,comboBox_3));
		comboBox_2.addItemListener(e->selectcomboBox_2(comboBox_2,comboBox_3));
		comboBox_3.addItemListener(e->selectcomboBox_3(comboBox_3));
 
		selectcomboBox_1(comboBox_1, comboBox_2);
		selectcomboBox_2(comboBox_2, comboBox_3);
 
 
		JComboBox<String> comboBox_1 = new JComboBox<String>(new DefaultComboBoxModel<String>());
		comboBox_1.setBounds(276, 149, 160, 20);
		frmSaisieDesHeures.getContentPane().add(comboBox_1);
		PreparedStatement stmt1 = null;
		ResultSet rs1 = null;
 
		try {
			stmt1 = Connexion_BDD.get().prepareStatement("SELECT DISTINCT NomEtPrenom FROM MOD_SalariesPresents_Vue01 where id");
 
			rs1 = stmt1.executeQuery();
 
			while (rs1.next()) {
 
				comboBox_1.addItem(rs1.getString("NomEtPrenom"));
			}
 
			rs1.close();
		} catch (SQLException e1) {
			// 
			e1.printStackTrace();
		}
 
		//JComboBox<String> comboBox_2 = new JComboBox<String>();
		comboBox_2.setBounds(81, 149, 113, 20);
		frmSaisieDesHeures.getContentPane().add(comboBox_2);
 
		try(PreparedStatement stmt2 = Connexion_BDD.get().prepareStatement("SELECT Matricule FROM MOD_SalariesPresents_Vue01")){
 
			ResultSet rs2 = stmt2.executeQuery();
 
			while (rs2.next()) {
 
				comboBox_2.addItem(rs2.getString("Matricule"));
			}
 
			rs2.close();
		} catch (SQLException e1) {
			// 
			e1.printStackTrace();
		}
 
 
		JComboBox<SectionSalarie> comboBox_3 = new JComboBox<>();
		comboBox_3.setBounds(522, 149, 212, 20);
		frmSaisieDesHeures.getContentPane().add(comboBox_3);
		comboBox_3 .setRenderer(new DefaultListCellRenderer() {
 
			public DefaultListCellRenderer getListCellRendererComponent(JList<?> list, Object value, int index, boolean isSelected,
					boolean cellHasFocus) {
 
				if ( value instanceof SectionSalarie ) {
					SectionSalarie sectionSalarie = (SectionSalarie ) value;
					setText(sectionSalarie.getSectionSalarie()+"-"+sectionSalarie.getLibellesection()); 
				}
 
				 return this;
			}
 
 
		});
 
		try (PreparedStatement stmt3 = Connexion_BDD.get().prepareStatement("SELECT DISTINCT id,SectionSalarie,LibelleSection FROM MOD_SalariesPresents_Vue01 ")) {
			   ResultSet rs3 = stmt3.executeQuery();
 
			 while((rs3.next())) {
 
		          comboBox_3.addItem(new SectionSalarie(rs3.getInt("id"), rs3.getString("SectionSalarie"), rs3.getString("LibelleSection")));
		 SectionSalarie sectionsalarie = (SectionSalarie)comboBox_3.getSelectedItem();
 
		 sectionsalarie.getId();			     
		 sectionsalarie.getSectionSalarie();
		 sectionsalarie.getLibellesection();
 
			 } 
 
			  rs3.close();
 
		} catch (SQLException e1) {
			// 
			e1.printStackTrace();
		}

merci

Citation:

Envoyé par Alex1407

Comment simplifier mes 3 combobox , est il possible d'avoir qu'une seul requête pour les 3 ?

Si 2 combo sont "liées", c'est que les items de l'une dépendent d'un item dans l'autre (celui sélectionné), item dont l'id va te servir à faire à le where. Autant, la deuxième et la troisième pourrait être liées chacune à la première, et donc on pourrait avoir la même requête pour les deux (d'ailleurs, inutile d'exécuter 2 fois la requête : on remplit une liste qui sert à remplir les 2 combo), mais sinon il y a forcément au moins 2 requêtes différentes. Après, on peut imaginer avec une seule méthode :
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 List<Salarie> getSalaries(Integer id) { String sql = <le select from table> if ( id!=null) { sql+=" where id = ?"; } List<Salarie> list = new ArrayList<>(); Connection cnx = Connection.getConnection(); try { try(PreparedWhere pw = cnx.preparedWhere(sql)) { if ( id!=null ) { pw.setInteger(1, id); } ResultSet rs = pw.executeQuery(); while( rs.nex() ) { list.add( new Salarie( ... ) ); } } finally(){ Connection.release(cnx); } return list; }
Ainsi pour remplir la première combo tu auras getSalaries(null), et pour les autres getSalaries( selectedSalarie.getId() )

Bonjour,

j'essaie depuis hier soir de mettre en place les combo-box mais en vain. Rien ne fonctionne, les list ne sont pas reconnu...
Merci de votre aide

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
47
48
 
		JPanel panel = new JPanel();
 
		JComboBox<Salarie> comboBox_1 = new JComboBox<>();
		JComboBox<Salarie> comboBox_2 = new JComboBox<>();
		JComboBox<SectionSalarie> comboBox_3 = new JComboBox<>();
 
		panel.add(comboBox_1);
		panel.add(comboBox_2);
		panel.add(comboBox_3);
 
		frmSaisieDesHeures.add(panel);
		String selection = combobox_1.getSelectedItem();
		String selectionCombo2 = (String) comboBox_2.getSelectedItem();
		comboBox_2.removeAllItems();
 
 
		JComboBox<String> comboBox_1 = new JComboBox<String>(new DefaultComboBoxModel<String>());
		comboBox_1.setBounds(276, 149, 160, 20);
		frmSaisieDesHeures.getContentPane().add(comboBox_1);
 
		List<Salarie> list = new ArrayList<>();
		List<Salarie> getSalaries(Integer id) {
 
			   String sql = ("SELECT id,Matricule,NomEtPrenom,LibelleSection FROM MOD_SalariesPresents_Vue01");
			   if ( id!=null) {
			      sql+=" where id = ?";
			   }
 
 
			   Connection cnx = Connexion_BDD.get();
			   try {
			       try(PreparedStatement pw = Connexion_BDD.get().prepareStatement(sql)) {
 
			              if ( id!=null ) {
			                  pw.setInt(1, id);
			              }
			              ResultSet rs = pw.executeQuery();
			              while(rs.next()) {
			                    list.add(new Salarie(rs.getInt("id"),rs.getString("Matricule"),rs.getString("NomEtPrenom"),rs.getString("LibelleSection")));
			              }
 
			       }finally {
			      cnx.close();
			   }
			   return list;
			   }
		}

Ne me mets pas de bribes de code parce que je ne sais comment les interpréter. Si je prends ton dernier code, j'ai une séquence d'instructions dont je ne sais si elles sont dans une méthode ou pas. Si je suppose qu'elles sont dans une méthode, alors tu as une méthode détfinie dans une méthode (List<Salarie> getSalaries(Integer id)) ce qui n'existe pas en Java.

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
47
48
49
50
51
52
53
54
55
56
57
public class JeSaisPasQuoi {
 
public void buildUI() {
JPanel panel = new JPanel();
 
		JComboBox<Salarie> comboBox_1 = new JComboBox<>();
		JComboBox<Salarie> comboBox_2 = new JComboBox<>();
		JComboBox<SectionSalarie> comboBox_3 = new JComboBox<>();
 
		panel.add(comboBox_1);
		panel.add(comboBox_2);
		panel.add(comboBox_3);
 
		frmSaisieDesHeures.add(panel);
		String selection = combobox_1.getSelectedItem();
		String selectionCombo2 = (String) comboBox_2.getSelectedItem();
		comboBox_2.removeAllItems();
 
 
		JComboBox<String> comboBox_1 = new JComboBox<String>(new DefaultComboBoxModel<String>());
		comboBox_1.setBounds(276, 149, 160, 20);
		frmSaisieDesHeures.getContentPane().add(comboBox_1);
 
		List<Salarie> list = getSalaries(null); // liste de tous les salaries
 
	        list.forEach(item-> comboBox_1.addItem(item)); // on ajoute tous les items de list dans la combo
 
 
}
 
		List<Salarie> getSalaries(Integer id) {
 
			   String sql = "SELECT id,Matricule,NomEtPrenom,LibelleSection FROM MOD_SalariesPresents_Vue01";
			   if ( id!=null) {
			      sql+=" where id = ?";
			   }
 
 
			   Connection cnx = Connexion_BDD.get();
 
			       try(PreparedStatement pw = Connexion_BDD.get().prepareStatement(sql)) {
 
			              if ( id!=null ) {
			                  pw.setInt(1, id);
			              }
			              ResultSet rs = pw.executeQuery();
			              while(rs.next()) {
			                    list.add(new Salarie(rs.getInt("id"),rs.getString("Matricule"),rs.getString("NomEtPrenom"),rs.getString("LibelleSection")));
			              }
 
			       }finally {
			      cnx.close();
 
			   return list;
			   } 
 
}

list.forEach(item-> comboBox_1.addItem(item)) est une façon de faire. Tu peux aussi itérer :

Code:

1
2
3
for(Salarie item : list) {
     comboBox_1.addItem(item);
}

ou encore plein d'autres solutions. Comme changer le modèle aussi : comboBox_1.setModel(new DefaultComboBoxModel<>(list.toArray(new Salarie[list.size()])));

29/09/2016, 13h22
Alex1407

Citation:

Ne me mets pas de bribes de code parce que je ne sais comment les interpréter

c'est quoi des bribes ? le code que j'ai mis est le code de ma jframe.

je ne sais pas ce que je peux enlever, j'essaye plein de chose et je m'y perd.

Citation:

Envoyé par Alex1407

c'est quoi des bribes ? le code que j'ai mis est le code de ma jframe.

Je veux dire indique moi au moins les signatures de méthodes, et l'accolade fermante. Si le code est trop long, tu mets que la partie intéressant et le reste tu mets des /** ... ici du code qui fait ci ou ça ... **/. Et précise-moi où se trouve cette méthode (le nom de la classe à minima).

Si je reprends ton dernier code, je rajoute en jaune ce que j'aurais préféré avoir,
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 47 48 49 50 51 52 53 54 55 56 57 58 59 60 public class MachinTruc { public void methodTrucBidule() { /** ... là du code qui fait des trucs qui ne concernent pas la question ... **/ JPanel panel = new JPanel(); JComboBox<Salarie> comboBox_1 = new JComboBox<>(); JComboBox<Salarie> comboBox_2 = new JComboBox<>(); JComboBox<SectionSalarie> comboBox_3 = new JComboBox<>(); panel.add(comboBox_1); panel.add(comboBox_2); panel.add(comboBox_3); frmSaisieDesHeures.add(panel); String selection = combobox_1.getSelectedItem(); String selectionCombo2 = (String) comboBox_2.getSelectedItem(); comboBox_2.removeAllItems(); JComboBox<String> comboBox_1 = new JComboBox<String>(new DefaultComboBoxModel<String>()); comboBox_1.setBounds(276, 149, 160, 20); frmSaisieDesHeures.getContentPane().add(comboBox_1); List<Salarie> list = new ArrayList<>(); List<Salarie> getSalaries(Integer id) { String sql = ("SELECT id,Matricule,NomEtPrenom,LibelleSection FROM MOD_SalariesPresents_Vue01"); if ( id!=null) { sql+=" where id = ?"; } Connection cnx = Connexion_BDD.get(); try { try(PreparedStatement pw = Connexion_BDD.get().prepareStatement(sql)) { if ( id!=null ) { pw.setInt(1, id); } ResultSet rs = pw.executeQuery(); while(rs.next()) { list.add(new Salarie(rs.getInt("id"),rs.getString("Matricule"),rs.getString("NomEtPrenom"),rs.getString("LibelleSection"))); } }finally { cnx.close(); } return list; } } /** ... là du code qui fait des trucs qui ne concernent pas la question ... **/ } }
Et donc là je vois tout de suite que ce que tu me montres ne peut pas compiler : il y a une méthode dans une méthode !

Alors qu'avec ton message, je me suis dit soit il a mis bout à bout deux codes qui n'ont rien à voir dans la même balise code, soit il a vraiment mis un code de méthode dans une méthode... c'est quand même mieux quand on n'est pas obligé de supposer toutes les possibilités. Enfin, si tu veux avoir une réponse directe et précise rapidement.
Accessoirement, ça me donne des renseignements annexes : si la méthode est static, si elle attend un paramètre en retour, etc... qui peuvent nous permettre de te donner une réponse encore plus précise, ou des alertes sur des problèmes que tu risques d'avoir.

29/09/2016, 15h19
Alex1407

3 pièce(s) jointe(s)

d'accord j'essayerais d'être plus clair.

dois-je mettre mon code dans une nouvelle classe ?

Je mets mes fichiers en pièces jointes pour être plus clair et ne pas dire de bêtise.

merci

Pas la peine de le mettre dans une autre classe (ce serait plus propre de séparer l'ui et le dao, mais ton programme a 3 classes, ce n'est pas un projet d'application professionnelle avec 25000 classes).

Mais surtout il faut que le code soit dans une méthode à part. Là, je vois que tu as simplement virer la signature de la méthode et l'accolade finale :
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 List<Salarie> list = getSalarie(null); // liste de tous les salaries list.forEach(item-> comboBox_1.addItem(item)); // on ajoute tous les items de list dans la combo String sql = ("SELECT id,Matricule,NomEtPrenom,LibelleSection FROM MOD_SalariesPresents_Vue01"); if ( id!=null) { sql+=" where id = ?"; } Connection cnx = Connexion_BDD.get(); try { try(PreparedStatement pw = Connexion_BDD.get().prepareStatement(sql)) { if ( id!=null ) { pw.setInt(1, id); } ResultSet rs = pw.executeQuery(); while(rs.next()) { list.add(new Salarie(rs.getInt("id"),rs.getString("Matricule"),rs.getString("NomEtPrenom"),rs.getString("LibelleSection"))); } }finally { cnx.close(); } return list;
Ça ne risque pas de mieux fonctionner. Ça ne risque pas de compiler puis que tu te retrouves à avec un return en plein milieu de ta méthode initialize(), ce qui est formellement interdit.

Je te remets le canevas de ce que devrais être ta classe Fenetre :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 public class Fenetre { public static void main(String[] args) { } public Fenetre() { initialize(); } private void initialize() { } private List<Salarie> getSalaries(Integer id) { } private List<SectionSalarie> getSectionSalaries(Integer id) { } }
Les méthodes (et le constructeur) sont bien séparés. Il n'y en a aucun l'un dans un autre. Maintenant, il te suffit de remettre le code dans les méthodes. Et dans la sous partie de initialize() qui suit, j'ai barrée ce qui devrait être dans private List<Salarie> getSalaries(Integer id) { //ICI }
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 JComboBox<String> comboBox_1 = new JComboBox<String>(new DefaultComboBoxModel<String>()); comboBox_1.setBounds(276, 149, 160, 20); frmSaisieDesHeures.getContentPane().add(comboBox_1); List<Salarie> list = getSalarie(null); // liste de tous les salaries list.forEach(item-> comboBox_1.addItem(item)); // on ajoute tous les items de list dans la combo String sql = ("SELECT id,Matricule,NomEtPrenom,LibelleSection FROM MOD_SalariesPresents_Vue01"); if ( id!=null) { sql+=" where id = ?"; } Connection cnx = Connexion_BDD.get(); try { try(PreparedStatement pw = Connexion_BDD.get().prepareStatement(sql)) { if ( id!=null ) { pw.setInt(1, id); } ResultSet rs = pw.executeQuery(); while(rs.next()) { list.add(new Salarie(rs.getInt("id"),rs.getString("Matricule"),rs.getString("NomEtPrenom"),rs.getString("LibelleSection"))); } }finally { cnx.close(); } return list;
Ensuite, pour les autres requêtes, tu devrais commencer à déplacer la partie qui concerne que la requête dans une méthode à part.
[entre parenthèse]Tu devrais d'ailleurs plus découper ton code en méthode, parce que ta méthode initialize() de 530 lignes, c'est vraiment trop long. Même quelqu'un d'expériementé va difficilement trouver où se trouve un problème lorsqu'il survient. Quand on découpe, on isole chaque partie, ce qui fait que lorsque quelque chose ne va pas, on se paluche pas 500 lignes, mais 40 tout au plus. Ensuite, on peut rapidement module (commenter une méthode, inverser l'appel de 2 méthodes, ajouter un élément intermédiaire, remplacer une méthode, par une méthode de test/debug, etc... en cas de problème). Avec 500 lignes tu peux de retrouver avec des commentaires dans tous le sens et ne plus savoir ce qu'il faut conserver, virer ou décommenter.[fin de la parenthèse]
Tu appliques le même principe plus ou moins (utilsier une classe pour représenter la donnée d'une table), retourner une liste. Remplir une combo avec cette liste.
Tu pourras ensuite mutualiser (fusionner 2 méthodes qui traitent la même tâble avec une requête légèrement différente (avec id ou sans, comme dans getSalarie()).

1 pièce(s) jointe(s)

Bonjour,

j'essaie de mieux organiser mon code.

Par exemple le code ci dessous , je dois le mettre dans quel parti de ma jframe puisque j'ai de l'interface et du code ?
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 JComboBox<String> comboBox_2 = new JComboBox<String>(); comboBox_2.setBounds(81, 149, 113, 20); frmSaisieDesHeures.getContentPane().add(comboBox_2); try(PreparedStatement stmt2 = Connexion_BDD.get().prepareStatement("SELECT Matricule FROM MOD_SalariesPresents_Vue01")){ ResultSet rs2 = stmt2.executeQuery(); while (rs2.next()) { comboBox_2.addItem(rs2.getString("Matricule")); } rs2.close(); } catch (SQLException e1) { // e1.printStackTrace(); }
Citation:

ce serait plus propre de séparer l'ui et le dao

je voulais séparer mon code mais je n'ai pas réussi de plus je n'ai jamais fait de programme avec une interface si imposante

Merci

C'est exactement le même principe que pour l'autre combo :

Il y a une partie ui
Code:

1 2 3 4 5 6 JComboBox<String> comboBox_2 = new JComboBox<String>(); comboBox_2.setBounds(81, 149, 113, 20); frmSaisieDesHeures.getContentPane().add(comboBox_2); // ici un appel à une méthode pour remplir la combo
La méthode (DAO) pour remplir la combo va être du genre :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 public List<String> getSalariesPresents()) { List<String> list = new ArrayList<>(); try(PreparedStatement stmt2 = Connexion_BDD.get().prepareStatement("SELECT Matricule FROM MOD_SalariesPresents_Vue01")){ ResultSet rs2 = stmt2.executeQuery(); while (rs2.next()) { list.addI(rs2.getString("Matricule")); } } catch (SQLException e1) { // e1.printStackTrace(); } return list; }
Ici j'ai fait une liste de String, parce que ça suffit à stocker un matribule. Ensuite, si tu veux pouvoir récupérer ou modifier autre chose que le matricule, fais une classe "SalariesPresents" et génére une List<SalariesPresnent>, comme dans l'autre cas qu'on a vue.

30/09/2016, 14h20
Alex1407

1 pièce(s) jointe(s)

Plus rien ne marche , il me souligne les List, le nom de mes combo box.

plus j'avance moins ca marche :(

merci

Dans ton fichier Fenetre3, à la fin de la classe il y a un bout de code d'une autre classe (est-ce vraiment comme ça dans ton code ou tu as tout concaténé dans le fichier juste pour mettre qu'un seul fichier joint ?) :
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 import java.awt.List; import java.sql.PreparedStatement; import java.sql.ResultSet; import java.sql.SQLException; import javax.swing.DefaultListCellRenderer; import javax.swing.JButton; import javax.swing.JComboBox; import javax.swing.JList; import DB.Connexion_BDD; public class SalariePresent { JComboBox<String> comboBox_2 = new JComboBox<String>(); comboBox_2.setBounds(81, 149, 113, 20); frmSaisieDesHeures.getContentPane().add(comboBox_2); try(PreparedStatement stmt2 = Connexion_BDD.get().prepareStatement("SELECT Matricule FROM MOD_SalariesPresents_Vue01")){ ResultSet rs2 = stmt2.executeQuery(); while (rs2.next()) { comboBox_2.addItem(rs2.getString("Matricule")); } rs2.close(); } catch (SQLException e1) { // e1.printStackTrace(); }
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 Jprivate List<SectionSalarie> getSectionSalaries(Integer id) { comboBox_3 .setRenderer(new DefaultListCellRenderer() { public DefaultListCellRenderer getListCellRendererComponent(JList<?> list, Object value, int index, boolean isSelected, boolean cellHasFocus) { if ( value instanceof SectionSalarie ) { SectionSalarie sectionSalarie = (SectionSalarie ) value; setText(sectionSalarie.getSectionSalarie()+"-"+sectionSalarie.getLibellesection()); } return this; } }); try (PreparedStatement stmt3 = Connexion_BDD.get().prepareStatement("SELECT DISTINCT id,SectionSalarie,LibelleSection FROM MOD_SalariesPresents_Vue01 ")) { ResultSet rs3 = stmt3.executeQuery(); while((rs3.next())) { comboBox_3.addItem(new SectionSalarie(rs3.getInt("id"), rs3.getString("SectionSalarie"), rs3.getString("LibelleSection"))); SectionSalarie sectionsalarie = (SectionSalarie)comboBox_3.getSelectedItem(); sectionsalarie.getId(); sectionsalarie.getSectionSalarie(); sectionsalarie.getLibellesection(); } rs3.close(); } catch (SQLException e1) { // e1.printStackTrace(); } }
Le premier bloc en jaune est uné sequence de code : on ne peut pas mettre une séquence de code dans le corps d'une classe, seulement dans une méthode, ou un constructeur éventuellement.
Le second bloc en jaune, c'est une méthode qui est censé retourner une liste de salariés, et elle commence par manipuler un combobox pour lui affecter un renderer ! Et la variable comboBox3 n'est pas définie en plus (elle l'est dans une autre méthode d'une autre classe; mais à l'endroit que tu l'utilises, c'est comme si elle n'existait pas). En l'occurence c'est du code d'UI, et je t'ai dit de le mettre juste après la création de la combo (new JComboBox() je veux dire).

Ensuite, il y a un mélange de code ici :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 try (PreparedStatement stmt3 = Connexion_BDD.get().prepareStatement("SELECT DISTINCT id,SectionSalarie,LibelleSection FROM MOD_SalariesPresents_Vue01 ")) { ResultSet rs3 = stmt3.executeQuery(); while((rs3.next())) { comboBox_3.addItem(new SectionSalarie(rs3.getInt("id"), rs3.getString("SectionSalarie"), rs3.getString("LibelleSection"))); SectionSalarie sectionsalarie = (SectionSalarie)comboBox_3.getSelectedItem(); sectionsalarie.getId(); sectionsalarie.getSectionSalarie(); sectionsalarie.getLibellesection(); } rs3.close(); } catch (SQLException e1) { // e1.printStackTrace(); }
Tu est dans la boucle de recupération d'informations du ResultSet, pour remplir une combo (qui n'existe pas), et tu fais appel à une selection de combo, dont tu appelles les getters juste pour le fun (parce le but d'appeler un getter c'est de récupérer une information, alors que là tu ignores simplement le retour des getters).
On manipule la sélection d'une combo lorsque la sélection change : donc sur écoute d'évenement. Certainement pas à l'endroit où on remplit la combo.

Je ne peux pas t'aider :

si tu n'as pas les bases : comment est fait une classe, où met-on le code, comment s'organisent les méthodes et le classes, comment appeler une méthode et récupérer un résultat produit par cette méthode, etc
si tu prends des blocs de code que tu mets n'importe où, en dépit de la logique aussi bien fonctionnelle qu'élémentaire.
Je t'ai montré une méthode pour récupérer les données lues dans la base par une requête dans une liste, je t'ai montré différentes façon d'utiliser cette liste pour remplir une JComboBox, je t'ai donné un petit programme ou on voit comment réagir à la sélection d'une combo pour l'exploiter pour faire quelque chose (en l'occurence changer les items d'une combo). Je vois bien que tu es perdu, mais si tu ne découpes pas ton problème en éléments séparés, ou que tu le fais mais que tu mélanges des bouts de ces éléments séparés ensembles, je ne vois pas trop comment je peux t'aider. Si je t'ai montré comment récupérer le résultat d'une requête dans une liste d'une part et d'autre part comment remplir une combo d'autre part, c'est bien pour que ces notions restent séparées, et ne se mélange pas. Donc commence par ne pas les mélanger. Ensuite, programmer c'est assembler les différents élements sans les mélanger : une méthode en appelle une autre et récupère son resultat, une méthode d'une classe créé un objet d'une autre classe et en stocke la référence, les connexions entre méthodes, entre classes/instances, ou entre méthodes et classes, sont limitées et il n'y a pas de bout de l'un qui doit aller dans l'autre.

30/09/2016, 15h27
Alex1407

Merci beaucoup pour votre aide, a chaque fois que j'ai demandé de l'aide vous me l'avez apporté.

Mais le Java a eu raison de moi cette fois-ci, c'est trop compliquer pour moi, interfaces, pas interfaces etc....

Je mets résolu, le poste aidera surement d'autres personnes qui on une logique plus développer que moi.

Merci beaucoup en tout cas ;)
30/09/2016, 15h31
joel.drigo

Il ne faut pas pour autant abandonner. Et il n'y a rien de tellement spécifique à Java : dans la plupart des langages, tu auras les mêmes structures (et le code devra aller dans des méthodes, des procédures, des scripts, des sous-programmes, enfin des entités séparées et délimitées, quelque soit leur forme). MSAccess étant plus orienté BD, tu auras des outils interactifs qui te rendront les choses plus abordables si tu prefères (et probablement des menus pour lier automatiquement 2 combos).
30/09/2016, 16h26
Alex1407

je n'abandonne pas, je laisse le projet de coté quelque temps. Je me suis tourné vers le Java pour son côté professionnel, on peut faire pas mal de chose.
MS Access est pour moi pas du tout professionnel. De plus je suis en apprentissage et je me vois mal leur présenter quelque chose de "bidon".
Je vais reprendre les bases et à partir de là, je pourrais avancer.

Merci

Verification formulaire et requete d'insertion

Bonjour,

je suis en train de terminer mon application , mais j'ai quelque soucis pour l'insertion dans ma base de données. Dans le code que j'ai mis, je vérifie que mes jtextfield "tpsmachine, tpsmo, tpschgt, duree" ne sont pas vide,je fais appel a la méthode getTimeWithStringValue qui convertie leur contenue. Puis je vérifie le contenue des autres jtexfield et j’insère dans la base.je problème c'est que j'insère dans la base alors que tous les champs ne sont pas remplis.
Je voudrais tous vérifier dans une seul boucle et sortir que quand le formulaire est bien remplis par l'utilisateurs et faire l'insertion dans la base ? je sais pas si ma démarche est correcte. j'ai essayer plusieurs chose sans succès.

Merci de votre aide

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
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
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
btnEnregistrer.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent arg0) {
				if (tpsmachine.getText().length() != 0 && tpsmo.getText().length() != 0
						&& tpschgt.getText().length() != 0 && duree.getText().length() != 0) {
					tpsmachine.setText("" + getTimeWithStringValue(tpsmachine.getText()));
					tpsmo.setText("" + getTimeWithStringValue(tpsmo.getText()));
					tpschgt.setText("" + getTimeWithStringValue(tpschgt.getText()));
					duree.setText("" + getTimeWithStringValue(duree.getText()));
				} else {
 
					JOptionPane.showMessageDialog(null, "Veuillez remplir les champs horaire");
					return;
				}
 
				PreparedStatement ps = null;
 
				try {
					/**
                                         * Permet de controle si tous les champs sont remplis
                                         */
					if (jtextfieldAutoComplete.getText().trim().length() > 0 && textField_1.getText().trim().length()>0 && textField_2.getText().trim().length()>0 &&
						  automachine.getText().trim().length() >0  &&tpschgt.getText().trim().length() >0 && 
						 complet.getText().trim().length() >0 && quantite.getText().trim().length() >0 && nbarret.getText().trim().length() >0 &&
						tpsmachine.getText().trim().length() >0 && tpsmo.getText().trim().length()>0 )
					{		
					ps = Connexion_BDD.get().prepareStatement(
					"insert into dbo.MODTESTTABLE ([Dates],[NomEtPrenom], [Matricule], [SectionSalarie], [NomRecouvrement], [MatriculeRecouvrement], [Poste], [ComPoste], [TravailExcept], [ComTravailExcept], [TempsDejeuner], [Machine], [MCUmachine], [TempsChgt], [CodeArticle], [Quantite], [Nbarret], [TempsMachine], [TempsMO]) values (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)");
 
					ps.setString(1, (((JTextField) dateChooser.getDateEditor().getUiComponent()).getText()));
					ps.setString(2, (String) jtextfieldAutoComplete.getText());
					ps.setString(3, (String) textField_1.getText());
					ps.setString(4, (String) textField_2.getText());
					ps.setString(5, (String) nom.getText());
					ps.setString(6, (String) matricule.getText());
					ps.setString(7, (String) comboBox_1.getSelectedItem());
					ps.setString(8, (String) temps_travail.getText());
					ps.setString(9, (String) comboBox_2.getSelectedItem());
					ps.setString(10, (String) texte.getText());
					ps.setString(11, (String) duree.getText());
					ps.setString(12, (String) automachine.getText());
					ps.setString(13, (String) mcu.getText());
					ps.setString(14, (String) tpschgt.getText());
					ps.setString(15, (String) complet.getText());
					ps.setString(16, (String) quantite.getText());
					ps.setString(17, (String) nbarret.getText());
					ps.setString(18, (String) tpsmachine.getText());
					ps.setString(19, (String) tpsmo.getText());
 
					ps.executeUpdate();
					ps.close();
 
					UpdateTable();
 
					}
					else
					{
						JOptionPane.showMessageDialog(null,"Tous les Champs doivent etre remplis");
 
 
						if(jtextfieldAutoComplete.getText().trim().length() > 0 && textField_1.getText().trim().length()>0 && textField_2.getText().trim().length()>0 &&
						  automachine.getText().trim().length() >0  &&tpschgt.getText().trim().length() >0 && 
						 complet.getText().trim().length() >0 && quantite.getText().trim().length() >0 && nbarret.getText().trim().length() >0 &&
						tpsmachine.getText().trim().length() >0 && tpsmo.getText().trim().length()>0 ){
 
						}else{
 
							ps = Connexion_BDD.get().prepareStatement(
									"insert into dbo.MODTESTTABLE ([Dates],[NomEtPrenom], [Matricule], [SectionSalarie], [NomRecouvrement], [MatriculeRecouvrement], [Poste], [ComPoste], [TravailExcept], 
                                                            [ComTravailExcept], [TempsDejeuner], [Machine], [MCUmachine], [TempsChgt], [CodeArticle], [Quantite], [Nbarret], [TempsMachine], [TempsMO]) values (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, 
                                                            ?, ?)");
 
 
							ps.setString(1, (((JTextField) dateChooser.getDateEditor().getUiComponent()).getText()));
							ps.setString(2, (String) jtextfieldAutoComplete.getText());
							ps.setString(3, (String) textField_1.getText());
							ps.setString(4, (String) textField_2.getText());
							ps.setString(5, (String) nom.getText());
							ps.setString(6, (String) matricule.getText());
							ps.setString(7, (String) comboBox_1.getSelectedItem());
							ps.setString(8, (String) temps_travail.getText());
							ps.setString(9, (String) comboBox_2.getSelectedItem());
							ps.setString(10, (String) texte.getText());
							ps.setString(11, (String) duree.getText());
							ps.setString(12, (String) automachine.getText());
							ps.setString(13, (String) mcu.getText());
							ps.setString(14, (String) tpschgt.getText());
							ps.setString(15, (String) complet.getText());
							ps.setString(16, (String) quantite.getText());
							ps.setString(17, (String) nbarret.getText());
							ps.setString(18, (String) tpsmachine.getText());
							ps.setString(19, (String) tpsmo.getText());
 
							ps.executeUpdate();
							ps.close();
 
							UpdateTable();
						}
					}
 
 
				} catch (SQLException e) {
					e.printStackTrace();
 
					JOptionPane.showMessageDialog(null,
							"Erreur lors de l'enregistrement ! Veuillez contactez l'administrateur");
					return;
 
				}
				/**
                                 * PERMET LE RESET DU FORMULAIRE
                                 */
 
				dateChooser.setDate(new Date());
				jtextfieldAutoComplete.setText("");quantite.setText("");
				textField_1.setText("");textField_2.setText("");
				nom.setText("");matricule.setText("");
				comboBox_1.setSelectedIndex(0);temps_travail.setText("");
				comboBox_2.removeAllItems();texte.setText("");
				duree.setText("");; automachine.setText("");
				mcu.setText("");tpschgt.setText("");
				complet.setText("");quantite.setText("");
				nbarret.setText(""); tpsmachine.setText("");
				tpsmo.setText("");
			}
 
		});