Servlet: Ecrire dans un fichier txt des données issues d'un formulaire HTML

Version imprimable

Bonjour,

Je suis en train de réaliser une servlet pour gérer une collection de livres. Et mon soucis est d'écrire dans un fichier .txt le enregistrements provenant des formulaires. J'ai récupéré tous les enregitrements grâce à la méthode getParameter mais je n'arrive pas à érire le données dans mon fichier.

Voici mon code :
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 String titre = request.getParameter("titre"); if (titre.length() == 0) { out.println("L'enregistrement du titre de ce livre est vide" + "<br>"); } else { out.println("<font size=\"3\">"); out.println("Titre :" + titre + "<br>"); } FileWriter writer = null; try{ writer = new FileWriter("collection.txt", true); writer.write(titre,0,titre.length()); }catch(IOException ex){ ex.printStackTrace(); }finally{ if(writer != null){ writer.close(); } }
Ce n'est pas le code en entier mais le bouts interessant pour mon test, histoire de savoir si cela fonctionne.

Merci d'avance

26/05/2008, 18h55
guigui5931

Quand tu dis que tu n'y arrive pas , tu as une exception ou le fichier est seulement vide ?
26/05/2008, 20h05
charlix

Le plus drôle c'est que je n'ai mm pas de fichier. Normalement, le fichier devrait se trouver dans le même dossier que les classes ? Sinon peut être qu'il existe mais je ne sais où.
26/05/2008, 20h57
charlix

Ba en fait, après avoir répondu à ton post je me suis dit, fait une recherche windows, ça ne mange pas de pain. Et en fait, ba je l'ai trouvé dans la racine de Tomcat.

Je vais faire des tests pour voir si le programme fait bien ce que je veux.

Le programme écris bien dans le fichier mais seulement lorsque je met qu'un seul parametre comme par exemple titre. Mais cela ne fonctionne pas lorsque je fais ceci :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
 
    	    	BufferedWriter ecrivain = null;
        try{
             ecrivain = new BufferedWriter (new FileWriter("collection_livres.txt", true));
             ecrivain.write("Titre : " + titre + "\n",0,titre.length() + 10);
             ecrivain.write("Isbn : " + isbn + "\n",0,isbn.length() + 9);
             ecrivain.write("Auteur : " + auteur + "\n",0,auteur.length() + 11);
             ecrivain.write("Année : " + annee + "\n",0,annee.length() + 10);
             ecrivain.write("Nombre de pages : " + nombrepages + "\n",0,nombrepages.length() + 18);
             ecrivain.write("Prix : " + prix + "\n",0,prix.length() + 9);
        }catch(IOException ex){
            ex.printStackTrace();
        }finally{
          if(ecrivain != null){
             ecrivain.close();
          }
        }

J'ai une erreur qui me dit que :

Code:

1
2
3
4
5
6
7
 
java.lang.StringIndexOutOfBoundsException: String index out of range: 15
	java.lang.String.getChars(Unknown Source)
	java.io.BufferedWriter.write(Unknown Source)
	Visionneuse.doGet(Visionneuse.java:148)
	javax.servlet.http.HttpServlet.service(HttpServlet.java:690)
	javax.servlet.http.HttpServlet.service(HttpServlet.java:803)

La ligne 148 est celle ou se trouve ecrivain.write ...titre...
Normalement cette exception n'apparait seulement lorsque il y trop de caractere pour ce qu'il y a de allouer ? Je pense avoir bien compté pourtant.

Si quelqu'un a une idée.

28/05/2008, 18h37
charlix

Pour ceux que cela intéressent j'ai réussi en changeant de méthode. A la place d'utiliser la méthode write qui ne fonctionnais pas parce que je ne sais pourquoi, j'ai utilisé simplement la méthode println toujours avec filewriter.
28/05/2008, 19h06
hlr
A titre d'info, tu utilises mal la méthode write de BufferedWriter
Code:

1 2 write(String s, int off, int len) ecrivain.write("Titre : " + titre + "\n",0,titre.length() + 10);
Tu donnes la mauvaises longueur : "Titre : " + "\n" a une longueur 9 et pas 10, du coup tu essaies 1 caractère de trop.
Une bonne pratique (pour plus tard...) : utilises une variable de type String
Code:

1 2 String maLigne = "Titre : " + titre + "\n" ecrivain.write(maLigne, 0, maLigne.length());
C'est quand même mieux non ?
29/05/2008, 09h36
charlix

Bonjour,

Ouai c'est vrai que pour toutes les lignes, il y avait un caractère de trop mais je pensait au début que le retour chariot comptait pour 2 caractères ("\n"). Mais cela n'avait aucune espece d'importance, ce n'était pas cela qui génait.

Citation:

Une bonne pratique (pour plus tard...) : utilises une variable de type String
Code :
String maLigne = "Titre : " + titre + "\n"
ecrivain.write(maLigne, 0, maLigne.length());

Mais merci de m'avoir expliquer pourquoi cela ne fonctionnait pas avec la méthode write().
29/05/2008, 09h47
hlr

Un bon développeur, quand il rencontre un problème, ne se contente pas de faire en sorte que ça marche, mais surtout pourquoi ça n'a pas marché.
J'ai déjà rencontré plusieurs cas de ce type : un jour, un mec me montre son code (c'était du C, mais cela peut être extrapolé à n'importe quel langage) : il a déclaré une variable au début de sa fonction. Je me suis aperçu qu'il ne l'utilise jamais dans son code. Je lui ai demandé de l'enlever, mais il ne voulait pas... Il me disait que sans cette variable, son programme plantait :roll: Il n'a pas cherché à savoir pourquoi ça plantait, il faisait juste du forcing
29/05/2008, 10h09
charlix

Je ne sais pas pourquoi j'ai le droit à un jugement comme cela, j'ai cherché pourquoi cela ne fonctionnais pas, et je n'ai pas trouvé. Donc j'ai essayé un autre méthode, je ne voit pas ce qu' il y a de mal à ça.

Et oui, je ne suis qu'un développeur débutant.

Mais merci quand même.
29/05/2008, 10h12
hlr

Citation:

Envoyé par charlix

Je ne sais pas pourquoi j'ai le droit à un jugement comme cela, j'ai cherché pourquoi cela ne fonctionnais pas, et je n'ai pas trouvé. Donc j'ai essayé un autre méthode, je ne voit pas ce qu' il y a de mal à ça.

Et oui, je ne suis qu'un développeur débutant.

Mais merci quand même.

Je ne te juge pas, je te donne les bonnes pratiques pour un développeur
Je n'ai jamais dit que tu codais mal ou que tu fais tout de travers. J'ai juste dit qu'il faut que tu essaies de comprendre pourquoi ça n'a pas marché. Parce que dire "ça marche avec println et pas avec write" ne suffit pas. Si ça ne marche pas avec write, c'est qu'il y a une raison et il est encore temps d'essayer de le comprendre. Le jour où tu seras vraiment confronté à l'utilisation obligatoire de write (genre écrire des byte[]), là tu seras content
29/05/2008, 10h22
charlix

Ouai j'avais bien compris mais étant donné que je suis en école d'informatique avec des projets plein le dos, donc lorsqu'il y a quelque chose qui fontionne pour ce je souhaite faire je passe un peu moins de temps à comprendre pourquoi cela ne fonctionne pas. C'est pour cela que je viens sur ce site : avoir des explications sur ce que je ne comprend pas lorsque je n'y arrive pas tout seul.

Merci de ton aide.
29/05/2008, 10h29
hlr

Citation:

Envoyé par charlix

Ouai j'avais bien compris mais étant donné que je suis en école d'informatique avec des projets plein le dos, donc lorsqu'il y a quelque chose qui fontionne pour ce je souhaite faire je passe un peu moins de temps à comprendre pourquoi cela ne fonctionne pas. C'est pour cela que je viens sur ce site : avoir des explications sur ce que je ne comprend pas lorsque je n'y arrive pas tout seul.

Merci de ton aide.

"Etre dans une école" n'est pas la bonne excuse ;-) D'ailleurs, c'est le meilleur moment pour apprendre.
Bon passons... l'essentiel c'est que tu aies compris le principe