je n'arrive pas a calculer la moyenne d'un tableau

Version imprimable

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
/**
 * @(#)calculmoy.java
 *
 * calculmoy application
 *
 * @author 
 * @version 1.00 2017/10/25
 */
import java.util.*; 
public class calculmoy {
    public static void charger(int [] t)
    {
    int rep=0;
    int indice=0;
    int fin=0;
    while (rep!=-1)	
    {
    	rep=Terminal.lireInt();
    	t[indice]=rep;
    	indice++;
    	fin=indice--;
    }
    for (indice=fin;indice<=t.length-1;indice++)
    {
 
    	t[indice]=-1;
    }
    }
    public static void affichage(int[]t)
    {
    	for(int i=0;i<=19;i++)
    	{
    		System.out.println(t[i]);
    	}
    }
    public static void moyenne(int[]t)
    {
    	double moyenne=0;
    	int nb=0;
    	int indice=0;
    	while(t[indice]!=-1)
    	{
    		moyenne=moyenne+t[indice];
    		indice++;
    		nb=nb++;
    	}
    	System.out.println( moyenne/nb);
    }
    public static void main(String[] args) {
    	int [] tab=new int[20];
    	charger (tab);
    	affichage(tab);
    	moyenne(tab);
    	System.out.println("Hello World!");
    }
}

le programme saisisi des nombres et -1 pour finir et je veux calculer la moyenne des nombres (non-1) saisi
j'obtiens comme resultat NaN

25/10/2017, 17h33
joel.drigo
L'affichage de ton tableau devrait te mettre la puce à l'oreille !

Quel est l'affichage si tu ajoutes
Code:

1 2 System.out.println( moyenne ); System.out.println( nb );
juste avant

Code:

System.out.println( moyenne/nb);

Que penses-tu de cet affichage et des conséquences sur le résultat du calcul de la moyenne, mathématiquement parlant ?

Ensuite, la piste pour corriger le problème :
Code:

1 2 indice++; fin=indice--;
Peux-tu décrire ce que font ces deux lignes ? Quelles sont les conséquences sur le tableau, en tenant compte du code de la méthode charger ?

le resultat est 0/0

franchement j'ai le nez devant je ne vois pas

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
 public static void moyenne(int[]t)
    {
    	double moyenne=0;
    	int nb=0;
    	int indice=0;
    	int somme=0;
    	while(t[indice]!=-1)
    	{
    	somme=somme+t[indice];
    		indice++;
    		nb=nb++;
    	}
    	System.out.println(somme );
		System.out.println( nb );
    	System.out.println( somme/nb)

25/10/2017, 18h27
joel.drigo

Pourquoi tu me montres un autre code maintenant ? Commence par répondre aux questions que je t'ai posées à propos de ton premier code !

Et là tu me dis que le résultat est 0/0. Donc zéro divisé par zéro. Peut-on diviser un nombre par 0 ?

Dans ton nouveau code, tu accumules le total dans la variable somme qui est du type int. En Java, un int divisé par un int donne un int. 10/3 donne 3. 10/7 donne 1. Un int divisé par un int plus grand donne toujours 0, parce que la division d'un nombre positif par un nombre plus grand donne un nombre positif inférieur à 1 et que le seul int positif inférieur à 1 c'est 0.

jai changé le type int en double mais le resultat ne change 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
import java.util.*; 
public class calculmoy {
    public static void charger(double [] t)
    {
    double rep=0;
    int indice=0;
    int fin=0;
    while (rep!=-1)	
    {
    	rep=Terminal.lireInt();
    	t[indice]=rep;
    	indice++;
    	fin=indice--;
    }
    for (indice=fin;indice<=t.length-1;indice++)
    {
 
    	t[indice]=-1;
    }
    }
    public static void affichage(double[]t)
    {
    	for(int i=0;i<=19;i++)
    	{
    		System.out.println(t[i]);
    	}
    }
    public static void moyenne(double[]t)
    {
    	double moyenne=0;
    	double nb=0;
    	int indice=0;
   		double somme=0;
    	while(t[indice]!=-1)
    	{
    	somme=somme+t[indice];
    		indice++;
    		nb=nb++;
    	}
    	System.out.println(somme );
		System.out.println( nb );
    	System.out.println( somme/(nb));
    }
    public static void main(String[] args) {
    	double [] tab=new double[20];
    	charger (tab);
    	affichage(tab);
    	moyenne(tab);
    	System.out.println("Hello World!");
    }
}

25/10/2017, 19h00
joel.drigo
Oui, m'enfin le problème de type tu l'avais sur le second code, pas sur le premier. Je t'ai posé un certain nombre de questions pour te guider vers la solution.

Dont celle-ci :
Citation:
Envoyé par joel.drigo

Ensuite, la piste pour corriger le problème :

Code:

1 2 indice++; fin=indice--;

Peux-tu décrire ce que font ces deux lignes ? Quelles sont les conséquences sur le tableau, en tenant compte du code de la méthode charger ?
25/10/2017, 19h08
jmc41320

indice++ est l'index de la boucle
et fin=indice-- recule d'un cran car avec la condition while je vais un cran trop loin
25/10/2017, 19h25
jmc41320

je crois que je n'entre pas dans la boucle de moyenne.

pourquoi nb reste a 0 ?

j'ai trouvé mais pourquoi indice++ n'est pas egal a indice=indice+1
25/10/2017, 19h41
joel.drigo
Citation:

Envoyé par jmc41320

indice++ est l'index de la boucle

La bonne réponse est : ça incrémente la variable indice. Donc c'est l'équivalent de indice=indice+1;

Citation:

Envoyé par jmc41320

et fin=indice-- recule d'un cran car avec la condition while je vais un cran trop loin

La bonne réponse c'est ça met la valeur de indice dans la variable fin et ça décrémente indice. Donc c'est l'équivalent de :
Code:

1 2 fin=indice; indice=indice-1;
Donc tes deux lignes sont l'équivalent de
Code:

1 2 3 4 indice=indice+1; fin=indice; indice=indice-1;
Autrement dit

Code:

fin=indice+1;

Donc indice ne varie pas, donc vaut toujours 0. Et fin non plus sauf au premier coup, donc vaut toujours 1.

Donc en gros, tout ce que tu lis au clavier tu le mets dans le premier élément de ton tableau. Y compris le -1 que tu saisis pour dire que tu finis la saisie.

Donc l'élément situé à l'indice 0 vaut -1. Et le reste vaut 0 (la valeur par défaut) jusqu'à ce que tu fasses la boucle suivante :
Code:

1 2 3 4 5 for (indice=fin;indice<=t.length-1;indice++) { t[indice]=-1; }
ce qui revient à remplir le reste du tableau de -1 (de l'élement 1 à la fin), donc sauf le premier élément, qui vaut déjà -1.

Donc tu un tableau rempli de -1.

Donc dans ton calcul de moyenne, on n'entre pas dans la boucle :

Code:

while(t[indice]!=-1)

donc nb vaut 0. Et ensuite la division de 0 par 0 donne NaN qui veut dire Not a Number. Normal.
25/10/2017, 19h48
joel.drigo

Citation:

Envoyé par jmc41320

je crois que je n'entre pas dans la boucle de moyenne.

Donc c'est bien ça.

Citation:

Envoyé par jmc41320

pourquoi nb reste a 0 ?

Parce que tu n'entres pas dans la boucle, et que nb est incrémenté dans la boucle, donc aucun risque qu'il varie. Je t'ai expliqué plus haut pourquoi on entre pas dans la boucle. Et c'est normal. Ta méthode de moyenne est bonne. Simplement il n'y a aucune note dans le tableau. Et ce cas, il faut que tu le prévois en testant si nb vaut 0 après la boucle : dans ce cas là, on ne dois pas chercher à calculer la moyenne mais afficher un message du genre : "aucune note saisie, calcul de moyenne impossible". Parce que même si tu corriges la méthode charger, rien n'empêche de saisir -1 dès le début.

Citation:

Envoyé par jmc41320

j'ai trouvé mais pourquoi indice++ n'est pas egal a indice=indice+1

Comme je te l'ai dit plus haut, le problème n'est pas indice++, mais indice--, qui annule indice++.