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