[ C ] Help, i'm gonna kill myself.

Marsh Posté le 18-06-2002 à 15:45:26

Mais comment que c'est casse couille les pointeurs .. franchement je comprends que dalle, je reste une éternité sur une pauvre portion de code. Je les utilise et je sais m^^eme pas ce que ça représente ...

Comment peut t on savoir où se trouve un erreur de type mémoire .. Quand je débugue, j'ai un truc du genre , User BReakpoint called from code at 0x77f etc ... ça représente quoi par rapport a mon code ???

Code :

#include <stdio.h>
#include <string.h>
#include <stdlib.h>
#define RESULT_FILE "result.txt"
#define LINERETURN "\n"
#define SIZETAB 2000
#define MAXCHAR 100
#define BATFILE "greptab.bat"
#define CPY_TO_FILE " >> "
char *tables[SIZETAB];
char input_f[100];
char path[100];
char *bat;
/* declaration des fonctions */
void launch_bat();
void help(void);
void writefile(char *mot);
char *insert(char *table);
int del_fichier_res(char *fileres);
int del_bat_file(void);
void fAnalyse(char *fileres);
/*Fonction qui scanne le fichier contenant la liste des chaines a rechercher dans les sources*/
void fRun(char *fichier)
{
FILE *fic;
FILE *sortie;
char temp[32];
char *cmd;
char *retour;
char *fileres;
int j;
int i=0;
char *tables_res;
char *info;
//formation de la chaine pour le chemin du fichier resultat
fileres = malloc((strlen(RESULT_FILE)+strlen(path))*sizeof(c
har));
sprintf(fileres,"%s%s",path,RESULT_FILE);
retour=malloc(strlen(CPY_TO_FILE)*sizeof(char));
strcpy(retour,CPY_TO_FILE);
info=malloc(strlen("echo " )*sizeof(char));
strcpy(info,"echo " );
fic = fopen(fichier,"r" );
if(fic != NULL)
{
// system("md result" );
do
{
fscanf(fic, "%s", temp);
tables[i]=malloc(strlen(temp)*sizeof(char));
strcpy(tables[i],temp);
i++;
}
while(!feof(fic));
//Pour se situer reelement a la derniere ligne
i--;
fclose(fic);
bat = malloc((strlen(path)+strlen(BATFILE))*sizeof(char)
);
sprintf(bat,"%s%s",path,BATFILE);
printf("effacement du fichier bat %s\n",bat);
if(del_bat_file()==0)
{
printf("le fichier bat %s a ete efface\n",bat);
}
else
{
printf("pas de fichier bat %s trouve\n",bat);
}
printf("effacement du fichier resultat %s\n",fileres);
if(del_fichier_res(fileres)==0) printf("le fichier resultat %s a ete efface\n",fileres);
else printf("pas de fichier resultat %s trouve\n",fileres);
sortie = fopen(bat,"a" );
if(sortie != NULL)
{
for(j=0;j<i;j++)
{
cmd = malloc(strlen(tables[j])*sizeof(char));
strcpy(cmd,insert(tables[j]));
tables_res = malloc((strlen(info)+strlen(tables[j])+strlen(file
res)+strlen(retour)+strlen(LINERETURN))*sizeof(cha
r));
sprintf(tables_res,"%s%s%s%s%s",info,tables[
j],retour,fileres,LINERETURN);
fputs(tables_res, sortie);
fputs(cmd, sortie);
}
fclose(sortie);
}
launch_bat(bat);
// fAnalyse(fileres,i);
}
else printf("le fichier %s n'a pas pu etre ouvert\n\n",input_f);
}
int del_bat_file(void)
{
char *command;
char delcom[5];
strcpy(delcom,"del " );
command = malloc((strlen(delcom)+strlen(bat))*sizeof(char));
sprintf(command,"%s%s",delcom,bat);
if (system(command) == 0)
{
return(0);
}
else
{
return(-1);
}
}
void launch_bat(void)
{
FILE *batfile;
batfile = fopen(bat,"r" );
if (batfile == NULL)
{
printf ("le fichier %s contenant les commandes à executer n'a pas pu être trouvé\n",BATFILE);
}
else
{
system(bat);
}
}
char *insert(char *table)
{
char *cmdd;
char *cmdf;
char *cmd;
char *separateur = "/";
char *lResultFileName;
char *searchdir;
char *filename;
lResultFileName = malloc(strlen(RESULT_FILE)*sizeof(char));
strcpy(lResultFileName,RESULT_FILE);
filename = malloc((strlen(path)+strlen(lResultFileName))*size
of(char));
sprintf(filename,"%s%s",path,lResultFileName);
cmdd=malloc(strlen("grep -i -c -d " )*sizeof(char));
cmdf=malloc(strlen(" >> " )*sizeof(char));
strcpy(cmdd,"grep -i -c -d " );
strcpy(cmdf," >> " );
searchdir = malloc((strlen(path)+strlen("*.*" ))*sizeof(char));
sprintf(searchdir,"%s%s%s"," ",path,"*.*" );
cmd = malloc((strlen(cmdd)+strlen(table)+strlen(searchdi
r)+strlen(cmdf)+strlen(filename)+strlen(LINERETURN
))*sizeof(char));
sprintf(cmd,"%s%s%s%s%s%s",cmdd,table,searchdir
,cmdf,filename,LINERETURN);
// free(lResultFileName);
// free(filename);
// free(searchdir);
return(cmd);
}
int del_fichier_res(char *fileres)
{
char *command;
char delcom[6];
strcpy(delcom,"del " );
command = malloc((strlen(delcom)+strlen(path))*sizeof(char))
;
sprintf(command,"%s%s",delcom,fileres);
if (system(command) == 0) return(0);
else return(-1);
}
void main(int argc, char *argv[])
{
argc = 3;
argv[1]="C:\\wrkcesar\\database\\list.txt
";
argv[2]="C:\\wrkcesar\\database\\";
if (argc != 3) help();
else
{
strcpy(input_f,argv[1]);
strcpy(path,argv[2]);
fRun(argv[1]);
}
}
void help(void)
{
printf("Correct syntax is :\n\tgreper <list.file> <search.path>" );
exit(0);
}

C'est surement très laid, c'est premier exe en C.

Est ce que vous pourriez juste me taper deux trois chtites remarques comme les erreurs a pas faire, les trucs que j'ai fais etc ..

Ce code est censé greper des mots ( qui sont contenues dans un fichier ) dans un répertoire et ses sous repertoires .. et de mettre le résultat dans un fichier resultat.txt

D'abord je tente de lire ma liste de mots pour construire un .bat qui éxécuté va remplir mon fichier resultat.txt

Je m'en sors pas du tout ..

Merci

Message édité par paranoidandroid le 18-06-2002 à 15:59:40

Reply

Marsh Posté le 18-06-2002 à 15:45:26

Reply

Marsh Posté le 18-06-2002 à 16:23:32

c surement un débordement mémoire.
quand tu fais un *scanf (ici, un fscanf), pour tes arguments (les derniers paramètres), tu doit mettre l'adresse, pas l'argument lui-même, donc tu vas plutot mettre :

Code :

fscanf(fic, "%s", &temp);

ensuite, vas voir sur l'autre topic pour la lecture de fichier :
http://forum.hardware.fr/forum2.ph [...] h=&subcat=

Essaye déja ça avant de te tuer ! :hello:

Message édité par El_gringo le 18-06-2002 à 16:24:13

Reply

Marsh Posté le 18-06-2002 à 16:28:36

et puis, dans ton main, ça va pas du tout:

Code :

void main(int argc, char *argv[])
{
argc = 3;
argv[1]="C:\\wrkcesar\\database\\list.txt
";
argv[2]="C:\\wrkcesar\\database\\";
if (argc != 3) help();
else
{
strcpy(input_f,argv[1]);
strcpy(path,argv[2]);
fRun(argv[1]);
}
}

T'utilises int argc et char *argv[] en écriture, j'aurais même pas pensé que ça pourrait compiler.
ces trucs, c fait pour récupérer les arguments que t'as passé à ton exe en ligne de comande. T'as pas à aller bricoler dedant. Pourquoi tu mets des valeurs dedant !? j'comprend pas ? c quoi ces : "C:\\wrkcesar\\database\\list.txt" et "C:\\wrkcesar\\database\\" ?

Reply

Marsh Posté le 18-06-2002 à 16:35:56

el_gringo a écrit a écrit :

c surement un débordement mémoire.
quand tu fais un *scanf (ici, un fscanf), pour tes arguments (les derniers paramètres), tu doit mettre l'adresse, pas l'argument lui-même, donc tu vas plutot mettre :

Code :

fscanf(fic, "%s", &temp);

ensuite, vas voir sur l'autre topic pour la lecture de fichier :
http://forum.hardware.fr/forum2.ph [...] h=&subcat=

Essaye déja ça avant de te tuer ! :hello:

Non c pas nécessaire, temp est déjà un pointeur vers une zone mémoire, par contre elle est peut être pas assez grande.

---------------
Le Tyran

Reply

Marsh Posté le 18-06-2002 à 16:39:17

paranoidandroid a écrit a écrit :

Code :

void main(int argc, char *argv[])
{
argc = 3;
argv[1]="C:\\wrkcesar\\database\\list.t
xt
";
argv[2]="C:\\wrkcesar\\database\\";
if (argc != 3) help();
else
{
strcpy(input_f,argv[1]);
strcpy(path,argv[2]);
fRun(argv[1]);
}
}
void help(void)
{
printf("Correct syntax is :\n\tgreper <list.file> <search.path>" );
exit(0);
}

:ouch:
tu nous fais quoi la ??
argc et argv sont utilisés pour récupérer les paramètres passés en ligne de commande, t'as pas à y accéder en écriture !!!

---------------
J'ai un string dans l'array (Paris Hilton)

Reply

Marsh Posté le 18-06-2002 à 16:44:02

el_gringo a écrit a écrit :

et puis, dans ton main, ça va pas du tout:

Code :

void main(int argc, char *argv[])
{
argc = 3;
argv[1]="C:\\wrkcesar\\database\\list.txt
";
argv[2]="C:\\wrkcesar\\database\\";
if (argc != 3) help();
else
{
strcpy(input_f,argv[1]);
strcpy(path,argv[2]);
fRun(argv[1]);
}
}

T'utilises int argc et char *argv[] en écriture, j'aurais même pas pensé que ça pourrait compiler.
ces trucs, c fait pour récupérer les arguments que t'as passé à ton exe en ligne de comande. T'as pas à aller bricoler dedant. Pourquoi tu mets des valeurs dedant !? j'comprend pas ? c quoi ces : "C:\\wrkcesar\\database\\list.txt" et "C:\\wrkcesar\\database\\" ?

Je bidouille dedans pour l'executer en mode débug ...

Avec VC++6 si je fais fais F5 je m'arrête des la fin du main ... Alors c'est pour le forcer a =entrer dans le read .. en quelque sorte ..

Reply

Marsh Posté le 18-06-2002 à 16:51:51

Bah, avec VC++, tu vas dans les Setting de ton projet, onglet debug, le 3e champ de texte en partant du haut : "program arguments"
ici, tu mets C:\\wrkcesar\\database\\list.txt C:\\wrkcesar\\database\\
ça sera plus simple et vachement moins vilain !

Reply

Marsh Posté le 18-06-2002 à 16:53:23

paranoidandroid a écrit a écrit :

Je bidouille dedans pour l'executer en mode débug ...

Avec VC++6 si je fais fais F5 je m'arrête des la fin du main ... Alors c'est pour le forcer a =entrer dans le read .. en quelque sorte ..

Voui mais non !
Déjà tu ne peux pas faire

Code :

mon_pointeur_de_caractère = "toto" ;

en C.
Mais

Code :

strcpy(mon_pointeur_de_caractère,"toto" ) ;

Et encore, ce n'est pas suffisant : il faut que mon_pointeur_de_caractère définissent une zone mémoire libre de taille supérieure ou égale à ceux que tu veux y faire rentrer dedans.
argc et argv étant des variables réservées, je ne m'y risquerai même pas.

---------------
Get the Cooool... Get the cool shoe-shine !

Reply

Marsh Posté le 18-06-2002 à 17:02:21

C est pas joli, joli :heink: :eek2:

Mais le probleme principal c est que tout oubli la place pour le '\0' lorsque tu fais des malloc de chaine de caractère

---------------
WoIP - Video and Voice over IP - http://www.woip.net/

Reply

Marsh Posté le 18-06-2002 à 17:45:21

MrTonio a écrit a écrit :

C est pas joli, joli :heink:

Mais le probleme principal c est que tout oubli la place pour le '\0' lorsque tu fais des malloc de chaine de caractère

Je sais que c'est pas beau ...
Je m'y recolle ce soir, que du c

Reply

Marsh Posté le 18-06-2002 à 17:45:21

Reply

Marsh Posté le 18-06-2002 à 17:45:44

el_gringo a écrit a écrit :

Bah, avec VC++, tu vas dans les Setting de ton projet, onglet debug, le 3e champ de texte en partant du haut : "program arguments"
ici, tu mets C:\\wrkcesar\\database\\list.txt C:\\wrkcesar\\database\\
ça sera plus simple et vachement moins vilain !

Merci pour cette précision ...

Reply

Marsh Posté le 18-06-2002 à 20:57:22

Y'a pas une taille maximale lorsque on lit une ligne dans un fichier ... genre 256 je sais plus

Reply

Marsh Posté le 18-06-2002 à 21:11:20

Loaded 'C:\WINNT\System32\ntdll.dll', no matching symbolic information found.
Loaded 'C:\WINNT\system32\KERNEL32.DLL', no matching symbolic information found.
The thread 0x5C0 has exited with code 57 (0x39).
The program 'E:\BTS\pti\greper\Debug\grep.exe' has exited with code 57 (0x39).

j'ai tout compris ...

Reply

[ C ] Help, i'm gonna kill myself.

Sujets relatifs:

Leave a Replay