Problème lors de l'écriture dans un fichier texte

Version imprimable

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:

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,

27/04/2013, 16h12
Metalman

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

Ensuite :

Code:

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)

Cela fait donc ça :
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 #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:

AABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789BABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789CABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789DABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789EABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789FABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789GABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789HABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789IABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789JABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789KABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789LABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789MABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789NABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789OABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789PABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789QABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789RABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789SABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789TABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789UABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789VABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789WABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789XABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789YABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789ZABCDEFGHIJKLMNOPQRSTUVWXYZ01234567890ABCDEFGHIJKLMNOPQRSTUVWXYZ01234567891ABCDEFGHIJKLMNOPQRSTUVWXYZ01234567892ABCDEFGHIJKLMNOPQRSTUVWXYZ01234567893ABCDEFGHIJKLMNOPQRSTUVWXYZ01234567894ABCDEFGHIJKLMNOPQRSTUVWXYZ01234567895ABCDEFGHIJKLMNOPQRSTUVWXYZ01234567896ABCDEFGHIJKLMNOPQRSTUVWXYZ01234567897ABCDEFGHIJKLMNOPQRSTUVWXYZ01234567898ABCDEFGHIJKLMNOPQRSTUVWXYZ01234567899ABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789

Encore pire :'(

Edit : Code édité.

27/04/2013, 16h41
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 !
27/04/2013, 17h09
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.
27/04/2013, 17h43
Iron-Avenger
Citation:

Envoyé par 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 !

Donc à la place de ça :
Code:

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:

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

_________________________________________________________________

Citation:

Envoyé par Winjerome

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:

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
27/04/2013, 19h07
Metalman

Eh bah... un 2e fputc avec en paramètre '\n' !
28/04/2013, 10h46
Iron-Avenger

Citation:

Envoyé par Metalman

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

J'ai déjà essayé :

Code:

fputc('\n', fichier);

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

Problème résolu avec :

Code:

char retour = '\n';

Code:

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