I studied struts2 and never used struts 1, but I guess the nesting of the two forEach is not quite right. AdminName should be an attribute of the product. Should it be changed to ${product.adminName}? I'm not sure if it was changed like this, but the source is unknown, and the reason should be here. If not, look at how the return value of hql query is distributed and debug breakpoints.