Pointeur sur List

Version imprimable

06/01/2009, 11h13
ludo00002

Pointeur sur List

Bonjour,
Dans une unité j'ai un pointeur sur une Liste :TList *list;
Dans cette même unité j'ai des fonctions qui permettent d'ajouter/insérer/supprimer des éléments dans cette liste :
ex: insert(list,Objet);

Je souhaiterais maintenant créer deux unités.
Dans chacune des unités je voudrais stocker une liste TList.
Je déclare donc TList *liste1; dans le 1ère et TList *liste2; dans la seconde. Chaque liste sera accessible via les mêmes fonctions définies dans la 1ère unité.

Comment doit-je modifier la 1ère unité pour pouvoir écrire dans Liste1 ou Liste2?
La liste ne sera plus stockée dans la 1ère unité mais alternativement dans liste1 ou list2
J'espère que j'ai été assez claire dans mes explications.
06/01/2009, 13h58
matafan

Non, ce n'est pas clair du tout. Qu'appelles-tu "unité" : unité de compilation ? Si c'est ça donne nous la liste de tes .h et de tes .c et ce qu'ils contiennent (en gros).

Voici une version réduite de unit1.c :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 
typedef struct _list_item
{
   const char *name;
   const char *label;
} list_item;
 
TList *list; //Déclaration à changer 
 
void append_item(const char *name, const char *label)
{
     list_item *item = calloc(1, sizeof(list_item));
     list_item->name = strdup(name);
     list_item->label = strdup(label);
     list = list_append(list,item);
}

unit1.h

Code:

1
2
3
 
     void append_item(const char *name, const char *label);
     void select_list(TList *new_list);//fonction à implémenter

unit2.c

Code:

1
2
3
4
5
6
7
8
9
10
 
#include unit1.h
 
TList *list1;
 
void show()
{
       select_list(list1);//fonction à implémenter dans unit1.c
       append_item("First Name List1","First label List1");
}

unit3.c

Code:

1
2
3
4
5
6
7
8
9
10
 
#include unit1.h
 
TList *list2;
 
void show()
{
       select_list(list2);//fonction à implémenter dans unit1.c
       append_item("First Name List2","First label List2");
}

Finalement j'ai trouvé, je passe le pointeur sur la liste par adresse et voici le code modifié:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
typedef struct _list_item
{
   const char *name;
   const char *label;
} list_item;

TList **list; //Déclaration d'un pointeur sur pointeur 

void append_item(const char *name, const char *label)
{
     list_item *item = calloc(1, sizeof(list_item));
     list_item->name = strdup(name);
     list_item->label = strdup(label);
     *list = list_append(*list,item);
}
void select_list(TList **new_list)
{
    list=new_list;
}

unit1.h

Code:

1
2
3
     void append_item(const char *name, const char *label);
     void select_list(TList **new_list);

unit2.c

Code:

1
2
3
4
5
6
7
8
9
10
#include unit1.h

TList *list1;

void show()
{
       select_list(&list1);//Passage de l'adresse de la liste
       append_item("First Name List1","First label List1");
}

unit3.c

Code:

1
2
3
4
5
6
7
8
9
10
#include unit1.h

TList *list2;

void show()
{
       select_list(&list2);//Passage de l'adresse de la liste
       append_item("First Name List2","First label List2");
}

07/01/2009, 18h05
3DArchi
Une question bête : pourquoi maintenir une variable globale sur la liste en cours ? Pourquoi ne pas passer la liste aux fonctions concernées :
Code:

1 2 3 void append_item(TList*,const char *name, const char *label); ...
07/01/2009, 19h23
ludo00002

Dans le programme d'origine c'est fait comma ça. Je respecte sa structure. La liste est utilisée dans beaucoup d'autres procédures.
08/01/2009, 09h05
3DArchi

Salut,
J'y voit un risque en utilisant une telle variable globale de contexte de t'emmêler à un moment entre la liste courante et celle que tu veux modifier. A mon sens, ton code est dangereux sous cet aspect. Après, je comprend qu'on ne te donne pas forcément le temps de refactorer le tout :(
08/01/2009, 10h47
ludo00002

Tu as sans doute raison. Je vais voir pour changer toutes mes procédures d'accès à la liste, ça évitera des problèmes plus tard.