petit soucis dans le main..

**Bibouda** · 08/12/2005, 19h13

Bonsoir , j'ai écrit un programme qui m'affiche la longueur de chaque ligne rencontrée en entrée , pour ce ci j'ai 2 fichiers , une fonction lireligne qui affiche la longueur d'une ligne , et un programme main pour l'éxécuter , voici les codes :

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

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
 
int lireligne(char ligne[], int taille);
 
int main(void)
{
   int l = 0;
   int c;
 
   while ((c=getchar() !=EOF && lireligne > 0 )
   {
      if ( c == '\n' )
         ++l;
      else
         l = 0;
   }
   printf("%d\n", l);
}

message d'erreur pour la compilation :

main.c : warning ordered comparaison of pointer with integer zero , control reaches end of non void function .

Est ce que par hasard çà serait parce que j'ai oublié le return 0 ?
merci

**gege2061** · 08/12/2005, 19h16

Envoyé par Bibouda

Est ce que par hasard çà serait parce que j'ai oublié le return 0 ?

Oui

**Médinoc** · 08/12/2005, 19h18

Tu compares une fonction (son adresse pas son résultat) à zéro, et oui, tu as oublié des return 0 par endroits...

**Bibouda** · 08/12/2005, 19h34

alors médinoc comment comparer le résultat qui est dans la fonction , à 0 , au lieu de comparer son adresse à 0 , et au passage pourquoi quand j'écris lireligne > 0 c'est traduit comme adresse de lireligne > 0?
Moi je veux que le résultat retourné par lireligne soit > 0 ...

merci

**Nasky** · 08/12/2005, 20h06

Hum... bizarre que tu ne comprennes pas ça quand même

lireligne c'est l'adresse. Tu peux donc créer un pointeur sur fonction et faire :

Code :

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

1
2
3
 
int (*ptr) (char *, int);
ptr = lireligne ;

Mais toi, tu veux le résultat de la fonction. Donc il faut l'appeler et utiliser les parenthèses et les paramètres de celle-ci... Ca donnerait un truc du style :

Code :

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

1
2
3
4
lireligne(s,n) > 0
 
// s étant un char[]
// n étant un int

C'est basique tout ça...

Nas'

**Bibouda** · 08/12/2005, 20h12

excuse moi j'ai pas encore vu les pointeurs..interdiction d'en utiliser , j'ai corrigé et çà marche pas encore , je comprends pas ( mon livre est très mal fait , le langage C norme ansi de k&r , trop dur pour débuter ) :

donc j'ai remplacé et j'ai tjs erreur de compilation :

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

erreur :

'ligne' undeclared.....
taile undeclared.....

je comprends pas , faut déclarer les arguments ? pourquoi vu que les arguments sont déjà déclarés sur la 1ere ligne :

int lireligne(char ligne[], int taille);

**Nasky** · 08/12/2005, 20h16

Oula... t'as un GROS problème là si tu ne comprends pas ça

Je te conseille de prendre un cours sur Developpez, bien que le livre que tu poosèdes soit très bien (mais il faut relire plusieurs fois les choses pour bien comprendre au début).

La première ligne te donne le prototype c'est à dire le nom de la fonction, le type de retour et le type des arguments (même si on peut y faire figurer le nom).
Mais une fois que tu veux utiliser la fonction ( <=> appeler la fonction), il faut avoir déclaré les arguments à passer, bien évidemment!

Nas'

**Bibouda** · 08/12/2005, 20h34

hé hé , alors j'ai remplacé et j'ai tjs une erreur de compilation :

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
21
22
 
int lireligne(char ligne[], int taille);
 
int main(void)
 
{
 
int l = 0;
int c;
char ligne[];
int taille;
 
while ((c=getchar() !=EOF && lireligne(ligne, taille) > 0 )
{
if ( c == '\n' )
++l;
else
l = 0;
}
 
printf("%d\n", l);
}

erreur de compilation :

error array size missing in ligne .

çà veut dire qu'il faut que je définisse une taille maximale à mon tableau (ligne) je parie , genre je rahoute cette ligne :

#define MAXILGNE 250

et dans la déclaration je mets

char ligne[MAXLIGNE] , non?

**diogene** · 08/12/2005, 21h31

çà veut dire qu'il faut que je définisse une taille maximale à mon tableau (ligne) je parie , genre je rahoute cette ligne :
#define MAXILGNE 250
et dans la déclaration je mets
char ligne[MAXLIGNE] , non?

Oui

excuse moi j'ai pas encore vu les pointeurs..interdiction d'en utiliser ,

Il y a de drôle de prof tout de même! (et ce sont des collègues !!!

)
Toutefois, vu ta manière de gérer les fonctions, il y a des choses qu'il te faut revoir de toute urgence!
Bonne continuation

**Bibouda** · 09/12/2005, 00h06

ben tu peux blamer denis ritchie , le créateur du langage C , car cet exercice est de lui , dans son livre 2eme édition , au passage j'ai laissé un message de le forum algorithmique en rapport total avec cet exercice , si quelqu'un veut jeter un oeil , c'est en rapport avec le C

**Emmanuel Delahaye** · 09/12/2005, 00h06

Envoyé par Bibouda

voici les codes :

Merci de ne pas recopier le code à la main, mais de le copier/coller

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
 
Compiling: main.c
main.c:11: warning: no previous prototype for 'lireligne'
main.c: In function `main_':
main.c:30: warning: ordered comparison of pointer with integer zero
main.c:31: error: syntax error before '{' token
main.c:27: warning: unused variable `l'
main.c: At top level:
main.c:37: error: syntax error before string constant
main.c:37: warning: type defaults to `int' in declaration of `printf'
main.c:37: warning: function declaration isn't a prototype
main.c:37: error: conflicting types for 'printf'
main.c:37: note: a parameter list with an ellipsis can't match an empty parameter name list declaration
main.c:37: error: conflicting types for 'printf'
main.c:37: note: a parameter list with an ellipsis can't match an empty parameter name list declaration
main.c:37: warning: data definition has no type or storage class
Process terminated with status 1 (0 minutes, 0 seconds)

On pert du temps là...

Pourquoi du code aussi compliqué ?

[EDIT]
Oups, j'ai besoin de vacances là... Que des bétises aujourd'hui... J'avais compris "compter les lignes"... Désolé.

**Bibouda** · 09/12/2005, 00h21

alors là je n'ai rien comprs à ton message lol , enfin mon code ne fait pas ce que j'essaye de faire en tout cas , mon code me prend une chaine en entrée et me retourne le chiffre 0 quand j'appuie sur la touche entrée , pourtant l'algo que j'ai en tête m'a l'air juste :

On parcourt la ligne ( qui n'est autre qu'un tableau de caractère ), on compte les caractères avec une fonction déjà programmée . Donc si on a n caractère à la 1ere ligne , j'incrémente la case 0 d'un tableau de n et j'affiche la valeur de la case 0 , si j'ai n caractère à la 2eme ligne , j'incrémente la case 1 n fois ...

Le programme doit donner ceci :

blabla
youp
kikikikiki

6 4 10 ( la ligne 1 a 6 lettres , la 2 a 4 lettres ... )

**Emmanuel Delahaye** · 09/12/2005, 00h31

Envoyé par Bibouda

alors là je n'ai rien comprs à ton message lol

Oui, j'avais mal compris. Oublie ça, désolé.

Bon voilà une version plus simple de ton affaire...

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
21
22
23
24
25
26
27
28
29
30
31
32
33
 
#include <stdio.h>
 
int lireligne(char s[], int lim)
{
   int i = 0;
   int c;
 
   while ((c = getchar()) != EOF && c != '\n')
   {
      if (i < lim - 1)
      {
         s[i] = c;
         ++i;
      }
   }
   s[i] = '\0';
 
   return i;
}
 
int main(void)
{
   int n = 0;
   char s[123];
 
   while ((n = lireligne(s, sizeof s)) != 0)
   {
      printf("%d\n", n);
   }
 
   return 0;
}

Ca compte les longueurs de lignes à la volée...

**diogene** · 09/12/2005, 00h33

blabla
youp
kikikikiki
6 4 10 ( la ligne 1 a 6 lettres , la 2 a 4 lettres ... )

N'est ce pas donc plutot

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
int main()
{
   int l = 0;
   int c;
   while ((c = getchar()) != EOF)
   {
      if (c != '\n')++l;
      else
      {
        printf("%d\n", l);
        l=0;
       }
   }
   return 0;
}

**Bibouda** · 09/12/2005, 00h43

non car je dois utiliser la fonction lireligne , dont j'ai mis le code dans le 1er post , juste au dessus

**Bibouda** · 09/12/2005, 00h53

emmanuel , interdiction d'utiliser sizeof en ce qui me concerne

**diogene** · 09/12/2005, 00h54

non car je dois utiliser la fonction lireligne , dont j'ai mis le code dans le 1er post , juste au dessus

Fais un effort tout de même!!!!!

On veut bien aider pas faire ton job. Le code analyse à ta convenance une chaîne de caractères lue au clavier. Tu dois pouvoir l'adapter pour une chaîne placée dans un tableau par ta fonction lireligne ou au minimum tu peux essayer de le faire. Si tu n'y arrives pas, alors on t'aideras.
Bonne nuit

**Bibouda** · 09/12/2005, 01h06

pourquoi ce code que j'ai fait ne va pas alors :

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;
 
   while ((c=getchar() !=EOF && lireligne(ligne, taille) > 0 )
   {
      if ( c == '\n' )
         ++l;
      else
         l = 0;
   }
   printf("%d\n", l);
return 0;
}

**Emmanuel Delahaye** · 09/12/2005, 09h15

Envoyé par Bibouda

pourquoi ce code que j'ai fait ne va pas alors :

Code :

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

1
2
3
4
5
 
char ligne[150];
int taille;
<...>
   while ((c=getchar() !=EOF && lireligne(ligne, taille) > 0 )

La variable 'taille' n'est pas initialisée. Passer une valeur indéfinie à une fonction invoque un comportement indéfini. Autrement dit, c'est un bug.

**Bibouda** · 09/12/2005, 15h56

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 :

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

pourtant quand je lis le programme en français çà me donne :

tant que c prend un caractère différent de EOF , et que le nombre de caractères est supérieur à 0 , et si le caractère est un passage à la ligne , alors on incrémente l ( qui est la longueur d'une ligne ) , sinon on dit que l = 0 , et ensuite on affiche la valeur de la ligne , alors qu'est ce qui va pas?

Mon idée : je pense que mon programme ne prend en compte que une ligne ( donc j'ai mal fait ma boucle ou sa structure j'en sais rien ) .

petit soucis dans le main..

C

Vue hybride

Discussions similaires

Partager

Partager