[jCheckBox] Probleme avec isSelected()

Bonjour a tous

J'utilise plusieurs jCheckBox et lorsqu'elle sont cochées je veu faire une action.
Pour savoir si elle sont cochées je fait
Code:

1 2 3 4 5 6 7 8 Porcentage pourcent = new pourcentage(); ... if(Pourcent.jCheckBox1.isSelected() == true){ .... }
Le probleme c'est que quelle soit cochées ou non , le test n'est pas validé :cry: .
Si quelqu'un c'est quoi faire dites le moi

merci

à quoi correspond ton objet Pourcentage? c'est lui qui contient tes ckeckBox?
si ce n'est pas le cas, c'est de la que vient ton problème...

Pourcentage est la classe ou sont declarer les jCheckBox

Code:

1
2
3
4
5
6
7
8
 
public class pourcentage extends JFrame{
 
jCheckBox1 = new jCheckBox1();
 
public pourcentage(){
 
jCheckBox1.setText("Bleu");

C'est bien la bonne méthode pour savoir si un JCheckBox est sélectionné -> l'erreur est ailleurs, poste un peu plus de code qu'on voit la betise...

Pa contre, ton code a l'air zarb' :

Code:

jCheckBox1 = new jCheckBox1();

faute de frappe ou c'est un objet à toi qui fait office de checkBox ?

Sinon, pèle mèle : nom de classe commence par une majuscule, nom de variable par une minuscule, et c'est pas super de nommer une variable en préfixant par son type swing (pour le cas où ça devrait changer) : c'est pas mal (je trouve) de préfixer par le type, mais préfère "cb" par exemple à la place de "JCheckBox".

1erement, la methode isSelected renvoie un booleen, donc pour faire le test, suffit de faire :
Code:

1 2 if(Pourcent.jCheckBox&.isSelected())
Mais je pense pas que ton erreur vienne de là.
Comme te l'a demande NikauSenpeï, poste le code autour de ton test.

Code:

1 2 3 4 5 6 7 Porcentage pourcent = new pourcentage(); ... if(Pourcent.jCheckBox1.isSelected() == true){ .... }
Pourquoi t'as mis une majuscule à P de Pourcent dans le if()
alors que quand tu déclares l'objet c'est p minuscule ??
Code:

1 2 Porcentage pourcent = new pourcentage();

alors la class Pourcentage
c'est la ou je declare les different objet de ma fenetre et l'action a effectuer en cliquant sur les boutons

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
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
 
public class MonPourcentage extends JFrame {
  JPanel contentPane;
  GridLayout gridLayout1 = new GridLayout();
  JPanel jPanel1 = new JPanel();
  JPanel jPanel2 = new JPanel();
  JPanel jPanel3 = new JPanel();
  JPanel jPanel4 = new JPanel();
  JButton jButton1 = new JButton();
  JCheckBox jCheckBox1 = new JCheckBox();
  GridLayout gridLayout2 = new GridLayout();
  JCheckBox jCheckBox2 = new JCheckBox();
  JCheckBox jCheckBox3 = new JCheckBox();
  JCheckBox jCheckBox4 = new JCheckBox();
  JCheckBox jCheckBox5 = new JCheckBox();
  JCheckBox jCheckBox6 = new JCheckBox();
  JCheckBox jCheckBox7 = new JCheckBox();
  JCheckBox jCheckBox8 = new JCheckBox();
  JCheckBox jCheckBox9 = new JCheckBox();
  JCheckBox jCheckBox10 = new JCheckBox();
  JButton jButton2 = new JButton();
  GridLayout gridLayout3 = new GridLayout();
  JLabel jLabel1 = new JLabel();
  JLabel jLabel6 = new JLabel();
  JLabel jLabel7 = new JLabel();
  JLabel jLabel2 = new JLabel();
  JLabel jLabel8 = new JLabel();
  JLabel jLabel3 = new JLabel();
  JLabel jLabel10 = new JLabel();
  JLabel jLabel4 = new JLabel();
  JLabel jLabel5 = new JLabel();
  JLabel jLabel9 = new JLabel();
  private Image imgCap = null;
  public ISOLUX iso = null;
 
  public MonPourcentage(){}
 
  public MonPourcentage(Image img){
    super();
    this.imgCap = img;
    jButton1.addMouseListener(new MonPourcentage_jButton1_mouseAdapter(this));
    jButton2.addMouseListener(new MonPourcentage_jButton2_mouseAdapter(this));
    setVisible(true);
 
    contentPane = (JPanel) this.getContentPane();
    gridLayout3.setColumns(4);
    gridLayout3.setRows(3);
    jPanel4.setLayout(gridLayout3);
    jButton2.setText("Calculer");
    jCheckBox10.setText("Bleu C.");
    jCheckBox9.setText("Noir");
    jCheckBox8.setText("Vert F.");
    jCheckBox7.setText("Jaune");
    jCheckBox6.setText("Orange");
    jCheckBox5.setText("Violet");
    jCheckBox4.setText("Bleu F.");
    jCheckBox3.setText("Turquoise");
    jCheckBox2.setText("Vert C.");
    gridLayout2.setColumns(2);
    gridLayout2.setRows(5);
    jPanel2.setLayout(gridLayout2);
    jCheckBox1.setText("Rouge");
    jButton1.setText("Choisir la Référence");
    gridLayout1.setColumns(1);
    gridLayout1.setRows(4);
    contentPane.setLayout(gridLayout1);
    this.setSize(new Dimension(213, 263));
    this.setTitle("Pourcentage");
    contentPane.add(jPanel1, null);
    jPanel1.add(jButton1, null);
    contentPane.add(jPanel2, null);
    jPanel2.add(jCheckBox1, null);
    jPanel2.add(jCheckBox6, null);
    jPanel2.add(jCheckBox7, null);
    jPanel2.add(jCheckBox2, null);
    jPanel2.add(jCheckBox8, null);
    jPanel2.add(jCheckBox3, null);
    jPanel2.add(jCheckBox10, null);
    jPanel2.add(jCheckBox4, null);
    jPanel2.add(jCheckBox5, null);
    jPanel2.add(jCheckBox9, null);
    contentPane.add(jPanel3, null);
    jPanel3.add(jButton2, null);
    contentPane.add(jPanel4, null);
    jPanel4.add(jLabel1, null);
    jPanel4.add(jLabel4, null);
    jPanel4.add(jLabel2, null);
    jPanel4.add(jLabel3, null);
    jPanel4.add(jLabel7, null);
    jPanel4.add(jLabel5, null);
    jPanel4.add(jLabel10, null);
    jPanel4.add(jLabel9, null);
    jPanel4.add(jLabel8, null);
    jPanel4.add(jLabel6, null);
 
    jLabel1.setForeground(Color.white);
    jLabel1.setText("??");
    jLabel2.setForeground(Color.white);
    jLabel2.setText("??");
    jLabel3.setForeground(Color.white);
    jLabel3.setText("??");
    jLabel4.setForeground(Color.white);
    jLabel4.setText("??");
    jLabel5.setForeground(Color.white);
    jLabel5.setText("??");
    jLabel6.setForeground(Color.white);
    jLabel6.setText("??");
    jLabel7.setForeground(Color.white);
    jLabel7.setText("??");
    jLabel8.setForeground(Color.white);
    jLabel8.setText("??");
    jLabel9.setForeground(Color.white);
    jLabel9.setText("??");
    jLabel10.setForeground(Color.white);
    jLabel10.setText("??");
 
    jLabel1.setBackground(new Color(Color.HSBtoRGB((float) 0.00, (float) 1.0, (float) 0.75)));
    jLabel1.setOpaque(true);
    jLabel4.setBackground(new Color(Color.HSBtoRGB((float) 0.09, (float) 1.0, (float) 0.85)));
    jLabel4.setOpaque(true);
    jLabel2.setBackground(new Color(Color.HSBtoRGB((float) 0.15, (float)1.0,(float)0.85)));
    jLabel2.setOpaque(true);
    jLabel3.setBackground(new Color(Color.HSBtoRGB((float) 0.21, (float) 1.0, (float) 0.75)));
    jLabel3.setOpaque(true);
    jLabel7.setBackground(new Color(Color.HSBtoRGB((float) 0.27, (float) 1.0, (float) 0.65)));
    jLabel7.setOpaque(true);
    jLabel5.setBackground(new Color(Color.HSBtoRGB((float) 0.45, (float) 1.0, (float) 0.75)));
    jLabel5.setOpaque(true);
    jLabel10.setBackground(new Color(Color.HSBtoRGB((float) 0.57, (float) 1.0, (float) 0.75)));
    jLabel10.setOpaque(true);
    jLabel9.setBackground(new Color(Color.HSBtoRGB((float) 0.66, (float) 1.0, (float) 0.75)));
    jLabel9.setOpaque(true);
    jLabel8.setBackground(new Color(Color.HSBtoRGB((float) 0.77, (float) 1.0, (float) 0.75)));
    jLabel8.setOpaque(true);
    jLabel6.setBackground(new Color(Color.HSBtoRGB((float) 0.83, (float) 1.0, (float) 0.25)));
    jLabel6.setOpaque(true);
 
 
  }
 
 
  void jButton1_mouseClicked(MouseEvent e) {
    Parcourir choisir = new Parcourir();
    boolean packFrame = false;
    if (packFrame) {
      choisir.pack();
    }
    else {
      choisir.validate();
    }
 
    choisir.setVisible(true);
 
  }
 
 
  void jButton2_mouseClicked(MouseEvent e) {
    MaComparaison cap = new MaComparaison(imgCap);
    this.iso = new ISOLUX(imgCap,new Dimension(imgCap.getHeight(null),imgCap.getWidth(null)));
    cap.pack();
  }
 
}
 
 
 
class MonPourcentage_jButton1_mouseAdapter extends java.awt.event.MouseAdapter{
  MonPourcentage adaptee;
 
  MonPourcentage_jButton1_mouseAdapter(MonPourcentage adaptee) {
    this.adaptee = adaptee;
  }
  public void mouseClicked(MouseEvent e) {
    adaptee.jButton1_mouseClicked(e);
  }
 
 
}
 
class MonPourcentage_jButton2_mouseAdapter extends java.awt.event.MouseAdapter {
  MonPourcentage adaptee;
 
  MonPourcentage_jButton2_mouseAdapter(MonPourcentage adaptee) {
    this.adaptee = adaptee;
  }
  public void mouseClicked(MouseEvent e) {
    adaptee.jButton2_mouseClicked(e);
  }
}

et ca c'est la classe ou j'utilise les test pour savoir si les JCheckBox sont cochées ou pas

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
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
 
package jwebcam;
 
import javax.swing.*;
import java.awt.*;
import java.awt.image.*;
import java.text.DecimalFormat;
 
 
public class Comparaison extends Thread{
  protected BufferedImage Cap = null;
  protected BufferedImage Ref = null;
  protected HSB[] TabCap = null;
  protected HSB[] TabRef = null;
  protected int width, height = 0;
 
  public Comparaison (Image image){
    this.Ref = ImagetoBufferedImage(new ImageIcon("reference.jpg").getImage());
    this.Cap = ImagetoBufferedImage(image);
 
    this.width = Cap.getWidth(null);
    this.height = Cap.getHeight(null);
 
    TabCap = new HSB[width*height];
    TabRef = new HSB[width*height];
  }
 
  /**convertie une Image en BufferedImage*/
   public BufferedImage ImagetoBufferedImage(Image img){
 
     if( img instanceof BufferedImage)
       return (BufferedImage)img;
     else{
       BufferedImage BI =new BufferedImage(img.getWidth(null),img.getHeight(null),BufferedImage.TYPE_INT_RGB);
 
       Graphics g= BI.createGraphics();
       g.drawImage(img,0,0,null);
       g.dispose();
       return BI;
     }
   }
 
   /**détermine les 3 couleurs rouge vert bleu et les transforme
  * en leur composante HSB
  */
 public HSB[] luminosité(int[] tabPix){
   int vert = 0, bleu = 0, rouge = 0;
   Color couleur = new Color(rouge,vert,bleu);
   float[] tab = new float[3];
   int pix=0;
   HSB[] tabResult = new HSB[width*height];
 
   for (int i = 0; i < width * height; i++) {
     pix = tabPix[i];
     bleu = pix & 0xff;
     vert = (pix >> 8) & 0xff;
     rouge =(pix >> 16) & 0xff;
     couleur.RGBtoHSB(rouge,vert,bleu,tab);
     tabResult[i] = new HSB(tab[0],tab[1],tab[2]);
     }
   return tabResult;
 }
 
 
 /**récupère la valeur de la composante RGB de chaque pixels*/
   public int[] RGBPixel(BufferedImage BI, int x, int y, int width, int height) {
 
     int tab[] = new int[width * height];
     BI.getRGB(x, y, width, height, tab, 0, width);
 
     return tab;
 
  }
 
 
  /**traçage de l'ISO_LUX*/
   public double[] ISO(HSB[] tabResult){
     int z = 0,x = 0, y = 0,d=0;
     double cptL1=0, cptL2=0, cptL3=0, cptL4=0, cptL5=0, cptL6=0, cptL7=0, cptL8=0, cptL9=0, cptL10=0, TotalPixels=0;
     double h=1.0;
     double[] tab = new double[10];
     //MesCouleur cool= new MesCouleur();
 
 
     for (int i = 0; i < height; i++) {
       for (int j = 0; j < width; j++) {
 
 
         if ( (tabResult[z].getLum()) >= 0.98 && tabResult[z].getLum()<= 1.0){
           this.Cap.setRGB(y, x, Color.HSBtoRGB( (float) 0.00, (float) 1.0, (float) 0.75));
           tab[0]=cptL1++;
         }
         if ( (tabResult[z].getLum()) >= 0.90 && tabResult[z].getLum() < 0.98){
           this.Cap.setRGB(y, x, Color.HSBtoRGB( (float) 0.09, (float) 1.0, (float) 0.85));
           tab[1]=cptL2++;
         }
         if ( (tabResult[z].getLum()) >= 0.80 && tabResult[z].getLum() < 0.90){
           this.Cap.setRGB(y, x, Color.HSBtoRGB((float)0.15,(float)1.0,(float)0.85));
           tab[2]=cptL3++;
         }
         if ( (tabResult[z].getLum()) >= 0.70 && tabResult[z].getLum() < 0.80){
           this.Cap.setRGB(y, x, Color.HSBtoRGB((float)0.21,(float)1.0,(float)0.75));
           tab[3]=cptL4++;
         }
         if ( (tabResult[z].getLum()) >= 0.60 && tabResult[z].getLum() < 0.70){
           this.Cap.setRGB(y, x, Color.HSBtoRGB((float)0.27,(float)1.0,(float)0.65));
           tab[4]=cptL5++;
         }
         if ( (tabResult[z].getLum()) >= 0.50 && tabResult[z].getLum() < 0.60){
           this.Cap.setRGB(y, x, Color.HSBtoRGB((float)0.45,(float)1.0,(float)0.75));
           tab[5]=cptL6++;
         }
         if ( (tabResult[z].getLum()) >= 0.40 && tabResult[z].getLum() < 0.50){
           this.Cap.setRGB(y, x, Color.HSBtoRGB((float)0.57,(float)1.0,(float)0.75));
           tab[6]=cptL7++;
         }
         if ( (tabResult[z].getLum()) >= 0.30 && tabResult[z].getLum() < 0.40){
           this.Cap.setRGB(y, x, Color.HSBtoRGB((float)0.66,(float)1.0,(float)0.75));
           tab[7]=cptL8++;
         }
         if ( (tabResult[z].getLum()) >= 0.20 && tabResult[z].getLum() < 0.30){
           this.Cap.setRGB(y, x, Color.HSBtoRGB((float)0.77,(float)1.0,(float)0.75));
           tab[8]=cptL9++;
         }
         if ( (tabResult[z].getLum()) >= 0.00 && tabResult[z].getLum() < 0.20){
           this.Cap.setRGB(y, x, Color.HSBtoRGB((float)0.83,(float)1.0,(float)0.25));
           tab[9]=cptL10++;
         }
 
         /*if((tabResult[z].getLum()==1.0) && (tabResult[z].getSat()==0.00) && (tabResult[z].getTeinte()==0.00)){
           tabC[d++] = new centre(y,x);
           //this.BI.setRGB(y, x, 16777215);
         }*/
 
         y++;
         z++;
         h=1.0;
      }
    y = 0;
    x++;
 
    }
    //saveJPEG("D:\\JBuilder\\travail\\JWebCam\\capture.jpg");
 
    return tab;
 
 
  }
 
 
  public void CalculePourcen(double[] tab1, double[] tab2){
    MonPourcentage Pour = new MonPourcentage();
    DecimalFormat df = new DecimalFormat("#0.##");
 
 
    if (Pour.jCheckBox1.isSelected() == false) {}
      else {
        Pour.jLabel1.setText("" + (df.format( (tab1[0] * 100) / tab2[0])));
      }
      if (Pour.jCheckBox2.isSelected() == false) {}
      else {
        Pour.jLabel1.setText("" + (df.format( (tab1[1] * 100) / tab2[1])));
      }
      if (Pour.jCheckBox3.isSelected() == false) {}
      else {
        Pour.jLabel1.setText("" + (df.format( (tab1[2] * 100) / tab2[2])));
      }
      if (Pour.jCheckBox4.isSelected() == false) {}
      else {
        Pour.jLabel1.setText("" + (df.format( (tab1[3] * 100) / tab2[3])));
      }
      if (Pour.jCheckBox5.isSelected() == false) {}
      else {
        Pour.jLabel1.setText("" + (df.format( (tab1[4] * 100) / tab2[4])));
      }
      if (Pour.jCheckBox6.isSelected() == false) {}
      else {
        Pour.jLabel1.setText("" + (df.format( (tab1[5] * 100) / tab2[5])));
      }
      if (Pour.jCheckBox7.isSelected() == false) {}
      else {
        Pour.jLabel1.setText("" + (df.format( (tab1[6] * 100) / tab2[6])));
      }
      if (Pour.jCheckBox8.isSelected() == false) {}
      else {
        Pour.jLabel1.setText("" + (df.format( (tab1[7] * 100) / tab2[7])));
      }
      if (Pour.jCheckBox9.isSelected() == false) {}
      else {
        Pour.jLabel1.setText("" + (df.format( (tab1[8] * 100) / tab2[8])));
      }
      if (Pour.jCheckBox10.isSelected() == false) {}
      else {
        Pour.jLabel1.setText("" + (df.format( (tab1[9] * 100) / tab2[9])));
      }
      Pour.pack();
 
  }
 
 
  /**thread de traitement*/
   public void run() {
 
      int tabLum[] = new int[width*height];
      int tabLum2[] = new int[width*height];
      double tab1[] = new double[10];
      double tab2[] = new double[10];
 
      tabLum=RGBPixel(Cap,0, 0, width, height);
      TabCap=luminosité(tabLum);
      tab1=ISO(TabCap);
 
      tabLum2=RGBPixel(Ref,0, 0, width, height);
      TabRef=luminosité(tabLum2);
      tab2=ISO(TabRef);
 
      CalculePourcen(tab1,tab2);
    }
 
}

voila en esperant que vous pourrez m'aider

merci

Dans ta methode CalculePourcen, tu crees une nouvelle instance de MonPourcentage. Or le constructeur que tu appelles ne fait rien.
Je pense que ce que tu esperais que ca fasse, c'est que tu recuperes une copie de l'objet existant.
Ce n'est pas du tout le cas, donc c'est normal les tests sur tes checkbox echouent vu que ce ne sont pas les bonnes que tu testes.

ok ca a marcher merci beaucoup

mais le truc c'est que ma classe MonPourcentage a deja ete apelle dans une autre de mes classe et je ne sais pas comment faire pour avoir acces aux parametre de mes objet de la classe MonPourcentage que j'ai appeller en premier depuis la classe Comparaison
Est ce que je doit les passer en parametres ou y a t il un autres moyens

merci encore

Je ne sais pas comment est la structure de ton appli, mais tu peux tout simplement passer en parametre de ta classe Comparaison l'instance de ta classe MonPourcentage qui est creee dans ton autre classe.

ok merci pour tout :wink: