Discussion :

[madbox]

Bonjour,

Je viens de commancer à apprendre le c++ avec le livre du zero (ancien nom) de Monsieur NEBRA et SCHALLER

Au niveau du chapitre sur les tableau dynamique j'ai voulu creer un petit programme qui prend la forme suivante :

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
#include <iostream>
#include <vector>
 
using namespace std;
 
int main()
{
    vector<double> tableauNoteUserDynamique(0);
    char yesNoUserNeed();
 
    do
    {
        cout << "Veuillez entrer votre note : " << endl;
        cin >> tableauNoteUserDynamique.push_back();
        cout << "Avez vous d'autres note a enregistrer yes(y) no (n)" << endl;
        cin >> yesNoUserNeed;
 
    } while (yesNoUserNeed == y);
 
    return 0;
}

l'objectif est de permettre à un utilisateur de rentre autant de note qu'il veuille.
Je suis rentrer dans un boulce do while peut être mal écrite avec l’égalité mais l'erreur vient
de la ligne
cin>>tableauNoteUserDynamique.push_back();
D:\c++\madbox\fonction\tableau\moyenneTabUserDynamique\main.cpp|14|error: no matching function for call to 'std::vector<double>::push_back()'|
Voila si quelqu'un pouvais éclairer ma lanterne merci d'avance.
Veuillez excuser mon orthographe et surtout les grossièretés que je peux écrire en C++
Cordialement madbox

[Bktero]

Tu utilises effectivement la méthode std::vector<double>::push_back(). Qu'essayes-tu de mettre dans le vecteur ? Quelle valeur ? Regarde bien ton code, ce qu'il fait, pas ce que tu penses qu'il fait.

Concentre-toi sur cette ligne et trouve ce qui ne va pas :

Code :

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

cin >> tableauNoteUserDynamique.push_back();

PS : Il y a d'autres problèmes dans ton code mais commençons par régler celui-là

[ternel]

Pour t'aider, tu peux regarder une description de vector, telle que: la classe vector sur cppreference.com.

[madbox]

apres relecture du code je me dit que je devrais le développer comme suit :

Code :

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

1
2
3
4
5
6
7
8
9
10
do 
   {
        double userNote();
        cout << "Veuillez entrer votre note : " << endl;
        cin >> userNote();
        tableauNoteUserDynamique.push_back(userNote);
        cout << "Avez vous d'autres note a enregistrer yes(y) no (n)" << endl;
        cin >> yesNoUserNeed;
 
    } while (yesNoUserNeed == y);

J'ai surement voulu prendre un raccourci illogique même si dans mon esprit nébuleux cela paraissait possible.
Je vais tenter de compiler ça se soir pour tester.
Et oui je pense que le while est totalement faux mais l'idée y est je ferai quelques recherche et reviendrai vers vous si nécessaire.
Merci et à bientôt sur le poste.

[ternel]

Il y a du mieux dans l'idée générale: passer par une variable intermédiaire.
Par contre, ton compilateur va hurler, et je te laisse lire sa diatribe.

Au passage, tu devrais regarder ce qu'est le "most vexing parse", c'est l'une de tes erreurs.

[madbox]

Voila le code refait et qui fonctionne dans la version basique

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
#include <iostream>
#include <vector>
using namespace std;
 
double moyenneTabDynamique(vector<double>tableau)
{
    double moyenne;
    for (int i(0);i<tableau.size;i++)
    {
        moyenne+=tableau[i];
    }
    moyenne/=tableau.size;
    return moyenne;
}
 
int main()
{
 
    vector<double>tableauNoteUserDynamique;
    char userChoice;
    do
    {
        cout<<"Veuillez entrer votre note : "<<endl;
        double noteUser(0);
        cin>>noteUser;
        tableauNoteUserDynamique.push_back(noteUser);
        cout<<"Avez vous d'autres note a enregistrer yes(y) no (n)"<<endl;
        cin>>userChoice;
    }while(userChoice=='y');
    moyenneTabDynamique(tableauNoteUserDynamique);
    return 0;
}

l'ont m'as vivement conseiller de changer la varibale userYesNo par useChoice plus lisible.
une histoire de

Code :

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

 std::

à mettre un peu partout que je n'ai pas encore comprise et assimiler.
ce code si fonctionne mais si je tente de mettre la fonction dans un cpp et un .h j'ai des soucis.
Mais je vais soumettre se problème différent dans une nouvelle discusion

[Bktero]

Cette histoire de std:: c'est le principe des namespace : http://cpp.developpez.com/faq/cpp/?page=Les-namespaces

C'est effectivement un meilleur nommage : c'est le choix de l'utilisateur. Ce n'est pas une indication pour dire si on a besoin d'un choix de l'utilisateur.

De la même manière, tu appelles ta variable 'tableau', pourtant ce n'est pas un tableau : c'est un vecteur !!!

[ternel]

Mieux encore, ce sont des notes, autant choisir ce nom.
Il vaut mieux nommer une variable par son utilité que par son type. C'est à dire privilégier le fond à la forme.

[Bktero]

C'est particulièrement juste pour un type aussi générique que vector !