我正在继承一个代码库,该代码库主要是JSP scriplets,很快就会变得不那么可读,例如这里的分页链接的片段:
Visa:
<a class="link" href="javascript:doSubmit('MenyNavigation', 'REW_<%= thisPage %>')" >
<%
if(headPointer - nrOfHitsOnPage > 0){
out.print(" << ");
if(tailPointer - nrOfHitsOnPage <= 0 ){
out.print(1);
} else {
out.print(tailPointer - nrOfHitsOnPage);
}
out.print("-");
out.print(tailPointer);
} %>
</a>
<% if(resultListSize > 0 ) { %>
<%=(tailPointer + 1)%> - <%= headPointer %>
<%}%>
<a class="link" href="javascript:doSubmit('MenyNavigation', 'FWD_<%= thisPage %>')" >
<%
if(headPointer < resultListSize){
out.print(1 + headPointer);
out.print("-");
if(headPointer + nrOfHitsOnPage > resultListSize){
out.print(resultListSize);
} else {
out.print(nrOfHitsOnPage + headPointer);
}
out.print(" >> ");
} %>
</a>
当这个代码显示文本“显示1-10 of 20”时,每页有10个元素,总共20个元素,同样适用于其他设置,但我认为这个解决方案不是非常可重复使用。代码已经在我们项目的一个地方使用了,我们决定如果我们有一个问题的解决方案,如果它们出现在不同的地方,我们将使用相同的解决方案,现在我必须为&gt;做分页。 5个具有搜索功能的弹出窗口(!),我想知道是否使用此代码,制作解析解决方案或从头开始构建。
控制器Java类也没有那么高的凝聚力:
public class MenyNavigationActionHandler extends PandoraActionHandler {
private ArendeProcessSessionData sessionData;
private final static Logger logger = Logger
.getLogger(MenyNavigationActionHandler.class);
protected IFormData getFormData() {
return null;
}
protected IFormPattern getPattern() {
return EmptyFormPattern.getInstance();
}
protected void performAction(ISessionHandler sessionHandler, Action action) {
String returnPage = null;
try {
sessionData = (ArendeProcessSessionData) sessionHandler.getSessionData();
if(sessionData.returnToUnsavedData(sessionHandler, action)){
return;
}
if (sessionData.getMenuManager() == null) {
MenuManager menuManager = new MenuManager();
sessionData.setMenuManager(menuManager);
}
// Om man bläddar i träfflista på formell kontrollista
if (action.getActionCommand().endsWith(ArendeProcessLocator.getFormellKontrollLista())) {
FormellKontrollInfo fInfo = sessionData.getFormellKontrollInfo();
FormellKontrollSearchAdmin fSearch = fInfo.getFormellKontrollSearchAdmin();
if(fInfo != null) {
if (action.getActionCommand().startsWith("FWD")) {
fSearch.setDirection(PandoraConstants.FWD_DIRECTION);
} else if (action.getActionCommand().startsWith("REW")) {
fSearch.setDirection(PandoraConstants.REW_DIRECTION);
} else {
fSearch.setDirection(PandoraConstants.NO_DIRECTION);
}
}
returnPage = ArendeProcessLocator.getFormellKontrollLista();
action.setReturnPage(returnPage);
} else if (action.getActionCommand().equalsIgnoreCase(
ArendeProcessLabelConstants.SUB_MENU_VISA_REGISTRERADE)) {
sessionData.setNyinkomnaLista(null);
MenuManager menuManager = sessionData.getMenuManager();
returnPage = menuManager.getLatestDestination(action.getActionTarget());
if (returnPage == null) {
returnPage = action.getCurrPage();
}
action.setReturnPage(returnPage);
} else {
// Standard hantering av menyval. Target från Navigation-objekt hämtas från action.getActionModifier
MenuManager menuManager = sessionData.getMenuManager();
if(action.getActionTarget() != null && !action.getActionTarget().trim().equals("")){
returnPage = menuManager.getLatestDestination(action.getActionTarget());
}else{
returnPage = menuManager.getLatestDestination(sessionData.getLatestActionTarget());
}
if (returnPage == null) {
returnPage = action.getCurrPage();
}
action.setReturnPage(returnPage);
}
if(action.getCurrPage().endsWith(ArendeProcessLocator.getNyregistrering())){
// Nytt önskemål, Kom ihåg innehållet på registrera sidan när vi går till listan 2010-12-02
sessionData.getNyregistrering().setKundReferens(sessionHandler.getRequestParameter(PandoraFieldConstants.FIELD_KUNDREFERENS));
sessionData.getNyregistrering().setSokEfternamn(sessionHandler.getRequestParameter(PandoraFieldConstants.FIELD_SOKEFTERNAMN));
sessionData.getNyregistrering().setSokFornamn(sessionHandler.getRequestParameter(PandoraFieldConstants.FIELD_SOKFORNAMN));
sessionData.getNyregistrering().setForetag(sessionHandler.getRequestParameter(PandoraFieldConstants.FIELD_FORETAG));
sessionData.getNyregistrering().setSokandeLandKod(sessionHandler.getRequestParameter(PandoraFieldConstants.FIELD_SELECT_SOKANDELAND));
sessionData.getNyregistrering().setAnkomstDatum(sessionHandler.getRequestParameter(PandoraFieldConstants.FIELD_ANKOMSTDAG));
sessionData.getNyregistrering().setOriginal("original".equals(sessionHandler.getRequestParameter(PandoraFieldConstants.FIELD_HANDLINGSFORM))? true : false);
sessionData.getNyregistrering().setArendeNummer(sessionHandler.getRequestParameter(PandoraFieldConstants.FIELD_ARENDENUMMER));
//printer ??
}
if(!action.getCurrPage().endsWith(ArendeProcessLocator.getNyinkomnaLista2()) && action.getReturnPage().endsWith((ArendeProcessLocator.getNyregistrering()))){
// Rensa minnet av registrera sidan om det inte är listan med registrerade man kommer ifrån
sessionData.getNyregistrering().setKundReferens("");
sessionData.getNyregistrering().setSokEfternamn("");
sessionData.getNyregistrering().setSokFornamn("");
sessionData.getNyregistrering().setForetag("");
sessionData.getNyregistrering().setSokandeLandKod("");
sessionData.getNyregistrering().setAnkomstDatum("");
sessionData.getNyregistrering().setOriginal(true);
sessionData.getNyregistrering().setArendeNummer("");
}
if(action.getActionCommand().equals(ArendeProcessLocator.getNyinkomnaLista())){
sessionData.setNyinkomnaLista(null);
}
} catch (Exception e) {
logger.error("MenyNavigationActionHandler: performAction() ", e);
}
}
}
我可以访问我的同事编写代码并可以让他指导我应该拥有哪些文件,这样分页就可以了,但我想要一个干净而精益的解决方案,没有任何划线,这将是很多工作开始这是从头开始而不是使用scriplets。
你觉得我应该怎么做?只需使用上面的scriplets,让它们适用于我的用例,然后尽可能重构JSTL标签和EL?还有其他任何评论如何重用Java / JSP代码进行分页?我们的框架是相当特殊和自定义的,我们不使用struts或spring或jsf,而是我们自己的框架,因为它依赖于会话数据而使事情变得复杂。任何答案或评论都表示赞赏。
答案 0 :(得分:1)
我认为这取决于这些代码的数量,以及您是否有时间花在使其更易于维护。我不认为除了你(和你的老板)之外的任何人都可以回答这个问题。
关于如何实现此类事情的问题,您可以在标准JSTL标记库之间进行选择,也可以实现自己的标记库或组合。如果将taglib与EL结合使用,您可以做得更多。并且会话属性不应该是特定问题。
虽然有一条建议。 JSTL和JSP EL越来越具有表现力,因此升级到实现最新版Servlet规范的servlet容器是值得的。