Discussion :

[basickiller]

bonjour a tous !

je suis actuellement en train de programmer un petit system qui me permet d'appeller une fonction grace a une chaine ...

petit exemple :

Code :

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

1
2
3
add_cmd("print",&print); // ajout la commande et de sa fonction
cmd("print"); // vas cherche si une commande s'appele printf puis appele sa fonction

pour sa j'utilise donc 2 type de variable :

Code :

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

1
2
3
void (*pt)(); // pour stoker le pointeur de la fonction
char* cmd; // pour stoker la chaine

le tout dans une class :

Code :

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

1
2
3
4
5
6
7
 
class BW_cmd_struc
{
    public :
        void (*pt)();
        char* cmd;
};

afin de simplifier la gestion j'utilise une liste doublement chain�e de structure :

Code :

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

1
2
3
4
5
6
7
 
struct dll
{
   struct dll *prev;
   struct dll *next;
   void *data;
};

jusque ici pas de probleme , je tape ma fonction add_cmd(); (elle est pas encore finaliser)

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
bool BWconsol::add_cmd(char* commande,void (*pFunction) ())
{
    if(commande == NULL)
    return false;
 
    //if(pFunction > 0)
    //return false;
 
 
    BW_cmd_struc *Packcmd = NULL;
    Packcmd = new BW_cmd_struc;
 
    if(Packcmd == NULL)
        printf("pas ok\n");
 
    Packcmd->cmd= commande;
    Packcmd->pt = pFunction;
 
    dll_insert(&commande_stok, &Packcmd);
 
    return true;
}

-> commande_stok est la liste doublement chain�e declarer dans la class principal
-> dll_insert(); me permet d'inserer une nouveaux noeud dans ma chaine et donc ici ma classe Packcmd

et quand je l'appelle tout marche bien (aucun plantage)

alors je passe a ma fonction cmd();

Code :

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

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
void BWconsol::cmd(char* commande)
{
    if(commande==NULL)
        return ;

    BW_cmd_struc *Packcmd = NULL;
    Packcmd = new BW_cmd_struc;
    
    int size = dll_sizeof(commande_stok); // permet de conaitre la taille de la chaine

    dll_first (&commande_stok); // me pemet de me placer au debut de la chaine
    while (size > 0)
    {
        printf("ok\n");
        Packcmd = (BW_cmd_struc*) dll_data(commande_stok); // je recupere la classe dans le void
        
        if(Packcmd == NULL)
            printf("pas ok\n");
        
        char *temp1 = Packcmd->cmd; // bug ici
        

        printf("ok\n");
        if(strcmp(temp1,commande)==0)
        {
            //(*Packcmd.pt) (); // enlever pour les tests
            return ;
        }
        
        dll_next (&commande_stok); // passe a l'element suivant
    }
    return ;
}

- les printf("ok\n"); me permet de voir ou est l'erreur

je lance donc mon programme (en mode console et mode debug sur dev-c++) :
- je voie le 1er "ok" dans la fonction cmd
- puis "pas ok" (normal ?)
- puis la dev-c++ me renvoie une erreur de segmentation fault pour la ligne marque "bug ici"

j'ai donc un probleme avec

Code :

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

Packcmd->cmd

et donc je pense que ma maniere de recupere ma class depuis un void* et mauvaise

j'ai essayer plusieur maniere de faire , mais rien n'a marcher alors je m'en remet a vous merci

[M�dinoc]

1�) Je peux voir exactement comment tu appelles add_cmd ? J'ai un doute � propos des cha�nes...
2�) Tu devrais utiliser un typedef pour test pointeurs de fonction: c'est facilement plus lisible
3�) Pour les cha�nes, je te conseillerais d'en stocker une copie dans tes objets (avec strdup() ou en utilisant une std::string...). Ainsi, au moins tu seras s�r qu'on ne passe pas un buffer r�utilis� � tes fonctions...

4�) J'ai trouv�! Erreur dans l'appel � dll_insert: Tu lui passes l'adresse de ton pointeur!

[khayyam90]

bien le bonsoir,

le "pas ok" implique logiquement l'erreur de segmentation. Je m'interroge sur la condition de ton while de BWconsol::cmd. size est fix�e avant la boucle et jamais r��valu�, donc si elle tourne une fois, c'est une boucle qui ne s'arr�te jamais et forc�ment, elle arrivera aux limites de ta liste chain�e � un moment ou � un autre.

ton code c++ ressemble beaucoup � du C. Et je me demande pourquoi tu n'as pas repris une des classes de conteneurs pr�sentes dans le c++, par exemple la std::map qui pourrait tout � fait convenir � tes besoins. Associer une chaine de caract�res � un pointeur de fonction, c'est bien �a que tu cherches � faire ?
Pourquoi r��crire ce qui existe d�j� et qui marche tr�s bien ? Tu prends le risque de rencontrer des erreurs.

[basickiller]

bien le bonsoir,

le "pas ok" implique logiquement l'erreur de segmentation. Je m'interroge sur la condition de ton while de BWconsol::cmd. size est fix�e avant la boucle et jamais r��valu�, donc si elle tourne une fois, c'est une boucle qui ne s'arr�te jamais et forc�ment, elle arrivera aux limites de ta liste chain�e � un moment ou � un autre.

effectivement il y a une erreur au niveau du size qui provoque une boucle infinit (size etant toujours supperieur a 0)
mais cela ne resout pas le bug ...
merci quand quand de me l'avoir dit ^^

ton code c++ ressemble beaucoup � du C. Et je me demande pourquoi tu n'as pas repris une des classes de conteneurs pr�sentes dans le c++, par exemple la std::map qui pourrait tout � fait convenir � tes besoins. Associer une chaine de caract�res � un pointeur de fonction, c'est bien �a que tu cherches � faire ?
Pourquoi r��crire ce qui existe d�j� et qui marche tr�s bien ? Tu prends le risque de rencontrer des erreurs.

j'utilise plus le c que le c++ , mais il reste quand meme du c++ (avec l'utilisation de class public /private)
en meme temps je cherche a apprendre ... meme si r�inventer la roue me serviras a pas grand chose ... je recode un peu tout ce qui me passe par la main ^^

1�) Je peux voir exactement comment tu appelles add_cmd ? J'ai un doute � propos des cha�nes...

a l'origin add_cmd et dans une class BWconsol , voila comme je l'appel :

Code :

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

1
2
3
4
 
BWconsol STDconsole;
STDconsole.add_cmd("print",&printff);
STDconsole.cmd("print");

2�) Tu devrais utiliser un typedef pour test pointeurs de fonction: c'est facilement plus lisible

je verrai ca un peu plus tard ^^

3�) Pour les cha�nes, je te conseillerais d'en stocker une copie dans tes objets (avec strdup() ou en utilisant une std::string...). Ainsi, au moins tu seras s�r qu'on ne passe pas un buffer r�utilis� � tes fonctions...

desole j'ai pas compris

4�) J'ai trouv�! Erreur dans l'appel � dll_insert: Tu lui passes l'adresse de ton pointeur!

le prototype de dll_insert est : dll_insert (dll_s **, void *);

ici :

Code :

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

dll_insert(&commande_stok, &Packcmd);

&commande_stok et obligatoire sinon il ne modifie pas commande_stok
et si je remplace &Packcmd par Packcmd le programme plante aussi et au meme endroit


------------

merci pour vos reponse ^^

[Rafy]

Comme tu travails en C++, il est pr�f�rable d'utiliser les std::string.
Au lieu de faire toi m�me ton propre conteneur, pourquoi tu n'utilise pas un std::map, tu pourrais ainsi associer � une chaine un pointeur de fonction...
Inspire toi du Design patern factory...

[Rafy]

Jette un coup d'oeil dans un bouquin : Modern C++ design d'Andrei Alexandrescu

[basickiller]

pas besion j'ai trouver ^^ je te repond puis j'explique

Comme tu travails en C++, il est pr�f�rable d'utiliser les std::string.
Au lieu de faire toi m�me ton propre conteneur, pourquoi tu n'utilise pas un std::map, tu pourrais ainsi associer � une chaine un pointeur de fonction...

je reprend un peu ce que j'ai dit plus haut je souhait refaire les fonction de base tout simple pour apprendre , je pense que j'apprendrais mieux le c/c++ si je m'amuse a refaire les fonction de base (enfin pas toute ^^) que de les utiliser directement

Inspire toi du Design patern factory...

desole je sais pas ce que c'est

--------------
bon alors mon probleme est resolut je m'explique :
j'ai modifier cette ligne : Packcmd = (BW_cmd_struc*) dll_data(commande_stok);
en : *Packcmd = *((BW_cmd_struc*) dll_data(commande_stok));

j'ai ensuite modifier ma class BW_cmd_struc comme ceci :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
class BW_cmd_struc
{
    public :
        void (*pt)();
        char* cmd;
        
    BW_cmd_struc& operator=(const BW_cmd_struc &b)
    {
        cmd = b.cmd;
        pt  = b.pt;
        return *this;
    }
};

et dans mon add_cmd();
j'ai modifier ceci : dll_insert(&commande_stok, &Packcmd);
en ceci : dll_insert(&commande_stok, Packcmd);

et j'obtient bien le resultat voulut (c'est a dire quand j'appel cmd() avec en parametre une chaine deja enregsitrer elle vme renvoie le fonction associer)

donc voila ca marche merci a tous ^^

j'ai juste une fuite de memoire a gerer mais ca c'est simple ...

bye ^^