Discussion :

[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 :

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

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

ou alors:

Code :

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

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

[Bibouda]

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

Code :

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

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;
}

[fearyourself]

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 :

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

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...

[Emmanuel Delahaye]

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.

Code :

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

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 ?

[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...

[fearyourself]

Le for dans ta fonction lireligne est:

Code :

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

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

Quand tu l'appelles tu fais:

Code :

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

1
2
 
 while ((c=getchar() !=EOF && lireligne(ligne, taille) > 0 )

Or taille = 0 donc finalement ta fonction lireligne ne fait rien...

Jc

[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...

[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...

[Bibouda]

avec ce code :

Code :

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

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

[Sib]

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

Code :

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

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

Il faut déclarer "taille" avant de l'appeler pour la déclaration de "ligne"

[Emmanuel Delahaye]

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

Code :

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

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.

[Sib]

Effectivement !

Dans ce cas, il vaut mieux utiliser un malloc

[Bibouda]

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

Code :

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

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

[fearyourself]

Utilises donc plutôt:

Code :

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

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;
}

[Bibouda]

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

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

gcc -W -Wall -ansi -pedantic

[fearyourself]

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