Discussion :

[Magnum8760]

Le résultat doit être comme ce qui suit (l'utilisateur encode le nb de semaines et le genre M et F) mais ce n'est pas le cas

Nombre de semaines: 4
Semaine 1: m
Semaine 2: f
Semaine 3: u
u n’est pas valide !
Semaine 3: m
Semaine 4: m

Tu as hébergé 75% d’hommes et 25% de filles sur les 4 dernières semaines.

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
 
#include <stdlib.h>
#include <stdio.h>
 
int main()
{
	int nb;
	int count=1;
 
	int m=0, f=0;
	char lettre;
	printf("Bonjour, entrez le nb de semaines : ");
	scanf("%d",&nb);
	do{
		nb=count;
		printf("Semaine %d : ",nb++);
		scanf("%c",&lettre);
		if("lettre"=='m'){
			m++;}
		else if("lettre"=='f')
			f++;
		else printf("\n%c n'est pas valide !\n",lettre);
	} while(count<=nb);
		printf("Tu as heberge %d %% de garcons et %d de filles sur %d %% dernieres semaines.",(m/nb)*100,(f/nb)*100,nb);
	return 0;
}

D'avance, un grand merci.

[foetus]

Réfléchis un peu en regardant vite fait je vois que 1) ta variable count n'est jamais incrémentée et 2) ta variable nb est à chaque fois réinitialisée à 1

Donc ton test count<=nb est toujours vrai.

Édiit : Tu demandes le nombre de semaines et tu t'empresses de l'écraser

[Magnum8760]

Desole, j'essaye d'incrementer mon nombre de semaine à chaque affichage en meme temps que mon compteur. Pour moi la programmation c'est nouveau, j'ai commence il y 5 jours.. 🙄

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
#include <stdlib.h>
#include <stdio.h>
 
int main()
{
	int nb=1;
	int count=1;
	int m=0, f=0;
	char lettre;
	printf("Bonjour, entrez le nb de semaines : ");
	scanf("%d",&nb);
	do{
		printf("Semaine %d : %c",nb,lettre);
		scanf("%c",&lettre);
		if("lettre"=='m'){
			m++;
			count++;
			nb++;}
		else if("lettre"=='f'){
			f++;
			count++;
			nb++;}
		else printf("\n%c n'est pas valide !\n",lettre);
	} while(nb>0);
		printf("Tu as heberge %d %% de garcons et %d de filles sur %d %% dernieres semaines.",(m/nb)*100,(f/nb)*100,nb);
	return 0;
}

[foetus]

Cela ne peut pas fonctionner

Tu incrémentes tes 2 variables count et nb en même temps.
Or ta variable nb, celle dans laquelle tu stockes le nombre de semaine, doit être considérée comme un maximum et donc elle doit rester invariable.

De plus ton test est faux et encore plus crasseux toutes tes variables sont positives (elles sont initialisées à 1 et toujours incrémentées)

[Magnum8760]

J'avais l'idée de créer une boucle et de demander à l'utilisateur de noter -1 (condition d'arrêt avec mon compteur supérieur à zéro) pour sortir de la boucle à la n ème semaine mais on nous demande ici d'inscrire le nombre de semaine à l'avance.

Je ne vois comment résoudre cet énoncé sachant qu'à chaque boucle mon genre m ou f doit être incrémenté de 1 pour qu'au bout de n boucle, j"affiche le % de m et % de f.
-----
Nombre de semaines: 4
Semaine 1: m
Semaine 2: f
Semaine 3: u
u n’est pas valide !
Semaine 3: m
Semaine 4: m

Tu as hébergé 75% d’hommes et 25% de filles sur les 4 dernières semaines.
-----

[edgarjacobs]

Hello,

Un petit pseudo-code

Code :

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

1
2
3
4
5
6
7
8
9
10
demander le nombre de semaine --> nbre_semaine
nbre_feminin <-- 0
nbre_masculin <-- 0
pour i allant de 1 à nbre_semaine
	demander masculin ou feminin --> genre
	si genre vaut f
		incrémenter nbre_feminin
	sinon
		incrémenter nbre_masculin
afficher les resultats

[foetus]

@edgarjacobs ton code est faux, parce qu'il faut une boucle while pour gérer les erreurs de saisie (exemple avec un u)

J'avais l'idée de créer une boucle et de demander à l'utilisateur de noter -1 (condition d'arrêt avec mon compteur supérieur à zéro) pour sortir de la boucle à la n ème semaine mais on nous demande ici d'inscrire le nombre de semaine à l'avance.

Avec un compteur de semaines, cela se fait également très bien

[edgarjacobs]

@edgarjacobs ton code est faux, parce qu'il faut une boucle while pour gérer les erreurs de saisie (exemple avec un u)

Non, ce genre de contrôle se fait autour de la ligne qui demande l'entrée, donc dans la boucle for

[foetus]

Non, ce genre de contrôle se fait autour de la ligne qui demande l'entrée, donc dans la boucle for

D'accord, mais tu alourdis ton code parce qu'au lieu d'avoir 1 boucle avec tous les tests, tu as 2 boucles mais les tests séparés en 2 : tests validation de l'entrée et tests effectifs

[Sve@r]

Bonjour

Je ne vois comment résoudre cet énoncé sachant qu'à chaque boucle mon genre m ou f doit être incrémenté de 1 pour qu'au bout de n boucle, j"affiche le % de m et % de f.

Et tu n'as pas pensé à créer une 4° variable nommée (par exemple) "i" qui partirait de 1 et qui serait incrémentée à chaque tour de boucle ; boucle qui s'arrêterait alors quand "i" atteint "nb" ????
Et en plus ça irait super bien car tu dois afficher à chaque tour de boucle le n° de la semaine en cours ; n° qui correspond alors exactement à la valeur de la variable "i"...

On peut aussi voir ça dans l'autre sens => créer une variable "i" destinée à stocker le n° de la semaine en cours (pour pouvoir ensuite l'afficher) et se rendre compte que fatalement, à un moment donné, cette variable sera égale au nb de semaines demandé et en déduire que cette égalité peut alors servir de détecteur pour sortir de la boucle...

[foetus]

Et tu n'as pas pensé à créer une 4° variable nommée (par exemple) "i" qui partirait de 1 et qui serait incrémentée à chaque tour de boucle ; boucle qui s'arrêterait alors quand "i" atteint "nb" ????
Et en plus ça irait super bien car tu dois afficher à chaque tour de boucle le n° de la semaine en cours ; n° qui correspond alors exactement à la valeur de la variable "i"...

Si, il y a pensé : c'est la variable count.
Le seul problème , c'est soit il inverse l'usage des 2 variables (premier code) et écrase définitivement le nombre de semaines demandé soit il incrémente les 2 en même temps (deuxième code effacé)

Il est tellement perdu qu'il a trouvé une 3ième façon : il n'utilise qu'une seule variable et donne un code de sortie à l'utilisateur (-1). Mais il n'arrive pas à avoir le nombre de semaines avec cette version

[Magnum8760]

Je comprend l'interet de toutes mes variables seulement je n'arrive pas à vider le buffer de mes variables nb_feminin et nb_masculin apres ma boucle. En fait ma selection dans ma boucle ne va pas du tout ...

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
#include <stdio.h>
#include <stdlib.h>
 
int main()
{
	int nb_semaine;
	int compteur =1;
	int i =1;
	int genre;
	int nb_feminin=0;
	int nb_masculin=0;
 
	printf("Nombre de semaine ?");
	scanf("%d",&nb_semaine);
 
	while(compteur>=1&&compteur<=nb_semaine){
		printf("Semaine %i :",i);
		printf("quel est mon genre ? %c",genre);
		scanf("%c",&genre);
		if(genre=='m')
			nb_masculin++;
		else if(genre=='f')
			nb_feminin++;
		else printf("erreur encodage");
		i++;
		compteur++;
	}
	printf("f vaut : %d",nb_feminin);
	printf("m vaut : %d",nb_masculin);
	return 0;
}

[Sve@r]

Je comprend l'interet de toutes mes variables seulement je n'arrive pas à vider le buffer de mes variables nb_feminin et nb_masculin apres ma boucle.

Tu entends quoi par "vider le buffer de mes variables" ? Déjà le terme "vider une variable" a peu de sens en informatique. Une variable contient toujours quelque chose tout le temps. Donc en général ce qu'on fait, c'est que ce qu'elle contient on s'en balance jusqu'à ce qu'on en ait besoin et dans ce cas, on veille alors à lui mettre au préalable la valeur dont on a/aura besoin. Bref une variable on ne lui met une valeur que si on a besoin de sa valeur un peu plus tard. Donc mettre quelque chose dans une variable en fin de programme ça ne sert absolument à rien.
Dit plus simplement

• on ne lit une variable que si on l'a remplie au préalable
• on ne remplit une variable que si on aura besoin de la lire après

En fait ma selection dans ma boucle ne va pas du tout ...

Ouais ben c'est à toi de donner les bons ordres à ton ordi. Accessoirement tes variables "i" et "compteur" ont exactement la même valeur au même moment tout le temps. Ce qui veut dire que l'une de ces deux variables ne sert à rien (puisque si t'as besoin de sa valeur tu l'as aussi dans l'autre).

Et un code ça se met entres balises

[Magnum8760]

J'entend afficher mes variables incrementer à la sortie de ma boucle 😐 Apres faire un pourcent, ca semble easy ...

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
#include <stdio.h>
#include <stdlib.h>
 
int main()
{
	int nb_semaine;
	int compteur =1;
	int i =1;
	int genre;
	int nb_feminin=0;
	int nb_masculin=0;
 
	printf("Nombre de semaine ?");
	scanf("%d",&nb_semaine);
 
	while(compteur>=1&&compteur<=nb_semaine){
		printf("Semaine %i :",i);
		printf("quel est mon genre ? %c",genre);
		scanf("%c",&genre);
		if(genre=='m')
			nb_masculin++;
		else if(genre=='f')
			nb_feminin++;
		else printf("erreur encodage");
		i++;
		compteur++;
	}
	printf("f vaut : %d",nb_feminin);
	printf("m vaut : %d",nb_masculin);
	return 0;
}

[Sve@r]

J'entend afficher mes variables incrementer à la sortie de ma boucle 😐

Ben à vue de nez il semble presque correct ce code. Excepté pour cette variable "i" que tu t'obstines à utiliser alors qu'elle est totalement inutile et ce "else if" qu'on écrirait plutôt "elif".
Le seul point noir c'est déjà l'affichage de la variable "genre" avant son remplissage et surtout son typage. Et aussi rajoute un fgetc(stdin) juste sous le "scanf" et là ça devrait fonctionner au poil.

Apres faire un pourcent, ca semble easy ...

Exact (si on n'oublie pas de faire le calcul en virgule flottante) => printf("%.02f%%", nb_feminin * 100.0 / nb_semaine).

[Magnum8760]

Effectivement j'avais deux compteur : i et compteur.

Merci pour le conseil : "%.02f%%"
Très utile.

fgetc(stdin)
Je ne connais pas et je n'ai pas encore vu cette fonction, j'ai trouvé que ça marche si tu met un espace devant ton flag " %c".

Problème résolu.

[Sve@r]

j'ai trouvé que ça marche si tu met un espace devant ton flag " %c".

Attention aux "petits trucs" qui "semblent marcher" mais qui ne marchent en réalité que le temps de le dire car c'est souvent une "fausse" bonne idée (ceci dit, mon fgetc(stdin) est lui aussi un "petit truc" donc lui-aussi à oublier).

Ce serait mieux si tu connaissais les relations liant scanf(), ce que tu tapes et "stdin" mais ce sera pour un peu plus tard quand tu auras progressé en C.

[foetus]

Ben à vue de nez il semble presque correct ce code

Non, il est faux en cas d'une erreur de saisie, il incrémente quand même le compteur

[Sve@r]

Non, il est faux en cas d'une erreur de saisie, il incrémente quand même le compteur

Oui enfin il y a plusieurs niveaux de "être correct".
Il y a le niveau "biberon" où "être correct" c'est avoir un code qui fait ce qu'on lui demande quand on entre les bonnes infos
Il y a le niveau "beginner" où "être correct" c'est avoir un code qui fait ce qu'on lui demande quand on entre les bonnes infos et qui arrive à gérer de petites erreurs
Il y a le niveau "padawan" où "être correct" c'est avoir un code qui fait ce qu'on lui demande quand on entre les bonnes infos et qui arrive à gérer de grosses erreurs
Et puis il y a le niveau "ère d'Ultron" où "être correct" c'est avoir un code qui s'écrit tout seul et qui s'adapte à toutes les situations possibles et impossibles

Bref j'adapte mon discours à la population. C'est sûr que si c'est toi qui avait proposé ça jamais je n'aurais dit que ce code "était correct". Et puis j'ai quand-même dit "presque"...