Discussion :

[Iron-Avenger]

Bonjour,

J'ai commencé un programme qui prend toutes les lettres de l'alphabet une par une, les mets dans une chaîne et écris cette chaîne dans un fichier texte (pour l'instant je test sa pour faire autre chose après).

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
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
int main()
{
    if("fichier.txt" != NULL)
    {
        remove("fichier.txt");
    }
    FILE* fichier = NULL;
    fichier = fopen("fichier.txt", "w");
    int i = 0;
    char alphabet[] = {'A','B','C','D','E','F','G','H','I','J','K','L','M','N','O','P','Q','R','S','T','U','V','W','X','Y','Z','0','1','2','3','4','5','6','7','8','9','\0'};
    char chaine1[1] = "";
 
    while (i<36)
    {
        chaine1[0] = alphabet[i];
        fputs(chaine1, fichier);
        i++;
        fclose(fichier);
    }
    return 0;
}

Le fichier texte devrait donc contenir "ABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789" mais non il contient "AABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789" soit un "A" en trop.

Quelqu'un aurait une idée d'où viens le problème ?

De plus j'aimerai rajouter un "\n" dans chaine1 afin que les lettres soient sur des lignes différentes, j'ai tenter plusieurs fonction mais sans succès. Une idée ?

Cordialement,

[Metalman]

Déjà : déplace ton fclose hors de la boucle...

Ensuite :

Code :

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

if("Dico.txt" != NULL)

Ça ne teste pas si le fichier existe ou non...
Tu dois tester l'ouverture du fichier (si fopen te renvoie NULL ou pas)

[Iron-Avenger]

Cela fait donc ça :

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
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
int main()
{
    FILE* fichier = NULL;
    fichier = fopen("fichier.txt", "r");
    if (fichier != NULL)
    {
        remove("fichier.txt");
    }
    fichier = fopen("fichier.txt", "w");
    int i = 0;
    char alphabet[] = {'A','B','C','D','E','F','G','H','I','J','K','L','M','N','O','P','Q','R','S','T','U','V','W','X','Y','Z','0','1','2','3','4','5','6','7','8','9','\0'};
    char chaine1[1] = "";
 
    while (i<36)
    {
        chaine1[0] = alphabet[i];
        fputs(chaine1, fichier);
        i++;
    }
    fclose(fichier);
    return 0;
}

Et mon fichier.txt contient ça :

Code :

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

AABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789BABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789CABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789DABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789EABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789FABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789GABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789HABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789IABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789JABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789KABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789LABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789MABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789NABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789OABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789PABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789QABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789RABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789SABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789TABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789UABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789VABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789WABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789XABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789YABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789ZABCDEFGHIJKLMNOPQRSTUVWXYZ01234567890ABCDEFGHIJKLMNOPQRSTUVWXYZ01234567891ABCDEFGHIJKLMNOPQRSTUVWXYZ01234567892ABCDEFGHIJKLMNOPQRSTUVWXYZ01234567893ABCDEFGHIJKLMNOPQRSTUVWXYZ01234567894ABCDEFGHIJKLMNOPQRSTUVWXYZ01234567895ABCDEFGHIJKLMNOPQRSTUVWXYZ01234567896ABCDEFGHIJKLMNOPQRSTUVWXYZ01234567897ABCDEFGHIJKLMNOPQRSTUVWXYZ01234567898ABCDEFGHIJKLMNOPQRSTUVWXYZ01234567899ABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789

Encore pire :'(

Edit : Code édité.

[Metalman]

Avant de remove, n'oublie pas de close !
Le mieux serait plutôt de voir comment trunk le fichier et le create !

D'après ça, il faudrait un "w+", plutôt qu'un w...

Cela fait exactement ce que tu veux : remettre à 0 le fichier s'il existe ou le créer !

[Invité]

Bonjour,

Tu souhaites écrire un caractère à la fois, or la fonction fputs écrit une chaîne de caractères, qui s'arrête donc au premier caractère '\0' rencontré et pas avant.
C'est fputc qu'il te faut utiliser ici.

[Iron-Avenger]

Avant de remove, n'oublie pas de close !
Le mieux serait plutôt de voir comment trunk le fichier et le create !

D'après ça, il faudrait un "w+", plutôt qu'un w...

Cela fait exactement ce que tu veux : remettre à 0 le fichier s'il existe ou le créer !

Donc à la place de ça :

Code :

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

1
2
3
4
5
6
fichier = fopen("fichier.txt", "r");
    if (fichier != NULL)
    {
        remove("fichier.txt");
    }
    fichier = fopen("fichier.txt", "w");

J'ai sa :

Code :

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

fichier = fopen("fichier.txt", "w+");

_________________________________________________________________

Bonjour,

Tu souhaites écrire un caractère à la fois, or la fonction fputs écrit une chaîne de caractères, qui s'arrête donc au premier caractère '\0' rencontré et pas avant.
C'est fputc qu'il te faut utiliser ici.

Parfait !

Avec ce que tu m'a conseillé :

Code :

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

1
2
3
4
5
    while (i<36)
    {
        fputc(alphabet[i], fichier);
        i++;
    }

J'ai bien "ABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789" =)

Ne manque plus qu'a savoir comment faire pour mettre un caractère par ligne :s

[Metalman]

Eh bah... un 2e fputc avec en paramètre '\n' !

[Iron-Avenger]

Eh bah... un 2e fputc avec en paramètre '\n' !

J'ai déjà essayé :

Code :

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

fputc('\n', fichier);

Mais cela ne fonctionne pas, j'ai toujours : "ABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789"

[Iron-Avenger]

Problème résolu avec :

Code :

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

char retour = '\n';

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
int main()
{
    FILE* fichier = NULL;
    fichier = fopen("fichier.txt", "w+");
    int i = 0;
    char alphabet[] = {'A','B','C','D','E','F','G','H','I','J','K','L','M','N','O','P','Q','R','S','T','U','V','W','X','Y','Z','0','1','2','3','4','5','6','7','8','9','\0'};
    char retour='\n';
    while (i<36)
    {
        fputc(alphabet[i], fichier);
        fputc(retour, fichier);
        i++;
    }
    fclose(fichier);
    return 0;
}

Merci beaucoup de votre aide =D