Discussion :

[olivier1508]

Bonjour à tous,

Je débute en c++ et travaille un peu avec les tableaux dynamiques et les fonctions.

Je voudrais faire un programme me permettant d'encoder, de supprimer le dernier ou d'afficher des élèves. Cela reste simple pour l'instant, j'y vais progressivement pour bien essayer de comprendre.

J'ai le code suivant, mais il ne fonctionne pas. J'arrive a rentrer des noms d'élèves, mais je n'arrive pas à les entrer dans le tableau.

Merci pour votre aide.

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
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
#include <iostream>
#include <vector>
 
using namespace std;
 
void encode_eleve(vector<string> eleves)
{
    string nom;
    cout << "encoder un eleve "; cin >> nom;
    eleves.push_back(nom);
    cout << endl << " taille du tableau "<< eleves.size();
 
}
void supprime_eleve(vector<string> eleves)
{
     eleves.pop_back();
     cout << "le dernier eleve a ete supprime " << endl;
}
void affiche_eleve(vector<string> eleves)
{
    for (int i=0; i< eleves.size();i++)
    {
    cout << "eleve numero " << i << "  " << eleves[i];
    }
}
int main()
{
    vector <string> eleves;
    int choix (0);char rep ('o');
    string nom;
    while (rep=='o')
    {
 
    cout << "bonjour que voulez-vous faire ? " << endl << endl;
    cout << "1. encoder des eleves  " << endl;
    cout << "2. supprimer des eleves " << endl;
    cout << "3. afficher des eleves " << endl;
    cin >> choix;
 
    while (choix <1 || choix >3)
    {
    cout << "veuillez introduire un numéro correct "; cin >> choix;
    }
    if (choix==1)
    {
        encode_eleve(eleves);
 
    }
    if (choix==2)
    {
        supprime_eleve (eleves);
 
    }
    if (choix==3)
    {
        affiche_eleve(eleves);
 
    }
    cout << "Voulez-vous encore faire une opération ?  (o/n)"; cin >> rep;
    }
 
}

[Médinoc]

C'est parce que tes fonctions travaillent à chaque fois sur une copie du vecteur, étant donné qu'il est passé par valeur. Renseigne-toi sur le passage par référence en C++.

[olivier1508]

Bonjour merci pour votre réponse,

je vais consulter les références, mais entre temps, j'avais trouvé un moyen d'y arriver. Je ne sais pas s'il est très orthodoxe, mais il fonctionne. Je suis en train de voir pointeurs et références. j'arriverais peut-être à faire ce programme plus facilement par la suite.

A bientôt

Merci

Olivier

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
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
#include <iostream>
#include <vector>
 
using namespace std;
 
string encode_eleve(vector<string> eleves)
{
    string nom;
    cout << "encoder un eleve "; cin >>nom;
    return nom;
 
}
vector <string> supprime_eleve(vector<string> eleves)
{
     eleves.pop_back();
     cout << "le dernier eleve a ete supprime " << endl;
     return eleves;
}
void affiche_eleve(vector<string> eleves)
{
    for (int i=0; i< eleves.size();i++)
    {
    cout << "eleve numero \t" << i+1 << "  " << eleves[i]<< endl;
    }
}
int main()
{
    vector <string> eleves;
    int choix (0);char rep ('o');
    string nom;
    while (rep=='o')
    {
 
    cout << "bonjour que voulez-vous faire ? " << endl << endl;
    cout << "1. encoder des eleves  " << endl;
    cout << "2. supprimer des eleves " << endl;
    cout << "3. afficher des eleves " << endl;
    cin >> choix;
 
    while (choix <1 || choix >3)
    {
    cout << "veuillez introduire un numéro correct "; cin >> choix;
    }
    if (choix==1)
    {
        nom= encode_eleve(eleves);
        eleves.push_back(nom);
 
    }
    if (choix==2)
    {
        eleves=supprime_eleve (eleves);
 
    }
    if (choix==3)
    {
        affiche_eleve(eleves);
 
    }
    cout << "Voulez-vous encore faire une opération ?  (o/n)"; cin >> rep;
    }
 
}

[Médinoc]

Là ça reste incorrect, mais surtout dans le cas actuel encode_eleve() accepte un paramètre dont elle ne fait rien!

[prgasp77]

Bonjour Olivier,

sais-tu que tu peux configurer ton compilateur pour qu'il t'aide à trouver ces petites erreurs qui ont un grand impact. Pour chacune de ces erreurs ton compilateur te dira : "He mec ! tu prend un paramètre et tu ne l'utilises même-pas ... regarde, *ici*". On ne s'en passe plus.

Comment activer ces warnings ? c'est compilateur-dépendant.

• GCC : -Wall -Wextra -Werror (documentation)
• VC++ : /Wall /W4 /WX (documentation)

Je te conseille vivement de prendre l'habitude de prêter attention et de corriger tous tes warnings. C'est une bonne habitude, et ça permet de repérer plus tôt des erreurs.

[jzyra]

Bonjour,

Il faut que tu passe tes vecteurs en référence.
Pour cela, il faut utiliser le & devant le nom de tes arguments.

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
42
43
44
45
46
47
48
49
50
51
 
#include <iostream>
#include <vector>
 
using namespace std;
 
void encode_eleve(vector<string> &eleves) {
  string nom;
  cout << "encoder un eleve "; cin >> nom;
  eleves.push_back(nom);
  cout << endl << " taille du tableau "<< eleves.size();
}
 
void supprime_eleve(vector<string> &eleves) {
  eleves.pop_back();
  cout << "le dernier eleve a ete supprime " << endl;
}
 
void affiche_eleve(const vector<string> &eleves) {
  for (int i=0; i< eleves.size();i++) {
    cout << "eleve numero " << i << "  " << eleves[i];
  }
}
 
int main() {
  vector <string> eleves;
  int choix (0);char rep ('o');
  string nom;
 
  while (rep=='o') { 
    cout << "bonjour que voulez-vous faire ? " << endl << endl;
    cout << "1. encoder des eleves  " << endl;
    cout << "2. supprimer des eleves " << endl;
    cout << "3. afficher des eleves " << endl;
    cin >> choix;
 
    while (choix <1 || choix >3) {
      cout << "veuillez introduire un numéro correct "; cin >> choix;
    }   
    if (choix==1) {
      encode_eleve(eleves);
    }   
    if (choix==2) {
      supprime_eleve (eleves);
    }   
    if (choix==3)
    {   
      affiche_eleve(eleves);
    }   
    cout << "Voulez-vous encore faire une opération ?  (o/n)"; cin >> rep;
}

Ce code fonctionne.