Discussion :

[jmc41320]

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

[joel.drigo]

L'affichage de ton tableau devrait te mettre la puce à l'oreille !

Quel est l'affichage si tu ajoutes

Code :

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

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

juste avant

Code :

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

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 :

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

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 ?

[jmc41320]

le resultat est 0/0

franchement j'ai le nez devant je ne vois pas

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

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

[jmc41320]

jai changé le type int en double mais le resultat ne change pas

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
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!");
    }
}

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

Ensuite, la piste pour corriger le problème :

Code :

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

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 ?

[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

[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

[joel.drigo]

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;

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 :

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

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

Donc tes deux lignes sont l'équivalent de

Code :

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

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

Autrement dit

Code :

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

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 :

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

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 :

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

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.

[joel.drigo]

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

Donc c'est bien ça.

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.

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