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 :

difference entre chaine ecrite en dur ("..") et char *


Sujet :

C

Vue hybride

Message précédent Message précédent   Message suivant Message suivant
  1. #1
    Membre confirmé
    Profil pro
    Ingénieur développement logiciels
    Inscrit en
    Septembre 2008
    Messages
    141
    Détails du profil
    Informations personnelles :
    Localisation : France

    Informations professionnelles :
    Activité : Ingénieur développement logiciels

    Informations forums :
    Inscription : Septembre 2008
    Messages : 141
    Par défaut difference entre chaine ecrite en dur ("..") et char *
    Bonjour

    j'ai actuellement un probleme avec un tableau de char... J'ai une structure dans laquelle j'ai un pointeur de pointeur qui est un tableau destiné a stocker des char*

    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
     
     
    pert->sommets = (char **) malloc(sizeof(char *) * nbSommetsMax);
    J'ai une fonction qui lit dans un fichier, et qui remplit un tableau de char (char *) en fonction des caracteres lus. A l'interieur de la fonction ce tableau est correctement rempli et je l'affecte donc au tableau 'sommet'

    Le probleme est qu'en dehors de la fonction se tableau semble vide...

    je vous mets ci-dessous le code la fonction en question :

    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
    34
    35
    36
    37
    38
    39
    40
    41
    42
    43
    44
    45
    46
    47
    48
    49
    50
    51
    52
    53
    54
    55
    56
    57
    58
    59
    60
    61
    62
    63
    64
    65
    66
    67
    68
    69
    70
    71
    72
    73
    74
    75
    76
    77
    78
    79
    80
    81
    82
    83
    84
    85
    86
    87
    88
    89
    90
    91
    92
    93
    94
    95
    96
    97
    98
    99
    100
    101
    102
    103
    104
    105
    106
    107
    108
    109
    110
    111
    112
    113
    114
    115
    116
    117
    118
    119
    120
    121
    122
    123
    124
    125
    126
    127
    128
    129
    130
    131
    132
    133
    134
    135
    136
    137
    138
    139
    140
    
    TypPert *creerSommetsFichierPert(char *fic) {
    
    	char      ligne_lue[512], 	/* la ligne qui est lue dans le fichier */
    			  *sommet,		  	/* le sommet en cours de lecture */
    			  *nomTache,	  	/* l intitule de la tache */
    			  *dureeChar,      	/* la duree de la tache en char */
    		      *pos;	          	/* pour determiner si la ligne lue contient # */
    
    	FILE      *fichier;       	/* Pointeur sur le fichier */
    
    	int  	  indiceSommet,	    /* indice du tableau de char 'sommet' */
    			  indiceNomTache,   /* indice du tableau de char 'nomTache' */
    			  indiceDureeTache, /* indice du tableau de char 'nomTache' */
    			  i,			    /* indice de la boucle For */
    			  numSommet,	   	/* numero du sommet en cours */
    			  duree,		  	/* la duree de la tache */
    			  champs;		  	/* le n° du champs qu'on lit dans le fichier (nom, intitule ou duree) */
    
    	int caractere,
    		nb;
    
    	char *testeur;
    
    	TypPert *pert;
    
    	booleen trouve;			  	/* TRUE une fois que l on a lu le sommet sur la ligne */
    
    
    	/* Ouverture du fichier en lecture */
    	fichier = fopen(fic, "r");
    
    	if (fichier != NULL) {
    
    		pert = creerPert(nbSommetsFichierPert(fic));
    		numSommet = 0;
    
    		/* Tant qu il y a une ligne a lire */
    		while (fgets(ligne_lue, 512, fichier) != NULL) {
    
    			pos = strchr(ligne_lue, 35); /* 35 = # */
    
    			if (pos == NULL) {
    
    				/* initialisation des differents tableaux et indices */
    				sommet = (char *) malloc(sizeof(char));
    				nomTache = (char *) malloc(sizeof(char));
    				dureeChar = (char *) malloc(sizeof(char));
    				indiceSommet = 0;
    				indiceNomTache = 0;
    				indiceDureeTache = 0;
    				numSommet++;
    				champs = 1;
    				trouve = FALSE;
    				nb = 0;
    				for(i = 0; i < 512 && trouve == FALSE ; i++){
    
    					/* si on a traite les 3 premiers champs ou si la fin de ligne est atteinte */
    					if ((champs > 3) || (ligne_lue[i] == '\0')) {
    
    						trouve = TRUE;
    
    					} else {
    
    						if (ligne_lue[i] != ',') {
    
    							switch (champs) {
    
    							case 1 :
    								/* on traite le 1er champs */
    								caractere = ligne_lue[i];
    								if(caractere != ' ') {
    
    									sommet[indiceSommet] = caractere;
    									nb++;
    									indiceSommet++;
    									if(ligne_lue[i+1] != ',' && ligne_lue[i+1] != ' '){
    										sommet = realloc(sommet, indiceSommet+1 * sizeof(char));
    									}
    
    
    								}
    
    								break;
    
    							case 2 :
    								/* on traite 2eme champs */
    								nomTache[indiceNomTache] = ligne_lue[i];
    								indiceNomTache++;
    								nomTache = realloc(nomTache, indiceNomTache+1);
    								break;
    
    							case 3 :
    								/* on traite 3eme champs */
    								dureeChar[indiceDureeTache] = ligne_lue[i];
    								indiceDureeTache++;
    								dureeChar = realloc(dureeChar, indiceDureeTache+1);
    								break;
    
    							default :
    								break;
    
    
    							}
    
    						} else {
    
    							/* si on lit une virgule on lit le champs suivant */
    							champs++;
    
    						}
    
    					}
    
    				}
    
    				/* on affecte la valeur des 3 champs au sommet dans le PERT */
    				char *buffer = strncpy(buffer, sommet, nb);
    				pert->sommets[numSommet - 1] = buffer;
    				free(buffer);
    				pert->intitule[numSommet - 1] = nomTache;
    				duree = atoi(dureeChar);
    				insertionSommetPert(pert, numSommet, duree);
    				free(sommet);
    				free(nomTache);
    				free(dureeChar);
    
    			}
    
    		}
    
    		fclose(fichier);
    
    
    	}
    
    	return pert;
    
    }
    La ligne en gras est le moment ou j'affecte ma chaine de caractere (buffer) a mon tableau de pointeur de pointeur. Et lorsque je recupere ma structure en dehors de la fonction est que je veux afficher, le tableau de pointeur de pointeur semble vide alors que si au lieu d'affecter le tableau 'buffer' j'affecte une chaine en dur (exemple "test") elle s'affiche correctement par la suite.

    Est ce que quelqu'un aurait une idée de la difference entre une chaine ecrite en dur et un tableau de char (char *)?

    merci

  2. #2
    Membre émérite

    Profil pro
    Inscrit en
    Septembre 2006
    Messages
    717
    Détails du profil
    Informations personnelles :
    Localisation : France

    Informations forums :
    Inscription : Septembre 2006
    Messages : 717
    Par défaut
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    char *buffer = strncpy(buffer, sommet, nb);
    Ça compile, ça ? Je vois au moins 3 erreurs sur cette simple ligne
    - déclaration d'une variable au milieu d'un bloc
    - utilisation d'une variable non initialisée
    - utilisation de strncpy sur une zone non-allouée

  3. #3
    Candidat au Club
    Profil pro
    Inscrit en
    Juillet 2009
    Messages
    2
    Détails du profil
    Informations personnelles :
    Localisation : France

    Informations forums :
    Inscription : Juillet 2009
    Messages : 2
    Par défaut
    Bonsoir,

    Je te reprends :

    Ça compile, ça ? Je vois au moins 3 erreurs sur cette simple ligne
    - déclaration d'une variable au milieu d'un bloc
    - utilisation d'une variable non initialisée
    - utilisation de strncpy sur une zone non-allouée
    Il n'y a pas de raison que cela ne compile pas, cela risque de provoquer une erreur a l'execution (il y a de fortes chances) .

    La seule erreur de compilation qu'il pourrait y avoir etant pour la declaration d'une variable au milieu d'un bloc, ce qui est autorise en C99, en revanche si son compilateur etait regle correctement, il lui afficherait des warnings significatifs pour les deux autres coquilles.

    Adroneus

  4. #4
    Membre averti
    Profil pro
    Inscrit en
    Avril 2010
    Messages
    27
    Détails du profil
    Informations personnelles :
    Localisation : France

    Informations forums :
    Inscription : Avril 2010
    Messages : 27
    Par défaut
    Hello,

    Petit résumé donc :

    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    4
     
    char *buffer = strncpy(buffer, sommet, nb);
    pert->sommets[numSommet - 1] = buffer;
    free(buffer);
    Décomposons ce code en code équivalent :
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    4
    5
     
    char *buffer;
    buffer = strncpy(buffer, sommet, nb);
    pert->sommets[numSommet - 1] = buffer;
    free(buffer);
    Ligne 1) tu déclares un pointeur buffer : il pointe vers n'importe ou et tu n'as aucune raison de modifier la valeur qu'il pointe (qui a de fortes chances de ne pas appartenir à ton programme)
    Ligne 2) Comme je viens de le dire, là tu tentes de modifier de la mémoire qui a très peu de chances de t'appartenir. (et si ça se trouve, tu es en train de modifier des données de ton programme!) => segmentation fault très probable
    Ligne 3) Tu sauves l'adresse à laquelle tu viens d'écrire. Cette adresse est toujours autant invalide qu'avant.
    Ligne 4) Double erreur
    - tu tentes de libérer une zone mémoire qui ne t'appartient pas (segmentation fault très probable)
    - tu tentes de libérer une zone mémoire auquel tu risques d'accéder plus tard (puisque tu l'as sauvegardée dans ton tableau pert->sommets) => segmentation fault très probable, mais plus tard dans l'éxecution du programme, le genre d'erreur relativement difficile à retrouver

  5. #5
    Membre éclairé Avatar de mikhailo
    Profil pro
    Inscrit en
    Mars 2010
    Messages
    78
    Détails du profil
    Informations personnelles :
    Âge : 41
    Localisation : France

    Informations forums :
    Inscription : Mars 2010
    Messages : 78
    Par défaut
    J'avais le même problème que toi (remplissage d'une structure déclarée en dehors de la fonction de remplissage, et qui semble vide en dehors de la fonction), et je l'ai résolu en déclarant la structure à remplir (le tableau sommets dans ton cas, si je comprends bien) en tant que variable globale. C'est un peu moche, mais ça marche en attendant une solution plus jolie =)

  6. #6
    Expert confirmé
    Avatar de diogene
    Homme Profil pro
    Enseignant Chercheur
    Inscrit en
    Juin 2005
    Messages
    5 761
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Localisation : France, Essonne (Île de France)

    Informations professionnelles :
    Activité : Enseignant Chercheur
    Secteur : Enseignement

    Informations forums :
    Inscription : Juin 2005
    Messages : 5 761
    Par défaut
    Pour compléter la remarque pertinente de bloomenthal :
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    4
    /* on affecte la valeur des 3 champs au sommet dans le PERT */
    char *buffer = strncpy(buffer, sommet, nb);
    pert->sommets[numSommet - 1] = buffer;
    free(buffer);
    - strncpy ne fait pas d'allocation mémoire (au contraire de strdup()) et copie simplement dans le buffer passé en premier argument (et renvoie la valeur de cet argument). buffer est un simple pointeur (non initialisé à l'appel de strncpy) et il n'y a pas de mémoire réservée pour la copie -> plantage potentiel.
    - Comme buffer n'est pas alloué dynamiquement, le free(buffer) va planter.

    De plus :
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    pert->intitule[numSommet - 1] = nomTache;
    ...
    free(nomTache);
    Après le free(nomTache), pert->intitule[numSommet - 1] contient une adresse invalide, la mémoire correspondante a été désallouée.

    Pouquoi ne pas avoir quelque chose comme :
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    4
    5
    sommet[indiceSommet] = '\0';  // termine la chaine
    nomTache[indiceNomTache] = '\0'; // idem
    pert->sommets[numSommet - 1] = sommet;
    pert->intitule[numSommet - 1] = nomTache;
    // et surtout ne pas faire de free() sur sommet et nomTache
    - Attention à certains détails :
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    sommet = realloc(sommet, indiceSommet+1 * sizeof(char));
    sans doute :
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    sommet = realloc(sommet, (indiceSommet+1) * sizeof(char));
    Heureusement que sizeof(char) vaut toujours 1 !

    -
    alors que si au lieu d'affecter le tableau 'buffer' j'affecte une chaine en dur (exemple "test") elle s'affiche correctement par la suite.
    La chaine en dur continue à exister après la sortie de la fonction. Ce serait le cas de la mémoire pointée par buffer si elle avait été allouée dynamiquement.


    - @mikhailo
    je l'ai résolu en déclarant la structure à remplir (le tableau sommets dans ton cas, si je comprends bien) en tant que variable globale
    Pourquoi donner de mauvais (et de plus faux) conseils ?

Discussions similaires

  1. Difference entre [Simple quote] & [Double quote]
    Par Invité dans le forum SQL
    Réponses: 3
    Dernier message: 24/07/2013, 12h24
  2. Recherche API indiquant differences entre chaines de caracteres
    Par julseb2007 dans le forum API standards et tierces
    Réponses: 2
    Dernier message: 14/08/2007, 14h24
  3. Réponses: 10
    Dernier message: 31/05/2007, 15h10

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