Bonjour à tous en cette bel journée !

Je désire savoir comment coder plus proprement mon appli J2EE car on m'a dit que je code mal.

Voici ce que je fais dans ma jsp:

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
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
 
<%@ page import="java.io.*" %>
<%@ page import="java.util.*" %>
<%@ page import="org.apache.poi.hssf.usermodel.HSSFWorkbook" %>
<%@ page import="org.apache.poi.hssf.usermodel.HSSFCell" %>
<%@ page import="org.apache.poi.hssf.usermodel.HSSFSheet" %>
<%@ page import="org.apache.poi.hssf.usermodel.HSSFRow" %>
<%@ page import="org.apache.poi.poifs.filesystem.POIFSFileSystem" %>
<%@ page import="java.text.DateFormat" %>
<%@ page import="java.text.ParseException" %>
<%@ page import="java.text.SimpleDateFormat" %>
<%@ page import="java.awt.*" %>
<%@ page import="org.jfree.chart.*" %>
<%@ page import="org.jfree.chart.axis.*" %>
<%@ page import="org.jfree.chart.entity.*" %>
<%@ page import="org.jfree.chart.labels.*" %>
<%@ page import="org.jfree.chart.plot.*" %>
<%@ page import="org.jfree.chart.renderer.category.*" %>
<%@ page import="org.jfree.chart.urls.*" %>
<%@ page import="org.jfree.data.category.*" %>
<%@ page import="org.jfree.data.general.*" %>
 
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<title>Extraction du Fichier Excel</title>
<link rel="stylesheet" type="text/css" href="style_de_base.css" media="screen">
</head>
<body>
	<h3>Extraction IETD</h3>
		<br/>
		chemin= <%=request.getParameter("file")%><br />
		<%
			String type_dossier;
			String ref_presta;
			String code_action;
			long N_BP_Symphonie;
			String Indice_Carat;
			String lib_offre;
			String entite;
			String televendeur;
			String recu = "";
			String expedie = "";
			String date_npai;
			String anomalie;
			int compteur = 0;
			Calendar cal1 = Calendar.getInstance();
			Calendar cal2 = Calendar.getInstance();
			Calendar cal3 = Calendar.getInstance();
			DateFormat formatter = new SimpleDateFormat("dd/MM/yyyy");
			java.util.Date dat1;
			java.util.Date dat2;
			java.util.Date dat3 = new Date(new GregorianCalendar().getTime().getTime()); 
			String datedujour=formatter.format(dat3);
			//compteur pour graphique
			int b2s = 0;
			int eurocrm = 0;
			int sym = 0;
			int tvmontpellier = 0;
			int tvlyon = 0;
			int tvquimper = 0;
 
		%>
		<table>
			<tr>
				<td>N° de la ligne:</td>
				<td>Dépassement de:(en jours)</td>
				<td>type_dossier</td>
				<td>référence prestataire</td>
				<td>code action</td>
				<td>N_BP_Symphonie</td>
				<td>Indice_Carat</td>
				<td>lib_offre</td>
				<td>entite</td>
				<td>televendeur</td>
				<td>recu</td>
				<td>expedie</td>
				<td>date_npai</td>
				<td>anomalie</td>
			</tr>
 
			<%
				try {
					POIFSFileSystem fs = new POIFSFileSystem(new FileInputStream(
							request.getParameter("file")));
					HSSFWorkbook wb = new HSSFWorkbook(fs);
					HSSFSheet sheet = wb.getSheetAt(0);
					HSSFRow row = null;
					HSSFCell cell = null;
					int numLigne = 1;
					int n;
					int num;
					for (Iterator rowIt = sheet.rowIterator(); rowIt.hasNext();) {
						row = (HSSFRow) rowIt.next();
						//nbre total de ligne
						n = row.getRowNum();
 
						//test pour eviter de lire les entetes
						if (n != 0 && row.getCell(8) != null && row.getCell(9) != null) {
							recu = row.getCell(8).getStringCellValue();
							expedie = row.getCell(9).getStringCellValue();
 
							//test pour eviter que la date d'expedition soit vides et avoir un parse error de la date
							//ATTENTION: demander à Hervé si la date recu est toujours renseigné dans les fichiers d'export IETD 
							if (expedie != ""){
								dat1 = formatter.parse(recu);
								dat2 = formatter.parse(expedie);
								cal1.setTime(dat1);
								cal2.setTime(dat2);
								//System.out.println("-----"+cal1.toString());
 
								int jours = 0;
								while (cal1.before(cal2)) {
									if(cal1.get(Calendar.DAY_OF_WEEK) != Calendar.SUNDAY && cal1.get(Calendar.DAY_OF_WEEK) != Calendar.SATURDAY)
										jours++;
									cal1.add(Calendar.DAY_OF_MONTH, 1);
 
								}
 
								//TEST
								if (jours > 2) {
									//pour afficher le nombre total de ligne non conforme
									compteur++;
									//pour afficher le numéro de la ligne non conforme
									num = row.getRowNum();
									num++;
 
									type_dossier = row.getCell(0)
											.getStringCellValue();
									ref_presta = row.getCell(1)
											.getStringCellValue();
									code_action = row.getCell(2)
											.getStringCellValue();
									N_BP_Symphonie = (long) row.getCell(3)
											.getNumericCellValue();
									Indice_Carat = row.getCell(4)
											.getStringCellValue();
									lib_offre = row.getCell(5).getStringCellValue();
									entite = row.getCell(6).getStringCellValue();
									televendeur = row.getCell(7)
											.getStringCellValue();
									//
									date_npai = row.getCell(10)
											.getStringCellValue();
									anomalie = row.getCell(11).getStringCellValue();
									//test pour le graphique
									if(entite.equals("B2S"))b2s++;
									if(entite.equals("EuroCRM")) eurocrm++;
									if(entite.equals("SYM")) sym++;
									if(entite.equals("TV Lyon")) tvlyon++;
									if(entite.equals("TV Montpellier")) tvmontpellier++;
									if(entite.equals("TV Quimper")) tvquimper++;
			%>
			<tr>
				<td><%=num%></td>
				<td><%=jours%></td>
				<td><%=type_dossier%></td>
				<td><%=ref_presta%></td>
				<td><%=code_action%></td>
				<td><%=N_BP_Symphonie%></td>
				<td><%=Indice_Carat%></td>
				<td><%=lib_offre%></td>
				<td><%=entite%></td>
				<td><%=televendeur%></td>
				<td><%=recu%></td>
				<td><%=expedie%></td>
				<td><%=date_npai%></td>
				<td><%=anomalie%></td>
			</tr>
 
		<%
			}
 
							for (Iterator cellIt = row.cellIterator(); cellIt
									.hasNext();) {
								cell = (HSSFCell) cellIt.next();
								//System.out.println("ligne n° "+cell.getRowIndex());//avoir le numéro de ligne
								break;
							}
						}else{//sinon si expedie==""
								if(row.getCell(11).getStringCellValue().isEmpty()==true){//et si l'anomalie est vide
									if(recu != ""){
							dat1 = formatter.parse(recu);
							dat3 = formatter.parse(datedujour);
							cal1.setTime(dat1);
							cal3.setTime(dat3);
 
							int jours = 0;
							while (cal1.before(cal3)) {
								if(cal1.get(Calendar.DAY_OF_WEEK) != Calendar.SUNDAY && cal1.get(Calendar.DAY_OF_WEEK) != Calendar.SATURDAY)
									jours++;
								cal1.add(Calendar.DAY_OF_MONTH, 1);
 
							}
 
							//TEST si la diff (entre la datedujour et la date recu) est supérieur à 2 jours
							if (jours > 2) {
 
								//pour afficher le nombre total de ligne non conforme
								compteur++;
								//pour afficher le numéro de la ligne non conforme
								num = row.getRowNum();
								num++;
 
								type_dossier = row.getCell(0)
										.getStringCellValue();
								ref_presta = row.getCell(1)
										.getStringCellValue();
								code_action = row.getCell(2)
										.getStringCellValue();
								N_BP_Symphonie = (long) row.getCell(3)
										.getNumericCellValue();
								Indice_Carat = row.getCell(4)
										.getStringCellValue();
								lib_offre = row.getCell(5).getStringCellValue();
								entite = row.getCell(6).getStringCellValue();
								televendeur = row.getCell(7)
										.getStringCellValue();
								//
								date_npai = row.getCell(10)
										.getStringCellValue();
								anomalie = row.getCell(11).getStringCellValue();
								//test pour le graphique
								if(entite.equals("B2S"))b2s++;
								if(entite.equals("EuroCRM")) eurocrm++;
								if(entite.equals("SYM")) sym++;
								if(entite.equals("TV Lyon")) tvlyon++;
								if(entite.equals("TV Montpellier")) tvmontpellier++;
								if(entite.equals("TV Quimper")) tvquimper++;
 
		%>
		<tr>
			<td><%=num%></td>
			<td><%=jours%></td>
			<td><%=type_dossier%></td>
			<td><%=ref_presta%></td>
			<td><%=code_action%></td>
			<td><%=N_BP_Symphonie%></td>
			<td><%=Indice_Carat%></td>
			<td><%=lib_offre%></td>
			<td><%=entite%></td>
			<td><%=televendeur%></td>
			<td><%=recu%></td>
			<td><%=expedie%></td>
			<td><%=date_npai%></td>
			<td><%=anomalie%></td>
		</tr><%
								}
									}
 
							}
							}
					}
				}//boucle for
				%></table>
 
		<br />
		total de ligne ayant dépassé 2 jours: <%=compteur%>
		<br/>
		Date du jour: <%=datedujour%>
		<br/>
		b2s: <%=b2s%>
		<br/>
		eurocrm: <%=eurocrm%>
		<br/>
		sym: <%=sym%>
		<br/>
		tvlyon: <%=tvlyon%>
		<br/>
		tvmontpellier: <%=tvmontpellier%>
		<br/>
		tvquimper: <%=tvquimper%>
		<br/>
		<%
			} catch (FileNotFoundException e) {
				e.printStackTrace();
			} catch (IOException e) {
				e.printStackTrace();
			}
 
		//graph
 
  final double[][] data = new double[][]{{b2s},{eurocrm},{sym},{tvlyon},{tvmontpellier},{tvquimper}};
 
  //final CategoryDataset dataset = DatasetUtilities.createCategoryDataset("Team ", "", data);
  DefaultCategoryDataset dataset1 = new DefaultCategoryDataset(); 
  dataset1.addValue(b2s, "b2s", "b2s"); 
  dataset1.addValue(eurocrm, "eurocrm", "eurocrm"); 
  dataset1.addValue(sym, "sym", "sym"); 
  dataset1.addValue(tvlyon, "tvlyon", "tvlyon");
  dataset1.addValue(tvmontpellier, "tvmontpellier", "tvmontpellier"); 
  dataset1.addValue(tvquimper, "tvquimper", "tvquimper");
 
  JFreeChart chart = null;
  BarRenderer renderer = null;
  CategoryPlot plot = null;
 
  final CategoryAxis categoryAxis = new CategoryAxis("Entité");
  final ValueAxis valueAxis = new NumberAxis("IETD non conforme");
  renderer = new BarRenderer();
 
  plot = new CategoryPlot(dataset1, categoryAxis, valueAxis, renderer); 
  plot.setOrientation(PlotOrientation.VERTICAL);
  chart = new JFreeChart("Graph par entité", JFreeChart.DEFAULT_TITLE_FONT, plot, true);
  chart.setBackgroundPaint(new Color(249, 231, 236));
 
  Paint p1 = new GradientPaint( 0.0f, 0.0f, new Color(16, 89, 172), 0.0f, 0.0f, new Color(201, 201, 244)); 
  renderer.setSeriesPaint(1, p1);
 
  Paint p2 = new GradientPaint(0.0f, 0.0f, new Color(255, 35, 35), 0.0f, 0.0f, new Color(40, 150, 70)); 
  renderer.setSeriesPaint(2, p2);
  plot.setRenderer(renderer);
 
  try {
	  final ChartRenderingInfo info = new ChartRenderingInfo (new StandardEntityCollection());
	  final File file1 = new File("barchart.jpeg");
	  ChartUtilities.saveChartAsPNG(file1, chart, 600, 400, info);
  } catch (Exception e) {
 		out.println(e);
  }
%>
 
  <img src="../barchart.png"/>	
</body>
</html>
On m'a dit de faire:

- Une classe "domaine" qui représente les valeurs contenues dans ta feuille Excel.

- Une classe "bridge" qui utilise POI pour lire le document Excel et te renvoyer les objets correctement instanciés avec les valeurs provenant de la feuille Excel.

- Une classe "contrôleur" qui fait appel à ta classe bridge et récupère un tableau d'objets domaine et le renvoie à la vue JSP.

- Je dois utiliser la JSTL dans la vue pour parcourir le tableau reçu.

Apparemment, on évite de mettre trop de code dans les JSP et on essaye généralement de ne pas mettre de "scriptlet" directement dans la vue.

Quelqu'un pourrait-il me donner une idée d'un code plus propre ?


Voici les classes que j'ai construites déja:


1. classe Feuille_Excel (domaine):
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
 
package Modele;
 
public class Feuille_Excel {
 
	public Feuille_Excel() {
		// TODO Auto-generated constructor stub
	}
	String type_dossier;
	String ref_presta;
	String code_action;
	long N_BP_Symphonie;
	String Indice_Carat;
	String lib_offre;
	String entite;
	String televendeur;
	String recu = "";
	String expedie = "";
	String date_npai;
	String anomalie;
 
 
 
 
	public String getType_dossier() {
		return type_dossier;
	}
	public void setType_dossier(String type_dossier) {
		this.type_dossier = type_dossier;
	}
	public String getRef_presta() {
		return ref_presta;
	}
	public void setRef_presta(String ref_presta) {
		this.ref_presta = ref_presta;
	}
	public String getCode_action() {
		return code_action;
	}
	public void setCode_action(String code_action) {
		this.code_action = code_action;
	}
	public long getN_BP_Symphonie() {
		return N_BP_Symphonie;
	}
	public void setN_BP_Symphonie(long n_BP_Symphonie) {
		N_BP_Symphonie = n_BP_Symphonie;
	}
	public String getIndice_Carat() {
		return Indice_Carat;
	}
	public void setIndice_Carat(String indice_Carat) {
		Indice_Carat = indice_Carat;
	}
	public String getLib_offre() {
		return lib_offre;
	}
	public void setLib_offre(String lib_offre) {
		this.lib_offre = lib_offre;
	}
	public String getEntite() {
		return entite;
	}
	public void setEntite(String entite) {
		this.entite = entite;
	}
	public String getTelevendeur() {
		return televendeur;
	}
	public void setTelevendeur(String televendeur) {
		this.televendeur = televendeur;
	}
	public String getRecu() {
		return recu;
	}
	public void setRecu(String recu) {
		this.recu = recu;
	}
	public String getExpedie() {
		return expedie;
	}
	public void setExpedie(String expedie) {
		this.expedie = expedie;
	}
	public String getDate_npai() {
		return date_npai;
	}
	public void setDate_npai(String date_npai) {
		this.date_npai = date_npai;
	}
	public String getAnomalie() {
		return anomalie;
	}
	public void setAnomalie(String anomalie) {
		this.anomalie = anomalie;
	}
}

ma classe bridge:
Code : Sélectionner tout - Visualiser dans une fenêtre à part
1
2
3
4
5
6
7
8
9
10
11
12
13
14

package Controlleur;

public class Bridge {

	public Bridge() {
		// TODO Auto-generated constructor stub
	}
	
//je sais pas ce qu'elle doit contenir		
	

}
ma classe controlleur:

Code : Sélectionner tout - Visualiser dans une fenêtre à part
1
2
3
4
5
6
7
8
9
10
11
12
13
 
package Controlleur;
 
public class Controlleur {
 
	public Controlleur() {
		// TODO Auto-generated constructor stub
	}
 
	//je sais pas ce qu'elle contient
 
 
}
Je veux partir sur de bonnes bases donc j'attend un vrai exemple.
Merci