namespace et class

problème :(

Sujet résolu

4 janvier 2008 à 15:28:07

Bonjour,

Je voudrais utiliser un namespace dans mon programme qui englobe les class et les fonctions, enfin tout quoi, un peu comme la lib standard ou SFML.

quand je fais :

namespace Hello
{
class Salut
{
public:
void menu ()
{
std::cout << "Salut !" << std::endl;
}
};
}

bien entendu, ça compile ( et ça fonctione

)

mais quand je fais :

namespace Hello
{
class Salut
{
public:
void menu ();
};
}

et que dans un autre fichier je fais :

void Hello::Salut::menu()
{
std::cout << "Salut !" << std::endl;
}

ça ne compile pas, le linker me sort des erreurs incomprehensible ...

Je suppose qu'il y a une façons précise de se servir d'un namespace ?

Merci d'avance

chlablak

4 janvier 2008 à 15:36:30

les erreurs stp

C++ Références | Idiomes | FAQ

Skypers

4 janvier 2008 à 15:39:54

Je n'ai jamais utilisé les namespaces, je compte le faire bientôt, mais tu es sûr que ce n'est pas

namespace Hello::void Salut::menu()
{
std::cout << "Salut !" << std::endl;
}

C'est juste une suggestion j'ai jamais utilisé !

chlablak

4 janvier 2008 à 15:41:44

non son code est juste a premiere vue, il nous faut les erreurs

C++ Références | Idiomes | FAQ

noob4ever

4 janvier 2008 à 15:47:16

Citation : -Skypers-

Je n'ai jamais utilisé les namespaces, je compte le faire bientôt, mais tu es sûr que ce n'est pas

namespace Hello::void Salut::menu()

{

std::cout << "Salut !" << std::endl;

}

C'est juste une suggestion j'ai jamais utilisé !

ça ne marche pas non plus

il me dit erreur de syntax.

Citation : Chlab_lak

non son code est juste a premiere vue, il nous faut les erreurs

ok ...

1>Linking...

1>menu.obj : error LNK2005: "int __cdecl Acht::BoundingBox(class sf::Event,class sf::Sprite)" (?BoundingBox@Acht@@YAHVEvent@sf@@VSprite@3@@Z) already defined in MainEntryPoint.obj

1>C:\Users\Adrien\Documents\Visual Studio 2008\Projects\sfml-dev-test\Debug\sfml-dev-test.exe : fatal error LNK1169: one or more multiply defined symbols found

1>Build log was saved at "file://c:\Users\Adrien\Documents\Visual Studio 2008\Projects\sfml-dev-test\sfml-dev-test\Debug\BuildLog.htm"

je suppose qu'avec ça il te faut le code ?

namespace Acht
{
int BoundingBox(sf::Event a, sf::Sprite b)
{
if ((a.Mouse.X >= b.GetLeft() + b.GetWidth())
|| (a.Mouse.X <= b.GetLeft())
|| (a.Mouse.Y >= b.GetTop() + b.GetHeight())
|| (a.Mouse.Y <= b.GetTop()))
{
return 0;
}
return 1;
}
class Menu
{
private:
sf::Image m_image[3];
sf::Sprite m_sprite[3];
sf::Event Event;
bool running;
public:
Menu(sf::RenderWindow& ref)
{
m_image[0].LoadFromFile("data/menu.jpg");
m_image[1].LoadFromFile("data/jouer.jpg");
m_image[2].LoadFromFile("data/quitter.jpg");
for (int i=0; i<=2; i++)
m_sprite[i].SetImage(m_image[i]);
m_sprite[1].SetLeft(200.f);
m_sprite[1].SetTop(160.f);
m_sprite[2].SetLeft(700.f);
m_sprite[2].SetTop(160.f);
running = true;
this->menu(ref);
}
void menu (sf::RenderWindow& );
};
}

et la définition :

#include "namespace.hpp"
void Acht::Menu::menu(sf::RenderWindow& screen)
{
while (running)
{
while (screen.GetEvent(Event))
{
switch (Event.Type)
{
case sf::Event::Close:
running = false;
break;
case sf::Event::MouseButtonPressed:
switch (Event.Mouse.Buttons)
{
case sf::Mouse::Left:
if (Acht::BoundingBox(Event, m_sprite[1]))
{
// jouer();
}
else if (Acht::BoundingBox(Event, m_sprite[2]))
running = false;
break;
}
break;
default:
break;
}
}
for (int i=0; i<=2; i++)
screen.Draw(m_sprite[i]);
screen.Display();
}
}

je le répète, si je définis la méthode menu directement dans la class, il n'y a aucun problème.

chlablak

4 janvier 2008 à 16:06:28

essaye de changer le nom de ta fonction Acht::BoundingBox() (ex: Acht::Bounding_Box() )

de plus j'ai un remarque sur ce code:

int BoundingBox(sf::Event a, sf::Sprite b)
{
if ((a.Mouse.X >= b.GetLeft() + b.GetWidth())
|| (a.Mouse.X <= b.GetLeft())
|| (a.Mouse.Y >= b.GetTop() + b.GetHeight())
|| (a.Mouse.Y <= b.GetTop()))
{
return 0;
}
return 1;
}

qui peut etre simplifier et modifier ( || -> && ) en:

bool BoundingBox(const sf::Event &a, const sf::Sprite &b)
{
return ((a.Mouse.X >= b.GetLeft() + b.GetWidth())
&& (a.Mouse.X <= b.GetLeft())
&& (a.Mouse.Y >= b.GetTop() + b.GetHeight())
&& (a.Mouse.Y <= b.GetTop()));
}

C++ Références | Idiomes | FAQ

noob4ever

4 janvier 2008 à 16:11:06

Citation : Chlab_lak

essaye de changer le nom de ta fonction Acht::BoundingBox() (ex: Acht::Bounding_Box() )

de plus j'ai un remarque sur ce code:

int BoundingBox(sf::Event a, sf::Sprite b)

{

if ((a.Mouse.X >= b.GetLeft() + b.GetWidth())

|| (a.Mouse.X <= b.GetLeft())

|| (a.Mouse.Y >= b.GetTop() + b.GetHeight())

|| (a.Mouse.Y <= b.GetTop()))

{

return 0;

}

return 1;

}

qui peut etre simplifier et modifier ( || -> && ) en:

bool BoundingBox(const sf::Event &a, const sf::Sprite &b)

{

return ((a.Mouse.X >= b.GetLeft() + b.GetWidth())

&& (a.Mouse.X <= b.GetLeft())

&& (a.Mouse.Y >= b.GetTop() + b.GetHeight())

&& (a.Mouse.Y <= b.GetTop()));

}

même erreur :

1>Linking...

1>menu.obj : error LNK2005: "int __cdecl Acht::Bounding_Box(class sf::Event,class sf::Sprite)" (?Bounding_Box@Acht@@YAHVEvent@sf@@VSprite@3@@Z) already defined in MainEntryPoint.obj

1>C:\Users\Adrien\Documents\Visual Studio 2008\Projects\sfml-dev-test\Debug\sfml-dev-test.exe : fatal error LNK1169: one or more multiply defined symbols found

j'ai pris note ta remarque concernant le BoundingBox merci

chlablak

4 janvier 2008 à 16:15:49

ce que je comprends de l'erreur c'est que ta fonction existe deja dans MainEntryPoint.o (pas ta fonction mais sa signature/symbole -> càd -> ?Bounding_Box@Acht@@YAHVEvent@sf@@VSprite@3@@Z).

essayes d'autre nom completement different

C++ Références | Idiomes | FAQ

Laurent Gomila

4 janvier 2008 à 16:26:17

BoundingBox doit soit être défini dans un .cpp, soit être déclarée inine si tu veux la garder entièrement dans l'en-tête.

chlablak

4 janvier 2008 à 16:31:00

je choisirais le .cpp.
mais pourquoi faut-il la mettre en inline si on la met dans le .hpp ?

C++ Références | Idiomes | FAQ

Nanoc

4 janvier 2008 à 16:35:20

La solution est très simple:

TYPEDERETOUR NOMNAMESPACE::NOMCLASSE::NOMFONCTION(ARGUMENTS)
{

}

Co-auteur du cours de C++. ||| Posez vos questions sur le forum ||| Me contacter.

MatteX

4 janvier 2008 à 16:39:50

Moi, j'ai toujours fait comme ceci :

MaClasse.h

#ifndef CLASSE_MACLASSE_H__
#define CLASSE_MACLASSE_H__
namespace MonEspace
{
class MaClasse
{
public:
bool MaMethode();
}
}
#endif

MaClasse.cpp

#include "MaClasse.h"
namespace MonEspace
{
bool MaClasse::MaMethode()
{
return true;
}
}

chlablak

4 janvier 2008 à 16:41:22

Citation : Chlab_lak

mais pourquoi faut-il la mettre en inline si on la met dans le .hpp ?

je crois avoir trouvé la reponse -> Lien

C++ Références | Idiomes | FAQ

noob4ever

4 janvier 2008 à 16:45:32

Citation : Laurent Gomila

BoundingBox doit soit être défini dans un .cpp, soit être déclarée inine si tu veux la garder entièrement dans l'en-tête.

merci à tous ça marche

, j'ai déclaré BoundingBox dans un cpp, mais ça veut dire quoi "inline" ? ça change rien ?

Aussi en passant puisque tu es là, je n'arrive pas a récuperer les dernières source sur ton svn je dois pas etre très doué : http://sfml.svn.sourceforge.net/viewvc/sfml/

enfaite je dois récuperer les fichiers 1 à 1 mais c'est un peu fastidieux ...

il y a pas une méthode pour tout récuperer d'un coup ?

merci

Anonyme

4 janvier 2008 à 16:50:32

une foncion inline est une fonction qui ne va jamais être appelé mais son code va plutôt être copié-collé aux endroits où elle aurait dû être appelé si elle n'avait pas été inline.

chlablak

4 janvier 2008 à 16:54:55

enfaite c'est ton compilateur qui decide, quand tu mets inline ce n'est qu'une proposition

C++ Références | Idiomes | FAQ

willish

4 janvier 2008 à 19:23:09

Citation

<...> mais ça veut dire quoi "inline" ? <...>

La fonction inline combine les avantages des fonctions ET des macros.
+ fonctions : Verification des paramètres par le compilateur
+ macros : Pas vraiment d'appel généré , c'est une sorte de traitement de texte .

Il est intéressant de déclarer une méthode inline quand son code est court.

noob4ever

4 janvier 2008 à 19:58:12

ok merci à tous

topic résolu

Laurent Gomila

5 janvier 2008 à 3:27:40

Si une fonction définie dans un en-tête n'est pas inline, elle va se retrouver copiée dans chaque fichier qui inclue cet en-tête --> multiple définition (l'éditeur de liens va retrouver son implémentation à plusieurs endroits).

Pour le SVN, il faut télécharger un client SVN (par exemple TortoiseSVN sous Windows) et faire un checkout du dépôt.

C'est expliqué sur la page sourceforge.net du projet, rubrique SVN.