Discussion :

[versus68]

Hello,

pour un projet en C, j'aimerais créer une liste de structure (pour pouvoir maintenir une liste d'utilisateur).

Voilà comment je procède :

Déclarer en globale :

Code :

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

1
2
GSList *listUser; (j'utilise la GSList de la glib)
FicheUser *fiche; (ma structure avec laquelle je veux faire ma liste)

Voici une fonction que j'utilise pour ajouter un user :

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
void choixAjoutUser(int type){
	int numUser;
	//FicheUser fiche;
 
	fiche = (FicheUser*) malloc(sizeof(FicheUser)+20);
 
	//listeuser affiche les users UNIX  
	listeUser();
 
	scanf( "%d", &numUser );	
 
 
	fiche->nom = (char*) malloc(sizeof(char*)*20);
	fiche->nom = tabNom[numUser];
	fiche->type = type;
	fiche->UID = tabUID[numUser];
	fiche->pid = 0;
	//on ajoute le user UNIX dans le tableau des users
	//tabUser[nbUser] = tabUID[UIDuser];
 
	//table[nbUser] = fiche;
	listUser = g_slist_append(listUser, fiche);
 
	//on ouvre un terminal pour le nouveau user :
	creationUser(tabNom[numUser],type);
	nbUser++;
}

Voici une fonction que j'utilise pour afficher mes users :

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
int listeUserProject(){
	int i, t;
	FicheUser* fiche; 
 
       i = 0;
 
    GSList *node = listUser;
 
    while(node){  
        fiche = node->data;  
 
        printf("Numéro : %d --- Nom : %s \t UID : %d \t PID : %d \t Status : %s \n",i,fiche->nom,fiche->UID,fiche->pid,tabType[fiche->type]);
        node = g_slist_next(node);
        i++;
    }
 
	if(i == 0){
	    printf("Il n'existe aucun utilisateur dans ce projet");
	}
    return i;
}

Est ce que c'est la bonne façon de faire ? Je trouve cela bizarre de faire un malloc sur une variable globale pour éviter qu'elle se fasse écraser lorsque la fonction est appelée une deuxième fois.

Merci pour vos éclaircissement

[Philou67430]

Dans ce code :

Code :

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

1
2
3
 
	fiche->nom = (char*) malloc(sizeof(char*)*20);
	fiche->nom = tabNom[numUser];

1- Je ne vois pas l'intérêt du malloc s'il est suivi d'un écrasement avec un autre pointeur (pire, c'est une fuite mémoire... puisqu'aucun free n'est fait, et que le pointeur retourné par le malloc n'est plus référencé par aucune variable)
2- le dimensionnement est basé sur sizeof(char *) (c'est à dire que la taille d'un pointeur de caractère, pas sur la taille d'un caractère)... il me semblerait plus logique de dimensionner selon la taille d'un char, voire, mieux, selon la taille du nom.

Je proposerais donc :

Code :

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

1
2
3
 
	fiche->nom = (char*) malloc(strlen(tabNom[numUser])+1);
	strcpy(fiche->nom, tabNom + numUser);

[versus68]

Merci pour ta réponse.

(c'est pas comme si dans mon projet, j'ai déclaré plein de tableau de char de cette manière en plus ^^, zouh, il va falloir corriger tout ça !)

Ok pour ce malloc j'ai saisi comment faire.

par contre celui qui me laisse perplexe, c'est celui que j'appelle lorsque je crée une nouvelle structure (fiche) lorsque j'ajoute un user.

ça a l'air de marcher, mais je ne sais pas si c'est bien clean... :s

[Philou67430]

A priori, ce malloc ne me semble pas suspect, mais sans connaitre ce qu'attends g_slist_append comme 2e paramètre (voir le manuel), je ne pourrais être plus catégorique.

[versus68]

Oukie.

d'après ce que j'ai compris, il suffit de lui passer un pointer vers un élément, n'importe lequel. Ensuite tout est geré par des "classes" de la glib.

ça a l'air de bien marché mon truc. Merci pour tes corrections sur mes malloc