Discussion :

[jjg65]

Bonjour à tous,
J'ai essayé de composer un petit programme très simple mais je suis confronté à un problème avec l'opérateur de surcharge << qui devrait afficher le contenu des données dans le tableau *tableau de la classe Tableau.
J'ai 2 messages d'erreurs dans la fonction membre :
friend ostream& operator << (ostream&, const& Tableau)
Quelqu'un aurait-il l'amabilité d'analyser le programme et de voir où est l'erreur.
Le programme fonctionne correctement sauf aux endroits ou les lignes de codes sont en gras.
Merci d'avance.

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
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
#include <iostream.h>
#include <stdlib.h>
#include <iomanip.h>

class Tableau{
	private:
	int *tableau;
	int max;
	int nombre_element;
	public:
	Tableau(int n);
	Tableau(const Tableau&);
	~Tableau();
	void initialisation(int);
	void affiche(int);
	void introduction_donnees(int);
	friend ostream& operator << (ostream&, const& Tableau);
};

Tableau::Tableau(int n = 0){
	max = n;
	nombre_element = 0;
	tableau = new int[n];
};

Tableau::Tableau (const Tableau& s) {
	max = s.max;
	nombre_element = s.nombre_element;
	tableau = new int[max];
	for (int i=0; i < max; i++)
		tableau[i] = s.tableau[i];
};



Tableau::~Tableau(){
	delete []tableau;
};

void Tableau::initialisation(int n){
	int i;
	for(i = 0; i < n; i++){
		tableau[i] = 0;
	};
}

void Tableau::affiche(int n){
	int i;
	for(i = 0; i < n; i++){
		cout<<Tableau::tableau[i]<<" ";
	};
};

void Tableau::introduction_donnees(int n){
	int i, donnee;
	for(i = 0; i < n; i++){
		cout<<"donnee numero "<<i+1<<":";
		cin>>donnee;
		tableau[i] = donnee;
		cout<<endl;
	};
};


ostream& operator << (ostream& o, const Tableau& a){
	
	for (int i= 0; i < a.max ; i++){
		o << a.tableau[i] << endl;
		return o;	
	};
	
};

int main(){
int i,n;
cout<<"veuillez introduire la taille de votre tableau sous forme d'un entier : ";
cin>>n;
cout<<""<<endl;
Tableau *un = new Tableau(n);
un->initialisation(n);
cout<<endl;
cout<<"affichage des valeurs contenues dans le tableau"<<endl;
un->affiche(n); 
cout<<""<<endl;
un->introduction_donnees(n);
un->affiche(n);
cout<<""<<endl;

cout<<"affichage avec operateur de surcharge <<"<<endl;
for(i = 0; i < n; i++){
		cout << un->tableau[i];
		cout << endl;
};
return 0;
}

[roulious]

Le probleme vient de la ligne

Code :

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

1
2
 
friend ostream& operator << (ostream &, const& Tableau);

Il faut la remplacer par

Code :

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

1
2
 
friend ostream& operator << (ostream &, Tableau const &);

Dans une déclaration, les symboles & et * doivent êtres placés après le type. Pour le const, c'est un peu plus compliqué : le const s'applique sur tout ce qui est situé à sa gauche, sauf s'il est tout à gauche. Dans ce cas, il sa'pplique au type immédiatement à sa droite. Pour plus de lisibilité, mieux vaut placer les const les plus à droite possible (sans bien sur changer la sémantique).

À part ça, il y a encore quelques erreurs :

• #include <iostream.h> devrait être remplacé par #include <iostream>, de même pour iomanip. Il faudra alors préfixer les membres du namespace std par std::. stdlib.h ne sert à rien, autant ne pas l'inclure.
• Pour utiliser la surcharge dans ton main, remplace
  

  Code :

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

  1
  2
   
   cout << un->tableau[i];

  par
  

  Code :

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

  1
  2
   
  std::cout << (*un);

[loufoque]

Il serait bon de penser à la balise [code], pour la lisibilité.
Pas besoin d'écrire

Code :

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

cout<<Tableau::tableau[i]<<" ";

,

Code :

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

cout << tableau[i] << " ";

suffit.

[jjg65]

Merci pour les infos !

[Ti-R]

Au lieu de

Code :

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

friend ostream& operator << (ostream &, Tableau const &);

Je préfère

Code :

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

friend ostream& operator << (ostream &, const Tableau &);

Mais après tout est question de goût et de couleur.

[rigobert]

Au lieu de

Code :

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

friend ostream& operator << (ostream &, Tableau const &);

Je préfère

Code :

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

friend ostream& operator << (ostream &, const Tableau &);

Mais après tout est question de goût et de couleur.

Pas tout à fait d'accord : la 1ère syntaxe relève du charabia ;
c'est normalement la syntaxe prévue pour les pointeurs constants; l'appliquer aux références n'a, AMHA, absolument aucun sens puisqu'une ref ne peut pas être ré-affectée !

Ce n'est pas une question de goût et de couleur