как отобразить данные, полученные от дао в JSP - PullRequest
0 голосов
/ 07 марта 2011

в JSP

<table width="100%" id='table1'  border="0" cellspacing="2" cellpadding="2">
    <tr class="tab-highlighted-2">
        <td class="tab-highlighted-2" width="10%">
          <div align="left">Project ID</div>
        </td>
        <td class="tab-highlighted-2" width="10%">
          <div align="left">Project Name</div>
        </td>
        <td class="tab-highlighted-2" width="20%">
          <div align="left">Cost Center</div>
        </td>
        <td class="tab-highlighted-2" width="20%">
          <div align="left">Manager</div>
        </td>
    </tr>
  <tr class="bg-row1">
  <c:forEach items="${resultList}" var="resultList"> 
        <td class="td-highlighted-2">
         <div align="left"><a href="UpdateProject.html">${resultList.Projid}</a></div>
        </td>
        <td class="td-highlighted-2">
          <div align="left">${resultList.Projname}</div>
        </td>
        <td class="td-highlighted-2">
          <div align="left">${resultList.Cost}</div>
        </td>
        <td class="td-highlighted-2">
          <div align="left">${resultList.Manager}</div>
        </td>  
    </tr>  
    </table> 

в дао

public final class SearchProjDAO   
{ 
    private static InitialContext context;  
    String CLASS_NAME="DBConnectionFactory";  

    public ArrayList  submitProjectDetails(SearchProjVO searchprojVO)   
    {  
        ArrayList ar = new ArrayList();
        String methodname="createConnection";
        Connection conn  = null;
        PreparedStatement psmt;
        try {
            System.out.println("in DAO");
            System.out.println("searchprojVO id=="+searchprojVO.getProjid());

            conn = DBConnection.getJNDIConnection();
            ResultSet rs = null;
            String query="select * from CR_EMPLOYEE_DETAILS";if(searchprojVO.getProjid()!=null || searchprojVO.getProjname()!=null || searchprojVO.getManager()!=null)
            query=query+" where ";
        if(searchprojVO.getProjid()!=null)
            query=query+" PROJ_ID="+searchprojVO.getProjid();
        if(searchprojVO.getProjname()!=null)
            query=query+"PROJ_NAME="+searchprojVO.getProjname();
        if(searchprojVO.getCost()!=null);
            query=query+"PROJ_COST="+searchprojVO.getCost();
        if(searchprojVO.getManager()!=null)
            query=query+"PROJ_MANAGER="+searchprojVO.getManager();

            psmt= conn.prepareStatement(query);
            rs=psmt.executeQuery();

            while(rs.next())
            {
                SearchProjVO projVO = new SearchProjVO();
                projVO.setProjid(rs.getString("PROJ_ID"));
                projVO.setManager(rs.getString("PROJ_NAME"));
                projVO.setProjname(rs.getString("PROJ_COST"));
                projVO.setManager(rs.getString("PROJ_MANAGER"));
             ar.add(projVO);
                                }

            System.out.println("conn==="+conn); 
        } catch (Exception e) {
            e.printStackTrace(System.err);
        } 

        return ar;
    }

}

1 Ответ

6 голосов
/ 07 марта 2011

Я замечаю несколько ошибок:

Здесь

<c:forEach items="${resultList}" var="resultList"> 

Вы переопределяете значение списка значением элемента списка каждый раз.Не делай этого.Дайте var уникальное имя переменной.Имя сущности является самым простым выбором.

<c:forEach items="${resultList}" var="project"> 

Обратите внимание, что я лично также переименую ничего не говорящий resultList в более понятный projects.


А вот,

<tr class="bg-row1">
    <c:forEach items="${resultList}" var="project"> 

поток неверный.Вы должны напечатать новую строку внутри каждой петли.Поменяйте их местами.

<c:forEach items="${resultList}" var="project"> 
    <tr class="bg-row1">

И здесь

${resultList.Projid}
${resultList.Projname}
${resultList.Cost}
${resultList.Manager}

имена свойств должны начинаться со строчной буквы (и исправить имя элемента так же, как в var).

${project.projid}
${project.projname}
${project.cost}
${project.manager}

Обратите внимание, что я лично также избавлюсь от префикса proj в некоторых именах свойств.


И, наконец, вы забылизакрытие </c:forEach>.

    </tr>
</c:forEach>

Не связанный с конкретной проблемой, ваш код JDBC чувствителен к атакам с использованием SQL-инъекций и приводит к утечке ресурсов.Исправьте это также.

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...