vector et string

**emmesse** · 31/01/2013, 05h28

bonjour à tous,

j'ai ces deux fichiers:
menu:

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
#ifndef menu
#define menu
 
#include <string>
#include <vector>
 
class menu{
   private:
      vector<std::string> options;
      int compteur;
   public:
      menu();
      void ajouterOpion(std::string chaineOption);
      void affiche() const;
      int demanderOption() const;
}
#endif

menu.cpp:

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 "menu"
 
#include <iostream>
#include <string>
using namespace std;
 
 
menu::menu:compteur(0){}
 
void menu::ajouterOption(string chaineOption){
   options[compteur++]=chaineOption;
}
 
vois menu::affiche() const {
   for(int i(1);i<=compteur;i++)
      cout<<i<<"-"<<options[i-1]<<endl;
}
 
int menu::demanderOption() const {
   int i;
   do{
      affiche();
      cout<<"Quel est votre choix? ";
      cin>>i;
      if(i<1)
         cout<<"Pas de valeur nulle ou négatie. Recommencer"<<endl;
      else if(i>compteur+1)
         cout<<"Choix hors limite"<<endl;
   }
   while(i<1||i>compteur+1);
   return i;
}

voici quand je compile:

Code :

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

1
2
3
4
5
6
7
8
9
10
In file included from menu.cpp:1:
menu:9: error: ISO C++ forbids declaration of ‘vector’ with no type
menu:9: error: expected ‘;’ before ‘<’ token
menu:12: error: expected unqualified-id before ‘)’ token
In file included from /usr/include/c++/4.4/bits/ios_base.h:43,
                 from /usr/include/c++/4.4/ios:43,
                 from /usr/include/c++/4.4/ostream:40,
                 from /usr/include/c++/4.4/iostream:40,
                 from menu.cpp:3:
/usr/include/c++/4.4/bits/locale_classes.h:45: error: expected unqualified-id before ‘namespace’

comment faire un tableau dynamique dont les éléments sont de la classe string?

quelqu'un a une idée?

**Ehonn** · 31/01/2013, 07h34

Bonjour

Voici quelques corrections:
Dans menu, à la ligne 9

Code :

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

std::vector<std::string> options;

Dans menu, à la ligne 12 (constructeur par défaut avec liste d'initialisation) (c'est ce qui est fait par défaut)

Code :

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

menu() : compteur(0) { }

Dans menu, à la ligne 16 (ne pas oublier le ;)

Code :

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

};

Dans menu.cpp, supprime la ligne 8 (qui est fausse syntaxiquement)
Dans menu.cpp, à la ligne 11

Code :

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

   options.push_back(chaineOption);

Dans menu.cpp, remplace les lignes 15 et 16 par :

Code :

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

1
2
   for (std::size_t i = 0; i < v.size(); ++i)
      cout << i << "-" << options[i] << endl;

Je te conseille de faire un tour par la FAQ C++ http://cpp.developpez.com/faq/cpp/

Classes: http://cpp.developpez.com/faq/cpp/?page=classes
Namespace : http://cpp.developpez.com/faq/cpp/?page=namespace
La STL (pour std::vector) : http://cpp.developpez.com/faq/cpp/?page=STL

Pour l'utilisation de std::vector, pour ajouter un élément, soit :
- on a déja réservé la taille, et on utilise v[i] = value; .
- on veut ajouter un élément, et on utilise v.push_back(value); .
Ta variable compteur a l'air inutile car elle fait la même chose que v.size() (?)
Pour plus de détails, regarde :
- http://www.cplusplus.com/reference/vector/vector/
- http://en.cppreference.com/w/cpp/container/vector

**r0d** · 31/01/2013, 10h10

Bonjour,

Je te conseille également, et très fortement, de renommer ton fichier "menu" en "menu.h" (et du coup, de modifier, dans menu.cpp, le

Code :

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

#include "menu"

par

Code :

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

#include "menu.h"

)

**Bousk** · 31/01/2013, 10h34

Bonjour,

Dans menu.cpp, supprime la ligne 8 (qui est fausse syntaxiquement)

Cette ligne est tout à fait correcte syntaxiquement parlant.

Le problème initial vient juste du fait que vector fait partie du namespace std, tout comme string que tu as bien écrit.
Les autres remarques sont toutefois légitimes, en particulier concernant le push_back !

**emmesse** · 31/01/2013, 10h39

merci pour vos cntributions

j'ai fait les modifications (je croyais que l'extension ".h" était du C)

Ehonn: quand tu dis v.size(), je dois utiliser options.size()?

maintenant, le compilateur me parle de classe anonyme ne pouvant pas supporter les méthodes et ne reconais plus le tableau "options"

cela vous dit quelque chose?

menu.cpp

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 "menu.h"
 
#include <iostream>
#include <string>
using namespace std;
 
 
 
 
void menu::ajouterOption(string chaineOption){
   options.push_back(chaineOption);
}
 
void menu::affiche() const {
   for (std::size_t i = 0; i < options.size(); ++i)
      cout << i << "-" << options[i] << endl;
}
 
int menu::demanderOption() const {
   int i;
   do{
      affiche();
      cout<<"Quel est votre choix? ";
      cin>>i;
      if(i<1)
         cout<<"Pas de valeur nulle ou négatie. Recommencer"<<endl;
      else if(i>options.size())
         cout<<"Choix hors limite"<<endl;
   }
   while(i<1||i>option.size);
   return i;
}

menu.h

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
#ifndef menu
#define menu
 
#include <string>
#include <vector>
 
class menu{
   private:
      std::vector<std::string> options;
 
   public:
      void ajouterOpion(std::string chaineOption);
      void affiche() const;
      int demanderOption() const;
};
#endif

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
In file included from menu.cpp:1:
menu.h:7: error: an anonymous struct cannot have function members
menu.h:9: error: member ‘std::vector<std::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::allocator<std::basic_string<char, std::char_traits<char>, std::allocator<char> > > > <anonymous class>::options’ with constructor not allowed in anonymous aggregate
menu.h:9: error: member ‘std::vector<std::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::allocator<std::basic_string<char, std::char_traits<char>, std::allocator<char> > > > <anonymous class>::options’ with destructor not allowed in anonymous aggregate
menu.h:9: error: member ‘std::vector<std::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::allocator<std::basic_string<char, std::char_traits<char>, std::allocator<char> > > > <anonymous class>::options’ with copy assignment operator not allowed in anonymous aggregate
menu.h:15: error: abstract declarator ‘<anonymous class>’ used as declaration
menu.cpp:10: error: explicit qualification in declaration of ‘void ajouterOption(std::string)’
menu.cpp: In function ‘void ajouterOption(std::string)’:
menu.cpp:11: error: ‘options’ was not declared in this scope
menu.cpp: At global scope:
menu.cpp:14: error: explicit qualification in declaration of ‘void affiche()’
menu.cpp:14: error: non-member function ‘void affiche()’ cannot have cv-qualifier
menu.cpp: In function ‘void affiche()’:
menu.cpp:15: error: ‘options’ was not declared in this scope
menu.cpp: At global scope:
menu.cpp:19: error: explicit qualification in declaration of ‘int demanderOption()’
menu.cpp:19: error: non-member function ‘int demanderOption()’ cannot have cv-qualifier
menu.cpp: In function ‘int demanderOption()’:
menu.cpp:27: error: ‘options’ was not declared in this scope
menu.cpp:30: error: ‘option’ was not declared in this scope

**emmesse** · 31/01/2013, 10h44

j'avais bien pensé au push_back, mais après avoir posté le message...
Sinon, si on indique using namespace std, on est quand-même obligé d'écrire std::size_t au lieu de size_t?

**Ehonn** · 31/01/2013, 10h58

Envoyé par r0d

Bonjour,

Je te conseille également, et très fortement, de renommer ton fichier "menu" en "menu.h" (et du coup, de modifier, dans menu.cpp, le

Code :

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

#include "menu"

par

Code :

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

#include "menu.h"

)

Ou menu.hpp c'est plus joli ^^

Envoyé par Bousk

Cette ligne est tout à fait correcte syntaxiquement parlant.

Je ne savais pas qu'on pouvait omettre les parenthèses dans menu::menu:compteur(0){}

Envoyé par matser

j'ai fait les modifications (je croyais que l'extension ".h" était du C)

En C++, l'extension n'a pas d'importance pour les fichiers d'entête.

Envoyé par matser

Ehonn: quand tu dis v.size(), je dois utiliser options.size()?

Oui, ça retourne la taille du vector (le nombre d'éléments) (il faut imaginer que v est une variable de type std::vector<T>)

Envoyé par matser

maintenant, le compilateur me parle de classe anonyme ne pouvant pas supporter les méthodes et ne reconais plus le tableau "options"

C'est une erreur bête du préprocesseur.
Lorsque tu protèges ton fichier d'entête, tu fais :

Code :

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

1
2
#ifndef menu
#define menu

Et lorsqu'il voit class menu {, il remplaces menu.
Tu peux donc écrire quelque chose comme :

Code :

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

1
2
#ifndef MENU_HPP
#define MENU_HPP

**r0d** · 31/01/2013, 11h02

Envoyé par matser

je croyais que l'extension ".h" était du C

En fait, tu peux mettre ce que tu veux. L'extension est une convention (c'est spécifié dans aucun standard mais tout le monde fais comme ça), et comme toute convention, il faut vraiment bien comprendre le pourquoi du comment avant de la transgresser. Donc en c++, la convention est: .h ou .hpp pour les en-têtes, .cpp pour les sources (il y en a d'autres, mais ceux-là sont les principaux).

Tu dois confondre avec les en-têtes de la bibliothèque standard (standard library, ou SL), qui, en C sont suffixés par ".h", et en c++ ne sont pas suffixés. Mais l'absence de suffixe ne concerne que les en-têtes de la SL.

**Bousk** · 31/01/2013, 11h14

Envoyé par Ehonn

Je ne savais pas qu'on pouvait omettre les parenthèses dans menu::menu:compteur(0){}

Effectivement j'avais mal vu, non les parenthèses des arguments sont bien sur obligatoires.
Mais il n'est nullement besoin de déplacer la déclaration dans le header pour autant.

**Ehonn** · 31/01/2013, 11h18

Envoyé par Bousk

Effectivement j'avais mal vu, non les parenthèses des arguments sont bien sur obligatoires.
Mais il n'est nullement besoin de déplacer la déclaration dans le header pour autant.

Oui, bien sûr.
Moi aussi j'avais mal regardé et n'avais pas compris que c'était une tentative d'écriture du constructeur (je pensais que c'était une tentative d'initialisation d'un supposé membre statique... j'avais pas vu les {}).
Du coup, c'était plus simple de mettre le reste du constructeur dans le fichier d'entête.

Envoyé par matser

j'avais bien pensé au push_back, mais après avoir posté le message...
Sinon, si on indique using namespace std, on est quand-même obligé d'écrire std::size_t au lieu de size_t?

En effet, le std:: est ici superflu.

**emmesse** · 31/01/2013, 11h36

bien changé "menu" par "MENU_HPP'
et aussi renomé le fichier d'entête (menu.hpp)
j'ai effectivement oublié les parenthèses à la ligne 30

j'ai corrigé le code source selon vos conseilss

la compilation ne fait plus d'histoires

Merci à tous, j'ai résolu le problème:

menu.hpp:

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
#ifndef MENU_HPP
#define MENU_HPP
 
#include <string>
#include <vector>
 
class menu{
   private:
      std::vector<std::string> options;
 
   public:
      void ajouterOption(std::string chaineOption);
      void affiche() const;
      int demanderOption() const;
};
#endif

menu.cpp

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 "menu.hpp"
 
#include <iostream>
#include <string>
using namespace std;
 
 
 
 
void menu::ajouterOption(string chaineOption){
   options.push_back(chaineOption);
}
 
void menu::affiche() const { //les menus commencent à 1. c'est pourquoi j'utilise options[i-1]
   for (size_t i = 1; i <= options.size(); ++i)
      cout << i << "-" << options[i-1] << endl;
}
 
int menu::demanderOption() const {
   int i;
   do{
      affiche();
      cout<<"Quel est votre choix? ";
      cin>>i;
      if(i<1)
         cout<<"Pas de valeur nulle ou négatie. Recommencer"<<endl;
      else if(i>options.size())
         cout<<"Choix hors limite"<<endl;
   }
   while(i<1||i>options.size());
   return i;
}

**Ehonn** · 31/01/2013, 11h57

Bien

Je n'ai pas compris ton histoire de i - 1.
Les indices vont de 0 à options.size() - 1 compris.
Lorsque tu fais un parcours, tu vas de 1 à options.size() compris et fais un accès en [i - 1], c'est strictement identique à ma version de la boucle (qui fait un accès en [i]).

Pour void affiche() const, tu peux aussi surcharger l'opérateur << pour std::ostream. Tu pourrais donc écrire std::cout << un_menu << std::endl;.

Petite optimisation, dans la signature de

Code :

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

void ajouterOption(std::string chaineOption);

, tu pourrais passer chaineOption en référence sur membre constant pour éviter une copie et interdire une modification. La signature serait donc void ajouterOption(std::string const & chaineOption);..

**emmesse** · 31/01/2013, 13h50

Lorsque tu fais un parcours, tu vas de 1 à options.size() compris et fais un accès en [i - 1], c'est strictement identique à ma version de la boucle (qui fait un accès en [i]).

c'est parce que la première option du menu, à l'affichage, doit correspondre à 1. En partant de 0: cout<< i+1<< " - " << options[i]

tu pourrais passer chaineOption en référence sur membre constant pour éviter une copie et interdire une modification. La signature serait donc void ajouterOption(std::string const & chaineOption);.

ok, je modifie la signature (et la definition)

Pour void affiche() const, tu peux aussi surcharger l'opérateur << pour std::ostream. Tu pourrais donc écrire std::cout << un_menu << std::endl;.

je n'ai pas compris pourquoi.
sinon voici comment cette classe peut être utilisé:
mainMenu.cpp

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
#include <iostream>
#include <string>
using namespace std;
#include "menu.hpp"
 
int main(){
   int resultatMenu;
   menu sudokuAccueil;
   sudokuAccueil.ajouterOption("premier choix");
   sudokuAccueil.ajouterOption("deuz");
   sudokuAccueil.ajouterOption("non");
   resultatMenu=sudokuAccueil.demanderOption();
   cout<<"vous avez choisi l'option "<<resultatMenu<<endl;
}

**emmesse** · 31/01/2013, 14h20

Ah si, ça y est, j'ai compris pourquoi surcharger <<...

**Ehonn** · 31/01/2013, 16h04

Oui, c'est mieux cout<< i+1<< " - " << options[i]Car la boucle a une écriture plus conventionnelle.

Pour les fflux (operateur << et >>)
http://cpp.developpez.com/faq/cpp/?page=SL#SL_operateur_affichage
Ç'est mieux que les méthodes affiche et demanderOption (et l'un n'empêche pas l'autre).

**emmesse** · 07/02/2013, 06h45

Merci à tous pour vos contributions. Voici ce que cela donne:
menu.hpp:

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
#ifndef MENU_HPP
#define MENU_HPP
 
#include <string>
#include <vector>
 
class Menu{
   private:
      std::vector<std::string> m_options;
      int m_choixOption;
   public:
      friend std::ostream& operator <<(std::ostream& flux, const Menu& menu);
      friend std::istream& operator >>(std::istream& flux, Menu& menu);
      void m_ajouterOption(std::string const & chaineOption);
      int m_getOption();
};
#endif

menu.cpp:

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
#include "menu.hpp"
 
#include <iostream>
#include <string>
using namespace std;
 
 
void Menu::m_ajouterOption(string const & chaineOption){
   m_options.push_back(chaineOption);
}
 
std::ostream &operator<<(std::ostream &flux,Menu const & menu){
   for (size_t i = 0; i <= menu.m_options.size()-1; i++)
      flux << i+1 << " - " << menu.m_options[i] << endl;
   return flux;
}
 
std::istream& operator >>(std::istream& flux, Menu& menu){
   int i,nombreOptions(menu.m_options.size());
   do{
      cout << "Quel est votre choix? ";
      flux >> i;
      if(i<1)
         cout << "Pas de valeur nulle ou négative. Recommencez" << endl;
      else if(i > nombreOptions)
         cout << "Choix hors limite" << endl;
   }
   while(i < 1 || i > nombreOptions);
   menu.m_choixOption=i;
   return flux;
}
 
int Menu::m_getOption(){
   return m_choixOption;
}

mainMenu.cpp:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
#include <iostream>
#include <string>
#include "menu.hpp"
using namespace std;
 
int main(){
   Menu sudokuAccueil;
   sudokuAccueil.m_ajouterOption("premier choix");
   sudokuAccueil.m_ajouterOption("deuz");
   sudokuAccueil.m_ajouterOption("non");
   cout<<sudokuAccueil;
   cin>>sudokuAccueil;
   cout<<"vous avez choisi l'option "<<sudokuAccueil.m_getOption()<<endl;
}

et voici un exemple d'éxécution:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
1 - premier choix
2 - deuz
3 - non
Quel est votre choix? 0
Pas de valeur nulle ou négative. Recommencez
Quel est votre choix? 4
Choix hors limite
Quel est votre choix? -4
Pas de valeur nulle ou négative. Recommencez
Quel est votre choix? 2
vous avez choisi l'option 2

merci à tous d'avoir participé à cette discussion

**emmesse** · 27/03/2013, 17h50

j'ai un peu modifié oprator >> pour qu'il répète le menu en cas de mauvaise réponse :

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
std::istream& operator >>(std::istream& flux, Menu& menu){
   int i,nombreOptions(menu.m_options.size());
   do{
      cout << "Quel est votre choix? ";
      flux >> i;
      if(i<1){
         cout << "Pas de valeur nulle ou négative. Recommencez" << endl << menu <<endl;
      }
      else if(i > nombreOptions){
         cout << "Choix hors limite" << endl << menu << endl;
      }
   }
   while(i < 1 || i > nombreOptions);
   menu.m_choixOption=i;
   return flux;
}

un petit exemple:

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
matser@ordiport:~/experiences/menu$ g++ -o quatre *cpp
matser@ordiport:~/experiences/menu$ ./quatre 
1 - premier choix
2 - deuz
3 - non
Quel est votre choix? 0
Pas de valeur nulle ou négative. Recommencez
1 - premier choix
2 - deuz
3 - non
Quel est votre choix? -5
Pas de valeur nulle ou négative. Recommencez
1 - premier choix
2 - deuz
3 - non
Quel est votre choix? 6
Choix hors limite
1 - premier choix
2 - deuz
3 - non
Quel est votre choix? 2
vous avez choisi l'option 2

**Bousk** · 27/03/2013, 20h30

Bonsoir,

je suis pas trop fan de cette surcharge. En plus de devoir stocker le résultat du menu dans la classe.
Je préfèrerais nettement une implémentation prototypée unsigned int Menu::MakeChoice() const;Mais dans le principe, ça permet d'appréhender la surcharge d'opérateur, et en particulier l'utilisation des stream, ce qui reste intéressant.

**emmesse** · 29/03/2013, 12h57

Merci Bousk,
voici ce que cela donne avec tes conseils:
menu.hpp:

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
#ifndef MENU_HPP
#define MENU_HPP
 
#include <string>
#include <vector>
 
class Menu{
   private:
      std::vector<std::string> m_options;
   public:
      friend std::ostream& operator <<(std::ostream& flux, const Menu& menu);
      int m_choixOption() const;
      void m_ajouterOption(std::string const & chaineOption);
};
#endif

menu.cpp:

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
#include "menu.hpp"
 
#include <iostream>
#include <string>
using namespace std;
 
 
void Menu::m_ajouterOption(string const & chaineOption){
   m_options.push_back(chaineOption);
}
 
ostream &operator<<(ostream &flux,Menu const & menu){
   for (size_t i = 0; i <= menu.m_options.size()-1; i++){
      flux << i+1 << " - " << menu.m_options[i];
      if(i<menu.m_options.size()-1)
         flux<<endl;
   }
   return flux;
}
 
int Menu::m_choixOption() const{
   int i,nombreOptions(m_options.size());
   do{
      cout << "Quel est votre choix? ";
      cin >> i;
      if(i<1){
         cout << endl << "Pas de valeur nulle ou négative. Recommencez" << endl << *this <<endl;
      }
      else if(i > nombreOptions){
         cout << endl << "Choix hors limite. Recommencez" << endl << *this << endl;
      }
   }
   while(i < 1 || i > nombreOptions);
   return i;
}

un petit exemple:

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
1 - premier choix
2 - deuz
3 - non
Quel est votre choix? 0
 
Pas de valeur nulle ou négative. Recommencez
1 - premier choix
2 - deuz
3 - non
Quel est votre choix? -2
 
Pas de valeur nulle ou négative. Recommencez
1 - premier choix
2 - deuz
3 - non
Quel est votre choix? 5
 
Choix hors limite. Recommencez
1 - premier choix
2 - deuz
3 - non
Quel est votre choix? 3
vous avez choisi l'option 3

vector et string

Langage C++

Discussions similaires

Partager

Partager