Forum:

EL algorithm help

Ranch Hand

Posts: 751

posted 19 years ago

Number of slices to send:

Optional 'thank-you' note:

Send

Hi guys, I need to forumulate my algorithm right... I can't seem to think of a better one and the layout of my page gets crazy. I have to do things as follows...

1. Iterate through a list
2. For every record in the list, display information.
3. Only 4 records per row...

I would be easy if I only had to display the records downward.. But I have to make another table row after every 4th record... My tables keep on getting scrammbled, so maybe... If you guys could help... Thank you!

Here's how I do the iteration...

<table border="5" width ="600" align="center">
 <tr>
  <td valign="top">
 
<table width="100%"  border="1" cellspacing="0" cellpadding="0">
      
     <c:forEach var="applicant" items="${searchResults.recordsOnPage}" varStatus="status">
      
      <c:if test="${ ( (status.index % 5) == 0 && (status.index) != 0) }"> 
       <tr>
      </c:if>        
         
      <c:if test="${ ( ( (status.index % 2) == 0) || ( (status.index % 3) == 0) || ( (status.index % 4) == 0) || !( (status.index % 5) == 0)  ) }"> 
        <td>
         
         <table align="center">
          <tr>
           <td><html:img page="/actions/loadimage.do?pictureId=${applicant.picture.pictureId}&pictureType=${applicant.picture.pictureType}"/></td>
          </tr>
          <tr>
           <td>Applicant ID: ${applicant.applicationInfo.applicantId}</td>
          </tr> 
          <tr>
           <td>Date Applied: ${applicant.applicationInfo.dateApplied}</td>
          </tr>
          <tr>
           <td>Availability: ${applicant.applicationInfo.applicationStatus}</td>
          </tr>
          <tr>
           <td><html:link page="/actions/viewapplicant.do?applicantId=${applicant.applicationInfo.applicantId}">View User</html:link></td>
          </tr>              
         </table>
         
        </td>
      </c:if>
 
      <c:if test="${ ( (status.index % 5) == 0 && (status.index) != 0) }">         
       </tr>
      </c:if>
        
     
      </c:forEach> 
        
 
    </table>
 
  </td>
 </tr>
</table>

[ June 19, 2006: Message edited by: Bear Bibeault ]

SCJP 1.5

Carl Trusiak

Sheriff

Posts: 3341

posted 19 years ago

Number of slices to send:

Optional 'thank-you' note:

Send

If you look at the html generated, it'll give you a big clue. I'm estamating based on a read through that your <tr> and </tr> are in the wrong order.

Remove the second c:if and change the first one to

You can still have a minor issue with the last row but that should be hard to solve.

I Hope This Helps
Carl Trusiak, SCJP2, SCWCD

Timothy Sam

Ranch Hand

Posts: 751

posted 19 years ago

Number of slices to send:

Optional 'thank-you' note:

Send

<c:if test="${ ( (status.index % 5) == 0 && (status.index) != 0) }">
</tr>
<tr>
</c:if>

Thanks to your suggestion but...

which actually means the outside if statement will have to be executed first ... Means I have to create a table row first before I could create a table data. But what I actually need is create 4 table data for every row...

I also tried c:when before, but to no luck... I could not figure out the solution...

SCJP 1.5

Christophe Verré

Sheriff

Posts: 14691

I like...

posted 19 years ago

Number of slices to send:

Optional 'thank-you' note:

Send

Make it look simpler

[My Blog]
All roads lead to JavaRanch

Timothy Sam

Ranch Hand

Posts: 751

posted 19 years ago

Number of slices to send:

Optional 'thank-you' note:

Send

Hi, I was able to solve the problem by putting it into paper first... I reversed the way I print the table rows...

SCJP 1.5

Richard Green

Ranch Hand

Posts: 536

posted 19 years ago

Number of slices to send:

Optional 'thank-you' note:

Send

nested tables are BAD!!!

why dont you switch to the DIV layout? http://www.snook.ca/archives/000177.php
[ June 19, 2006: Message edited by: Lynette Dawson ]

MCSD, SCJP, SCWCD, SCBCD, SCJD (in progress - URLybird 1.2.1)

With a little knowledge, a cast iron skillet is non-stick and lasts a lifetime.