demande petite aide pour newbi du C :)

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : demande petite aide pour newbi du C :)

nDeXok

Bonjour je suis en formation de programmeur micro. Et comme je débute, j'ai un peu du mal.

Voici l'énoncé de mon exercice :

Citation :

Ecrire une fonction qui permet d'entrer des caractères du clavier dans un tableau t, y compris espaces et tabulations, ce que ne peut faire l'instruction scanf("%s",t);. L'entrée des caractères se termine par un retour-chariot. La fonction sera définie comme suit :

Code :

lecture(t)
char *t;
{
...à écrire...
}

On prendra le programme principal suivant pour tester la fonction.

Code :

#include <stdio.h>
main()
{ char t[100];
lecture (t);
printf("%s",t);
}

Voici le code que j'ai commencé :

#include <stdio.h>

Code :

int main(void) {
char t[100];
lecture(t);
printf("%s\n",t);
printf("\n" );
return 0;
}
void lecture(t) {
char *t;
printf("Entrez des caractères : \r" );
gets(*t);
}

je sais qu'il est bourré de conneries. Quelqu'un peut t'il m'aider ou me filer le code pour que je puisse comprendre avec mon cours svp.
Un grand merci d'avance.

Publicité

Elmoricq

Modérateur

1. Ne jamais, jamais, jamais utiliser gets(). C'est un bug ambulant cette fonction, et elle est dépréciée depuis quelques années (elle n'est encore là que pour des raisons de compatibilité). Utiliser fgets() à la place.

2. Définition de la fonction lecture() fausse, il faut donner un type à l'argument "t" qu'elle reçoit. Ne pas redéfinir la variable t dans la fonction non plus, relis l'énoncé, tu as fait une erreur. Enoncé qui d'ailleurs est obsolète, car on ne définit plus les fonctions comme ça depuis longtemps:

void lecture(char *t) {
...
}

Relis ton cours, recherche des tutoriaux, et réessaie.

PS : on ne donne pas de programme tout fait sur ce forum, encore moins de réponses à des exercices, juste de l'aide, des exemples, ou des morceaux précis.

EDIT : plein d'edit, a y est c'est fini, désolé j'avais mal lu le sujet. [:elmoricq]

Message édité par Elmoricq le 05-09-2005 à 15:07:25

Emmanuel Delahaye

C is a sharp tool

nDeXok a écrit :

je suis en formation de programmeur micro.
Voici l'énoncé de mon exercice :

Citation :

Code :

lecture(t)
char *t;
{
...à écrire...
}

Une formation qui propose d'écrire une fonction de saisie aussi mal prototypée que gets() [1] n'est pas sérieuse. Change d'urgence d'organisme et fait toi rembourser.

Donne le nom de l'organisme, qu'on le mette en liste noire.

----------------------
[1] et en plus dans un style obsolète depuis 1989

Message édité par Emmanuel Delahaye le 05-09-2005 à 15:43:24

---------------
Des infos sur la programmation et le langage C: http://www.bien-programmer.fr Pas de Wi-Fi à la maison : http://www.cpl-france.org/

nDeXok

Je fais ça chez Educatel... vous avez un autre organisme à me conseiller en formations à distance ?

Message édité par nDeXok le 05-09-2005 à 16:00:32

Emmanuel Delahaye

C is a sharp tool

nDeXok a écrit :

Educatel

Eh beh !

---------------
Des infos sur la programmation et le langage C: http://www.bien-programmer.fr Pas de Wi-Fi à la maison : http://www.cpl-france.org/

Sve@r

Emmanuel Delahaye a écrit :

Une formation qui propose d'écrire une fonction de saisie aussi mal prototypée que gets() [1] n'est pas sérieuse. Change d'urgence d'organisme et fait toi rembourser.
[1] et en plus dans un style obsolète depuis 1989

C'est assez facile comme conseil. En général, il est difficile de se faire rembourser un truc commencé. Et va donc prouver que cette formation est nulle. Le moins qu'on puisse faire c'est d'essayer de limiter les dégats

nDeXok a écrit :

Voici le code que j'ai commencé :

Code :

#include <stdio.h>
int main(void) {
char t[100];
lecture(t);
printf("%s\n",t);
printf("\n" );
return 0;
}
void lecture(t) {
char *t;
printf("Entrez des caractères : \r" );
gets(*t);
}

Voici les soucis et erreurs (déjà dit par Elmoricq)
1) on ne redéfinit pas le paramètre comme si c'était une nouvelle variable. Il faut seulement le déclarer entre le nom de la fonction et l'accolade ouvrante (relis ton cours sur la syntaxe des fonctions)

Code :

void lecture(t)
char *t;
{
...
}

2) ce genre de syntaxe (on liste les paramètres puis on les déclare) est périmé depuis qu'on peut faire les deux à la fois

Code :

void lecture(char *t)
{
...
}

3) la fonction "gets" doit recevoir un "char *". Si tu lui passes "*t", tu lui passes un "char". Ce n'est pas ce qu'elle attend.
Quand tu écris "char *t", tu définis deux item
a) tu définis l'item "etoile t" de type "char"
b) tu définis l'item "t" de type "char étoile"
Les deux item ont leur utilité et sont utilisables (si on le fait correctement)
Donc si la fonction "gets" attend un "char étoile", passe lui un "char étoile" donc passe-lui simplement "t". C'est d'ailleurs ce que tu fais quand tu appelles "lecture" qui attend un "char étoile", tu lui passes "t". Pourquoi changer avec "gets" qui attend la même chose ???

Une fois que tu auras corrigé cela, tu auras répondu à l'énoncé de l'exercice. Le problème c'est que, en utilisant "gets", tu ne limites pas la saisie. Et ainsi ta fonction "lecture" pourra tenter d'engranger plus de caractères que ce que ta zone "t" peut en stocker (et le C ne vérifie pas ça).
Il vaut donc mieux ne pas (ne jamais) utiliser "gets" mais "fgets" qui permet de donner une limite à la saisie.

Message édité par Sve@r le 06-09-2005 à 11:10:50

nDeXok

merci bien

chrisbk

c'etait quoi l'idée geniale presidant a la creation de la vieille formulation ? (la void truc(t) char *t { ..} )

Message édité par chrisbk le 06-09-2005 à 13:58:56

moi23372

nDeXok a écrit :

merci bien

moi je dirais que FGETS ben c'est pas encore génial non plus!

faut utilisé GETCHAR()
une petite boucle et c'est parti

Code :

void lecture (char *t, int longueurmax)
{
int i=0;
do
{
t = getchar();
i++;
t++;
} while (i<longueurmax && *(t-1) != '\\n');
*(t-1) = '\\0';
}

Voila le programme est correcte mtn! ;-) quelques petite fautes de frappe vite corrigées

Message édité par moi23372 le 07-09-2005 à 13:36:29

Sve@r

moi23372 a écrit :

moi je dirais que FGETS ben c'est pas encore génial non plus!

faut utilisé GETCHAR()
une petite boucle et c'est parti

Code :

void lecture (char *t, int longueurmax)
{
int i=0;
do
{
t = getchar();
i++;
t++;
} while (i<longueurmax && *(t-1) != '\n');
*(t-1) = '\0';
}

N'oublie quand-même pas de rajouter le '\0' final, sinon ce n'est plus une chaîne !!!

Message édité par Sve@r le 07-09-2005 à 13:36:44

Publicité

chrisbk

*t = getchar();

bande de solicoque palmipede

Elmoricq

Modérateur

moi23372 a écrit :

moi je dirais que FGETS ben c'est pas encore génial non plus!

Pour quelle raison ?

chrisbk

pis il est bugueux ce code, nom d'un sexe d'autriche

Elmoricq

Modérateur

chrisbk a écrit :

pis il est bugueux ce code, nom d'un sexe d'autriche

Ah oui, je m'étais arrêté au "faut pas utiliser fgets()" moi [:mlc]

C'est vraiment la peine de s'embêter avec un getchar() si c'est pour écrire ce genre de chose. [:pingouino]

moi23372

chacun fait ce qu'il faut! Je ne suis pas certain que FGETS est standars sur tout les compilateurs! Or getchar est standars sur tout les compilo, donc portable!
Mon code n'est pas infesté de bug, même si j'ai oublié une étoile! Elle fait tout ce qu'il faut pour faire une saisie clavier tout simplement!

chrisbk

non, y plante, ton code

Emmanuel Delahaye

C is a sharp tool

moi23372 a écrit :

faut utilisé GETCHAR()
une petite boucle et c'est parti
Voila le programme est correcte mtn! ;-) quelques petite fautes de frappe vite corrigées

Correct ? Non. (Probablement jamais compilé, donc jamais testé...)

Ceci fonctione :
[EDIT] : Nouvelle version sécurisée.

Code :

#include <stdio.h>
#include <string.h>
char *get_line (char *t, size_t longueurmax)
{
char *p = NULL;
if (t != NULL && longueurmax > 0)
{
size_t i = 0;
int c = 0;
while (i < longueurmax - 1
&& (c = getchar ()) != '\n'
&& c != EOF)
{
t[i] = c;
i++;
}
t[i] = '\0';
/* purge ?*/
if (c != '\n')
{
while ((c = getchar ()) != '\n'
&& c != EOF)
{
}
}
if (c != EOF)
{
p = t;
}
}
return p;
}
int main (void)
{
char s[10] = "??????????";
get_line (s, sizeof s);
printf ("'%s' (%u)\n", s, (unsigned) strlen (s));
return 0;
}

Hello world
'Hello wor' (9)

[EDIT] Ouin: Pourquoi quand je tape '\n', ça affiche 'n' dans [fixed ] et [cpp ]? C'est nouveau ? Et '\0', ça fait un carré. C'est quoi ce bins!

Message édité par Emmanuel Delahaye le 07-09-2005 à 15:20:50

---------------
Des infos sur la programmation et le langage C: http://www.bien-programmer.fr Pas de Wi-Fi à la maison : http://www.cpl-france.org/

Elmoricq

Modérateur

moi23372 a écrit :

chacun fait ce qu'il faut! Je ne suis pas certain que FGETS est standars sur tout les compilateurs! Or getchar est standars sur tout les compilo, donc portable!

Euh faut arrêter, fgets() c'est du C89, ça fait plus de quinze ans que c'est standard.

moi23372 a écrit :

Mon code n'est pas infesté de bug, même si j'ai oublié une étoile! Elle fait tout ce qu'il faut pour faire une saisie clavier tout simplement!

C'est déjà un bon gros bug.

EDIT : Ah ouais zarb le coup du '\n' dans le [ fixed], je pensais que c'était une erreur de moi686554.

Message édité par Elmoricq le 07-09-2005 à 14:27:26

chrisbk

nan par contre je crois qu'il compte mal ses caracteres [:zaib3k]

moi23372

j'aime bien veux bien un gros bug! mais je ne l'ai pas testé, j'ai fais ça de mémoire en 30 secondes!
Mon code marche, je viens de le tester

Code :

/**********************************************************************************/
/*INPUT: adresse du vecteur + le nombre d'élément maximum. */
/*PROCESS: lecture d'une chaine de caracteres */
/*OUTPUT: retour du nombre d'éléments */
/**********************************************************************************/
int LireVchar (char *p, int n)
{
int i=0;
do
{
*p=getchar();
i++;
p++;
} while (i<n && *(p-1) != '\\n');
*(p-1) = '\\0';
return(i-1);
}

Message édité par moi23372 le 07-09-2005 à 14:33:11

Elmoricq

Modérateur

moi23372 a écrit :

j'aime bien veux bien un gros bug! mais je ne l'ai pas testé, j'ai fais ça de mémoire en 30 secondes!
Mon code marche, je viens de le tester

Code :

Euuh :

char *lecture(char *t, size_t maxlength)
{
if ( t )
fgets(t, maxlength, stdin);
return t;
}

Et puis ça ne mérite même pas une fonction ce truc. [:petrus75]

Message édité par Elmoricq le 07-09-2005 à 14:41:36

chrisbk

char toto[2]

LireVchar (toto, 2);

je sens le segfault

chrisbk

Ah nan j'avais pas vu la feinte, rien dit

Elmoricq

Modérateur

chrisbk a écrit :

Ah nan j'avais pas vu la feinte, rien dit

Ouais enfin ceci dit, suffit de faire : "LireVchar(NULL, -1);"

chrisbk

[:ddr555]

moi23372

oui ça va! quand on veut faire planter un programme il y a tjs moyen! Naturellement faut savoir programmer aussi, passer null c'est une erreur de débutant!
Et puis merde, c'est du C, les fonctions ne sont pas destinées ici à être réutillisé comme les classes! Vive la POO

Elmoricq

Modérateur

moi23372 a écrit :

Et puis merde, c'est du C, les fonctions ne sont pas destinées ici à être réutillisé comme les classes! Vive la POO

Ben et les shared-lib, elles servent à quoi alors ? [:itm]

chrisbk

ne pas vérifier NULL c'est une erreur de jeanjean [:vague nocturne]

bin oui c'est du C, un langage de merde par excellence, tout a fait, enfin meme en C++ faut vérifier les NULL

moi23372

oui mais la POO dispose des exceptions! Il y a tjs moyen d'intercepter ce genre d'erreur!
Puis il aurait ici suffit simplement de faire

if (t == null || n <= 0)
return -1;

Message édité par moi23372 le 07-09-2005 à 14:46:41

Emmanuel Delahaye

C is a sharp tool

moi23372 a écrit :

j'aime bien veux bien un gros bug! mais je ne l'ai pas testé, j'ai fais ça de mémoire en 30 secondes!
Mon code marche, je viens de le tester

C'est OK, mais ces '-1', c'est un peu compliqué . Et puis comme tu as un index (i), autant utiliser directement la notation [i] et laisser le pointeur tranquille (il n'est jamais bon de modifier un paramètre).

D'autre part, si on passe NULL, le comportement est indéfini.

---------------
Des infos sur la programmation et le langage C: http://www.bien-programmer.fr Pas de Wi-Fi à la maison : http://www.cpl-france.org/

Emmanuel Delahaye

C is a sharp tool

Elmoricq a écrit :

char *lecture(char *t, size_t maxlength)
{
if ( t )
fgets(t, maxlength, stdin);
return t;
}

Pas pareil (tu laisses le '\n');

Nota dans les deux cas, stdin n'est pas purgé en cas de débordement. En plus, on a aucun moyen de le savoir.

J'ai integré une purge à mon code. La fonction s'appelle get_line() et elle est libre ! Si on ne veut pas de la purge automatique, on prends fgets()...

Message édité par Emmanuel Delahaye le 07-09-2005 à 15:07:42

---------------
Des infos sur la programmation et le langage C: http://www.bien-programmer.fr Pas de Wi-Fi à la maison : http://www.cpl-france.org/

Publicité

FORUM HardWare.fr

Programmation

demande petite aide pour newbi du C :)

Sujets relatifs
[ASM]besoin d'aide	A l'aide sur VBA
Aide pour un débutant	Demande d'aide VBA : tableau dynamique et importation de données
Menu déroulant en css (à l'aide je bloque!!!!!!)	Besoin d'aide pour feuille de style.css
Aide pour MSMQ	[wxWidgets] \|\| Aide au survol
créer menu fixe + aide
Plus de sujets relatifs à : demande petite aide pour newbi du C :)

Page générée en 0.114 secondes