Non ottenere la dimensione corretta di un ArrayList in JSP
Domanda
Ho problemi a ottenere il giusto numero di elementi nella ArrayList alt
nella pagina JSP in basso. Quando visualizzo il JSP mostra che la dimensione è 1 (<%=alt.size()%>
) quando dovrebbe essere 3; Penso che sto aggiungendo quel metodo all'array nella classe del generatore, quindi non capisco perché sta mostrando 1.
Questa è la mia pagina jsp:
<%
ArrayList<Alert> a = AlertGenerator.getAlert();
pageContext.setAttribute("alt", a);
%>
<c:forEach var="alert" items="${alt}" varStatus="status" >
<p>You have <%=alt.size()%> Active Alert(s)</p>
<ul>
<li><a href="#" class="linkthree">${alert.alert1}</a></li>
<li><a href="#" class="linkthree">${alert.alert2}</a></li>
<li><a href="#" class="linkthree">${alert.alert3}</a></li>
</ul>
</c:forEach>
Questa è la classe che genera gli avvisi:
package com.cg.mock;
import java.util.ArrayList;
public class AlertGenerator {
public static ArrayList<Alert> getAlert() {
ArrayList<Alert> alt = new ArrayList<Alert>();
alt.add(new Alert("alert1","alert2","alert3"));
return alt;
}
}
Questa è la mia classe bean:
package com.cg.mock;
public class Alert {
String alert1;
String alert2;
String alert3;
public Alert(String alert1, String alert2,String alert3) {
super();
this.alert1 = alert1;
this.alert2 = alert2;
this.alert3 = alert3;
}
public String getAlert1() {
return alert1;
}
public void setAlert1(String alert1) {
this.alert1 = alert1;
}
public String getAlert2() {
return alert2;
}
public void setAlert2(String alert2) {
this.alert2 = alert2;
}
public String getAlert3() {
return alert3;
}
public void setAlert3(String alert3) {
this.alert3 = alert3;
}
}
Soluzione
Per ottenere 3 avvisi è possibile riprogettare come segue. Si noti che esiste solo una proprietà della classe di avviso. Puoi creare una nuova istanza dell'Avviso per ogni avviso.
package com.cg.mock;
public class Alert {
String alert1;
public Alert(String alert1) {
super();
this.alert1 = alert1;
}
public String getAlert1() {
return alert1;
}
public void setAlert1(String alert1) {
this.alert1 = alert1;
}
}
In AlertGenerator:
ArrayList<Alert> alt = new ArrayList<Alert>();
alt.add(new Alert("alert1");
alt.add(new Alert("alert2");
alt.add(new Alert("alert3");
return alt;
E su JSP:
<p>You have <%=alt.size()%> Active Alert(s)</p>
<ul>
<c:forEach var="alert" items="${alt}" varStatus="status" >
<li><a href="#" class="linkthree">${alert.alert1}</a></li>
</c:forEach>
</ul>
Nota che gli ul sono al di fuori del ciclo forEach.
Altri suggerimenti
Il problema è che hai una sola istanza di avviso in ArrayList, ma quel singolo avviso ha 3 proprietà: alert1, alert2 e alert3.
Dai un'occhiata alla linea:
alt.add(new Alert("alert1","alert2","alert3"));
Hai solo una riga di aggiunta e non è in loop.
Una possibile soluzione:
public class Alert {
private String description;
private String status;
private Date raisedOn;
public Alert(String description, String status) {
this.description = description;
this.status = status;
this.raisedOn = new Date();
}
public String getDescription() { return description; }
public String getStatus() { return status; }
public Date getRaisedOn() { return raisedOn; }
}
....
alt.add(new Alert("Disk Almost Full", "Warning"));
alt.add(new Alert("Disk Full", "Severe"));
...
...
<table>
<tr><th>Description</th><th>Status</th><th>Raised</th></td>
<c:forEach var="alert" items="${alt}">
<tr>
<td><c:out value="${alert.description}"/></td>
<td><c:out value="${alert.status}"/></td>
<td><c:out value="${alert.raisedOn}"/></td>
</tr>
</c:forEach>
</table>
Perché ti aspetti che restituisca 3
quando hai add
editato solo un elemento in List
?
ArrayList contiene SOLO un elemento Alert (l'elemento Alert contiene tre allarmi Stringhe.
Cambia il tuo JSP in:
<%
ArrayList<Alert> a = AlertGenerator.getAlert();
pageContext.setAttribute("alt", a);
%>
<p>You have <%=alt.size()%> Active Alert(s)</p>
<ul>
<c:forEach var="alert" items="${alt}" varStatus="status" >
<li><a href="#" class="linkthree">${alert.alert}</a></li>
</c:forEach>
</ul>
Cambia AlertGenerator.java in:
package com.cg.mock;
import java.util.ArrayList;
public class AlertGenerator {
public static ArrayList<Alert> getAlert() {
ArrayList<Alert> alt = new ArrayList<Alert>();
alt.add(new Alert("alert2"));
alt.add(new Alert("alert2"));
alt.add(new Alert("alert3"));
return alt;
}
}
Cambia Alert.java in:
package com.cg.mock;
public class Alert {
String alert;
public Alert(String alert) {
this.alert = alert;
}
public String getAlert() {
return alert;
}
public void setAlert(String alert) {
this.alert = alert;
}
}