un cin qui coince !

Version imprimable

Voir 40 message(s) de cette discussion en une page

un cin qui coince !

Bonjour.
J'ai un exo sur tableau dynamique récalcitrant à cause d'un cin

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
 
#include <iostream>
#include <vector>
using namespace std;
 
int main()
{
 vector<double> notesSur20;  
 for(int i(0); i<notesSur20.size(); ++i)
    {
     cout<<"donne une note sur 20 : ";
     cin >> notesSur20.push_back[0];  // ERROR : invalid types ‘<unresolved overloaded function type>[int]’ for array subscript
    }
 double moyenne(0); 
      for(int i=0; i<notesSur20.size(); ++i)
         {
           moyenne += notesSur20 [i]; 
         }
      moyenne /= notesSur20.size();  
     cout << "La moyenne de toutes ces notes est :  "<< moyenne << endl;
 return 0;
}

13/01/2017, 16h17
ternel
push_back est une fonction, donc push_back[20] est une erreur.
tu ne peux qu'appeler cette fonction (push_back(20)).

Qui plus est, l'opérateur >> n'est défini que pour cin et une référence modifiable vers certains types, dont double.

Il faut donc procéder en deux temps:
Code:

1 2 3 double d; cin >> d; notesSur20.push_back(d);
En fait, il y a deux sous parties à ta boucle.
- lire une note, qui pourrait etre une fonction retournant la note
- stocker la note lue dans une liste.

hiii, merci ternelll en plus que c'est clair: je pige!!!
Oui mais... heu, le code fonctionne, pas d'erreur:

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
 #include <iostream>
#include <vector>
using namespace std;
 
// moyenneDesValeursEntrees2
/* Calcul de la moyenne des valeurs contenues dans un tableau (dynamique)
 * tableau dynamique: vector<type>nom(taille); et en haut un include <vector> */
 
int main()
{
 vector<double> notesSur20; 
 double note ; // je déclare la variable note qui répondra au cout ci-dessous
 for(int i(0); i<notesSur20.size(); ++i)
    {
     cout<<"donne une note sur 20 : ";
     cin >> note;  // ici l'utilisateur tape sa note
     notesSur20.push_back(note);
    }
 double moyenne(0); 
      for(int i=0; i<notesSur20.size(); ++i)
         {
           moyenne += notesSur20 [i]; 
         }
      moyenne /= notesSur20.size(); 
 cout << "La moyenne de toutes ces notes est :  "<< moyenne << endl;
 return 0;
}

donc le job fonctionne mais ne me demande pas "donne une note sur 20 : " et m'envoie:
"La moyenne de toutes ces notes est : -nan
... s'est terminé avec le code 0 "

13/01/2017, 18h29
Matt_Houston

Quelle est la taille de ton vecteur au lancement du programme ?
13/01/2017, 21h36
moniroge

heu, je ne comprends pas trop la question;
la taille du tableau, au départ... nulle je suppose; c'est un tableau vide au départ.
13/01/2017, 22h09
Matt_Houston

Et donc ? ;)

Qu'en déduis-tu en traçant l'exécution de ton code, à la main ?

Un indice : une valeur NaN (Not-a-Number) résulte le plus souvent d'une division par zéro.
14/01/2017, 13h43
moniroge

Hi hi, tu me guides bien:
quand j'ai vu ta suggestion avant de me coucher, j'étais rouge de honte!!
ben oui, j'ai divisé ma moyenne par zéro avec ma fichue boucle!!! donc ça a tout arrêté!!!
Mais ce matin, j'écris à la main et:
pas du tout!! c'est en dehors de la boucle for que je fais la division:
moyenne /= notesSur20.size();
je divise la somme totale des cases par la taille du tableau
et ici, le code doit voir la taille totale du tableau et donc ne pas dire nan !!! :(
14/01/2017, 19h40
ternel

Et ta boucle, elle commence comment?
14/01/2017, 21h20
moniroge

Ben ma boucle pour moyenne , comme toute boucle d'itération, commence avec la case numéro 0
moyenne += notesSur20 [0] à qui elle prend la valeur mise dedans et la met dans la variable moyenne
continue avec la case numéro1 dont elle prend la valeur et l'ajoute dans moyenne à celle de la case 0
etc.
et quand elle arrive à la dernière case, [28] par exemple, moyenne a pour valeur la somme de toutes les valeurs,
la boucle for s'arrête et le programme passe à l'opération suivante:
moyenne /= notesSur20.size();
soit il va faire la division de la somme contenue dans moyenne par la taille du tableau : 29 pour mon exemple..
ça devrait marcher!!! ça ne marche pas, vous avez évidemment raison
et donc la faille dans mon code, je vois toujours rien
et même, il me semble que je ne comprends plus rien.
14/01/2017, 21h33
jo_link_noir

Ligne 13, quel est le résultat de notesSur20.size() ? Et par extension, quand s'arrête la boucle ?
14/01/2017, 22h18
moniroge
la ligne 13: for(int i(0); i<notesSur20.size(); ++i)
eh bien la boucle commence avec i=0: la première case du tableau
et elle se termine... quand l'utilisateur ne cin pas note; soit qu'il ne tape plus à "donne une note sur 20 : "
non ???
Tiens, que mon début de boucle vous turlupine, je vais essayer ceci:
Code:

1 2 int i(0); // je déclare la variable d'itération for(int i=0; i<notesSur20.size(); ++i)
Ben, toujours "la moyenne de toutes ces notes: -nan
Bon là, ja vais me coucher... d'un exo simple, je fais un sac de noeuds!!!

Bonsoir,

Tu vas trop vite dans ton raisonnement. Il ne faut pas penser en termes « je m'attends à ce que mon programme fasse ceci, cela et j'ai confiance dans ce que j'ai écrit » mais « ce que j'ai écrit ne va justement pas, et je l'analyse finement afin de savoir pourquoi il ne fait pas ce que j'attends ».

Concentrons-nous sur cette partie de code :
Code:

1 2 3 4 5 6 7 8 vector<double> notesSur20; double note ; // je déclare la variable note qui répondra au cout ci-dessous for(int i(0); i<notesSur20.size(); ++i) { cout<<"donne une note sur 20 : "; cin >> note; // ici l'utilisateur tape sa note notesSur20.push_back(note); }
Matt_Houston t'a posé une question tout à fait pertinente :

Citation:

Quelle est la taille de ton vecteur au lancement du programme ?

à laquelle tu as parfaitement bien répondu :

Citation:

la taille du tableau, au départ... nulle je suppose; c'est un tableau vide au départ.

et jo_link_noir a à nouveau mis le doigt dessus, mais sans grand succès... Et pourtant c'est tout le problème. ;)

Donc à nouveau : que vaut notesSur20.size() au départ de ta boucle ? Quelle incidence cela a sur ta boucle ?

réflexion...

Toujours pas de déclic ? Quelles sont alors les différentes étapes et instructions de ta boucle ?
Initialisation, condition, incrémentation, instructions... Décris-nous très précisément chacune de ces étapes.
Prends pour cela un papier et un crayon, et note pour chacune de ces étapes les états des variables/expressions qui interviennent.

Si tout va bien tu te rendras vite compte de quelque chose :)

Autre indice : le code :

Code:

1
2
3
4
5
6
7
8
vector<double> notesSur20; 
double note;
for(int i(0); i<notesSur20.size(); ++i)
{
	cout<<"donne une note sur 20 : ";
	cin >> note;  // ici l'utilisateur tape sa note
	notesSur20.push_back(note);
}

équivaut au code :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
vector<double> notesSur20; 
double note;
{
	int i(0);
	while(i<notesSur20.size())
	{
		cout<<"donne une note sur 20 : ";
		cin >> note;  // ici l'utilisateur tape sa note
		notesSur20.push_back(note);
		++i;
	}
}

16/01/2017, 15h27
moniroge

Merci à vous d'être aussi patient, hi hi...
et désolé de vous répondre avec retard... (tit problème de santé)
Voilà, j'ai à peine commencé à écrire sur un papier mon code (la moitié d'une ligne)
que hop!! la lumière (de vos projecteurs) :
ben oui, quand la boucle for commence, c'est avec i=0 et la taille du tableau est nulle!! or j'ai la condition: i<notesSur20.size(); soit une impossibilité qui => le nan à la console...
et donc le dernier des Neurones s'empressa de corriger:

Code:

for (int i=0; i<(notesSur20.size()+1); ++i)

et enfin, la console demande la série de notes...
Donc ce problème réglé...
Maintenant, je dois trouver moyen de demander qu'une fois la dernière note entrée,
le job fasse la moyenne....
bon, je m'y attèle.
16/01/2017, 15h30
Bousk

Non.
Tu demandes à boucler sur i de 0 à taille du tableau, ton tableau est nul, donc tu boucles pas.
Si par hasard tu bouclais, tu ajoutes une données au tableau dans la boucle => boucle infinie.
C'est pas un problème de programmation mais d'algorithmique.
16/01/2017, 15h32
ternel

En fait, il faut que tu trouves un moyen de trouver quelle est la dernière note.
Tu as deux solutions:
demander avant le nombre de notes à lire.
demander une réponse et voir si c'est une note, la fin de la liste ou encore une fausse manipulation.

Le premier te permet de rester avec cin, mais ce n'est pas la bonne utilisation.
Le second demandera que tu passes par un getline(cin, ligne) (ou ligne est une string), puis que tu analyses cette ligne (std::istringstream sera utile à cette fin)

Hello Bousk!!
Diantre, tout le forum vient m'aider!!! hi hi, c'est chouette!!!
mais bon, vous ne me connaissez pas, vous allez vite en avoir marre de moi...
Je suppose que Bousk n'a pas vu mon post de 15h27 ???
En attendant, j'ai fait du progrès que voici:
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 #include <iostream> #include <vector> using namespace std; int main() { vector<double> notesSur20; // tableau vide: pas de parenthèses ni de taille double note ; int i(0); double moyenne(0); // for(int i=0; i<notesSur20.size(); ++i) /* au départ de cette boucle, donc avec i=0, déjà un truc curieux: i doit * être < à la taille du tableau qui à ce moment est zéro: donc ça ne peut pas marcher. Je vais essayer ceci: */ for (int i=0; i<(notesSur20.size()+1); ++i) { cout<<"donne une note sur 20 : "; cin >> note; notesSur20.push_back(note); moyenne += notesSur20 [i]; moyenne /= notesSur20.size(); } if(note<0 || note>20 ) // j'aimerais bien aussi mettre note !=int pour qui taperait une lettre { cout <<"Vous avez entré "<< notesSur20.size() <<" notes."<<endl; cout << "La moyenne de toutes ces notes est : "<< moyenne << endl; } return 0; }
ah ben zut alors!! il entrait les notes et si je mettais une note 23, hop!
* il me disait: vous avez entré 6 notes: super-chouette!
puis pas de moyenne,:( et de plus, si je tapais la lettre a,
* le code moulinait dans la choucroute quelque secondes...:weird:
* à re-essayer, naouww, comme présenté ci-dessus, il ne me dit même plus combien de notes j'ai entrées :(

16/01/2017, 17h31
Matt_Houston

Non, Bousk répondait bien à ton post de 15h27. Lis bien attentivement les deux messages qui le suivent. Si tu ne comprends pas quelque chose dans les propos des intervenants, demande des compléments sur des points précis.
16/01/2017, 19h36
Invité
Pose-toi la bonne question : tu souhaites continuer à boucler tant que quoi ?
Dès que tu répondras de façon posée à cette question, tu auras ta bonne condition de boucle.

ternel t'a déjà donné une bonne piste de réflexion.

En complément, ce sujet de FAQ : Comment vérifier les valeurs saisies avec cin ?

Sinon concernant la partie suivante :
Code:

1 2 3 notesSur20.push_back(note); moyenne += notesSur20 [i]; moyenne /= notesSur20.size();
Hors code, reprends papier/crayon et prends les notes {5, 10, 15}.
Quel résultat obtiens-tu ? Quel résultat devrais-tu obtenir ?
19/01/2017, 16h23
moniroge

Hi!!! ne croyez pas que j'ai laissé tomber!!!
En fait, j'ai regardé chacune de vos remarques, conseils, recommandations, hi hi
et ça m'a demandé du temps pour faire évoluer mon code,
si, il a évolué... il a fonctionné... pour se planter plus loin...
et puis je découvre GitHub pour ne pas surcharger ici, est-ce que ça marche??? dites-moi, vous le voyez, mon code ???
parce que GitHub, c'est écrit dans un anglais épouvantable, tiens, je ne sais même pas supprimer, pas vu de delete....
Ah oui, mon code...
https://github.com/moniroje/moyenneD...aster/main.cpp
et aujourd'hui que je suis en Suse: CodeBlocks ne marche pas, et, surprise, QtCreator ne marche plus...
Donc demain, je retourne sur Debian et CodeBlocks.

Voir 40 message(s) de cette discussion en une page