problemes dans mon programme?

Bonjour à tous! Je suis nouvelle sur le forum et débutante en C ++
J'ignore si je peux poster ici mais...
En fait j'ai tenté d'effectuer un programme dont l'enoncé suit mais je ne comprend pas pourquoi il ne fonctionne pas..A moins que je l'ai mal interprété?
Donc si quelqu'un peut m'eclairer ce serait vraiment sympa.:ccool:
Enoncé


Une compagnie de location de voitures dispose de 8 voitures de locations. À la fin de chaque mois, elle fait entrer le nombre de km parcourus par chaque voiture. À chaque voiture, est attribué un nom ( A, B, C, D, E, F, G, H) :

Le programme doit etre en mesure de calculer et afficher le nombre total de km parcourus par chaque voiture et le nombre km total parcourus par toutes les voitures.

Utiliser une boucle contrôlée par valeur sentinelle entrer les données, . Plus précisément, à chaque itération le programme demande d’entrer un nom de voiture et un nombre de kilomètres, jusqu’à ce que l’utilisateur entre la lettre S et 0 (ce qui mettra fin au programme). Il doit y avoir une structure switch pour déterminer le traitement à faire pour chaque voiture. Dans le cas ou l'utilisateur entre un nom de voiture non compris entre { A, B, C, D, E, F, G, H}, le programme affichera un message d'erreur et continuera.


Programme que j'ai ecrit

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

# include <iostream>   using namespace std;   int main () { int kilom_voiture =1; char nom_voiture ; char A ; char B ; char C; char D; char E; char F; char G; char H ;   while ( kilom_voiture != 0 && nom_voiture != 'S' ) { cout << "Tapez le nom de la voiture"; cin >> nom_voiture;   int kilom_parcouruA =0; int kilom_parcouruB =0; int kilom_parcouruC =0; int kilom_parcouruD =0; int kilom_parcouruE=0; int kilom_parcouruF=0; int kilom_parcouruG=0; int kilom_parcouruH=0;   if(nom_voiture =='A', nom_voiture =='B', nom_voiture== 'C', nom_voiture == 'D', nom_voiture =='E', nom_voiture=='F', nom_voiture=='G',nom_voiture='H') { switch (nom_voiture) { case 'A': cout << " Entrez le nombre de kilometre parcouru par cette voiture " << endl; cin>>kilom_parcouruA; cout << " Le nombre total de Km parcouru par cette voiture est : " << kilom_parcouruA <<endl; break;   case 'B': cout << " Entrez le nombre de kilometre parcouru par cette voiture " << endl; cin>>kilom_parcouruB; cout << " Le nombre total de Km parcouru par cette voiture est : " << kilom_parcouruB <<endl; break; } } else cout << " Erreur " << endl; } cout << "Terminaison du programme\n"; return 0; }

---

Le truc c'est que quand j'execute il me demande d'initialiser les variables char mais...je ne comprends pas pourquoi et je ne vois pas comment... Et puis quand le programme marchait avant que je continu de le faire le programme ne se terminait pas quand j'entrais S et 0 mais se terminait si j'entrais seulement 0 je suis bloquée et perdue...
(logiciel : Microsoft Visual C++ Express 2010 version gratuite)

Met les balises CODE et /CODE et indente ton code pour plus de lisibilité stp.

Utilise aussi des tableaux plutôt que des variables du type nb_km_a, nb_km_b etc...

Code:

---

1 2	if(nom_voiture =='A', nom_voiture =='B', nom_voiture== 'C', nom_voiture == 'D', nom_voiture =='E', nom_voiture=='F', nom_voiture=='G',nom_voiture='H')

---

la virgule ignore la valeur de retour des instructions précédentes.
Comme tu effectues des comparaisons, ce que tu as fait est équivalant à :

Code:

---

1 2	if(nom_voiture=='H')

---

Tu dois remplacer la virgule par :
- || pour un ou inclusif
- && pour un et

Quand tu écris

Code:

---

1 2 3

char A; char B;

---

Cela a pour effet de réserver de l'espace en mémoire pour 2 variables type char avec une valeur non définie à l'intérieur vu que tu ne les initialises pas.
Il aurait fallu mettre par exemple :

Code:

---

1 2 3

char A = 'A'; char B = 'B';

---

Maintenant, tu n'utilises pas ces variables dans ton programme et tu n'en as, il est vrai, pas besoin donc supprimes juste toutes tes déclarations de char X.

Il aurait été beaucoup plus simple d'utiliser un map pour faire l'exercice proprement mais l'énoncé semble t'orienter vers quelque chose de différent. Si c'est dans le cadre d'études et que tu as vu les conteneurs STL, alors je te conseillerai de regarder du côté de std::map. Moi ou une autre personne du forum pourrons t'aider dans son utilisation si tu le souhaites.

Code:

---

1 2	cout << " Le nombre total de Km parcouru par cette voiture est : " << kilom_parcouruA <<endl;

---

N'indiques pas le résultat juste après. Il faut que tu fasses l'affichage une fois que toutes les données ont été saisies, comme un récapitulatif. Cela t'obligera à conserver les valeurs des variables en mémoire kilom_parcouruX et donc à les déclarer autre part et/ou sous une autre forme.

Tu peux aussi mettre tes lectures "clavier" dans le while :

Code:

---

1 2 3 4 5 6 7 8

const char sentinel = 'S'; while(cin >> nom_voiture) { if(nom_voiture == sentinel) return; else ... }

---

Bonjour et bienvenue sur le forum

J'ai corrigé l'indentation du code, c'est pas plus lisible comme ça ?

D'après ce que j'ai compris de l'énoncé, tu es partie dans la bonne direction.
Plusieurs remarques...

Code:

---

int kilom_voiture = 1;

---

Pourquoi initialiser à 1 et pas à 0 ?
Comme c'est une distance, la valeur sera toujours positive donc tu peux utiliser un unsigned int (entier non signé, c'est à dire entier positif)

Code:

---

1 2 3 4 5 6 7 8

char A ; char B ; char C; char D; char E; char F; char G; char H ;

---

A quoi va servir ces variables ?
Niveau écriture, tu mets des espaces un peu n'importe où (ce n'est pas interdit en C++ :)) mais pour faciliter la lecture, essaies d'être cohérent. Par exemple, pas d'espace avant le point-virgule, un espace avant et après le signe égal, etc.

Code:

---

1 2 3 4 5 6 7 8

int kilom_parcouruA =0; int kilom_parcouruB =0; int kilom_parcouruC =0; int kilom_parcouruD =0; int kilom_parcouruE=0; int kilom_parcouruF=0; int kilom_parcouruG=0; int kilom_parcouruH=0;

---

Idem pour unsigned et les espaces.

Beaucoup plus gênant : tu définies tes variables dans le bloc du while (pour rappel, un bloc, c'est le code compris entre deux crochets {}, donc ici le bloc commence après la ligne contenant le while et se termine à la ligne avec le "Erreur")
Une variable déclarée dans un bloc est une variable locale : elle sera détruite en sortant du bloc. Tu vois le problème ? A chaque itération du while (donc à chaque fois que tu redemandes le nom de la voiture, les variables kilm_xxx auront été détruites et seront recréées juste après et initialisées à 0.
Pour éviter cela, il faut déplacer la déclaration de ces variables avant le while.

Code:

---

if(nom_voiture =='A', nom_voiture =='B', nom_voiture== 'C', nom_voiture == 'D', nom_voiture =='E', nom_voiture=='F', nom_voiture=='G',nom_voiture='H')

---

C'est une solution mais elle est un peu lourde. Une autre solution, plus simple, est d'ajouter une clause default dans le switch. Cette ligne sera utilisée à chaque fois que le switch ne trouve pas le caractère.
La syntaxe :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18

switch ( Param ) { case option1: ... break;   case option2: ... break;   case option3: ... break;   ...   default: ... // pas besoin de break puisqu'on est à la fin du switch }

---

Code:

---

cin>>kilom_parcouruA;

---

Avec ce code, tu lis à chaque cycle la valeur et tu l'as met dans la variables. La distance précédente est donc perdue.
Il faut donc créer une variable dans le bloc de while uniquement pour entrer la valeur puis l'additionner à la variable correspondante à la bonne voiture :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16

int kilom = 0; switch (nom_voiture) { case 'A': cout << " Entrez le nombre de kilometre parcouru par cette voiture " << endl; cin>>kilom; kilom_parcouruA += kilom; cout << " Le nombre total de Km parcouru par cette voiture est : " << kilom_parcouruA <<endl; break;   case 'B': cout << " Entrez le nombre de kilometre parcouru par cette voiture " << endl; cin>>kilom; kilom_parcouruB += kilom; cout << " Le nombre total de Km parcouru par cette voiture est : " << kilom_parcouruB <<endl; break; }

---

Voila pour le moment. Il y aurait encore quelques petites choses à corriger mais on verra plus tard. N'hésite pas à poster de nouveau quand tu auras avancé ou si tu as des questions.

Grillé (un peu) par Xtrem_Voyageur

Citation:

---

Envoyé par Xtrem_Voyageur

Quand tu écris

Code:

---

1 2 3

char A; char B;

---

Cela a pour effet de réserver de l'espace en mémoire pour 2 variables type char avec une valeur non définie à l'intérieur vu que tu ne les initialises pas.
Il aurait fallu mettre par exemple :

Code:

---

1 2 3

char A = 'A'; char B = 'B';

---

---

Il n'y a pas de raison que cela génère une erreur. Les variables sont quand même correctement initialisées, même sans leurs affecter une valeur.
Mais il est vrai qu'on ne sait pas quelle erreur est réellement générée...

Citation:

---

Le truc c'est que quand j'execute il me demande d'initialiser les variables char mais...je ne comprends pas pourquoi et je ne vois pas comment...

---

Quand tu as un message d'erreur, indique le correctement pour comprendre le problème.

Citation:

---

Et puis quand le programme marchait avant que je continu de le faire le programme ne se terminait pas quand j'entrais S et 0 mais se terminait si j'entrais seulement 0 je suis bloquée et perdue...

---

Tu utilises comme condition dans ton while :

Code:

---

while ( kilom_voiture != 0 && nom_voiture != 'S' ) {

---

ce qui veut dire, en français, que tu continues ton while tant que kilom_voiture ne vaut pas 0 et que nom_voiture ne vaut pas 'S'. Si l'une ou l'autre des conditions n'est pas respectée, alors tu arrêtes le while.
Je te laisse chercher un peu la syntaxe correcte pour faire ce que tu souhaites


PS :

Citation:

---

Envoyé par Xtrem_Voyageur

Si c'est dans le cadre d'études

---

Je suppose (j'espère) que c'est dans le cadre d'étude, parce que niveau C++, c'est très moche... J'espère que c'est le tout début du cours, sinon :aie:

Non gb,tu as été plus précis que moi :ccool:
J'ai toujours du mal avec les codes de personnes qui débutent.
On ne sait jamais dans quel cadre elles viennent poster et si ce qui les intéresse c'est d'apprendre ou uniquement d'avoir une compilation error-free...
Du coup, j'hésite entre juste leur dire où est l'erreur, et leur donner 2-3 petits conseils à côté pour améliorer le reste tout en gardant à l'esprit qu'elles débutent et que cela ne servirait à rien de leur fournir une solution de plus haute volée.. (chose que l'on voit parfois sur le forum)

Citation:

---

PS :
Citation:
Envoyé par Xtrem_Voyageur Voir le message
Si c'est dans le cadre d'études
Je suppose (j'espère) que c'est dans le cadre d'étude, parce que niveau C++, c'est très moche... J'espère que c'est le tout début du cours, sinon

---

Oui, on ne sait jamais si la personne apprend de façon autodidacte pour son plaisir. Ou si cela entre dans le cadre d'études, ce qui permet de savoir ce qui a été vu et donc de ne pas trop sortir des sentiers battus de ce qu'elles savent quand on leur propose une solution.
Mais j'espère comme toi! Malheureusement, certains ont vu beaucoup plus en cours et ne savent pas mieux faire et viennent se décharger de leur travail ici pour mieux pouvoir s'avachir sur le banc d'école.

Attention !

Citation:

---

Envoyé par digitalout

Programme que j'ai ecrit

Code:

---

1 2 3 4 5 6 7 8 9 10

# include <iostream>   using namespace std;   int main () { … if(nom_voiture =='A', nom_voiture =='B', nom_voiture== 'C', nom_voiture == 'D', nom_voiture =='E', nom_voiture=='F', nom_voiture=='G',nom_voiture='H') … }

---

---

Citation:

---

Envoyé par Neckara

Code:

---

1 2	if(nom_voiture =='A', nom_voiture =='B', nom_voiture== 'C', nom_voiture == 'D', nom_voiture =='E', nom_voiture=='F', nom_voiture=='G',nom_voiture='H')

---

la virgule ignore la valeur de retour des instructions précédentes.
Comme tu effectues des comparaisons, ce que tu as fait est équivalant à :

Code:

---

1 2	if(nom_voiture=='H')

---

---

Je ne sais pas si c'est une faute de frappe en recopiant, mais regarde bien ta dernière clause :

Code:

---

,nom_voiture='H');

---

Tu as écrit « = » à la place de « == » ! Du coup, quelque soit le résultat des tests précédents, tu affectes 'H' à « nom_voiture » et comme tu as utilisé des virgules, l'expression entière prend cette même valeur. Et comme elle n'est pas nulle, le test sera toujours vrai.

Tu vas donc non seulement entrer dans ton bloc comme si l'utilisateur avait tapé une lettre valide, mais ton programme va se comporter comme si l'utilisateur tapait toujours « H ».

Bien vu :)
De toute façon, le mieux reste de virer ce if affreux

Je vous remercie beaucoup pour vos indications. Je les aies toutes prises en compte et j'ai réécrit mon programme. Il fonctionne mais j'ai toujours un soucis en fait c'est que il ne s’arrête pas quand j'ecrit S et 0 mais quand j'ecrit S seulement et pourtant j'ai essayé de modifier la syntaxe comme me l'a indiqué gbdivers

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 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 74 75 76 77 78 79 80 81 82 83 84 85 86 87 88 89 90 91 92 93 94 95 96 97 98 99 100 101 102 103 104 105 106 107 108 109 110 111 112 113 114 115 116 117 118 119 120 121 122

# include <iostream>   using namespace std;   int main () {       char nom_voiture = 'A' && 'B' && 'C' && 'D' && 'E' && 'F'&& 'G'&& 'H'; int kilom = 0; unsigned int kilomA = 0; unsigned int kilomB = 0; unsigned int kilomC = 0; unsigned int kilomD = 0; unsigned int kilomE = 0; unsigned int kilomF = 0; unsigned int kilomG = 0; unsigned int kilomH = 0;         { while (kilom != 0 || nom_voiture != 'S') {   // Demander a l'utilisateur d'entrer le nom de la voiture souhaitée   cout<< " Quel est le nom de la voiture" << endl; ; cin>> nom_voiture;   switch (nom_voiture) {   // Afficher et calculer le nombre de kilometres correspondant pour chaque voiture   case 'A': cout << " Entrez le nombre de kilometre parcouru par cette voiture " << endl; cin>>kilom; kilomA += kilom; cout << " Le nombre total de km parcouru par cette voiture est : " << kilomA << endl; break;   case 'B': cout << " Entrez le nombre de kilometres parcouru par cette voiture" << endl; cin>>kilom; kilomB += kilom; cout << " Le nombre total de km parcouru par cette voiture est : " << kilomB << endl; break;   case 'C': cout << " Entrez le nombre de kilometres parcouru par cette voiture" << endl; cin>>kilom; kilomC += kilom; cout << " Le nombre total de km parcouru par cette voiture est : " << kilomC << endl; break;   case 'D': cout << " Entrez le nombre de kilometres parcouru par cette voiture" << endl; cin>>kilom; kilomD += kilom; cout << " Le nombre total de km parcouru par cette voiture est : " << kilomD << endl; break;   case 'E': cout << " Entrez le nombre de kilometres parcouru par cette voiture" << endl; cin>>kilom; kilomE += kilom; cout << " Le nombre total de km parcouru par cette voiture est : " << kilomE << endl; break;   case 'F': cout << " Entrez le nombre de kilometres parcouru par cette voiture" << endl; cin>>kilom; kilomF += kilom; cout << " Le nombre total de km parcouru par cette voiture est : " << kilomF << endl; break;   case 'G': cout << " Entrez le nombre de kilometres parcouru par cette voiture" << endl; cin>>kilom; kilomG += kilom; cout << " Le nombre total de km parcouru par cette voiture est : " << kilomG << endl; break;   case 'H': cout << " Entrez le nombre de kilometres parcouru par cette voiture" << endl; cin>>kilom; kilomH += kilom; cout << " Le nombre total de km parcouru par cette voiture est : " << kilomH << endl; break;     // Dans e cas d'entre d'une voiture non comprise entre A-B-C-D-E-F-G-H, indiquer l'erreura l'utilisateur et continuer   default : cout << " Erreur de saisie" << endl;   }   }     // Dans le cas ou le nom de la voiture vaut S et son km 0, alors on met fin au programme.   cout << "Terminaison du programme " << endl;       }   // Afficher a la fin le nombre total de kilometres parcourus par toutes les voitures   int kilomtotal = 0;   kilomtotal = kilomA + kilomB + kilomC + kilomD + kilomE + kilomF + kilomG + kilomH;   cout << " Le nombre de km parcouru par toutes les voitures est : " << kilomtotal << endl;     return 0;     }

---

J"espere qu'il semble plus presentable que le premier et si ca va au niveau des indentations cette fois.

Sinon pour repondre, oui je suis bien dans un cadre d'etudes. Ca fait un mois que j'ai commencé ...J'en suis qu'aux bases en fait :)

:ccool:

Pour l'indentation, c'est mieux. Par contre, utilise les balises CODE (le bouton # dans la barre d'outils) pour que ton code s'affiche correctement.

Code:

---

char nom_voiture = 'A' && 'B' && 'C' && 'D' && 'E' && 'F'&& 'G'&& 'H';

---

Tu souhaites faire quoi avec cette ligne ?

D'autant plus que la première fois que tu utilises cette variable, tu fais :

Code:

---

cin>> nom_voiture;

---

c'est à dire que tu perds son contenu

Code:

---

int kilom = 0;

---

Pourquoi pas unsigned aussi ?

Code:

---

1 2	{ while (kilom != 0 || nom_voiture != 'S') {

---

Je ne voulais pas dire qu'il fallait utiliser un OU logic, c'est pas compliqué mais c'est pas ça
Le { avant le while ne sert à rien (mais c'est pas une erreur, juste inutile)

Code:

---

1 2	cout << " Entrez le nombre de kilometres parcouru par cette voiture" << endl; cin>>kilom;

---

Tu réécris ces 2 lignes pour chaque case alors que ces lignes ne dépendent pas du switch, tu peux améliorer ça

Code:

---

1 2	int kilomtotal = 0; kilomtotal = kilomA + kilomB + kilomC + kilomD + kilomE + kilomF + kilomG + kilomH;

---

Pas de raison d'écrire 2 lignes là

:plusser: