Discussion :

[Invité]

Voici la consigne de mon formateur :
"Ecrire un algorithme qui affiche la moyenne des nombres pairs et la moyenne des multiples de 3 d'un tableau d'entiers initialisés par des nombres entrés au clavier par l'utilisateur.
L'utilisateur entrera un maximum de 15 nombres et signifiera la fin de ses entrées en tapant -1 au clavier.
0 n'est pas considéré comme un nombre pair ou un multiple de 3."

Voici notre 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
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
// initialisation des variables indice j, longeur effective, le nombre lu...
tableau tab[15] :entier
var j, lgEff, nbLu, multiple2, multiple3, cptMultiple2, cptMultiple3

lgEff <- 0
multiple2 <- 0
multiple3 <- 0
cptMultiple2 <- 0
cptMultiple3 <- 0

lire nbLu
TANT QUE nbLu <> -1 FAIRE    // -1 signifiera la fin de ses entrées
	lgEff <- lgEff + 1
	tab[j] <- nbLu
		SI tab[j] <> 0 ALORS //0 n'est pas considéré comme un nombre pair ou un multiple de 3.
			
			SI tab[j] mod2 = 0 ALORS   // si nbLu est un nb pair
				multiple2 <- multiple2 + 1
				cptMultiple2 <- cptMultiple2 + 1
			FSI

			SI tab[j] mod3 = 0 ALORS   // si nbLu est un multiple de 3
				multiple3 <- multiple3 + 1
				cptMultiple3 <- cptMultiple3 + 1
			FSI
		FSI
	j <- j + 1
FTANT

SI cptMultiple2 = 0 ALORS
          ecrire 'pas de nombre pair'
SINON
          ecrire multiple2/cptMultiple2  //affiche la moyenne des nombres pairs
FSI

SI cptMultiple3 = 0 ALORS
          ecrire 'pas de nombre multiple de 3'
SINON
          ecrire multiple3/cptMultiple3  //affiche la moyenne des multiples de 3
FSI

Avez-vous des avis sur la construction de cet algorithme svp ? J'ai effetué cet algo avec des collègues et nous nous demandons s'il y aurait moyen de l'améliorer... On débute en algo et nous ne devons pas encore programmer en C pour le moment.

[Flodelarab]

Bonjour

"Lire" n'est pas dans une boucle. Dès lors, le code affiché ne répondra pas à la question puisqu'un seul nombre sera lu.

[Invité]

"Lire" n'est pas dans une boucle. Dès lors, le code affiché ne répondra pas à la question puisqu'un seul nombre sera lu.[/QUOTE]


Merci beaucoup pour cette remarque pertinente. J'imagine que la ligne ' lire nbLu ' devrait se situer entre les lignes 27 et 28 (entre j <- j + 1 et FTANT ). J'y réfléchirai et je reviens sur ce forum dès que possible.

[Galet]

Bonjour,
Pour alimenter tes réflexions, tu as plusieurs solutions :

- relire un nouveau nombre avant de reboucler,
- décaler ta lecture tout de suite après le début de ta boucle mais il faudra penser à ce que ta boucle soit bien traitée la première fois et gérer la réponse -1 à la première question.
- ne faire le test de rebouclage qu'à la fin

A toi de jauger la meilleure solution.

Tu dois aussi penser à limiter le nombre d'entrées...

Autre chose, essaies de séparer l'entrée de tes données de leur traitement, ce sera plus facile à écrire, à lire, à debugguer mais aussi à améliorer
- Entrée des données : Dialogue avec l'utilisateur
- Traitement des données et calcul des résultats
- Affichage des résultats. Dialogue avec l'utilisateur.
Beaucoup de traitement sont gérés de cette manière : Données => Traitement => Résultats...

Belle journée,

[tbc92]

Autre point, tu as ces 2 lignes :

Code :

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

1
2
multiple2 <- multiple2 + 1
cptMultiple2 <- cptMultiple2 + 1

Et à la fin, tu divises multiple2 par cptmultiple2 ; ok.
Mais je pense que tu as fait un copier/coller, et que tu as oublié de changer un petit truc.

[Guesset]

Bonjour

Comme signalé par tbc92, idem pour :

Code :

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

1
2
      multiple3 <- multiple3 + 1
      cptMultiple3 <- cptMultiple3 + 1

D'où l'intérêt de bien nommer les variables : peut être sumMultiple2 et sumMultiple3 au lieu de multiple2 et multiple3.

Salutations