besoin d'avis sur mon code

Version imprimable

bonjour,
je viens d'ecrire une fonction qui lit et ecrit dans un fichier
elle fait bien le travail mais ne s'arrete pas car j'ai ce message dela console "

Citation:

processus arrete

"

voici mon code
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 FILE* read_write(FILE* f){ FILE* f_lign = fopen("text_lign.txt","r+"); char c=fgetc(f); while((f!=NULL)&&(c!=EOF)){ if(((c==' ')||(c=='\n'))){ fputs("\n",f_lign); //printf("\n"); } else { fputc(c,f_lign); //printf("%c",c); } c=fgetc(f); } return f_lign; fclose(f_lign); }
merci d'avance

18/11/2015, 22h39
Médinoc

Citation:

Elle ne s'arrête pas car j'ai le message "processus arrêté"

:question:
18/11/2015, 22h44
chrislains

Citation:

Envoyé par Médinoc

:question:

car sa n'execute pas la suite du programme
pendant un moment on a l'impression qu'il ya un beug et ensuite sa m'affiche processus arrete
si tu veux je te met l'ensemble du code

voici l'ensemble de mon code

fichier full.c

Code:

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
#include <stdlib.h>
#include <stdio.h>
#include <assert.h>
#include <string.h>
#include "test.h"
#include "test.c"
#include "test_read_fich.c"
#define TAILLE_MAX 28
 
Pelt chain(FILE *p){
	Pelt li = malloc(sizeof(elt));
	char *c =malloc(sizeof(char));
	fgets(c,TAILLE_MAX,p);
	elt f=new_elt(c);
	li=new_list();
	li=add(li,f);
	while((*c)!=EOF){
		f=new_elt(c);	
		li=add(li,f);	
		fgets(c,TAILLE_MAX,p);
	}
 
	return li;
}
 
int main(){
	printf("afic 12");
	FILE* fic = NULL;
	printf("afic 13");
	FILE* f = NULL;
	printf("afic 14");
	fic = fopen("test.txt","r");
	printf("afic hjhjhj ");
	assert(fic!=NULL);
	printf("afic ");
	f=read_write(fic);
	printf("afic889 ");
	Pelt g=malloc(sizeof(elt));
	printf("afic0023 ");
	g=chain(f);
	affich_list(g);
	printf("afic good5848484 ");
	fclose(fic);
	return EXIT_SUCCESS;
}

fichier test_read.c

Code:

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
#include <stdlib.h>
#include <stdio.h>
#include <assert.h>
#include <string.h>
#include "test.h"
 
FILE* read_write(FILE* f){
	FILE* f_lign = fopen("text_lign.txt","r+");
	assert(f!=NULL);
	char c=fgetc(f);
	while((c!=EOF)){
		if(((c==' ')||(c=='\n'))){	
			fputs("\n",f_lign);
			//printf("\n");
		} else {
			fputc(c,f_lign);
			//printf("%c",c);
		}
		c=fgetc(f);
	}
	return f_lign;
	fclose(f_lign);
}
 
 
 
 
 
/**int main(){
        FILE* fic = NULL;
        fic = fopen("test.txt","r");
        assert(fic!=NULL);
        printf("afic ");
        read_write(fic);
        fclose(fic);
        return EXIT_SUCCESS;
}**/

fichier test.h

Code:

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
#include <stdlib.h>
#include <stdio.h>
#include <assert.h>
#include <string.h>
#ifndef TEST_H
#define TEST_H
 
 
typedef struct elt *Pelt;
typedef struct elt {
	char* nam;
	Pelt next;
}elt;
 
typedef struct liste{
	Pelt first;
	int size;
}*liste;
 
// fonction defini dans test.c
elt new_elt (char *n);
Pelt new_Pelt(elt e);
Pelt new_list();
Pelt add_vid(Pelt lister, elt te);
Pelt last_elt(Pelt l);
Pelt add_list_elt(Pelt list, elt e);
Pelt add(Pelt list, elt e);
Pelt add_Pelt(Pelt list, Pelt b);
void affich_list(Pelt r);
Pelt prev_elt(Pelt l, elt e);
Pelt suiv_elt(Pelt l, elt e);
Pelt ins_elt(Pelt l, elt e,elt h);
 
FILE* read_write(FILE* f);
//fin fonction defini dans test_read_fich.c
 
Pelt chain(FILE *p);
//fin fonction defini dans full_chain.c
#endif

Citation:

Envoyé par chrislains

voici l'ensemble de mon code

Code:

typedef struct elt *Pelt;

Bonjour

M8oi, quand je vois ça, j'arrête de lire. Cacher une étoile est la dernière des choses à faire. On ne sait plus si les variables sont des pointeurs ou pas. Or, savoir ce qu'on manipule est primordial en C.
Citation:
Envoyé par chrislains
Code:

1 2 3 Pelt chain(FILE *p){ Pelt li = malloc(sizeof(elt)); char *c =malloc(sizeof(char));
Et voilà. "c" est un pointeur (on se demande d'ailleurs pourquoi tu ne l'as pas défini directement comme "char c") et "li" est aussi un pointeur. Sauf que les deux pointeurs (qui sont du même rang) n'ont pas le même nombre (visible) d'étoiles. Comment veux-tu t'en sortir après ça...
Citation:
Envoyé par chrislains
Code:

1 2 3 4 Pelt chain(FILE *p){ char *c =malloc(sizeof(char)); fgets(c,TAILLE_MAX,p);
Ah oui, classe !!! :applo:
Citation:
Envoyé par chrislains
Code:

1 2 3 4 5 6 FILE* read_write(FILE* f){ FILE* f_lign = fopen("text_lign.txt","r+"); ... return f_lign; fclose(f_lign); }
Ouais, pourquoi pas !!! Ce qui me tue le plus, c'est que ça ne te choque même pas au moment où tu l'écris. Mais bon, vu ce que j'ai vu avant...

Autres remarques: ta gestion des erreurs est très mal faite. Tes assertions feront quitter salement ton code si elles ne sont pas respectées. Or, quand quelque chose se passe mal, il faut gérer (nettoyer les ressources allouées, etc) et non pas tout quitter comme un malpropre.
Exemple de gestion
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 <type>fonction(...) { char *xxx=NULL; FILE *yyy=NULL; int *zzz=NULL; errno=0; if ((xxx=malloc(...)) == NULL) goto fin: if ((yyy=fopen(...)) == NULL) goto fin: if ((zzz=malloc(...)) == NULL) goto fin: ... (travail) ... fin: if (xxx) free(xxx); if (yyy) fclose(yyy); if (zzz) free(zzz); return errno == 0 ?valeur_ok :valeur_speciale_indiquant_erreur (souvent NULL) }
Et bon ben si les types standards ont un suffixe "_t" dans leur nom, ce n'est peut-être pas pour rien. Donc toi, ce serait bien de mettre un préfixe "t_" à tes propres types...

19/11/2015, 19h24
fred1599

Citation:

M8oi, quand je vois ça, j'arrête de lire.

Finalement, tu l'as quand même fais, moi j'ai vraiment arrêté, je ne comprend absolument pas la plus value pédagogique d'insister sur cette mauvaise pratique...
19/11/2015, 20h40
chrislains

Citation:

Envoyé par Sve@r

Et bon ben si les types standards ont un suffixe "_t" dans leur nom, ce n'est peut-être pas pour rien. Donc toi, ce serait bien de mettre un préfixe "t_" à tes propres types...

En gros, si je crée un type que j'appelle ens, il faudrait que j’écrive t_ens ?
20/11/2015, 15h03
Sve@r

Citation:

Envoyé par chrislains

En gros, si je crée un type que j'appelle ens, il faudrait que j’écrive t_ens ?

Exactement. Déjà ça prêtera moins à confusion quand on lit le code et ça te laissera le token "ens" libre pour un nom de variable...
20/11/2015, 15h41
chrislains

Citation:

Envoyé par Sve@r

Exactement. Déjà ça prêtera moins à confusion quand on lit le code et ça te laissera le token "ens" libre pour un nom de variable...

Merci Sver@r pour tes réponses
Peux tu m'expliquer tes autres remarques ? Stp car je ne comprends toujours pas
20/11/2015, 18h25
Sve@r

Citation:

Envoyé par chrislains

car je ne comprends toujours pas

Pfff :boulet:
1) Tu fais un malloc de "1" alors que tu vas y stocker 28 caractère lus dans le fichier. D'ailleurs, pourquoi un malloc pour une zone qui n'est traitée que dans la fonction et qui ne ressort nulle part ???
2) Tu quittes la fonction avant de fermer le fichier
20/11/2015, 18h45
Médinoc

Et qui a moinssé Sve@r et pourquoi?
20/11/2015, 21h48
Sve@r

Bah, probablement chrislains quand je l'ai mis face à lui-même et que ce qu'il a vu ne lui a pas plu. Comme il ne pouvait pas se moinsser lui-même, il a moinssé le porteur de mauvaises nouvelles comme le faisaient autrefois les tyrans aux petits pieds...
26/11/2015, 08h57
chrislains

Citation:

Envoyé par Sve@r

Bah, probablement chrislains quand je l'ai mis face à lui-même et que ce qu'il a vu ne lui a pas plu. Comme il ne pouvait pas se moinsser lui-même, il a moinssé le porteur de mauvaises nouvelles comme le faisaient autrefois les tyrans aux petits pieds...

non je ne t'ai pas moinsser, au contraire je suis content d'avoir eu tes remarques, car je ne cherche qu'a apprendre et a m'ameliorer
je suis entrain de corriger mon programme ;)