嵌套<c:forEach>生成重复下载按钮问题求助

我来帮你分析问题根源，再给出针对性的修复方案：

问题核心原因 #

你的代码现在有两个关键逻辑错误：

1. 后端数据传递逻辑不对：你把所有费用条目对应的预览URL都塞进了同一个全局列表previewUrl里，前端每个表格行都会遍历这个完整列表，自然每行都会生成所有URL对应的按钮。
2. 前端循环逻辑错误：内层<c:forEach>遍历的是全局URL列表，而非当前行费用数据对应的单个URL。

修复步骤 #

第一步：调整Portlet后端代码

你需要把每个费用条目对应的预览URL，直接绑定到对应的expenseData对象上（如果你的expenseData类没有previewUrl字段，可以先添加这个字段和对应的setter方法；或者创建一个包含expenseData和previewUrl的DTO类）：

try {
    // 改用增强for循环，代码更简洁易读
    for (ExpenseData expData : expenseDataList) {
        long fileEntryId = expData.getFileEntryId();
        String previewURL = StringPool.BLANK;
        
        if (fileEntryId > 0) {
            FileEntry fileEntry = DLAppLocalServiceUtil.getFileEntry(fileEntryId);
            previewURL = DLUtil.getPreviewURL(fileEntry, fileEntry.getFileVersion(), themeDisplay, StringPool.BLANK);
        }
        // 把当前条目的预览URL绑定到对应expenseData对象
        expData.setPreviewUrl(previewURL);
        
        LOG.info("File Entries" + fileEntryId);
        LOG.info("Preview URl " + previewURL);
    }
    // 只需要传递绑定好URL的expenseDataList即可，无需单独传previewUrl列表
    renderRequest.setAttribute("expenseDataList", expenseDataList);
} catch (Exception e) {
    e.printStackTrace();
}

第二步：修改JSP前端代码

现在每个expData已经携带了自己的预览URL，不需要再用内层循环，直接读取当前行的URL生成单个按钮即可：

<c:forEach var="expData" items="${expenseDataList}"> 
    <tr> 
        <td> 
            <div class="custom-control custom-checkbox"> 
                <label>
                    <aui:input cssClass="custom-control-input expense select-all" type="checkbox" data-amount="${expData.expenseAmount}" data-expenseid="${expData.expenseId}" id="expenseCheckbox_${expData.expenseId}" name="expenseCheckbox" label="" /> 
                </label> 
            </div> 
        </td> 
        <!-- 省略其他列代码 -->
        <td> 
            <!-- 仅当previewUrl不为空时渲染按钮，避免无效空按钮 -->
            <c:if test="${not empty expData.previewUrl}">
                <aui:button 
                    icon="icon-download-alt" 
                    style="border:none; background-color: #1E47C2; color:white" 
                    data-previewurl="${expData.previewUrl}" 
                    cssClass="download" 
                    name="downloadButton" 
                    id="downloadButton_${expData.expenseId}" 
                />
            </c:if>
        </td> 
    </tr> 
</c:forEach>

额外优化说明 #

• 给复选框和按钮的id添加了${expData.expenseId}后缀，避免同一页面出现多个相同id的元素（这是HTML规范要求，相同id会导致JS选择元素出错）。
• 添加了<c:if>判断，只在有有效预览URL时才渲染按钮，避免出现无意义的空按钮。

这样修改后，每个表格行只会生成对应自己的一个下载按钮，不会再出现重复问题了。

内容的提问来源于stack exchange，提问作者Tirth Timaniya