[C++14] Probl�me avec un vector<unique_ptr>

Version imprimable

Bonsoir � tous :), je sais que j'ai d�j� �crit un post il y � quelque heures mais j'ai un probl�me sans aucun rapport et que j'essaye de r�soudre sans succ�s depuis hier :(
Alors voil�, j'ai des entit�s et l� je g�re leur physique, donc je me suis occup� des collions( pour l'instant en externe ), etc.. Et l� je m'occupe de la gravit�. J'ai donc ce code ci :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 void updatePhysics( std::vector<std::unique_ptr<Entity>> &v_entity, sf::Vector2f const& gameSize ) { ... std::vector<std::pair<bool, size_t>> entityInFall; ///COLLISION TEST ... for( size_t i{0}; i < v_entity.size(); ++i ) { if( v_entity[i]->position.y > 0 ) { entityInFall.push_back(std::make_pair(true,i)); } } ///GRAVITY TEST ... }
Ce code me donne cette erreur alors que si je le met dans mon main je n'es aucun probl�mes. Voici l'erreur en question :
Code:

1 2 B:\Workspace\C++\Gravity\Function.cpp|11|error: '(&(& v_entity)->std::vector<_Tp, _Alloc>::operator[]<std::unique_ptr<Entity>, std::allocator<std::unique_ptr<Entity> > >(i))->std::unique_ptr<_Tp, _Dp>::operator-><Entity, std::default_delete<Entity> >()->Entity::position' does not have class type|
Donc, il prend position pour une class ?
je comprend pas vraiment l'erreur, qulqu'un peux m'aider ?
Merci ^^"

31/03/2016, 01h16
rnr72

Personnellement, je n'ai encore jamais travaill� avec les unique_ptr, donc je ne suis pas s�r de pouvoir t'aider efficacement.

J'ai d�j� eu des erreurs ressemblants, je dirai que le compilateur consid�re ton v_entity[i]->position.y comme une fonction/m�thode.

On arrive � une partie de ton code qui me semble bizarre, c'est quoi ces accolades size_t i{0} ligne 8?

Je ne sais pas si ton probl�me vient de l� mais je n'est jamais vu �a avant.

�claire moi STP.
31/03/2016, 02h03
jo_link_noir

Je dirais qu'il manque l'include de la classe Entity.

@rnr72: la construction avec {} est la forme universalis�e des initialisations (direct-list-initialization).
31/03/2016, 09h35
rnr72

jo_link_noir, oui, pour des struct, des listes, tableaux, ... je n'avais jamais vu �a pour initialiser une variable simple dans un for. Merci pour l'info.
31/03/2016, 15h41
Disixlis

J'ai bien mis l'en-t�te Entity.hpp car d'autres membre de cette classe fonctionne ailleurs bien que ce ne soit pas la m�me utilisation.
Et pour le size_t i{0} dans le for, il faut savoir que c'est tout simplement du c++ moderne, les accolades, c'est la norme.
Donc, on ne fait plus unsigned int i = 0, mais unsigned int i{0} ;)
Bref, du coup vu que j'ai bien inclut mon Entity, o� pourrait �tre le probl�me ^^ mais en tout cas d�j� merci de m'avoir r�pondu.
31/03/2016, 16h18
rnr72

J'ai appris des choses, c'est cool!

Pourrais-tu s'il te plait nous fournir Entity.hpp et .cpp.
31/03/2016, 16h31
dragonjoker59

Moi je dis que c'est pas l'include d'Entity.hpp, mais l'include correspondant au type de Entity::position (il aurait r�l� sur Entity, pas sur Entity::position)
31/03/2016, 16h35
rnr72

Je suis d'accord avec dragonjoker59, c'est pour �a que j'aimerai voir Entity.hpp pour savoir exactement ce qu'est Entity::position.

J'ai bien lu vos message, donc voici mon code.
Entity.hpp

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
 
#ifndef Entity_included
#define Entity_included
 
#include <vector>
#include <SFML/Audio.hpp>
#include <SFML/Graphics.hpp>
 
class Entity : public sf::Drawable, public sf::Transformable
{
public:
 
    Entity( sf::Texture const &texture, sf::Vector2f const &position, sf::Vector2f const &origin );
    Entity( sf::Texture const &texture, sf::Vector2f const &position );
    Entity( sf::Texture const &texture );
 
    void scale( const sf::Vector2f& factor );
    sf::Vector2f position() const;
 
private:
 
    float const _fall_velocity;
    float _fall_acceleration;
    sf::Sprite entity_sprite;
    virtual void draw( sf::RenderTarget& target, sf::RenderStates states ) const;
 
 
};
 
#endif // Entity_included

Entity.cpp

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
 
#include "Entity.hpp"
 
#define FALL_VELOCITY 0.1
 
Entity::Entity( sf::Texture const &texture, sf::Vector2f const &position, sf::Vector2f const &origin ) : _fall_velocity{FALL_VELOCITY}, _fall_acceleration{0}
{
    entity_sprite.setTexture(texture);
    entity_sprite.setOrigin(origin);
    entity_sprite.setPosition(position);
}
 
Entity::Entity( sf::Texture const &texture, sf::Vector2f const &position ) : _fall_velocity{FALL_VELOCITY}, _fall_acceleration{0}
{
    entity_sprite.setTexture(texture);
    entity_sprite.setPosition(position);
}
 
Entity::Entity( sf::Texture const &texture ) : _fall_velocity{FALL_VELOCITY}, _fall_acceleration{0}
{
    entity_sprite.setTexture(texture);
}
 
void Entity::scale( const sf::Vector2f& factor )
{
    entity_sprite.scale(factor);
}
 
sf::Vector2f Entity::position() const
{
    return entity_sprite.getPosition();
}
 
void Entity::draw( sf::RenderTarget& target, sf::RenderStates states ) const
{
    states.transform *= getTransform();
    states.texture = entity_sprite.getTexture();
    target.draw(entity_sprite);
}

L'endroit ou j'ai le bug :
Function.cpp

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
 
#include "Function.hpp"
#include <tuple>
#include <windows.h> // adapter plus tard
 
void error( std::string const &error ) // utiliser ostream
{
    HANDLE consoleHandle = GetStdHandle(STD_OUTPUT_HANDLE);
    SetConsoleTextAttribute(consoleHandle, FOREGROUND_RED | FOREGROUND_INTENSITY );
    std::cout << error;
    SetConsoleTextAttribute(consoleHandle, FOREGROUND_RED | FOREGROUND_BLUE | FOREGROUND_GREEN | FOREGROUND_INTENSITY );
}
 
void updatePhysics( std::vector<std::unique_ptr<Entity>> &v_entity, sf::Vector2f const& gameSize )
{
    ...
    std::vector<std::pair<bool, size_t>> entityInFall;
    ///COLLISION TEST
    ...
 
    for( size_t i{0}; i < v_entity.size(); ++i )
    {
        if( v_entity[i]->position.y > 0 )
        {
            entityInFall.push_back(std::make_pair(true,i));
        }
    }
*/
    ///GRAVITY TEST
    ...
 
}
 
void loadAllTexture( std::map<std::string,sf::Texture> &m_texture )
{
    sf::Texture t;
    t.loadFromFile("B:/Workspace/C++/Gravity/bin/Release/src/Entity/100.png");
    m_texture["Nebuleux"] = t;
}
 
void addMobs( std::string name, std::vector<std::unique_ptr<Entity>> &v_entity, std::map<std::string,sf::Texture> const &m_texture )
{
    auto it = m_texture.find(name);
    if(it != m_texture.end())
    {
        auto e = std::make_unique<Entity>( it->second, sf::Vector2f(100,100) );
        e->scale(sf::Vector2f(0.17,0.17));
        v_entity.push_back( std::move(e) );
        return;
    }
    error("[ERROR][TEXTURE]The mob with texture name : " + name + " is invalid");// faire gestion logs
}

Voil� :)

31/03/2016, 17h08
bacelar

Citation:

sf::Vector2f position() const;

~~Virez-moi ces parenth�ses, right now !!!~~
C'est une fonction, il faut donc AJOUTER des parenth�ses lors de l'appel.

EDIT: je sais que mon intervention est un peu bourrine et que la vraie option est l'ajout de parenth�ses; mais les pouces rouges, c'est un peu beaucoup, non ? :pleure:
31/03/2016, 17h13
rnr72

J'avais bien raison, Entity::position est une fonction! le voila ton probl�me

essaie �a

Code:

v_entity[i]->position().y
31/03/2016, 18h14
Disixlis

Citation:

Envoy� par rnr72

J'avais bien raison, Entity::position est une fonction! le voila ton probl�me

essaie �a

Code:

v_entity[i]->position().y

Mais quesque je suis b�te !!!!! Merci � vous ! Je crois que je suis fatigu� x)