Discussion :

[Disixlis]

Bonjour � tous , je suis acctuellement entrain de cr�� un petit jeu et j'en suis dans la partie application des texture sur des entit�s.
Donc j'ai dans mon main ceci :

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
 
std::map<std::string,sf::Texture> globalTexture;
std::vector<std::unique_ptr<Entity>> globalEntity;

Et pour appliquer les texture j'ai essayer ceci :

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
6
7
8
9
10
11
12
 
void addMobs( std::string name, std::vector<std::unique_ptr<Entity>> &v_entity, std::map<std::string,sf::Texture> const &m_texture )
{
    for(m_texture::/*const_*/iterator it=m_texture.begin() ; it!=m_texture.end() ; ++it)
    {
        if ( *it.first == name );
        {
            auto e = std::make_unique<Entity>( *it.second, sf::Vector2f(100,100) );
            v_entity.push_back( std::move(e) );
        }
    }
}

Mais le probl�me et que �a ne fonctionne pas car je m'y connais pas trop avec les it�rateur x).
J'ai ces deux erreurs :

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
 
B:\Workspace\C++\Gravity\Function.cpp|12|error: 'm_texture' is not a class, namespace, or enumeration|
B:\Workspace\C++\Gravity\Function.cpp|12|error: 'it' was not declared in this scope|

J'avais pourtant vu ce code " for(m_texture::/*const_*/iterator it=m_texture.begin() ; it!=m_texture.end() ; ++it)" dans un exemple d'un post sur le forum, mais �a n'as pas l'aire de fonctionn� chez moi x)
Quelqu'un peux m'aider ?
Merci ^^

[rnr72]

Bonjour Disixlis,

le m_texture::iterator est �tonnant, je ne dis pas qu'il est faux mais... est faux. (merci VivienD)

Essaie avec std::map<std::string,sf::Texture>::iterator ou encore plus simple auto
sinon, au lieu de faire *it.first fais plut�t it->first
Edit:

Tu peux aussi remplacer ta boucle par :

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
6
auto it = m_texture.find(name);
if(it != m_texture.end())
{
            auto e = std::make_unique<Entity>( it->second, sf::Vector2f(100,100) );
            v_entity.push_back( std::move(e) );
 }

[VivienD]

[...]

le m_texture::iterator est �tonnant, je ne dis pas qu'il est faut mais...

[...]

m_texture::iterator est faux, techniquement. Pour r�cup�rer le type-membre iterator de n'importe quel conteneur de la classe container, il faut �crire container::iterator, quoi qu'il advienne. Naturellement, dans le cas pr�sent, on peut aussi �crire auto it = m_texture.begin().

[rnr72]

VivienD, nous sommes d'accord.

Merci de confirmer ce que je pensais.

[Bousk]

L'�criture vraiment correcte serait for(const auto& el : m_textures) enfin l� vu qu'on cherche une cl� dans une map, qui est forc�ment unique, un find serait encore mieux.

[Disixlis]

Merci � vous ! �a fonctionne ^^

[VivienD]

VivienD, nous sommes d'accord.

Merci de confirmer ce que je pensais.

Merci � vous ! �a fonctionne ^^

Il n'y a pas de quoi.

[Danny-k]

Comme l'a mentionn� Bousk, il est meilleur de faire un find, plut�t qu'une boucle for, car avec un find t'as une recherche en log(N), tandis qu'avec ton for, ta recherche se fait O(N), sans compter les probl�mes de caches d� aux pointeurs dans la map.

Tu peux �crire ceci :

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
6
7
8
9
10
11
 
//En mettant const std::string &, c'est mieux, on évite les recopie inutile
void addMobs(const std::string & name, std::vector<std::unique_ptr<Entity>> &v_entity, std::map<std::string,sf::Texture> const &m_texture )
{
    std::map<std::string,sf::Texture>::const_iterator it = m_texture.find(name);
    //Mieux
    //auto it = m_texture.find(name);
 
    if(it != m_texture.end())
        v_entity.emplace_back(std::make_unique<Entity>(it->second, sf::Vector2f(100, 100);
}

Il est meilleur de faire un emplace_back, on �vite un d�placement, ton unique_ptr sera construit sur place dans ton vector.