Discussion :

[stallaf]

Bonjour à tous,
S'exerçant sur les pointeurs, je dois écrire une fonction permettant d'entrer des caractères du clavier dans un tableau t, y compris espaces et tabulations (donc pas de scanf). L'entrée des caractères doit se terminer par un retour-chariot. La fonction doit être définie comme suit :

Code :

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

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

et comme test :

Code :

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

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

C'est l'enfer, j'arrive à rien (vive fgets : ). Pourrais-je être mis sur la voie...(soyons clair, je ne demande pas la solution).
Merci d'avance.

[Médinoc]

Code archaïque :

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

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

!!
Il date de quand, ton exercice?
Personne ne déclare plus le prototype d'une fonction de cette façon, et je ne suis même pas sûr que ce soit encore autorisé par le dernier standard...

[Médinoc]

De plus, avec un prototype pareil, la fonction qu'on te demande d'écrire est un bug au même titre que gets(), puisqu'elle ne permet pas de spécifier la taille de buffer...

[Melem]

Bonjour à tous,
S'exerçant sur les pointeurs, je dois écrire une fonction permettant d'entrer des caractères du clavier dans un tableau t, y compris espaces et tabulations (donc pas de scanf). L'entrée des caractères doit se terminer par un retour-chariot. La fonction doit être définie comme suit :

Code :

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

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

et comme test :

Code :

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

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

C'est l'enfer, j'arrive à rien (vive fgets : ). Pourrais-je être mis sur la voie...(soyons clair, je ne demande pas la solution).
Merci d'avance.

Alors là, je ne puis que te conseiller un tutoriel de langage C : Initiation au langage C.

je dois écrire une fonction permettant d'entrer des caractères du clavier dans un tableau t, y compris espaces et tabulations (donc pas de scanf)

scanf sait lire un caractère (format %c).

[stallaf]

Pourriez-vous quand même me donner une piste pour l'écriture d'une telle fonction ... conforme aux standards actuels ?

[Emmanuel Delahaye]

La fonction doit être définie comme suit :

Code :

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

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

Cette façon de définir les fonctions est obsolète depuis la normalisation du C en 1989. Si ton prof ne le sais pas, change de prof... De plus, il faut passer un paramètre de taille pour le tableau de destination :

Code :

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

1
2
3
4
int lecture (char* t, size_t size)
{ 
   /* ... a ecrire ... */
}

ensuite, une simple boucle avec getchar() devrait suffire...

[Melem]

Pourriez-vous quand même me donner une piste pour l'écriture d'une telle fonction ... conforme aux standards actuels ?

Si t'avais suivi le lien que je t'ai donné, tu n'aurais même pas posé cette question.

[stallaf]

Si t'avais suivi le lien que je t'ai donné, tu n'aurais même pas posé cette question.

Melem, je suis resté trop longtemps sur la page d'édition de mon message et je n'avais donc pas pris connaissance de ce message. Depuis, j'ai amélioré mes connaissances (je pense) grâce à ce lien.
Donc voici le code que j'ai écrit et qui fonctionne. J'ai cependant une question inscrite en commentaires.

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
#include<stdio.h>
 
char *lecture (char *t, size_t taille)
{
    size_t i = 0;
    char *p = t;
 
    printf("Taper une phrase de 30 caratèeres maximum :\n");
    while ((i  < taille - 1) && (*p != '\n'))
    {
        t[i] = *p;
        i++; // Pourquoi p++ ne fonctionne pas ?
        *p = getchar();
    }
    t[i] = '\0';
 
    return t;
}
 
int main(void)
{
    char t[31];
 
    lecture(t, sizeof(t));
    printf("La chaine vaut: %s\n", t);
 
    return 0;
}

Merci.

[Melem]

Attention, ta fonction de lecture est mal écrite. Essaie de faire une version plus simple.

Code :

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

1
2
3
4
5
printf("Taper une phrase de 30 caratèeres maximum :\n");
while ((i  < taille - 1) && (*p != '\n'))
{
    ...
}

Tu fais un test sur *p (p pointant initialement sur t[0]). Imagine un peu que t[0] contienne '\n' (ou que ta fonction rencontre un caractère '\n' dans t alors que la boucle doit encore continuer !) ! On lit un caractère et on fait un test sur ce caractère (il faut ignorer ce qu'il y a dans le tableau), s'il est bon et qu'on a encore assez de place, on le copie dans le tableau et on lit un autre caractère. Et ainsi de suite.

Pourquoi p++ ne fonctionne pas ?

Tu peux faire déplacer le pointeur si tu veux et dans ce cas tu n'as plus besoin de i pour accéder a tableau. Mais tu auras toujours besoin d'un compteur pour compter le nombre de caractères déjà copiés dans t. Montre ton code.

[stallaf]

Fonction simplifiée et réécrite : C'est mieux normalement

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
#include<stdio.h>
 
char *lecture (char *t, size_t taille)
{
    size_t i = 0;
    char *p = t;
 
    *p = getchar();
    while ((i  < taille - 1) && (*p != EOF) && (*p != '\n'))
    {
        t[*p] = *p;  
        p++;
        i++;
        *p = getchar();
    }
    t[i] = '\0';
    return t;
}
 
int main(void)
{
    char t[31];
 
    printf("Taper une phrase de 30 caratèeres maximum :\n");
    lecture(t, sizeof(t));
    printf("La chaine vaut: %s\n", t);
 
    return 0;
}

[souviron34]

euh..

ce serait pas :

Code :

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

1
2
 
       t[i] = *p;

et non

Code :

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

1
2
 
       t[*p] = *p;

??

[stallaf]

Le t[*p] = *p compile et fonctionne chez moi !!

[souviron34]

Le t[*p] = *p compile et fonctionne chez moi !!

premiere erreur fondamentale en programmation ; ca n'est pas parce que ca compile et ne plante pas que ca fonctionne...

chaine = "Toutou"

p = chaine

*p = 'T' = 84 (ASCII)

je ne crois pas que ce soit ce que tu veux.....

[stallaf]

Envoyé par souviron34
premiere erreur fondamentale en programmation ; ca n'est pas parce que ca compile et ne plante pas que ca fonctionne...

C'est vrai.
Je voulais simplement dire que mon code produisait ce que j'en attendais. Maintenant, il est vrai aussi que je ne l'ai pas testé de manière à mettre des erreurs en évidence.
Cela montre bien la difficulté pour un débutant de produire une code logique, propre, conforme aux standards et sans erreurs.
Bref, c'est en forgeant...
Merci à vous.