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 !

**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

**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.

**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;
}

**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?

**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.

vector et string

Langage C++

Vue hybride

Discussions similaires

Partager

Partager