Problème pour la création d'une fonction

Bonjour,
Débutant en langage C, je dois pour un exercice, créer une fonction qui permet d'entrer des caractères du clavier dans un tableau t, y compris espaces et tabulations.L'entrée des caractères se termine par un retour-chariot. La fonction sera définie comme suit:
lecture (t)
char *t;
{
... à écrire...
}

je vous lache mon code:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21

#include<stdio.h> /* creation de la fonction */ lecture(t) char *t; { for(t=0;t<'\0';t++);/* pour la lecture du tableau */ gets(lecture(t)); } /* programme de test */ main () { char t[100]; lecture(t); printf("%s",t);   sleep(30000); /* Le sleep est present pour garder la fenetre suffisement longtemps a l'ecran pour la lecture du résultat environs 30 secondes.*/ return 0; }

---

Le compilateur (Dev-C++ 4.9.9.2) me signale le warning suivant:
[Warning] passing arg 1 of `gets' makes pointer from integer without a cast

De plus le prog ne tourne pas, j'ai une superbe fenêtre d'erreur application avec le message suivant:
L'exception unknown software exeption (0xc00000fd) s'est produite dans l'application à l'emplacement 0x004012a3.
Cliquer sur ok pour terminer la programme.

Où se trouve mon erreur, sachant que je ne peux pas contacter le prof (cours par correspondance), parce qu'il n'y a pas de permanence professeurale le week-end.

Merci d'avance pour votre aide a tous.

A mon avis, c'est loin d'être le seul warning que tu auras si tu mettais de bonne option de compilation.


Celui que tu as vient du fait que dans lecture, t est de type char*

Et tu fais :

Code:

---

for(t=0;t<'\0';t++);

---

0 est clairement pas de ce type. Donc warning. Je te signales que t est une variable locale, donc après le retour de la fonction, les modifications ne seront pas gardées.

De plus,

Citation:

---

t<'\0'

---

Ne veut pas dire grand chose non plus. Même *t<'0' n'est à mon avis pas que c'est ce que tu souhaites.

Tu fais aussi : get(lecture()) et lecture ne retourne rien.

Moi, quand je compiles (gcc -Wall -W -Wmain), j'ai :

Code:

---

1 2 3 4 5 6 7 8 9 10 11

classique.c:4: warning: return type defaults to 'int' classique.c: In function 'lecture': classique.c:6: warning: ordered comparison of pointer with integer zero classique.c:7: warning: passing argument 1 of 'gets' makes pointer from integer without a cast classique.c:8: warning: control reaches end of non-void function classique.c: At top level: classique.c:11: warning: return type defaults to 'int' classique.c: In function 'main': classique.c:16: warning: implicit declaration of function 'sleep' /home/neoflo/tmp/cc2dS6ZY.o: In function `lecture': classique.c:(.text+0x1c): warning: the `gets' function is dangerous and should not be used.

---

Bon, je suis de bonne humeur, je vais t'expliquer chaque warning :

Code:

---

warning: return type defaults to 'int'

---

tu n'as pas spécifié le type de retour de ta fonction. Il est indiqué que par défaut, ta fonction retournera un int.

Code:

---

classique.c:6: warning: ordered comparison of pointer with integer zero

---

C'est ce que j'avais dit, tu compares un pointeur et un entier.

Code:

---

classique.c:7: warning: passing argument 1 of 'gets' makes pointer from integer without a cas

---

Comme gets prend en entrée un char* et que lecture retourne un int. Il y a une erreur de typage dans les paramètres.

Code:

---

classique.c:8: warning: control reaches end of non-void function

---

Comme le type a été défini comme int, il faut retourner un int.

Code:

---

classique.c:11: warning: return type defaults to 'int'

---

La fonction main a deux types possibles :

int main(void) ou int main(int argc, char* argv[]).

Tu n'as pas indiqué et il le mettra en int par défaut.

Code:

---

classique.c:16: warning: implicit declaration of function 'sleep'

---

Tu as oublié de définir le fichier d'entête qui déclare sleep. Au passage, sleep prend des secondes en entrées, et pas des millisecondes.

Code:

---

warning: the `gets' function is dangerous and should not be used.

---

Ca dit ce que ca dit, il est fortement déconseillé d'utiliser gets. Pourquoi ? Voir ici : http://c.developpez.com/faq/c/?page=...EEN_gets_fgets




---
C'est étrange que tu utilises l'ancienne notation pour la définition des fonctions 8O Pour bien définir les prototypes des fonctions, regarde ici : http://emmanuel-delahaye.developpez....c.htm#fonction ou http://c.developpez.com/faq/c/?page=..._prototype_def. Tu peux regarder tout le site, il est de très bonne qualité :D et il t'aidera bien, je l'espère.

Hello,

Citation:

---

Envoyé par jipé95

Bonjour,
Débutant en langage C, je dois pour un exercice, créer une fonction qui permet d'entrer des caractères du clavier dans un tableau t, y compris espaces et tabulations.L'entrée des caractères se termine par un retour-chariot. La fonction sera définie comme suit:
lecture (t)
char *t;
{
... à écrire...
}

---

Aïe, ça commence mal, c'est de l'écriture K&R du temps des dinosaures. Si c'est dans le cadre d'un cours ou d'une formation, le formateur n'est pas à jour. De plus, pour la tâche proposée, passer la taille du tableau hôte des données est plus qu'indiqué

Code:

---

1 2 3 4 5

int lecture (char * t /*, int taille_t*/) { /* ... à écrire... */ }

---

Citation:

---

Envoyé par jipé95

je vous lache mon code:

Code:

---

1 2 3 4 5 6 7 8 9

#include<stdio.h> /* creation de la fonction */ lecture(t) char *t; { for(t=0;t<'\0';t++);/* pour la lecture du tableau */ gets(lecture(t)); }

---

---

Oublie la fonction gets(), complètement. Interdis-toi de l'utiliser, cette fonction n'a aucun moyen d'offrir un contrôle par rapport à ce que saisit l'utilisateur, ainsi tu peux exploser un tampon de données (tableau par exemple) en fournissant plus de données qu'il ne peut en prendre. Utilise fgets() à la place pour une lecture par ligne (terminée par un retour-chariot, donc toute indiquée pour ta problématique). Tu peux ici utiliser aussi getchar() et fgetc()/getc() si tu veux stocker tes caractères 1 par 1. Sinon, tu fais un appel récursif de la fonction lecture() (qui ne va pas loin heureusement) sans t'en rendre compte et tu passes le retour de lecture (int) à gets() qui attend un char *, c'est je dois dire n'importe quoi.

Code:

---

1 2 3 4

int lecture(char * s, int nbElem) { fgets(s, nbElem, stdin); }

---

[/QUOTE]

Citation:

---

Envoyé par jipé95

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

/* programme de test */ main () { char t[100]; lecture(t); printf("%s",t);   sleep(30000); /* Le sleep est present pour garder la fenetre suffisement longtemps a l'ecran pour la lecture du résultat environs 30 secondes.*/ return 0; }

---

Le compilateur (Dev-C++ 4.9.9.2) me signale le warning suivant:
[Warning] passing arg 1 of `gets' makes pointer from integer without a cast

De plus le prog ne tourne pas, j'ai une superbe fenêtre d'erreur application avec le message suivant:
L'exception unknown software exeption (0xc00000fd) s'est produite dans l'application à l'emplacement 0x004012a3.
Cliquer sur ok pour terminer la programme.

Où se trouve mon erreur, sachant que je ne peux pas contacter le prof (cours par correspondance), parce qu'il n'y a pas de permanence professeurale le week-end.

Merci d'avance pour votre aide a tous.

---

Néanmoins, si tu souhaites une solution sans à avoir à ajouter la taille du tableau en paramètre, utilises par exemple un define. Exple:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16

#include <stdio.h> #define TMAX 100   void lecture(char * s) { fgets(s, TMAX, stdin); }   int main(void) { char t[TMAX]; lecture(t); printf("%s",t); return 0; }

---

Enfin, je te conseille d'acheter le fameux K&R pour apprendre le langage C. Même si il n'est pas forcément très péda, il m'a l'air plus au point que ton prof par correspondance.

A+

Citation:

---

Envoyé par jipé95

De plus le prog ne tourne pas, j'ai une superbe fenêtre d'erreur application avec le message suivant:
L'exception unknown software exeption (0xc00000fd) s'est produite dans l'application à l'emplacement 0x004012a3.

---

Je pense que c'est du au fait que ton code n'a pas la structure " requise "
pour un programme en C.

Tu devrais avoir, par exemple :

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

#include... #include... #define...   void PROCEDURE(int nombre) {   (instructions...)   }     int FONCTION1() { (d'autres instructions...) }   char *FoNcTion( int place ) {   int machin; double bidule;   (encore des instructions...) }   main() {   (toujours des instructions...)   }

---

EN GROS, il te faut toujours au minimum :
* #include <stdio.h> ( inclure une librairie qui contient des instructions
ex : strlen, strcpy, printf....

* main() ( module principal du programme)
ne pas oublier les accolades

Bonjour,
Merci a tous pour vos commentaires...

Je prend note et m'aperçois du même coup que je payes peut être pour rien....
Enfin, je vais perdurer quand même
Tchô:mouarf: :mouarf: :mouarf:

Citation:

---

Envoyé par Jean|Jean[FR]

EN GROS, il te faut toujours au minimum :
* #include <stdio.h> ( inclure une librairie qui contient des instructions
ex : strlen, strcpy, printf....

---

Ce n'est pas une bibliotheque (en francais library ne se traduit pas pas librairie mais par bibliotheque) mais un fichier d'entete. La bibliotheque contenant strlen, strcpy, printf, etc. est la bibliotheque standard du C.
Les deux notions de ficher d'entete et de bibliotheque sont differentes:
* un fichier d'entete contient des constantes, declaration de fonctions, etc. et est utilise lors de la phase de compilation d'un programme.
* une bibliotheque contient le code compile des fonctions et est utilise lors de l'edition de lien.

Citation:

---

Envoyé par Jean|Jean[FR]

* main()

---

int main(void)