problème de pointeur

**doudoustephane** · 06/05/2006, 07h30

Envoyé par BnY

oki c super ca marche

c'est deja ca, cool

Petite modif :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
 
void afficher_fichier(FILE *flot)
{
 char *chaine=NULL;
 
 while(fgets(chaine,sizeof(chaine),flot)!=NULL)
 {
  printf("\n%s",chaine);
 }
}

espace en plus dans ...ier(FILE ...
et paranthese, tu test, j'attends...

**BnY** · 06/05/2006, 07h34

toujours ce bon vieux D.A.Failed...

**doudoustephane** · 06/05/2006, 07h39

Envoyé par BnY

toujours ce bon vieux D.A.Failed...

essaye d'allouer ta variable chaine avant le while dans le genre :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
char *chaine =  (char *)malloc(taille*sizeof(char)+1);

avec taille valant un entier...

sinon, a quel endroit de ta fonction il plante exactement???, sur quelle ligne???

**BnY** · 06/05/2006, 07h48

il plantait juste a ce moment la ( un pti printf mindiquai que le flot etait non NULL et des lappel de la fonction afficher

AF... )mais la niquel plus de DAF lool ^^ bon a part un pti problm d'affichage vu qu'il saute des lignes tout seul au lieu d'afficher correctement les lignes du fichier...lool

je vais regarder ca de plus pres pour savoir pourquoi il fait ca

a moins que tu ais une idée tout de suite lol mais bon en tout cas niquel merci encore beaucoup

**BnY** · 06/05/2006, 07h57

lol c bon resolu c moi ki mettait un \n de trop par reflexe dan mon printf

bon ben voila c super tu viens de me sauver la vie :p lool

**doudoustephane** · 06/05/2006, 07h58

Envoyé par BnY

il plantait juste a ce moment la ( un pti printf mindiquai que le flot etait non NULL et des lappel de la fonction afficher

AF... )mais la niquel plus de DAF lool ^^ bon a part un pti problm d'affichage vu qu'il saute des lignes tout seul au lieu d'afficher correctement les lignes du fichier...lool

je vais regarder ca de plus pres pour savoir pourquoi il fait ca

a moins que tu ais une idée tout de suite lol mais bon en tout cas niquel merci encore beaucoup

mais de rien, je viens juste de passer une nuit blanche de plus...

t'inquiete pas va...

pour les saut de lignes, cela vient de la lecture du fichier proprement dis. fgets parfois lit et parfois ne lit pas le caractere '\n' (=retour a la ligne). donc, je pense que ca vient de cela mais la, je ne peux plus rien pour toi. au mieu, il faut que tu regarde si ta chaine contient le caractere ou pas et le supprimer dans le cas possitif... (tu as cela ici, dans la

C)...

j'y vais mais tiens moi au courant, je revois ca dans la journee...

**doudoustephane** · 06/05/2006, 08h02

Envoyé par BnY

lol c bon resolu c moi ki mettait un \n de trop par reflexe dan mon printf

bon ben voila c super tu viens de me sauver la vie :p lool

ok, tres bien, mais regarde quand meme le message que je viens de poster juste avant, je pense que tu devrais l'inclure (le traitement du caractere '\n')...

content d'avoir pu t'aider...

pense a mettre le tag

, c'est dans Outils de la discussion, en haut de la page, juste au dessus du premier message...

**BnY** · 06/05/2006, 08h20

en fait pour le \n j'ai une tite fonction qui est forte utile dans le cas ou j'utilise fgets sur stdin pour le \n saisie lorsqu'on tape "entrée"... mais la dans ce cas precis cette fonction ne m'interesse pas puisque je veux afficher le fichier donc autant garder ce \n a chaque fin de ligne pour que l'affichage des lignes se fasse automatiquement

non ??

ca fait plaisir maintenant des que je vois du DAF hop une ptite allocation de memoire sur ma chaine et c fini lool ...
ce qui pour moi etait un fleau toute la nuit est une tite routine maintenant loool

merci encore

et bonne journée ( enfin bonne soirée si tu lis ce message en rentrant lool )

**Franck.H** · 06/05/2006, 08h55

C'était juste en passant, tu peux aussi écrire ton malloc de cette manière:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

char * chaine = malloc (taille + 1);

Le type char vallant 1 octets de toutes façons, inutile de la préciser ici, d'ailleur les cast en C ne sont pas obligatoire et pas forcement necessaire dans ce cas de figure !

**Emmanuel Delahaye** · 06/05/2006, 12h34

Envoyé par BnY

par contre maintenant j'ai un message d'erreur :

debug assertion failed
file fgets.c
line 59
string!=NULL

C'est la bibliothèque du C qui se protège contre les fausses manips de l'utilisateur... C'est un peu brutal comme réaction, mais pourquoi pas. Tu as passé une adresse valant NULL à la fonction fgets() quelque part dans ton code. C'est évidemment une erreur à corriger.

**Emmanuel Delahaye** · 06/05/2006, 12h39

Envoyé par crocodilex

C'est parceque tu fais appel à la fonction assert() dans le fichier fgets.c

A l'évidence fgets() appartient à la bibliothèque, pas à l'application... Il n'est pas responsable de la présence du assert(). Par contre son erreur est d'avoir appelé un fgets() avec NULL...

Pour supprimer ce message il faut définir la constante symbolique NDEBUG, soit avec un #define ou bien avec l'option de compilation -D (si j'ai bonne mémoire)

Ben oui. Quand la fièvre est trop élevée, on casse le thermomètre et le patient va tout de suite mieux, c'est bien connu.

Quand un voyant d'alarme s'allume sur le poste de contrôle de la centrale de Chernobyl, on débranche le voyant... Problème réglé.

C'est vraiment une drôle de façon de raisonner...

**Emmanuel Delahaye** · 06/05/2006, 12h40

Envoyé par BnY

oki mais supprimer ce message évite-il aussi l'erreur ??

Evidemment non. Il y a un bug dons ton code. C'est tout. A toi de le corriger.

**Emmanuel Delahaye** · 06/05/2006, 12h50

Envoyé par BnY

par exemple la quand je fais le code suivant:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
 
 FILE *fpaero;
 
 fpaero=fopen("FicCaracteristiquesAeroport.txt","r");
 
 if(fpaero==NULL)
 {
  printf("nn");
 }

STOP! En cas d'erreur, il ne faut pas continuer, sinon, tu passes un NULL à fgets(), ce qui explique l'arrêt du programme par le assert() de fgets.c.

Le principe est le suivant (pseudo-code) :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
 
FILE *fp = fopen(...)
if (fp != NULL)
{
   /* OK, on continue */
 
   fclose (fp)
}
else
{
   /* Probleme avec le fichier (inconnu, inaccessible...) */
}

#define NDEBUG
il ne se passe rien de plus.. tu pourrais m'éclaircir stp

merci

Normal. Il faudrait recompiler la bibliothèque. Mais je ne te le conseille pas.

Le bug qui te concerne, c'est que fopen() retourne NULL. Il faut en trouver la cause :

le fichier n'est pas dans le répertoire d'exécution
le fichier est dans le répertoire d'exécution, mais sous un faux nom (un nom plus court genre "AirportData.txt" est plus facile à manipuler...)
le fichier est dans le répertoire d'exécution, mais il est déjà ouvert par une autre application
etc.

Quand à 'eclaircir', je pense que tu voulais dire 'éclairer'...

**Emmanuel Delahaye** · 06/05/2006, 12h55

Envoyé par BnY

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
if((flot=fopen("C:\Program Files\Microsoft Visual Studio\MyProjects\testlogincorrigé\tlogcorr\FicCaracteristiquesAvion.txt","r+"))==NULL)

Le caractère \ se code \\ en C. Sinon, Windows accepte aussi le séparateur /

**Emmanuel Delahaye** · 06/05/2006, 12h57

Envoyé par doudouallemand

rajoute apres ton "r+" un b ou un t en fonction de si ton fichier est binaire "r+b" ou ASCII (=texte) "r+t"

Ce "t" n'existe pas. C'est "r+".

**Emmanuel Delahaye** · 06/05/2006, 13h02

Envoyé par BnY

bon par contre je crois ke le debug assertion failed m'en veu puisqu'il reapparait juste apres au moment de la fonction afficher_fichier() que voici:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
 
 char *chaine=NULL;
 
 while(fgets(chaine,sizeof chaine,flot)!=NULL)

Ben c'est sur que si tu passes un NULL à fgets(), tu vas retomber sur le problème... Tu fais un peu n'importe quoi, non ? Tu t'imagines quoi en passant NULL ?

C'est pas très dur de lire la doc :

http://dpobel.free.fr/man/html/affic...969/man/fgets/

Il faut fournir l'adresse d'un tableau de char. Quand au sizeof, il n'a de sens que sur un tableau (pas sur un pointeur).

**Emmanuel Delahaye** · 06/05/2006, 13h06

Envoyé par BnY

ca fait plaisir maintenant des que je vois du DAF hop une ptite allocation de memoire sur ma chaine et c fini lool ...
ce qui pour moi etait un fleau toute la nuit est une tite routine maintenant loool

Et n'oublie pas de libérer ce qui a été alloué. Mais as-tu vraient besoin d'un tableau dynamique ? Un simple tableau de taille fixe ne suffit pas ?

**crocodilex** · 06/05/2006, 13h27

Envoyé par Emmanuel Delahaye

A l'évidence fgets() appartient à la bibliothèque, pas à l'application... Il n'est pas responsable de la présence du assert(). Par contre son erreur est d'avi=oir appelé un fgets() avec NULL...

Ben oui. Quand la fièvre est trop élevée, on casse le thermomètre et le patient va tout de suite mieux, c'est bien connu.

Quand un voyant d'alarme s'allume sur le poste de contrôle de la centrale de Chernobyl, on débranche le voyant... Problème réglé.

C'est vraiment une drôle de façon de raisonner...

Désolé, mais je n'avais ma boule de crystal pour savoir que le fichier fgets.c n'appartenait pas à notre ami BnY. Tu vas me dire que j'aurais pu le deviner. J'y avais pensé. Mais rien ne l'interdit de crée un fichier qui porte le même nom qu'une fonction de la lib C.
D'autre part, la macro assert() est très utile en phase de debug. Ca serait dommage de ne pas l'utiliser. Lorsqu'on estime que le logiciel est suffisement fiable, à ce moment là on peut se permettre de neutraliser cette macro.
Donc ce que je trouve bizarre c'est de laisser des infos de debug dans un produit censé debuggé et livré. Ca ne fait pas très pro.

**doudoustephane** · 06/05/2006, 17h14

Envoyé par CSoldier

Le type char vallant 1 octets de toutes façons, inutile de la préciser ici, d'ailleur les cast en C ne sont pas obligatoire et pas forcement necessaire dans ce cas de figure !

oui, mais le fait de le preciser ne rent-il pas le code plus portable au travers des differentes versions de windows notamment (avec l'arrive de windows vista et ces changements de taille de variable)???

sinon, merci pour l'info de fin BnY.

sinon, pour info Emmanuel Delahaye, c'est le probleme qu'on a regle durant la nuit et qui est explique plus haut dans les posts

**Emmanuel Delahaye** · 06/05/2006, 17h18

Envoyé par doudouallemand

oui, mais le fait de le preciser ne rent-il pas le code plus portable au travers des differentes versions de windows notamment (avec l'arrive de windows vista et ces changements de taille de variable)???

Non. La taille d'un char est 1 par définition.

C'est l'unité de compte. Une unité vaut 1. Point. C'est la définition.

problème de pointeur

C

Discussions similaires

Partager

Partager