Les données ne s'ajoutent pas dans le fichier XML

**pitchu** · 03/03/2015, 22h05

Bonjour,
J'ai une servlet qui récupérè les donnée d'un formulaire (qui fonctionne) et qui les ajoutes dans un fichier xml, mais le fichier xml ne se crée pas
La servlet appelle un fichier java qui ajoute les données xml.
Je n'ai pas de message d'erreur, je n'ai qu'une page blanche après avoir entrée les données sur la page web

la servlet

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
 
	protected void doGet(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
		// TODO Auto-generated method stub
 
		String nom,prenom,email,telPortable,telFixe,commentaire;
 
		response.setContentType("text/plain");
		//PrintWriter pw = response.getWriter(); 
 
		nom = request.getParameter("nom");
		prenom = request.getParameter("prenom");
		email = request.getParameter("email");
		telPortable = request.getParameter("tel_portable");
		telFixe = request.getParameter("tel_fixe");
		commentaire = request.getParameter("commentaire");
 
 
		new Ajouter(nom,prenom,email,telPortable,telFixe,commentaire); 
 
 
	}

Le fichier java qui ajoute les données dans le fichier xml

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
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
 
package packageTreatment;
 
 
import java.io.File;
import java.io.FileOutputStream;
import java.io.IOException;
 
import org.jdom2.Document;
import org.jdom2.Element;
import org.jdom2.JDOMException;
import org.jdom2.Parent;
import org.jdom2.input.SAXBuilder;
import org.jdom2.output.XMLOutputter;
 
public class Ajouter {
 
	static String nom;
	static String prenom; 
	static String email;
	static String telPortable;
	static String telFixe;
	static String commentaire ;
 
	public Ajouter(String parNom, String parPrenom, String parEmail, String parTelPortable, String parTelFixe, String parCommentaire)
	{
		nom = parNom ;
		prenom = parPrenom ; 
		email = parEmail;
		telPortable = parTelPortable;
		telFixe = parTelFixe;
		commentaire = parCommentaire;
 
	}
 
	public static void main(String[] args) throws Exception 
	{
		Document document = Ajouter.ajout("WebContent/annuaire.xml");
 
		enregistreFichier(document,"WebContent/NEWannuaire.xml");
 
	}
 
 
	static Document ajout(String fileName) throws JDOMException, IOException
	{ 
		Document document = null ;
		File file = new File(fileName);		
		SAXBuilder builder = new SAXBuilder(); 
 
		document = builder.build(file);
 
 
 
		//----------------- Création des éléments ------------------ \\
 
		Element personne = new Element("personne");
		personne.setAttribute("id",nom) ;
 
		Element varNom = new Element("nom");
		varNom.setText(nom);
 
		Element varPrenom = new Element("prenom");
		varNom.setText(prenom);
 
		Element varEmail = new Element("email");
		varNom.setText(email);
 
		Element varTelPortable = new Element("telPortable");
		varNom.setText(telPortable);
 
		Element varTelFixe = new Element("telFixe");
		varNom.setText(telFixe);
 
		Element varCom = new Element("commentaire");
		varNom.setText(commentaire);
 
		// ----------------- Création de la hiérarchie ---------------- \\
 
		personne.addContent(varNom);
		personne.addContent(varPrenom);
		personne.addContent(varEmail);
		personne.addContent(varTelPortable);
		personne.addContent(varTelFixe);
		personne.addContent(varCom);
 
		document.getRootElement().addContent(personne);
 
 
		return document ;
 
	}
 
    static void enregistreFichier(Document document,String fichier) throws Exception
    {
    	XMLOutputter sortie = new XMLOutputter();
    	sortie.output(document, new FileOutputStream(fichier));
    }
 
}

**joel.drigo** · 03/03/2015, 23h03

Salut,

Il faudrait revoir un peu les bases : le programme ne fait que ce que tu lui demandes de faire. Ni plus. Ni moins.

Quand tu fais

Code :

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

new Ajouter(nom,prenom,email,telPortable,telFixe,commentaire);

; ça exécute le constructeur de Ajouter, soit :

Code :

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

1
2
3
4
5
6
7
8
9
10
public Ajouter(String parNom, String parPrenom, String parEmail, String parTelPortable, String parTelFixe, String parCommentaire)
	{
		nom = parNom ;
		prenom = parPrenom ; 
		email = parEmail;
		telPortable = parTelPortable;
		telFixe = parTelFixe;
		commentaire = parCommentaire;
 
	}

Ni plus. Ni moins. Donc ça ne fait que mettre les valeurs que tu passes en paramètre dans les variables statiques. Et rien de plus. Il faut appeler la méthode qui fait l'ajout dans le fichier xml, pour que ça le fasse. Bon, là, tu as bidouillé un truc avec des variables statiques et une méthode main() qui n'a d'intérêt que pour lancer un programme, mais dans ton contexte, une simple méthode de sauvegarde avec des variables locales feraient la sauvegarde sans bidouille avec des statiques, qui te poseront problème en cas d'accès multiple (à priori une application web est utilisable par plus d'une personne à la fois.

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
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
 
package packageTreatment;
 
 
import java.io.File;
import java.io.FileOutputStream;
import java.io.IOException;
 
import org.jdom2.Document;
import org.jdom2.Element;
import org.jdom2.JDOMException;
import org.jdom2.Parent;
import org.jdom2.input.SAXBuilder;
import org.jdom2.output.XMLOutputter;
 
public class Ajouter {
 
 
	public static void ajouter(String parNom, String parPrenom, String parEmail, String parTelPortable, String parTelFixe, String parCommentaire)
	{
		Document document = Ajouter.ajout("WebContent/annuaire.xml", parNom, parPrenom, parEmail, parTelPortable,  parTelFixe, parCommentaire);
 
		enregistreFichier(document,"WebContent/NEWannuaire.xml");
 
	}
 
 
	private static Document ajout(String fileName, String parNom, String parPrenom, String parEmail, String parTelPortable, String parTelFixe, String parCommentaire) throws JDOMException, IOException
	{ 
		Document document = null ;
		File file = new File(fileName);		
		SAXBuilder builder = new SAXBuilder(); 
 
		document = builder.build(file);
 
 
 
		//----------------- Création des éléments ------------------ \\
 
		Element personne = new Element("personne");
		personne.setAttribute("id",nom) ;
 
		Element varNom = new Element("nom");
		varNom.setText(nom);
 
		Element varPrenom = new Element("prenom");
		varNom.setText(prenom);
 
		Element varEmail = new Element("email");
		varNom.setText(email);
 
		Element varTelPortable = new Element("telPortable");
		varNom.setText(telPortable);
 
		Element varTelFixe = new Element("telFixe");
		varNom.setText(telFixe);
 
		Element varCom = new Element("commentaire");
		varNom.setText(commentaire);
 
		// ----------------- Création de la hiérarchie ---------------- \\
 
		personne.addContent(varNom);
		personne.addContent(varPrenom);
		personne.addContent(varEmail);
		personne.addContent(varTelPortable);
		personne.addContent(varTelFixe);
		personne.addContent(varCom);
 
		document.getRootElement().addContent(personne);
 
 
		return document ;
 
	}
 
    private static void enregistreFichier(Document document,String fichier) throws Exception
    {
    	XMLOutputter sortie = new XMLOutputter();
    	sortie.output(document, new FileOutputStream(fichier));
    }
 
}

Pour commencer. Que tu appelles dans ta servlet comme ça :

Code :

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

Ajouter.ajouter(nom,prenom,email,telPortable,telFixe,commentaire);

Ensuite 3 problèmes, au moins :

Le chemin n'est pas bon : il me semble t'avoir parler de ça dans une autre discussion, là
Ensuite, tu lis un fichier, et tu écris dans un autre : si tu ajoutes 2 utilisateurs, le premier sera perdu (puisque tu repars d'un fichier qui n'est jamais modifié (il faut à la fin remplacer l'ancien fichier par le nouveau : voir renameTo().
Si plusieurs personnes ajoutent des utilisateurs en même temps, l'un d'eux ouvrira le fichier pour écrire dedans et les autres se feront jeter... (à régler tout simplement par synchronized)

Ah, j'oubliais : si tu as une page blanche en réponse c'est parce que ta servlet genère une réponse de type plain/text dans laquelle tu n'écris rien.

**pitchu** · 04/03/2015, 10h36

Bonjour,
Pour le chemin je ne comprends pas très bien , j'ai mis la même chose que dans l'autre discussion mais le chemin étant un string il m'affiche des erreurs

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
	public static void ajouter(String parNom, String parPrenom, String parEmail, String parTelPortable, String parTelFixe, String parCommentaire) throws Exception 
	{
		SAXBuilder sxb = new SAXBuilder(); 
		Document doc = sxb.build(new File(request.getServletContext().getRealPath("annuaire.xml"));
 
		Document document = Ajouter.ajout(doc,parNom, parPrenom, parEmail, parTelPortable,  parTelFixe, parCommentaire);
 
		enregistreFichier(document,doc);
 
	}

**joel.drigo** · 04/03/2015, 11h37

C'est bien ce que je disais : il faut reprendre les bases !

C'est quoi request dans la méthode ajouter ? C'est quelque chose qui n'a aucun sens pour Java, ou, à la limite une variable qui n'existe pas. Il faut passer l'instance de request, ou de ServletContext en paramètre de la méthode ajouter, pour pouvoir appeler une de ses méthodes dans la méthode ajouter().

**pitchu** · 04/03/2015, 15h14

Mais pour le chemin, il faut que ce soit un string là c'est un document pour les méthodes ajout et enregistreFichier.

fichier Ajouter

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
	public static void ajouter(String parNom, String parPrenom, String parEmail, String parTelPortable, String parTelFixe, String parCommentaire, HttpServletRequest request) throws Exception 
	{
		SAXBuilder sxb = new SAXBuilder(); 
		Document doc = sxb.build(new File(request.getServletContext().getRealPath("annuaire.xml")));
 
		Document document = Ajouter.ajout(doc,parNom, parPrenom, parEmail, parTelPortable,  parTelFixe, parCommentaire);
 
		enregistreFichier(document,doc);
 
	}

fichier Servlet

Code :

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

1
2
3
4
5
6
7
8
 
		...
		try {
			Ajouter.ajouter(nom,prenom,email,telPortable,telFixe,commentaire, request);
		} catch (Exception e) {
			// TODO Auto-generated catch block
			e.printStackTrace();
		}

**joel.drigo** · 04/03/2015, 15h50

Pour le nom du fichier, certes, mais pas son chemin absolu, que tu ne peux connaitre forcément à l'avance, puis qu'il dépend du nom du dossier de l'application web et du dossier d'installation du serveur (voir de la structure des dossiers en fonction du serveur).

D'ailleurs, pour des fichiers qu'on modifie, il vaut mieux les stocker dans un répertoire à part du dossier de l'application web (lors d'un redéploiement, les fichiers dans webcontent seront supprimés). : ainsi tu te prévois un dossier là où tu veux ton disque dont le chemin absolu est connu de ton application (éventuellement paramétrable), et tu n'as plus besoin de traiter le path en relatif par rapport à quoique ce soit (context de l'application web ou dossier de lancement de l'application).

Les données ne s'ajoutent pas dans le fichier XML

Servlets/JSP Java

Vue hybride

Discussions similaires

Partager

Partager