C++

Tableau de chaînes de caractères...

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : Tableau de chaînes de caractères...

ceyquem

E falso sequitur quodlibet

Bonjour,

j'ai un problème lorsque je veux passer à un module un tableau de chaînes de caractères pour qu'il le remplisse.

ce module est censé lire toutes les lignes d'un fichier index.txt et les placer dans le tableau correspondant s'appelant indexv.

le remplissage dans le module se passe bien (je peux lire le tableau depuis le module sans pb) mais quand on revient dans le main, je n'arrive plus à y accéder. Voici le code :

[g]main.c[/b]

Code :

#include<string.h>
#include "texindex.h"
int main(int argc, char** argv)
{
int i=0;
int indexc=0;
char** indexv;
//lecture du fichier d'index de mots :
readIndexTxt(argv[1],&indexc,&indexv);
printf("\nVérification : %d\n",indexc);
for(i=0;i<indexc;i++)
{
printf("Vérification : indexv[%d]\t:\t%s\n",i,indexv[i]);
}
printf("\n" );
return EXIT_SUCCESS;
}

texindex.c

Code :

#include "texindex.h"
void readIndexTxt(char* filename, int* indexc, char*** indexv)
{
int i=0;
FILE* f;
//tampon de lecture
char s[40];
if((f=fopen(filename,"r" ))==NULL)
{
printf("Erreur d'ouverture du fichier %s\n",filename);
exit(1);
}
//comptage des mots contenus dans le fichiers (un par ligne)
(*indexc)=0;
while((fgets(s,80,f))!=NULL) (*indexc)++; //ajout d'un mot
//allocation mémoire
(*indexv)=malloc((*indexc)*sizeof(char[40]));
//retour au début du fichier
rewind(f);
//remplissage de la liste de mots
printf("Chargement de la liste de mots" );
printf("\n" );
while((fgets(s,80,f))!=NULL)
{
(*indexv)[i]=s;
printf("Chargement de : indexv[%d]\t:\t%s",i,(*indexv)[i]);
i++;
}
fclose(f);
}

texindex.h

Code :

#include<stdlib.h>
#include<stdio.h>
void readIndexTxt(char* filename, int* indexc, char*** indexv);

makefile

Code :

all : main clean
main : main.o texindex.o
gcc -g main.o texindex.o -o main
main.o : main.c texindex.h
gcc -g -c main.c
texindex.o : texindex.c texindex.h
gcc -g -c texindex.c
clean : main
rm *.o

Et voici le résultat :

Code :

Chargement de la liste de mots
Chargement de : indexv[0] : tex
Chargement de : indexv[1] : latex
Chargement de : indexv[2] : miktex
Chargement de : indexv[3] : jadetex
Chargement de : indexv[4] : pstricks
Vérification : 5
Vérification : indexv[0] : pstr q@¼
Vérification : indexv[1] : pstr q@¼
Vérification : indexv[2] : pstr q@¼
Vérification : indexv[3] : pstr q@¼
Vérification : indexv[4] : pstr q@¼

Une idée :??: :??: ??

Merci !

Publicité

antp

Super Administrateur
Champion des excuses bidons

(*indexv)[i]=s;

tu copies le pointeur vers une chaine stockée dans une variable locale

strcpy((*indexv)[i], s);
me semble plus indiqué

---------------
mes programmes ·· les voitures dans les films ·· apprenez à écrire

nonokao

moooort o foooot !!!

oulala ! t'as fais deux bêtises, je t'explique dès que j'ai le temps

Deaddy

commence par simplifier:

void readIndexTxt(char* filename, int* indexc, char** indexv)

et
//lecture du fichier d'index de mots :
readIndexTxt(argv[1],&indexc,indexv);

Taz

bisounours-codeur

sizeof(char[40])

beurk

sizeof ma_variable
sizeof(mon_type)

nonokao

moooort o foooot !!!

la première, c'est un détail, mais dangeureux quand même,
tu déclares

Code :

char s[40];

donc tu réserves 40 caractères et tu autorises fgets à y écrire 80 caractères,

Code :

fgets(s,80,f)

, mets une chaine de plus de 40 caractères sur une ligne dans ton fichier et ton pc va être content, surtout si l'impact n'est pas tout de suite visible.

Taz

bisounours-codeur

y au aussi autre problèem. il faut faire l'allocation en 2 fois

une boucle pour allouer n pointeurs de caractères
une autre pour allouer m caracteres pour chacune de ces n pointeurs.

si tu fais les 2 d'un coup, il faut proprement tu n'as qu'un char* et pas un char* donc la double indirection ne fonctionne pas

nonokao

moooort o foooot !!!

deuxième erreur :

tu utilises indexv comme un tableau de pointeur sur des chaines

Code :

printf("Vérification : indexv[%d]\t:\t%s\n",i,indexv[i]);

hors tu réserve je ne sais pas trop koi en mémoire, un truc à cheval entre les deux mallocs nécessaire

Code :

(*indexv)=malloc((*indexc)*sizeof(char[40]));

oui, deux mallocs sont nécessaire,
un pour le tableau de pointeurs
le deuxième dans une boucle pour allouer l'espace pour chaque chaine ; idéalement, tu peux allouer juste l'espace nécessaire pour la longueur de chaque chaine, ou prévoir plus (40 par exemple) si tu as l'intention de les modifier, de les rallonger ...
bref je te laisse corriger ton code

nonokao

moooort o foooot !!!

bien entendu il existe la possibilité d'alloué tout l'espace nécessaire à indexv puis de l'indexer par paquets de 40 caractères, seulement il faut soit changer sa déclaration, soit changer la manière d'y acceder
sur ce, @+, je repasserais dès que j'ai un peu de temps voir si tu t'en sors :wahoo:

nonokao

moooort o foooot !!!

dernier détail, bien que ton compilateur ajoute le code nécessaire à ton application pour que losrque celle-ci se termine, toutes les ressources allouées soient libérées, il est préférable de libérer chaque allocation de mémoire avec la fonction free() dès que tu as fini d'utiliser ta super liste de mots en tex :lol: surtout si ton programme fais autre chose après et qu'il n'en a plus besoin.

Publicité

ceyquem

E falso sequitur quodlibet

voilà j'ai trouvé comment faire un tableau dynamiques de chaines de caractères de longuer statique :

Code :

char (*x)[20];
int nombreDeChaines;
nombreDeChaines = 4;
x = (char (*)[20])malloc(sizeof(char[25])*nombreDeChaines);
strcpy(x[0],"salut" );
strcpy(x[1],"allo" );
printf("%s\n%s\n",x[0],x[1]);
// ...
free(x);

Maintenant, jen'arrive pas à passer le tableau x en argument à ma fonction readIndexTxt dans laquelle je voudrais faire la partie allocation et remplissage du code précédent

Pouvez-vous m'éclairer ??

Merci

Deaddy

cf mon post plus haut

theshockwave

I work at a firm named Koslow

Deaddy a écrit :

cf mon post plus haut

si il alloue la mem dans sa fct, il est obligé de faire un char *** ... sinon, il ne pourra pas modifier le pointeur passé en param !

Deaddy

exact ! mais dans ce cas (le tablo n'a pas été alloué) ca sert à rien de le passer en param

autant faire un truc du style:
char** readIndexTxt(char* filename, int* indexc)
{
char **x;
...
allocation de x
...
pas de free(x);
return x;
}

Musaran

Cerveaulté

ceyquem a écrit :

Code :

char (*x)[20];
int nombreDeChaines;
nombreDeChaines = 4;
x = (char (*)[20])malloc(sizeof(char[25])*nombreDeChaines);
...

Allocation de [25] mais cast en [20], ça va pas marcher...
C'est pas interdit de simplifier:

Code :

typedef char Chaine[20];
int nombreDeChaines = 4;
Chaine* x= (Chaine *)malloc(sizeof(Chaine)*nombreDeChaines);

---------------
Bricocheap: Montage de ventilo sur paté de mastic silicone

ceyquem

E falso sequitur quodlibet

j'ai résolu mon pb en utilisant :

Code :

typedef char str40[40+1];

et en faisant

Code :

strcpy((*indexv)[i],s);

merci à tous pour votre aide !

FORUM HardWare.fr

Programmation

C++

Tableau de chaînes de caractères...

Sujets relatifs
Comment définir le nombre de caractères maximal pour une <textarea> ?	Tableau dynamique à deux entrées avec vector
nombre de sous chaines dans une chaine	tableau à 2 dimension dynamique de CString
Comment enregistrer une chaine de caracteres saisie au clavier ?	Passage erroné d'un tableau de chaînes de caractères à une procédure..
[PHP] Trier un tableau associatif sur le champ de son choix ?	partager une chaîne de caractères
Plus de sujets relatifs à : Tableau de chaînes de caractères...

Page générée en 0.091 secondes