IdentifiantMot de passe
Loading...
Mot de passe oublié ?Je m'inscris ! (gratuit)
Navigation

Inscrivez-vous gratuitement
pour pouvoir participer, suivre les réponses en temps réel, voter pour les messages, poser vos propres questions et recevoir la newsletter

 C Discussion :

Variable globale et malloc, explication


Sujet :

C

Vue hybride

Message précédent Message précédent   Message suivant Message suivant
  1. #1
    Membre averti
    Inscrit en
    Mai 2008
    Messages
    29
    Détails du profil
    Informations forums :
    Inscription : Mai 2008
    Messages : 29
    Par défaut Variable globale et malloc, explication
    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

  2. #2
    Expert confirmé

    Homme Profil pro
    Ingénieur développement logiciels
    Inscrit en
    Avril 2009
    Messages
    3 577
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 59
    Localisation : France, Bas Rhin (Alsace)

    Informations professionnelles :
    Activité : Ingénieur développement logiciels
    Secteur : Aéronautique - Marine - Espace - Armement

    Informations forums :
    Inscription : Avril 2009
    Messages : 3 577
    Par défaut
    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);

  3. #3
    Membre averti
    Inscrit en
    Mai 2008
    Messages
    29
    Détails du profil
    Informations forums :
    Inscription : Mai 2008
    Messages : 29
    Par défaut
    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

  4. #4
    Expert confirmé

    Homme Profil pro
    Ingénieur développement logiciels
    Inscrit en
    Avril 2009
    Messages
    3 577
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 59
    Localisation : France, Bas Rhin (Alsace)

    Informations professionnelles :
    Activité : Ingénieur développement logiciels
    Secteur : Aéronautique - Marine - Espace - Armement

    Informations forums :
    Inscription : Avril 2009
    Messages : 3 577
    Par défaut
    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.

  5. #5
    Membre averti
    Inscrit en
    Mai 2008
    Messages
    29
    Détails du profil
    Informations forums :
    Inscription : Mai 2008
    Messages : 29
    Par défaut
    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

Discussions similaires

  1. Variable globale / Propriété des threads
    Par rgarnier dans le forum XMLRAD
    Réponses: 4
    Dernier message: 03/10/2003, 10h49
  2. Cacher Variable Url avec Variable Globale
    Par danuz dans le forum XMLRAD
    Réponses: 6
    Dernier message: 26/09/2003, 17h20
  3. Variable "globale" (dans plusieurs templa
    Par mattmat dans le forum XSL/XSLT/XPATH
    Réponses: 3
    Dernier message: 17/06/2003, 19h22
  4. question sur les variables globales et les thread posix
    Par souris_sonic dans le forum POSIX
    Réponses: 5
    Dernier message: 13/06/2003, 13h59
  5. les variables globales static
    Par gRRosminet dans le forum C
    Réponses: 8
    Dernier message: 27/04/2002, 08h34

Partager

Partager
  • Envoyer la discussion sur Viadeo
  • Envoyer la discussion sur Twitter
  • Envoyer la discussion sur Google
  • Envoyer la discussion sur Facebook
  • Envoyer la discussion sur Digg
  • Envoyer la discussion sur Delicious
  • Envoyer la discussion sur MySpace
  • Envoyer la discussion sur Yahoo