petit soucis dans le main..

Version imprimable

Voir 40 message(s) de cette discussion en une page

09/12/2005, 16h09
fearyourself
Sans vraiment regarder le code, si tu as une fonction lireligne, je suppose que le premier argument est la chaîne de caractères et le deuxième la taille du tableau (donc le maximum d'éléments qu'on peut mettre dedans...), en principe il faudrait plutôt:
Code:

1 2 3 4 //avec un const, c'est tellement mieux const int taille = 150; char ligne[taille];
ou alors:
Code:

1 2 3 4 5 6 //Meme un define c'est mieux! #define TAILLE 150 .... char ligne[TAILLE]; ...

revoici le code de la fonction lireligne si tu veux plus de précisions sur ce qu'elle fait :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
 
/* fonction */
#include <stdio.h>
 
int lireligne(char s[], int lim)
{
   int c, i;
 
   for ( i=0; i < lim - 1 && (c=getchar()) != EOF && c!='\n'; ++i)
      s[i] = c;
   if ( c == '\n' )
   {
      s[i] = c;
      ++i;
   }
   s[i] = '\0';
   return i;
}

Un bug de logique, tu récupères le '\n' dans lireligne et apres t'esperes encore le récupérer dans ton main...

En supposant que je ne touche pas à lireligne...

Ton main devrait être (pas testé par contre):
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 int main(void) { int l = 0; int c; const int taille = 150; char ligne[taille]; while ((c=getchar() !=EOF)) { l = lireligne(ligne, taille); //on incremente d'un pour récupérer le caractere lu par getchar ++l; //on affiche la taille de cette ligne printf("%d\n", l); } return 0; }
EDITION: remarquez que mon ++l prend en compte le '\n' ce qui n'est pas forcément voulu... Suffirait de l'enlever sinon...

PS: par contre, c'est mal habile et ta fonction lireligne n'est pas utile dans le sens ou elle remplit un tableau qui n'est même pas utilise...

Citation:

Envoyé par Bibouda

j'ai corrigé mais mon programme ne donne pas ce que je veux , il prend des caractères en entrée et me retourne un 0 quand j'appuie sur entrée :

Normal, tu lui dis de prendre 0 caractères.
Citation:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 int lireligne(char ligne[], int taille); int main(void) { int l = 0; int c; char ligne[150]; int taille = 0; while ((c=getchar() !=EOF && lireligne(ligne, taille) > 0 ) { if ( c == '\n' ) ++l; else l = 0; } printf("%d\n", l); return 0; }
Tu penses que de passer une taille de 0 à la fonction, c'est correct ? Quel est le rôle de ce paramètre ? As-tu compris comment fonctionnait lireligne() ? Tout laisse penser que tu as récupéré cette fonction sans savoir à quoi elle sert ni comment on s'en sert..

Autre chose, dans le while(...), à quoi sert le '(c=getchar() !=EOF)' ? C'est pas déjà fait dans lireligne(), ça ?

10/12/2005, 02h11
Bibouda

oui évidemment mais tu m'as dit d'initialiser la variable taille , donc je 'lai mise à 0 , maintenant la fonction lireligne je sais ce qu'elle fait , elle renvoit le nombre de caractère contenus dans un tableau de caractères ( une ligne ) , alors :

le while ( lireligne(ligne, taille) > 0 ) ne veut rien dire si j'ai bien compris?

le taille dans cette fonction c'est la valeur qu'elle retourne , mais je dis bien qu'elle est supérieure à 0 , et c'est la variale l qui compte les caractères , je m'emmèle les pinceaux là , tu me dis que mon code prend 0 caractère mais je comprends pas pourquoi...
10/12/2005, 02h22
fearyourself
Le for dans ta fonction lireligne est:

Code:

for ( i=0; i < lim - 1 && (c=getchar()) != EOF && c!='\n'; ++i)

Quand tu l'appelles tu fais:
Code:

1 2 while ((c=getchar() !=EOF && lireligne(ligne, taille) > 0 )
Or taille = 0 donc finalement ta fonction lireligne ne fait rien...

Jc
10/12/2005, 03h15
Bibouda

donc il faut que ma fonction lireligne dans le main me compte les caractères de toutes les lignes , et si je faisais comme çà qu'en penses tu :

while ( l = lireligne(ligne,taille) > 0)

c'est mieux je crois mais il reste à définir taille , et je ne sais vraiment pas quelle valeur lui donner...
10/12/2005, 03h24
fearyourself

Comme beaucoup de fonctions qui ont un tableau en argument, il faut donner la taille de celui-ci pour eviter les depassements memoires, comme je l'ai deja dit dans un precedent post, tu donnes a la variable taille la taille du tableau, ie le nombre d'elements du tableau...
:wink:

avec ce code :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 int lireligne(char ligne[], int taille); int main(void) { int l = 0; int c; char ligne[taille]; int taille = 150; while ((c=getchar() !=EOF)) { l = lireligne(ligne, taille); ++l; printf("%d\n", l); } return 0; }
j'ai cette erreur à la compilation :

taille undeclared .... unused variable ligne...

et fear , emmanuel a dit dans un post plus haut que le c = getchar() dans le while n'était pas nécessaire vu que lireligne utilise déjà getchar , qu'en penses tu?
merci

12/12/2005, 01h38
Sib
Pour le message concernant la variable "taille" non déclarée, l'erreur est toute bête :
Code:

1 2 int taille = 150; char ligne[taille];
Il faut déclarer "taille" avant de l'appeler pour la déclaration de "ligne" ;)
12/12/2005, 01h57
Emmanuel Delahaye
Citation:
Envoyé par Sib

Pour le message concernant la variable "taille" non déclarée, l'erreur est toute bête :

Code:

1 2 int taille = 150; char ligne[taille];

Il faut déclarer "taille" avant de l'appeler pour la déclaration de "ligne" ;)
Attention, tu mets en oeuvre un concept nouveau du C99 (VLA) qui n'est pas forcément au point sur ton implémentation.
12/12/2005, 02h05
Sib

Effectivement !

Dans ce cas, il vaut mieux utiliser un malloc :)

alors j'ai reccorigé et j'ai tjs un message d'erreur ( je vais jamais y arriver ) :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 int lireligne(char ligne[], int taille); int main(void) { int l = 0; int c; int taille = 150; char ligne[taille]; while ((c=getchar() !=EOF)) { l = lireligne(ligne, taille); ++l; printf("%d\n", l); } return 0; }
warning : ISO C90 forbids variable -size array ligne

PS : interdiction d'utiliser malloc pour moi ou tout autre chose que des tableaux , des boucles , et la fonction lireligne

Utilises donc plutôt:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
 
#define TAILLE 150
int lireligne(char ligne[], int taille);
int main(void)
{
   int l = 0;
   int c;
    char ligne[TAILLE];
 
   while ((c=getchar() !=EOF))
   {
     l =  lireligne(ligne, TAILLE);
     ++l;
    printf("%d\n", l);
   }
return 0;
}

12/12/2005, 19h41
Bibouda

error taille undeclared...
warning unused variable ligne...

pour compiler j'utilise un makefile , qui compile avec ces options :

gcc -W -Wall -ansi -pedantic
12/12/2005, 19h45
fearyourself

:oops: Mon erreur, j'ai corrigé mon code...

Voir 40 message(s) de cette discussion en une page