Question Sur l'Opérateur d'affichage C++ <<

Version imprimable

Bonjour,
Voici mon code:

Code:

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
 
#include <iostream>
#include <cmath>
using namespace std;
 
enum Pavillon { JollyRogers, CompagnieDuSenegal, CompagnieDOstende };
 
enum Etat { Intact, Endommage, Coule };
 
int sq(int x)
{
  return x*x;
}
 
class Coordonnees
{
public:
  Coordonnees(int un_x, int un_y) : x_(un_x), y_(un_y) {}
  int x() const {
    return x_;
  }
  int y() const {
    return y_;
  }
  void operator+=(Coordonnees const& autre); 
private:
  int x_;
  int y_;
};
 
class Navire
{
private:
	Coordonnees position_;
protected:
	Pavillon pavillon_;
	Etat etat_;
public:
	Navire(int x, int y, Pavillon p) :
		position_(x, y), 
		pavillon_(p), 
		etat_(Etat::Intact) {};
	Coordonnees position() const { return position_;  }
	void avancer(int de_x, int de_y) {
		position_ += Coordonnees(de_x, de_y);
	}
	void renflouer() { etat_ = Etat::Intact; }
	friend std::ostream& operator <<(std::ostream&, const Coordonnees&); 
	void afficher(ostream& sortie) const {
		sortie << " en " << position_
			   << " battant pavillon " << pavillon_
			   << ", " << etat_;
	}
};
ostream& operator<< (ostream& sortie, Coordonnees const& c) {
	sortie << "(" << c.x() << ", " << c.y() << ")";
	return sortie;
}

J'ai eu deux erreurs:
L'une au niveau de :

Code:

Navire(int x, int y, Pavillon p) :

et l'autre au niveau:

Code:

sortie << " en " << position_

erreur: cannot bind 'std::basic_ostream<char>' lvalue to 'std::basic_ostream<char>&&'

Avez-vous une idée ?
Merci d'avance pour l'aide.
Cordialement.

18/04/2014, 16h22
Invité

Bonjour,

La ligne

Code:

friend std::ostream& operator <<(std::ostream&, const Coordonnees&);

n'a rien à faire dans la classe Navire, elle doit être placée dans la classe Coordonnees.
18/04/2014, 16h30
chercheur111
En fait, j'ai pas le droit de modifier la classe Coordonnees.
Je dois commencer le développement de la solution juste après :
Code:

1 2 class Navire {
18/04/2014, 16h34
Invité

Alors supprime carrément cette ligne :)
Elle ne sert de toute façon pas vu que tu utilises les méthodes d'accès x() et y() dans la définition. Elle le serait si tu voulais utiliser directement les attributs privés x_ et y_.
18/04/2014, 16h56
chercheur111

Merci pour vos réponse. c'est résolu le deuxième point et il me reste le premier point concernant le constructeur..
Avez vous une idée ?
Merci beaucoup.
18/04/2014, 17h00
Invité

Déjà, quel est le message d'erreur ?

Je m'excuse.. en fait c'est au niveau du constructeur de la classe fille:

Code:

1
2
3
4
5
6
 
class Pirate : public Navire {
public:
	Pirate(int x, int y, Pavillon p) 
		: Navire(x, y, p) {};
};

Code:

1
2
3
4
 
bateaux.cc:80:7: note:   candidate expects 1 argument, 0 provided
bateaux.cc:80:7: note: constexpr Pirate::Pirate(Pirate&&)
bateaux.cc:80:7: note:   candidate expects 1 argument, 0 provided

Merci d'avance.

18/04/2014, 17h24
Invité

Le message d'erreur entier aiderait plus..
En tout cas cela semble provenir de l'utilisation que tu fais du constructeur de Pirate tu ne lui fournirais aucun argument alors que lui en demanderait (3 j'imagine avec ce que tu montre), c'est tout ce que je peux te dire avec ces trois petites lignes.
Le début du message te dira où exactement.

Ok, merci.
J'ai une dernière question svp:
Classe Navire:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 ostream& afficher(ostream& sortie) const { sortie<<" en ("<<position_.x()<<", "<<position_.y()<<") battant pavillon "<<pavillon_<<", "<<etat_; return sortie; } ... }; ostream& operator<< (ostream& sortie, Navire const& n) { n.afficher(sortie); return sortie; }
Classe Pirate: fille de Navire
Code:

1 2 3 4 5 void afficher(ostream& sortie) const { sortie << "bateau pirate"; Navire::afficher(sortie); }
à l'affichage d'un bateau pirate:
Code:

1 2 Pirate ship1(0, 0, JollyRogers); cout << ship1 << endl;
j'ai pas eu l'expression "bateau pirate" au début de la phrase !

Merci d'avance.

18/04/2014, 18h40
Invité

Rends ta méthode afficher() virtuelle, et ça devrait aller mieux :)

Merci. J'ai fait les changements nécessaires mais j'ai pas eu l'affichage convenable:
Classe Navire:
Code:

1 2 3 4 5 virtual ostream& afficher(ostream& sortie) const { sortie<<" en ("<<position_.x()<<", "<<position_.y()<<") battant pavillon "<<pavillon_<<", "<<etat_; return sortie; }
Dans l'affichage:
à l'affichage d'un bateau pirate:
Code:

1 2 3 Pirate ship1(0, 0, JollyRogers); cout << ship1 << endl;
bateau pirate ...... le pavillon_ est 0 et l'état_ est 00x6f....
Ma surcharge (externe) de l’opérateur d’afﬁchage << pour les Pavillons:
Code:

1 2 3 4 5 6 7 8 9 ostream& operator<< (ostream& sortie, Pavillon const& p) { switch(p) { case Pavillon::JollyRogers: sortie << "pirate"; break; case Pavillon::CompagnieDuSenegal: sortie << "français"; break; case Pavillon::CompagnieDOstende: sortie << "autrichien"; break; default: sortie << "pavillon inconnu"; } return sortie; }
Et j'ai mis:

Code:

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

dans la classe Navire juste avant la méthode afficher..

Merci d'avance.

18/04/2014, 20h34
Invité

Citation:

Envoyé par chercheur111

Et j'ai mis:

Code:

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

dans la classe Navire juste avant la méthode afficher..

Non cette ligne n'a rien à y faire, de même que précédemment.

Déclarer une fonction en friend dans une classe permet de pouvoir accéder à ses attributs privés directement sans aucun besoin de getter. Mais cela ne concerne que la classe elle-même.

Pavillon est un enum, à portée "globale", tu n'as donc aucun besoin de ce genre de ligne, où que ce soit.

EDIT : à voir l'accolade qui ferme, tu sembles avoir mis sa définition dans la classe Navire ? Non, il faut la mettre à l'extérieur, indépendamment.

Oui, ceci est à l'extérieur de la classe Navire (après la classe Navire).
Code:

1 2 3 4 5 6 7 8 ostream& operator<< (ostream& sortie, Pavillon const& p) { switch(p) { case Pavillon::JollyRogers: sortie << "pirate"; break; case Pavillon::CompagnieDuSenegal: sortie << "français"; break; case Pavillon::CompagnieDOstende: sortie << "autrichien"; break; default: sortie << "pavillon inconnu"; } return sortie; }
Le problème est que la méthode afficher de Navire ne reconnait pas << de Pavillon .. est-ce qu'il y une solution pour résoudre ce pb sans déplacer le code de << de Pavillion avant la classe Navire ?

18/04/2014, 21h34
Invité

Si tu peux mettre la déclaration avant oui, sinon faire utiliser à Navire quelque chose dont il ignore l'existence va être un peu dur ;)

D'accord. Merci bien pour toutes vos remarques.
Il me reste une erreur dans le cadre de cette même problématique:
Dans la classe Navire j'ai une méthode :
Code:

1 2 3 4 5 6 7 8 void rencontrer(Navire& n) { if(etat_ != Etat::Coule && n.etat_ != Etat::Coule && pavillon_ != n.pavillon_ && distance(n) < Navire::rayon_rencontre) { attaque(n); n.replique(*this); }
Qui m'a généré l'erreur suivante:

Code:

fatal error in "test_navire_rencontrer": memory access violation at address: 0x7fffa8889ff8: no mapping at fault address

Je ne sais pas si vous avez une idée sur la cause de telle erreur .. Merci d'avance pour votre réponse.
Cordialement.

19/04/2014, 13h15
Invité

Là comme ça je ne vois rien de suspect.
19/04/2014, 13h22
Bousk

Utilise un debuger, il s'arrêtera sur la ligne qui pose problème et tu pourras constater ce que tu as en mémoire.
19/04/2014, 13h36
chercheur111

J'utilise Geany .. est-ce qu'il offre cette fonctionnalité ?

Vraiment l'erreur m'a bloqué ..
J'essaye d'analyser avec vous, peut être j'aurais la solutio
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 Class Navire{ // double distance(Navire const&, Navire const&); double distance(Navire const& n) { return distance(*this, n); } }; double distance(Navire const& n1, Navire const& n2) { Coordonnees a(n1.position()), b(n2.position()); return distance(a,b); } double Navire::distance(Navire const& n1, Navire const& n2) { return ::distance(n1, n2); }
Ce code ne me génère pas l'erreur, mais je voudrais avoir une solution comme suit:
Code:

1 2 3 4 5 6 7 8 9 ............... double distance(Navire const& n1, Navire const& n2) { Coordonnees a(n1.position()), b(n2.position()); return distance(a,b); } double Navire::distance(Navire const& n1, Navire const& n2) { return distance(n1, n2); }
Ce code me génère le problème du mémoire...
Mon objectif est de trouver une solution qui marche sans utiliser " ::" devant return..

En renommant les fonctions pour y voir plus clair
Le premier code :
Code:

1 2 3 4 5 6 7 double foo(Navire const& n1, Navire const& n2) { Coordonnees a(n1.position()), b(n2.position()); return distance(a,b); } double Navire::distance(Navire const& n1, Navire const& n2) { return foo(n1, n2); }
Le second
Code:

1 2 3 4 5 6 7 double foo(Navire const& n1, Navire const& n2) { Coordonnees a(n1.position()), b(n2.position()); return distance(a,b); } double Navire::distance(Navire const& n1, Navire const& n2) { return distance(n1, n2); }
Boucle infinie, stack overflow, toutes ces joyeusetés.

Si les fonctions ont le même nom, les "::" sont obligatoires (et c'est tout à fait légal en C++. Ce n'est pas une mauvaise pratique.).
Si tu ne veut pas de "::", il va falloir renommer une fonction.

edit : une solution, sans renommer, serait d'avoir une fonction membre
Code:

1 2 3 4 5 6 7 8 9 10 struct Navire { double distance(const Navive& rhs) const { Coordonnees a(position()), b(rhs.position()); return distance(a,b); } static double distance(const Navire& lhs, const Navire& rhs) { return lhs.distance(rhs); } };

19/04/2014, 15h27
Invité

Ton problème vient du fait que tu appelles récursivement la méthode Navire::distance infiniment, tu fais donc "exploser" la pile des appels.

Pourquoi ne pas passer un seul argument et calculer la distance avec l'instance this ? Le compilateur n'aura alors aucun problème à savoir lequel appeler. Sinon il n'existe pas d'autre solution que de rajouter ::

@Iradrille:
J'ai pensé à faire ceci:
Code:

1 2 3 4 5 6 7 8 9 namespace myns { double distance(const Coordonnees &, const Coordonnees&) {...} double distance(const Navire&, const Navire& {...} } double Navire::distance(const Navire& other) { return myns::distance(...); }
mais dans le main(), j'ai ceci:

Code:

cout << "Distance : " << distance(ship1, ship2) << endl;

recommandé par l'exercice.. ;(

Je vais voir avec votre proposition de renommer les méthodes ..

Code:

1 2 3 4 5 6 namespace myns { double distance(const Coordonnees &, const Coordonnees&) {...} double distance(const Navire&, const Navire& {...} } myns::distance(...);
Ou
Code:

1 2 3 4 double distance(const Coordonnees &, const Coordonnees&) {...} double distance(const Navire&, const Navire& {...} ::distance(...);
C'est strictement la même chose, dans un cas, tes fonctions sont dans un namespace myns, dans l'autre dans le namespace global.
Dans les deux cas tu utilises des "::", et dans les deux cas ce n'est pas dérangeant (et même obligatoire). En quoi ces "::" te dérangent exactement ?

Voici l'erreur que m'a générée les ::

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
*** Exécution arrêtée : exception de type Exception :
Compilation échouée
====== Sortie de g++ (1): ======
In file included from /tmp/tmprck_fu/test.cc:153:0:
/tmp/tmprck_fu/student.cc: In member function ‘double h8f38ea9190dd6f1c464fbf7a78f3979b::Navire::distance(const h8f38ea9190dd6f1c464fbf7a78f3979b::Navire&, const h8f38ea9190dd6f1c464fbf7a78f3979b::Navire&)’:
/tmp/tmprck_fu/student.cc:227:10: error: ‘::distance’ has not been declared
   return ::distance(n1, n2);
          ^
compilation terminated due to -Wfatal-errors.
 
====== Sortie de clang++ (1): ======
In file included from /tmp/tmprck_fu/test.cc:153:
/tmp/tmprck_fu/student.cc:227:12: fatal error: no member named 'distance' in the global namespace
  return ::distance(n1, n2);
         ~~^
1 error generated.

19/04/2014, 21h28
chercheur111

J'ai renommé la méthode et ça marche..
Merci bien pour toutes vos réponses :)