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 :

Problème avec les pointeurs


Sujet :

C

  1. #1
    Membre averti
    Inscrit en
    Octobre 2006
    Messages
    33
    Détails du profil
    Informations forums :
    Inscription : Octobre 2006
    Messages : 33
    Par défaut Problème avec les pointeurs
    Bonjour a tous,

    Je débute en C et je doit faire un prog qui lit des lignes dans un fichier, trouver les chaines de caractères qui m'interessent et les écrire dans un autre fichier.

    Voici mon code:

    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
    int main(){
    	if( (cuID = (char**)malloc(CUID_TAB_SIZE * sizeof(*cuID) )) == NULL){
    		printf("l'allocation de la mémoire a échoué!!\n");
    		exit(-1);
    	}
    getCustomerID_String(buffer, cuID);
    fprintf(sortie, "cuID[%d] = %s\n", j, cuID[j]);
    }
     
    void getCustomerID_String(char *_query, char **_cuID){
    	if( (_cuID[i] = (char*)malloc(strlen(_query) + 1)) == NULL){
    		printf("L'allocation de la mémoire a échoué!!\n");
    		freemem(query, increment, pattern);
    	}
    	else{
    		strcpy(_cuID[i], "ma chaine trouver");
    }
    Lors du realloc j'obtient une belle erreur de type :

    *** glibc detected *** ./getCustomerID_String: realloc(): invalid pointer: 0xb7ee2278 ***
    ======= Backtrace: =========
    /lib/i686/cmov/libc.so.6[0xb7df8624]
    /lib/i686/cmov/libc.so.6(realloc+0x242)[0xb7dfcc82]
    /lib/i686/cmov/libc.so.6(realloc+0x42)[0xb7dfca82]
    ./getCustomerID_String[0x80489ea]
    ./getCustomerID_String[0x8048c80]
    /lib/i686/cmov/libc.so.6(__libc_start_main+0xe5)[0xb7da0455]
    ./getCustomerID_String[0x8048741]
    ======= Memory map: ========
    08048000-0804a000 r-xp 00000000 08:01 475103 /home/mathieu/
    0804a000-0804b000 rw-p 00001000 08:01 475103 /home/mathieu
    09ce6000-09d07000 rw-p 09ce6000 00:00 0 [heap]
    b7d6f000-b7d7b000 r-xp 00000000 08:01 40883 /lib/libgcc_s.so.1
    b7d7b000-b7d7c000 rw-p 0000b000 08:01 40883 /lib/libgcc_s.so.1
    b7d89000-b7d8a000 rw-p b7d89000 00:00 0
    b7d8a000-b7edf000 r-xp 00000000 08:01 50066 /lib/i686/cmov/libc-2.7.so
    b7edf000-b7ee0000 r--p 00155000 08:01 50066 /lib/i686/cmov/libc-2.7.so
    b7ee0000-b7ee2000 rw-p 00156000 08:01 50066 /lib/i686/cmov/libc-2.7.so
    b7ee2000-b7ee5000 rw-p b7ee2000 00:00 0
    b7ef0000-b7ef4000 rw-p b7ef0000 00:00 0
    b7ef4000-b7ef5000 r-xp b7ef4000 00:00 0 [vdso]
    b7ef5000-b7f0f000 r-xp 00000000 08:01 40882 /lib/ld-2.7.so
    b7f0f000-b7f11000 rw-p 0001a000 08:01 40882 /lib/ld-2.7.so
    bfbfc000-bfc11000 rw-p bffeb000 00:00 0 [stack]
    Quelqu'un pourrait il me dire pourquoi?
    Merci.

  2. #2
    Membre averti
    Inscrit en
    Octobre 2006
    Messages
    33
    Détails du profil
    Informations forums :
    Inscription : Octobre 2006
    Messages : 33
    Par défaut
    J'ai omis de mettre les déclarations de mes variables mais tous le monde saura que ces dernières sont correctement initialisée, juste celle la si çà intéresse quelqu'un :

    Je fais ca aussi au debut de mon main :

    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    4
    if( (cuID = (char**)malloc(CUID_TAB_SIZE * sizeof(*cuID) )) == NULL){
    		printf("l'allocation de la mémoire a échoué!!\n");
    		exit(-1);
    }

  3. #3
    Membre averti
    Inscrit en
    Octobre 2006
    Messages
    33
    Détails du profil
    Informations forums :
    Inscription : Octobre 2006
    Messages : 33
    Par défaut
    désolé pour les balises codes!!!!

  4. #4
    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
    Tu parles d'un realloc qui ne figure pas dans le code !

    Tu utilises des variables globales cuID , buffer, j, i, query, increment, pattern et une fonction freemem() dont on n'a pas le code.
    L'utilisation de variables globales fait qu'on ne peut rien dire sans l'intégralité du code. La première chose à faire est de les supprimer, c'est une habitude à prendre dès qu'on débute. Les globales sont des pestes !

    Sinon, une anomalie ici :
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    4
    5
    6
    void getCustomerID_String(char *_query, char **_cuID){
    	if( (_cuID[i] = (char*)malloc(strlen(_query) + 1)) == NULL){
    ...
    	}
    	else{
    		strcpy(_cuID[i], "ma chaine trouver");
    tu alloues de la place pour ranger la chaîne _query et tu places dedans une autre chaine; Si elle est plus grande, ça va planter.

  5. #5
    Membre averti
    Inscrit en
    Octobre 2006
    Messages
    33
    Détails du profil
    Informations forums :
    Inscription : Octobre 2006
    Messages : 33
    Par défaut
    Voila l'integralité de mon code j'espère que ca vous aideras...

    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
    #include <stdlib.h>
    #include <stdio.h>
    #include <string.h>
    #include <time.h>
     
    #define CUID_TAB_SIZE	20
     
    void freemem(char *_query, char *_increment, char *_pattern){
    	free(_query);
    	free(_increment);
    	free(_pattern);
    }
     
     
    void getCustomerID_String(char *_query, char **_cuID){
    	char *query = NULL;
    	char *start;
    	char *stop;
    	char * increment = NULL;
    	char *pattern = NULL;
    	int i = 0;
     
    	if( (query = (char*)malloc(strlen(_query)+1)) == NULL){
    		printf("L'allocation de la mémoire a échoué\n");
    		exit(-1);
    	}
    	else
    		strcpy(query, _query);
     
    	if( (pattern = (char*)malloc(5)) == NULL){
    		printf("L'allocation de la mémoire a échoué\n");
    		exit(-1);
    	}
    	strcpy(pattern, "cu");
     
    	if( (increment = (char*)malloc(5)) == NULL){
    		printf("L'allocation de la mémoire a échoué\n");
    		exit(-1);
    	}
     
    	for(i = 1 ; i <= CUID_TAB_SIZE; i++){
    		if( (snprintf(increment, 2, "%d", i)) < 0){
    			printf("La conversion a echouer\n");
    			freemem(query, increment, pattern);
    			exit(-1);
    		}
    		strcat(pattern, increment);
    		strcat(pattern, "=");
     
    		if((start = strstr(query, pattern)) == NULL){
    //			printf("La chaine %s n'a pas était trouvés!!!!\n", pattern);
    			break;
    		}
    		else{
    			if((stop = strstr(start, "&")) == NULL)
    				printf("Le caractère '&' n'a pas était trouvés!!!!\n");
    			else{
    				if( (_cuID[i] = (char*)realloc((char*)_cuID[i], i * strlen(start) + 1)) == NULL){
    //				if( (_cuID[i] = (char*)malloc(strlen(_query) + 1)) == NULL){
    					printf("L'allocation de la mémoire a échoué!!\n");
    					freemem(query, increment, pattern);
    				}
    				else{
    					strcpy(_cuID[i], "");
    					strncpy(_cuID[i], start + strlen(pattern), (strlen(start) - strlen(stop)) - strlen(pattern) );
    					strcpy(query, stop);
    					strcpy(start, stop);
    				}
    			}
    		}
    		strcpy(pattern, "cu");
    	}
    	freemem(query, increment, pattern);
    }
     
     
    int main(){
     
    	int j;
    	char **cuID;
    	char *MyFile = "ExempleLog";
    	char *buffer;
    	FILE *fd;
    	FILE *sortie;
    	char buf[1024];
    	time_t begin, end;
    	struct tm * t;
     
    	begin = time(NULL);
    	t = localtime(&begin);
    	printf("Début d'execution à %02uh %02umin %02usec\n", t->tm_hour, t->tm_min, t->tm_sec);
     
    	if( (fd = fopen(MyFile, "r")) == NULL){
    		printf("Impossible d'ouvrir le fichier %s\n", MyFile);
    		exit(-1);
    	}
    	if( (sortie = fopen("customerFile", "w+")) == NULL){
    		printf("Impossible d'ouvrir le fichier customerFile\n");
    		fclose(fd);
    		exit(-1);
    	}
     
    	if( (cuID = (char**)malloc(CUID_TAB_SIZE * sizeof(*cuID) )) == NULL){
    		printf("l'allocation de la mémoire a échoué!!\n");
    		exit(-1);
    	}
     
    	if( (buffer = (char*)malloc(1024)) == NULL){
    		printf("l'allocation de la mémoire a échoué!!\n");
    		exit(-1);
    	}
     
    	while( (fgets(buf, 1024, fd)) != NULL){
    		j = 1;
    		strncpy(buffer, buf, strlen(buf)+1);
    		getCustomerID_String(buffer, cuID);
    		while(cuID[j] != NULL){
    			if( (strcmp(cuID[j], "") != 0)){
    				fprintf(sortie, "cuID[%d] = %s\n", j, cuID[j]);
    				strncpy(cuID[j], "", strlen(cuID[j]));
    			}
    			j++;
    		}
    	}
     
    	if( (fclose(fd)) != 0)
    		printf("Fermeture du fichier fd impossible\n");
    	if( (fclose(sortie) != 0))
    			printf("Fermeture du fichier sortie impossible\n");
    	free(cuID);
     
    	end = time(NULL);
    	t = localtime(&begin);
    	printf("Début d'execution à %02uh %02umin %02usec\n", t->tm_hour, t->tm_min, t->tm_sec);
     
    	return 0;
    }
    Fichiers attachés Fichiers attachés

  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
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    4
    5
    6
    7
    8
    9
    10
    void getCustomerID_String(char *_query, char **_cuID){
    ...
    	for(i = 1 ; i <= CUID_TAB_SIZE; i++){
    ...
    				if( (_cuID[i] = (char*)realloc((char*)_cuID[i], i * strlen(start) + 1)) == NULL){
    ...
    int main(){
    ...
    	if( (cuID = (char**)malloc(CUID_TAB_SIZE * sizeof(*cuID) )) == NULL){
    ...
    main a alloué CUID_TAB_SIZE éléments (je n'ai pas vu de realloc sur cuID).
    Or la boucle va de 1 (pourquoi 1 et pas 0 ?) à CUID_TAB_SIZE qui est hors du tableau dont les indices doivent être entre 0 etCUID_TAB_SIZE-1

  7. #7
    Membre averti
    Inscrit en
    Octobre 2006
    Messages
    33
    Détails du profil
    Informations forums :
    Inscription : Octobre 2006
    Messages : 33
    Par défaut
    En fait mon probleme est que je veux garder mon char **cuID tout au long de l'éxecution du programme donc je le passe en parametre à ma fonction getCustomerID_String. Je l'utilise de la facon suivant cuID[i] = "test";

    Est ce judicieux de jouer avec des types char **??
    N'y a t il pas une solution plus simple de passer un type char *cuID[] à une fonction??

  8. #8
    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
    En fait mon probleme est que je veux garder mon char **cuID tout au long de l'éxecution du programme donc je le passe en parametre à ma fonction getCustomerID_String. Je l'utilise de la facon suivant cuID[i] = "test";
    Donc, ta boucle devrait être
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    for(i = 0; i < UID_TAB_SIZE; i++){
    Et ton problème vient très probablement de là.

    J'espère que tu n'écris pas cuID[i] = "test" mais strcpy(cuID[i] ,test")

    Est ce judicieux de jouer avec des types char **??
    N'y a t il pas une solution plus simple de passer un type char *cuID[] à une fonction??
    tu as un tableau de pointeurs sur char et il est normal que tu obtiennes des char **
    Par contre, tu uses et peut-être abuses de l'allocation dynamique inutilement
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    if( (pattern = (char*)malloc(5)) == NULL){
    if( (increment = (char*)malloc(5)) == NULL)
    Pourquoi ne pas faire des tableaux automatiques ?
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    char pattern[5];
    char increment[5];

  9. #9
    Membre averti
    Inscrit en
    Octobre 2006
    Messages
    33
    Détails du profil
    Informations forums :
    Inscription : Octobre 2006
    Messages : 33
    Par défaut
    Merci de tes réponses mais j'ai totalement revu mon programme et je suis enfin arrivé a mes fins...

    En tout cas merci pour ton aide ca m'a aidé.

  10. #10
    Expert éminent
    Avatar de Emmanuel Delahaye
    Profil pro
    Retraité
    Inscrit en
    Décembre 2003
    Messages
    14 512
    Détails du profil
    Informations personnelles :
    Âge : 68
    Localisation : France, Paris (Île de France)

    Informations professionnelles :
    Activité : Retraité

    Informations forums :
    Inscription : Décembre 2003
    Messages : 14 512
    Par défaut
    Citation Envoyé par matdakillah Voir le message
    Voila l'integralité de mon code j'espère que ca vous aideras...
    Il ne manque plus qu'un exemple de fichier...
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    4
    5
    6
     
    DÚbut d'execution Ó 10h 09min 04sec
    Impossible d'ouvrir le fichier ExempleLog
     
    Process returned -1 (0xFFFFFFFF)   execution time : 0.097 s
    Press any key to continue.
    Nota : les valeurs de retour portables pour main() sont
    • 0
    • EXIT_SUCCESS
    • EXIT_FAILURE

    Sur ma machine, -1 n'est pas valable...

Discussions similaires

  1. Problème avec les pointeurs intelligents de boost.
    Par Le Barde dans le forum Boost
    Réponses: 2
    Dernier message: 05/09/2007, 12h47
  2. Petit problème avec les pointeurs et variable
    Par mitherkiller dans le forum C
    Réponses: 5
    Dernier message: 09/03/2007, 22h05
  3. problème avec les pointeurs en c
    Par dialloma dans le forum C
    Réponses: 14
    Dernier message: 01/01/2007, 21h22
  4. probléme avec les pointeurs
    Par killer_instinct dans le forum C++
    Réponses: 6
    Dernier message: 11/12/2006, 11h37
  5. [TTreeView] Problème avec les pointeurs d'objet
    Par BlackWood dans le forum Composants VCL
    Réponses: 2
    Dernier message: 02/07/2004, 14h31

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