Récupérer les valeurs d'un JComboBox

Version imprimable

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:

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:

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:

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?? :mrgreen:

10/07/2012, 14h33
Robin56

Citation:

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 ?
10/07/2012, 14h43
Lukoda

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

run:

Citation:

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)
10/07/2012, 14h48
Robin56

Donc tu as une erreur NullPointerException à la ligne 41 de ton fichier EditeurTxt.java. Et c'est quoi cette ligne ?
10/07/2012, 14h50
supergeoffrey

Peux-tu fournir plus de code?

Petit conseil en passant : Mets le

Code:

this.setVisible(true);

à la fin de ton constructeur.

Ma ligne 41 est

Code:

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

Merci Super Geoffrey j'ai suivie ton conseil :)

La classe Police:

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
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:

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();
    }

Donc comme on peut le voir dans cet extrait :
Code:

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.

10/07/2012, 15h19
Lukoda

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

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

Code:

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...

10/07/2012, 15h52
supergeoffrey

Citation:

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
10/07/2012, 16h00
Lukoda

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?
10/07/2012, 16h03
Robin56

Citation:

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)

Citation:

Envoyé par Lukoda

Code:

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

10/07/2012, 16h19
Lukoda

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.
10/07/2012, 17h38
supergeoffrey
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:

1 2 3 4 public ClasseChoixPol getChoixPol(){ return choixPol; }
et tu peux y accéder dans police en faisant

Code:

polc.getChoixPol()

ChoixPol est le nom de ma JComboBox1.

Code:

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

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

Code:

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:

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...

Bonjour,

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

En plus il y a une erreur de frappe :

Code:

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

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

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 }