Récupérer les valeurs d'un JComboBox

**Lukoda** · 10/07/2012, 15h20

Bonjour,

Je suis en train d'essayer de créer une sorte de bloc note.
J'en suis donc a la partie pour changer la police, la taille et le style.

J'ai donc utiliser des jComboBox pour écrire les différentes polices, idem pour les tailles et les styles.

J'ai donc 2 JFrame, une pour la fenêtre bloc note normale et une pour la fenêtre de police.

Je cherche donc a savoir comment je pourrai récupérer la valeur sélectionnée dans le JComboBox, pour la transmettre sur ma fenêtre principale pour que la police change?

Je vous transmet donc des bouts de code:

Ma classe Police:

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
 public Police() {
 
        initComponents();
        choixPol.removeAllItems();
        choixStyle.removeAllItems();
        choixTaille.removeAllItems();
        this.setVisible(true);
        choixPol.addItem("Arial");
        choixPol.addItem("Cambria");
        choixPol.addItem("Impact");
        choixStyle.addItem("Gras");
        choixStyle.addItem("Italique");
        choixTaille.addItem("8");
        choixTaille.addItem("10");
        choixTaille.addItem("12");
        choixTaille.addItem("14");
        choixTaille.addItem("18");
        choixTaille.addItem("32");
}

Code :

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

1
2
3
4
  public String getPolice(){
            String po=choixPol.getSelectedItem().toString();
            return po;
        }

Ma classe Fenetre (que j'ai créée à partir de la palette):

Code :

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

1
2
3
4
5
6
7
8
9
public Fenetre() {
        this.setVisible(true);
        initComponents();
        initMenu();
        this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        Font cambri = new Font(pol.getPolice(), Font.BOLD, 12);
        jTextPane1.setFont(cambri);
       
    }

Ce que j'ai mis en gras est faux

Quelqu'un aurait-il la solution??

**Robin56** · 10/07/2012, 15h33

Envoyé par Lukoda

Ce que j'ai mis en gras est faux

Ça veut dire quoi ? Le code ne compile pas ? Le code ne procure aucun changement ? Le code émet une erreur à l'exécution ? ..

Peux tu nous détailler davantage les symptômes ?

**Lukoda** · 10/07/2012, 15h43

Ça ne marche pas (la police ne change pas) et ça me met cette erreur:

run:

Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
at editeurdetexte.EditeurTxt.<init>(EditeurTxt.java:41)
at editeurdetexte.EditeurTxt$8.run(EditeurTxt.java:315)
at java.awt.event.InvocationEvent.dispatch(InvocationEvent.java:251)
at java.awt.EventQueue.dispatchEventImpl(EventQueue.java:701)
at java.awt.EventQueue.access$000(EventQueue.java:102)
at java.awt.EventQueue$3.run(EventQueue.java:662)
at java.awt.EventQueue$3.run(EventQueue.java:660)
at java.security.AccessController.doPrivileged(Native Method)
at java.security.ProtectionDomain$1.doIntersectionPrivilege(ProtectionDomain.java:76)
at java.awt.EventQueue.dispatchEvent(EventQueue.java:671)
at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:244)
at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:163)
at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:151)
at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:147)
at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:139)
at java.awt.EventDispatchThread.run(EventDispatchThread.java:97)

**Robin56** · 10/07/2012, 15h48

Donc tu as une erreur NullPointerException à la ligne 41 de ton fichier EditeurTxt.java. Et c'est quoi cette ligne ?

**supergeoffrey** · 10/07/2012, 15h50

Peux-tu fournir plus de code?

Petit conseil en passant : Mets le

Code :

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

this.setVisible(true);

à la fin de ton constructeur.

**Lukoda** · 10/07/2012, 16h01

Ma ligne 41 est

Code :

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

Font cambri = new Font(pol.getPolice(), Font.BOLD, 12);

Merci Super Geoffrey j'ai suivie ton conseil

La classe Police:

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
public class Police extends javax.swing.JFrame {
        JComboBox combo = new JComboBox();
        Font cambria = new Font("Arial", Font.BOLD, 12);
        EditeurTxt edit;
 
    public Police() {
 
        initComponents();
        choixPol.removeAllItems();
        choixStyle.removeAllItems();
        choixTaille.removeAllItems();
        initMenu();
        choixPol.addItem("Arial");
        choixPol.addItem("Cambria");
        choixPol.addItem("Impact");
        choixStyle.addItem("Gras");
        choixStyle.addItem("Italique");
        choixTaille.addItem("8");
        choixTaille.addItem("10");
        choixTaille.addItem("12");
        choixTaille.addItem("14");
        choixTaille.addItem("18");
        choixTaille.addItem("32");
        this.setVisible(true);
 
    }
     public String getPolice(){
            String po=choixPol.getSelectedItem().toString();
            return po;
        }
 
 
 
    private void initMenu(){
            choixPol.addActionListener(new ActionListener() {
 
            @Override
            public void actionPerformed(ActionEvent e) {
                System.out.println(choixPol.getSelectedItem()); //je l'ai créé pour voir si ca me renvoyais ce qu'y est sélectionné dans le jcombobox et ca marche.
            }
        });
    }

La classe EditeurTxt:

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
public class EditeurTxt extends javax.swing.JFrame {
    Police pol;
    JTextPane saisie = new JTextPane();
    JScrollPane ascenceur = new JScrollPane(saisie);
 
    public EditeurTxt() {
        this.setVisible(true);
        initComponents();
        initMenu();
        this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
         Font cambri = new Font(pol.getPolice(), Font.BOLD, 12);
        jTextPane1.setFont(cambri);
 
    }
private void initMenu() {
 
enr.addActionListener(new ActionListener() {
 
            @Override
            public void actionPerformed(ActionEvent e) {
                JFileChooser jfc = new JFileChooser();
                jfc.setCurrentDirectory(new File("."));
                String approve = "Enregistrier";
                int res = jfc.showDialog(jfc, approve);
                if (res == JFileChooser.APPROVE_OPTION) {
                    String monFichier = jfc.getSelectedFile().toString();
                    if (monFichier.endsWith(".txt") || monFichier.endsWith(".TXT")) {;
                    } else {
                        monFichier = monFichier + ".txt";
                    }
                    try {
                        FileWriter fw = new FileWriter(monFichier);
                        try (BufferedWriter bw = new BufferedWriter(fw)) {
                            bw.write(jTextPane1.getText());
                        }
                    } catch (IOException ex) {
                        Logger.getLogger(EditeurTxt.class.getName()).log(Level.SEVERE, null, ex);
                    }
 
                }
            }
        });
 
quit.addActionListener(new ActionListener() {
 
            @Override
            public void actionPerformed(ActionEvent e) {
                System.exit(0);
            }
        });
poli.addActionListener(new ActionListener() {
            private boolean JMenu;
 
            @Override
            public void actionPerformed(ActionEvent e) {
                //if((JButton)e.getSource()==bouton;
                if(e.getSource()==poli){
                    new Police().setVisible(true);
                }
            }
        });
 
        this.pack();
    }

**Robin56** · 10/07/2012, 16h12

Donc comme on peut le voir dans cet extrait :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
public class EditeurTxt extends javax.swing.JFrame {
    Police pol;
    JTextPane saisie = new JTextPane();
    JScrollPane ascenceur = new JScrollPane(saisie);
 
    public EditeurTxt() {
        this.setVisible(true);
        initComponents();
        initMenu();
        this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
         Font cambri = new Font(pol.getPolice(), Font.BOLD, 12);
        jTextPane1.setFont(cambri);
 
    }

Tu n'initialises pas l'objet Police "pol" et tu l'utilises après en faisant "pol.getPolice()" donc oui ça va péter et tu vas avoir une NullPointerException.

**Lukoda** · 10/07/2012, 16h19

Heu d'accord, mais quand tu dis initialiser je dois faire quoi? Mettre "Police pol = null" ?

**Lukoda** · 10/07/2012, 16h38

Je n'ai plus d'erreurs mais ca ne marche toujours pas. J'ai changé ma classe EditeurTxt():

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 class EditeurTxt extends javax.swing.JFrame {
    Police polc=null;
    JTextPane saisie = new JTextPane();
    JScrollPane ascenceur = new JScrollPane(saisie);
 
    private EditeurTxt() {
        this.setVisible(true);
        initComponents();
        initMenu();
    }
    public EditeurTxt(Police pol) {
        this.polc= pol;
        this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        Font cambri = new Font(polc.getPolice(), Font.BOLD, 12);
        jTextPane1.setFont(cambri);
 
    }

Je ne trouve toujours pas mon erreur...

**supergeoffrey** · 10/07/2012, 16h52

Envoyé par Lukoda

Heu d'accord, mais quand tu dis initialiser je dois faire quoi? Mettre "Police pol = null" ?

Instancier c'est créer un objet donc "Police pol = new Police()" sinon tu vas avoir des NullPointerException

**Lukoda** · 10/07/2012, 17h00

Merci pour ta réponse, cependant, quand je suis ta méthode, lorsque je lance mon programme, mes 2 fenêtres s'ouvrent simultanément...

Avec le code que j'ai mis précédemment je n'ai plus de NullPointerException, mais ça ne change toujours pas ma police...alors que mon System.out.println renvoie correctement le nom de la police que j'ai sélectionné dans le JComboBox...

Je ne comprend pas. L'erreur vient probablement de mon getPolice qu'en pensez-vous?

**Robin56** · 10/07/2012, 17h03

Envoyé par Lukoda

Heu d'accord, mais quand tu dis initialiser je dois faire quoi? Mettre "Police pol = null" ?

Non justement. NullPointerException veut bien dire ce qui est écris, c'est à dire que l'on fait référence à un élément d'un objet qui est null.

Si tu mets pol = null, et que tu appelles pol.getPolice(), c'est comme si tu faisais null.getPolice(). Tu fais donc référence à une méthode getPolice() sur un objet non initialisé. Il faut donc initialiser un objet via l'instruction new pour pouvoir faire des références dessus (sauf certains cas particuliers).
(cf. réponse de supergeoffrey)

**supergeoffrey** · 10/07/2012, 17h17

Envoyé par Lukoda

Code :

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

1
2
3
4
5
 
            @Override
            public void actionPerformed(ActionEvent e) {
                System.out.println(choixPol.getSelectedItem()); //je l'ai créé pour voir si ca me renvoyais ce qu'y est sélectionné dans le jcombobox et ca marche.
            }

Ton action fonctionne, il faudrait changer l'affichage par du fonctionnel

**Lukoda** · 10/07/2012, 17h19

Merci pour vos réponses.

Oui Supergeoffrey, c'est ce que j'essaie de faire, mais choixPol est dans la classe Police. Donc ça me complique les choses.

**supergeoffrey** · 10/07/2012, 18h38

Je trouve pas choixpol dans le code que tu nous as fourni

si choixPol est un attribut ajoute un accesseur dessus comme ci dessous en changeant "ClasseChoixPol" par son vrai type.

Code :

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

1
2
3
4
 
public ClasseChoixPol getChoixPol(){
  return choixPol;
}

et tu peux y accéder dans police en faisant

Code :

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

polc.getChoixPol()

**Lukoda** · 11/07/2012, 09h22

ChoixPol est le nom de ma JComboBox1.

Code :

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

1
2
3
choixPol.addItem("Arial");
        choixPol.addItem("Cambria");
        choixPol.addItem("Impact");

J'ai essayé ce code hier.
Dans Police:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
     public String getPolice(){
         String font="Cambria";
            if(choixPol.getSelectedItem()=="Cambria"){
                font="Cambria";
            }if(choixPol.getSelectedItem()=="Arial"){
                font="Aria";
            }if(choixPol.getSelectedItem()=="Impact"){
                font="Impact" ;
            }
            return font; 
            }

EditeurTxt:

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
public class EditeurTxt extends javax.swing.JFrame {
    Police polc= new Police();
    JTextPane saisie = new JTextPane();
    JScrollPane ascenceur = new JScrollPane(saisie);
 
    private EditeurTxt() {
        polc.setVisible(false);
        this.setVisible(true);
        initComponents();
        initMenu();
        this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        Font laPolice = new Font(polc.getPolice(), Font.BOLD, 26);
        jTextPane1.setFont(laPolice);
 
    }

Jusque là, mon bloc note ne prenait même pas le style (gras) et la taille que je lui avait donné avec mon "polc.getPolice()".
Avec ce code, mon texte est bien de taille 26 et gras, mais il ne prend toujours pas la police...

**Mathieu.J** · 11/07/2012, 11h53

Bonjour,

On ne compare pas les Strings avec == mais avec .equals()

En plus il y a une erreur de frappe :

Code :

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

font="Aria";  // il manque le L

Je ne comprends pas l'intérêt de tous tes IF, pourquoi ne pas faire directement :

Code :

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

1
2
3
4
String fontName = choixPol.getSelectedItem();
if(fontName == <Valeur "non sélectionnée", peut être NULL>) { // Utiliser equals si la valeur <non sélectionnée> est != NULL
    fontName = "Cambria"; // valeur par défaut
}

Récupérer les valeurs d'un JComboBox

AWT/Swing Java

Discussions similaires

Partager

Partager