Discussion :

[Moux88]

Bonjour je lis la valeur "Red" de chaque pixel sous une surface.
Je mets toutes ces valeurs dans une ArrayList et j'aimerais faire la moyenne de toutes les valeurs.

Voici mon code :

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
 
ArrayList listRed = new ArrayList();
 for(int i = 0; i< B;i++){
            for(int j = 0; j< B - i; j++){
 
 
                int p = imgglobal.getRGB(i, j);
 
                int r = (p>>16)&0xff;
 
 
                listRed.add(r);
 
                System.out.println(""+ listRed);
            }
         }

J'ai essayé ceci mais sans succès j'ai deux erreurs :

Code :

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

1
2
3
4
5
 for(int i=0; i < listRed.length ; i++)
                sommeRed = sommeRed + listRed[i];
 
 
                double moyenne = sommeRed / listRed.length;

Une erreur pour length "Cannot find symbol"
Une erreur pour

Code :

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

sommeRed = sommeRed + listRed[i];

"array required, but Arraylist found"

merci bien

[wax78]

Tu déclare une ArrayList et tu l'utilise comme si c'était un simple tableau. Forcement le compilo il aime pas.

Les List (arraylist et autres) utilise .size() et .get(index) et non .length et [index].

[Moux88]

Ah super merci.

J'ai changé mon code

Code :

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

1
2
3
4
5
for(int i=0; i < listRed.size() ; i++)
                sommeRed = sommeRed + listRed.get(i);
 
 
                double moyenne = sommeRed / listRed.size();

Mais j'ai maintenant cette erreur :

bad operand types for binary operateur '+'

sommeRed est un "int" et listred.get(i) aussi non ?

[wax78]

Justement non, car tu ne l'as pas spécifié et le compilateur ne sais pas quoi faire.

Alors soit tu explicite clairement le type d'objet dans la liste comme ceci :

Code :

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

ArrayList<Integer> listRed = new ArrayList<>();

Soit tu fais un cast :

Code :

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

(int)listRed.get(i);

(Mais la première méthode est préférable)

[Moux88]

Ah ok je comprends maintenant, j'avais oublié qu'on pouvait la déclarer comme ça.
merci beaucoup.

[Alkhan]

Bonjour,

Justement non, car tu ne l'as pas spécifié et le compilateur ne sais pas quoi faire.

Alors soit tu explicite clairement le type d'objet dans la liste comme ceci :

Code :

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

ArrayList<Integer> listRed = new ArrayList<>();

Soit tu fais un cast :

Code :

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

(int)listRed.get(i);

(Mais la première méthode est préférable)

J'ajouterai une petite précision afin de les débutants ne se perdent pas !
Toutes les Collections ne stock pas de type simple mais bien des objets. Moux88, lorsque tu fais :

Code :

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

1
2
int r = 12;
listRed.add(r)

ce n'est pas un int qui est ajouter mais bien un Integer, qui lui est un objet !
le code équivalent serait :

Code :

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

1
2
int r = 12;
listRed.add(Integer.valueOf(r));

Mais ça c'est fait implicitement et c'est ce que l'on appel l'autoboxing !