Discussion :

[ludo00002]

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.

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

[ludo00002]

Voici une version réduite de unit1.c :

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
 
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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

unit2.c

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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");
}

[ludo00002]

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

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
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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

unit2.c

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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");
}

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

Sélectionner tout - Visualiser dans une fenêtre à part

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

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

[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

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