Discussion :

[magnetic]

bonjour a tous
je veut ouvrir un fichier texte qui contient un entier ,metre cette entier dans une variable puis avant de fermer le fichier texte je veut qu il incremente cette entier (si il y avait un 1 alors on aura 2 avant la fermeture du fichier ) j ai essaie avec ce code mais ca ne marche pas
merci encore une fois

Code :

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

1
2
3
4
5
6
7
8
9
10
 BufferedReader fi=new BufferedReader(new FileReader("file.txt"));
                String s1=fi.readLine();
                filename=filename+s1;
                fi.close();
                PrintWriter fich=new PrintWriter(new FileWriter("file.txt"));
                int a1=Integer.parseInt(s1);
                a1=a1+1;
                s1=a1.toString();
                fi.Println(s1);
                fi.close();

[spekal]

J'ai l'impression qu'il y a tant de choses qui ne marchent pas dans ce code que je ne sais pas par laquelle commencer ! Par exemple : à quoi sert filename ? D'où sort la méthode Println du BufferedReader ?? Etc.

Donc, la première étape que je te suggère est de faire des bouts de code qui compilent bien. Tu n'es pas loin de la solution dans ton code (ou du moins, d'une solution possible), mais il y a tant de petites erreurs d'étourderie, qu'il est impossible de savoir ce que tu as voulu faire exactement.

Fais des petites itérations dans ton développement : tu écris un code qui compile, tu l'exécutes, tu le contrôles avec des bouts de tests, etc.

[le y@m's]

j ai essaie avec ce code mais ca ne marche pas

c'est à dire ? As-tu des messaes d'erreur ?

Code :

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

1
2
fi.Println(s1);
fi.close();

Tu écris avec ton BufferedReader... je pense que tu voulais utiliser ton PrintWriter fich.


PS : il est plus convenable de fermer les flux dans un bloc finally

[NeptuS]

Il y a quelques autres erreurs :

String s1=fi.readLine();

d'après la javadoc :

public String readLine()
throws IOException

Tu dois donc mettre cette ligne dans un try-catch.

filename=filename+s1;

Comme on te l'a dit, cette ligne ne sert (apparemment) à rien.

fi.close();

à mettre dans un bloc finally.

int a1=Integer.parseInt(s1);

d'après la javadoc :

public static int parseInt(String s,
int radix)
throws NumberFormatException

Tu dois donc mettre cette ligne dans un try-catch.

a1=a1+1;

idem que a1++; (on n'est plus en C )

s1=a1.toString();

inutile

fi.Println(s1);

faux : on ne met pas de majuscule aux noms de méthodes .... donc leur appel ne prend pas de majuscule non plus :

Code :

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

fich.println(""+a1);

fi.close();

à mettre dans un bloc finally.


à part ça, le reste est bon .... la première ligne en fait ^^.

Bon courage.