Discussion :

[Emilie MARQUOIS-OGEZ]

Bonjour,

Pour commencer, merci à ceux qui m'ont déjà aidé.
Ensuite, voici le code modifié. Pouvez-vous me dire si ça va ?

Pour info, j'ai déclaré et créé mes ArrayList dans ma classe. C'est pourquoi ça n'apparaît pas ici.

Merci beaucoup.

Emilie

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
 
// Calcul de la regularite d'envoi de messages
    public void calculRegularite() {
 
        double diff; //variable contenant la difference des donnees avec la moyenne
        double somme = 0;
        double moyenne = 0;
        double ecartType = 0;
 
        // différence entre deux iterations puis ajout de l'info dans l'arrayList interIterations
        for (int j = 0; j < iterationNumbers.size(); j++) {
            if(iterationNumbers.size() != j + 1) {
                interIterations.add((Integer) iterationNumbers.get(j+1) - (Integer) iterationNumbers.get (j));
            }
        }
 
        // calcul de la somme des itérations
        for (int i=0; i < interIterations.size(); i++) {
            double valeurIter = (Integer)interIterations.get(i);
            somme  = somme + valeurIter;
            valeurIter = valeurIter + 1;
        }
 
        //calcul de la moyenne
        moyenne = somme / interIterations.size();
 
        //calcul de l'écart type
        for (int i = 0; i < interIterations.size(); i++) {
          diff = pow((Integer) this.interIterations.get(i) - moyenne, interIterations.size());
          ecartType = diff;
        }
        ecartType = sqrt(ecartType / iterationNumbers.size());
    }

[mhamedbj]

est-ce que ton résultat est bon ?

[jhaythem]

optimise tes boucles en fesant une decrementation plus tot...
et pour voir si ca marche tu n as qu a l executer et voir si le resultat est bon ou pas
cheers,

[Benouze]

A quoi sert la ligne en gras ?

Code java :

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

1
2
3
4
5
for (int i=0; i < interIterations.size(); i++) {
   double valeurIter = (Integer)interIterations.get(i);
   somme  = somme + valeurIter;
   valeurIter = valeurIter + 1;
}

[Tasumy]

Bonjour,
j'ai quelques remarques qui pourraient t'aider à améliorer ton code et corriger tes erreurs ...

A quoi te sert réellement ta variable valeurIter? elle n'est indispensable nul part.

Comme je te l'ai déjà dit, dans ton cas, le calcul de l'écart type a pour formule :

racine_carrée[ [(x1-M)²+(x2-M)²+...+(xn-M)²]/n ]
avec M la moyenne, n la taille de ta liste et x(1 à n) une valeur de cette liste.

Or, toi tu fais :

racine_carrée[ (xn-M)(^n)/m ]
avec M la moyenne, n la taille de ta liste, xn la dernière valeur de cette liste et m la taille d'une autre liste.

Je ne sais pas si tu l'as testé mais le résultat ne doit pas être très juste...

Courage, tu y es presque.

[Emilie MARQUOIS-OGEZ]

Merci à tous et plus particulièrement à Tasumy qui est très pédagogue !
Je fais les tests et si je n'y arrive pas, je reviens vers vous.

Je pensais que valeurIter m'était utile pour faire la somme. Il semble que non.

Code :

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

1
2
3
4
5
 for (int i=0; i < interIterations.size(); i++) {
   double valeurIter = (Integer)interIterations.get(i);
   somme  = somme + valeurIter;
   valeurIter = valeurIter + 1;
}