Appel fonction dans classe enfant

Version imprimable

Bonjour, j'ai une classe qui peut contenir des composants enfants, lors de la gestion d'événements je passe les événements aux composants enfants comme ceci.

Code:

1
2
3
4
5
6
 
virtual void pushEvent(sf::Event event) {
       for (unsigned int i = 0; i < children.size(); i++) {
             children[i]->pushEvent(event);
       }
}

Le soucis est que pour les composants enfants, la méthode pushEvent n'est pas appelée.

Code:

1
2
3
4
5
6
7
8
 
void TextArea::pushEvent (sf::Event event) {
                LightComponent::pushEvent(event);
                //N'affiche rien, méthode pas appelée.
                if (tmp_text == "test")                  
                    std::cout<<"push event!"<<std::endl;
                getListener().pushEvent(event);
            }

Es-tu certain que ton composant est dans ton vector?
Es-tu certain que ton tmp_test=="test"?
Es-tu certain que la fonction parente doivent être virtual? (je te pose la question, parce que dans la lib de GUI que je viens de développer, en SFML, ce n'est pas nécessaire)

Pourquoi ne transmets-tu pas une référence (constante) de l'événement à la fonction?
En effet, la boucle d'événements doit être faite dans le thread d'affichage, et tout temps perdu réduit la réactivité.
un sf::event est une structure assez conséquente pour mériter une référence.

Et ta conception est limitée à un unique observateur (getListener) par composant. C'est dommage, à moins de le renommer en controleur, et de permettre à celui-ci d'avoir plusieurs observateurs

Ton code pourrait ressembler à:
Code:

1 2 3 4 5 6 7 void TextArea::do_pushEvent (sf::Event const& event) { //pas la peine d'appeler la fonction parente, do_pushEvent est protected virtual, et est appelée par LightEvent::pushEvent qui n'est pas virtuelle. Listenable::pushEvent(event); #ifndef NDEBUG std::cout<<"push event!"<<std::endl; #endif }
edit, au passage, pour itérer un vecteur, avec un compilateur de moins de 5 ans, on écrit plutot for(auto const& child: chilren).
La référence constante portant sur les pointeurs, pas forcément les pointés (ca dépend du type de pointeur utilisé).

28/01/2016, 12h51
Invité

Citation:

Es-tu certain que ton composant est dans ton vector?

Oui vu qu'il s'affiche.

Citation:

Es-tu certain que ton tmp_test=="test"?

Oui

Citation:

Es-tu certain que la fonction parente doivent être virtual? (je te pose la question, parce que dans la lib de GUI que je viens de développer, en SFML, ce n'est pas nécessaire)

Non elle ne doit pas être nécessairement virtuelle.

Citation:

Pourquoi ne transmets-tu pas une référence (constante) de l'événement à la fonction?

Oui, je pourrais.

Citation:

Et ta conception est limitée à un unique observateur (getListener) par composant. C'est dommage, à moins de le renommer en controleur, et de permettre à celui-ci d'avoir plusieurs observateurs

Ce n'est pas un observateur étant donné que je ne l'utilise que pour gérer les événements propre au composant, pas les événements utilisateur. (C'est juste un interpréteur de commandes)
Si l'utilisateur souhaite faire plusieurs observateurs il doit juste créer une interface (Par exemple l'interface ActionListener) en prenant soin de définir la méthode abstraite qui devra être redéfinie lors du déclenchement de l'événement.

Par contre j'ai modifié le code des text area, pour que la gestion du focus soit automatique et donc que l'utilisateur n'est plus à s'occuper de cela.

Citation:

edit, au passage, pour itérer un vecteur, avec un compilateur de moins de 5 ans, on écrit plutot for(auto const& child: chilren).
La référence constante portant sur les pointeurs, pas forcément les pointés (ca dépend du type de pointeur utilisé).

Je n'aime pas utiliser auto, je préfère toujours connaître le type de l'objet.

Ce qui est bizarre c'est que je n'ai que ce problème pour le composant de type panneau, pour les autres composants la méthode est bien appelée.
28/01/2016, 13h41
ternel

Si tu ne veux pas utiliser auto, tu peux aussi écrire for (my_own_ptr<child> const& child : children).
auto est la solution la plus propre, car la variable sera forcément du bon type. Mais il n'est pas nécessaire.
28/01/2016, 14h54
Invité

Ok.

Bref je trouve ça bizarre quand même que la fonction pushEvent est appelée pour une classe dérivée mais pour l'autre classe dérivée elle ne l'est pas.

Bref je laisse donc ceci en suspens jusqu'à ce que je comprenne pourquoi.
28/01/2016, 14h56
ternel

Si on avait un peu plus de code, on pourrait te dire.
Tu n'aurais pas du masquage, une faute de frappe dans le nom?

Rajoute un message dans la fonction de la classe de base.

Citation:

Si on avait un peu plus de code, on pourrait te dire.
Tu n'aurais pas du masquage, une faute de frappe dans le nom?

Non je n'ai pas fait de faute de frappe dans le nom.

Citation:

Rajoute un message dans la fonction de la classe de base.

En fait c'est ici que ça ne va pas :

Code:

1
2
3
4
5
6
7
8
9
10
 
void Panel::onUpdate(RenderWindow* window, sf::Event& event) {
       LightComponent::onUpdate(window, event);
       std::cout<<"update"<<std::endl;
}
void Panel::pushEvent(const sf::Event& event) {
       std::cout<<"push event"<<std::endl;
       LightComponent::pushEvent(event);
       getListener().pushEvent(event);
}

Par contre il va bien là dedans :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
 
void TextArea::onUpdate(RenderWindow* window, sf::Event& event) {
                if (window == &getWindow() && event.type == sf::Event::TextEntered) {
                    getListener().bindCommandSlotParams("CTEXTENTERED", static_cast<char>(event.text.unicode));
                }
            }            
            void TextArea::pushEvent (const sf::Event& event) {
                LightComponent::pushEvent(event);
                if (tmp_text == "test")
                    std::cout<<"push event!"<<std::endl;
                getListener().pushEvent(event);
            }

Sauf si mes text area sont enfants de panels alors là évidemment ça ne fonctionne pas non plus.
Tout deux héritent de la même classe de base.