Discussion :

[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 ::

[chercheur111]

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

Code :

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

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 :

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

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

recommandé par l'exercice.. ;(

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

[Iradrille]

Code :

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

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

Ou

Code :

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

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 ?

[chercheur111]

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

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

[chercheur111]

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