Discussion :

[cetiop]

Salut, voilà mon programme :

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
#include <stdio.h>
#include <stdlib.h>
int main()  
{
char texte_tape[80];
 
printf ("");
gets (texte_tape);
 
 
 
 
FILE *cr = fopen("c:\\essai.txt","w");                                       
fprintf (cr,texte_tape);  
fclose (cr);     
 
return 0;
}

Mais le problème est que je ne peux taper qu'une ligne,car quand j'appuie sur entrée, le programme est quitté au lieu d'allé à la ligne.J'aurais aimé savoir comment faire pour que je puisse tapé plusieurs ligne de texte?

Merci pour votre aide :-)

[Elboras]

Un systeme tout con pourrait etre de faire comme certains programmes sous unix, ils attendent une ligne avec un seul point pour s'arreter.
Tu ecris donc ton texte et tu attends une ligne avec un point pour tarreter.

Cependant je te conseillerais de faire attention, tu cree un buffer de 80 et tu ne le limite pas actuellement, renseigne toi sur les buffers overflow et autres problemes qui peuvent etre lier a cela.

Bonne journee

[cetiop]

Ca va merci pour ta réponse.
Je vais étudier cela.

[odsen.s]

Salut,

Il ne faut jamais utiliser gets(). Voici un extrait de man gets :

BOGUES
N'utilisez jamais gets(). Comme il est impossible de savoir à l'avance
combien de caractères seront lus par gets(), et comme celui-ci écrira
tous les caractères lus, même s'ils débordent du tampon, cette fonction
est extrèmement dangereuse à utiliser. On a déjà utilisé ce dysfonc-
tionnement pour créer des trous de sécurité. UTILISEZ TOUJOURS fgets()
À LA PLACE DE gets().

Pour ton problème, je vois bien une lecture caractère par caractère grâce à fgetc(stdin) tant que le caractère lu n'est pas EOF.
Attention à ne pas lire plus de caractères que le tableau ne peut en contenir.

[Franck.H]

Salut,

Il ne faut jamais utiliser gets(). Voici un extrait de man gets :


Pour ton problème, je vois bien une lecture caractère par caractère grâce à fgetc(stdin) tant que le caractère lu n'est pas EOF.
Attention à ne pas lire plus de caractères que le tableau ne peut en contenir.

Et au pire des cas, pour éviter les buffer overflow, tu peux faire un buffer alloué dynamiquement et faire une réallocation dynamique avec une croissance par progression géométrique: http://www.bourguet.org/realloc.pdf soit pour illustrer (merci à Jean-Marc.Bourguet en passant) :

Code :

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

new_size=old_size + old_size/2

Cela dit, si tu cherches à faire un programme ultra rapide et optimisé... les fonctions d'allocation sont parmis les plus lentes, même si pour toi cela reste rapide

[Faiche]

Mais le problème est que je ne peux taper qu'une ligne,car quand j'appuie sur entrée, le programme est quitté au lieu d'allé à la ligne.J'aurais aimé savoir comment faire pour que je puisse tapé plusieurs ligne de texte?

Merci pour votre aide :-)

Bon, tout le monde est tellement concentré sur tes possibles buffer overflow que personne n'a pensé à te demander si tu savais ce qu'est une boucle.

Si tu appliques ton code une fois, il s'exécute une fois. Si tu fais une boucle, il s'exécute boucle fois.

[Thierry Chappuis]

Une solution simple:

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
34
35
36
37
38
39
40
41
42
43
44
#include <stdio.h>
#include <stdlib.h>
 
#define LINE_LENGTH 80
#define NB_LINES 10
 
void check_and_clean(int c);
 
int main(void)
{
    char s_buffer[LINE_LENGTH * NB_LINES] = "";
    int c;
    int i; /* loop index */
 
    c = fgetc(stdin);
    for (i = 0; i < sizeof s_buffer -1 && c != EOF; i++)
    {
        s_buffer[i] = c;
        c = fgetc(stdin);
    }
    check_and_clean(c);
 
    printf("Vous avez entré:\n\n%s\n", s_buffer);
    return EXIT_SUCCESS;
}
 
/**
 * Vérifie si l'entrée a été tonquée et vide le tampon du flux d'entrée standard
 * si nécessaire.
 */
void check_and_clean(int c)
{
    if (c != EOF)
    {
        /* Le tampon est plein */
        fprintf(stderr, "Buffer is full, entry has been truncated...!\n");
 
        /* On vide le tampon d'entrée */
        while ((c = fgetc(stdin)) != '\n' && c != EOF)
        {
            continue;
        }
    }
}

Après, ça dépend pas mal de ce que tu veux faire. Pour une simple saisie multilignes avec une taille de tampon fixe, le code proposé ci-dessus fonctionne.

Thierry

[Faiche]

Code :

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

1
2
 
    char s_buffer[LINE_LENGTH * NB_LINES] = "";

C'est plus prudent de mettre tout le buffer à \0 (avec une boucle ou avec memset).

La dans ton code, tu ne met pas de \0 à la fin, donc le printf craque.

[Thierry Chappuis]

Code :

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

1
2
 
    char s_buffer[LINE_LENGTH * NB_LINES] = "";

C'est plus prudent de mettre tout le buffer à \0 (avec une boucle ou avec memset).

La dans ton code, tu ne met pas de \0 à la fin, donc le printf craque.

Oui,

L'initialisation suivante:

Code :

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

char s_buffer[LINE_LENGTH * NB_LINES] = "";

est équivalente à:

Code :

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

char s_buffer[LINE_LENGTH * NB_LINES] = {0};

qui initialise toute les cases du tampon à la valeur 0.

Comme la boucle s'exécute seulement lorsque i est inférieur à sizeof s_buffer - 1, je ne vois pas où est problème. Le code proposé est sûr de ce point de vue.

Thierry

[Faiche]

effectivement, je connaissais pas cette initialisation, sorrite :p