在servlet的jsp文件中显示购物车中的项目

Question

我正在用Java构建一个电子商务简单解决方案，我陷入了一个关键时刻。 我无法显示正确的项目。 我正在使用具有doGet shoppingcart类，products类和viewcart.jsp文件的servlet。 为了在servlet之间传递项目，我使用了上下文的setAttribute。 这是我到目前为止已建立的文件。 谁能帮我解决这个问题？ 每次我按添加到汽车时，都会添加该元素，但看起来所有先前的元素都被我添加的最后一个元素覆盖。

package com.kd.ecommerce;

import java.io.IOException;
import java.sql.Connection;
import java.sql.ResultSet;
import java.sql.SQLException;
import java.sql.Statement;

import javax.servlet.ServletContext;
import javax.servlet.ServletException;
import javax.servlet.annotation.WebServlet;
import javax.servlet.http.HttpServlet;
import javax.servlet.http.HttpServletRequest;
import javax.servlet.http.HttpServletResponse;

/**
 * Servlet implementation class addtocart
 */
@WebServlet("/addtocart")
public class addtocart extends HttpServlet {
private static final long serialVersionUID = 1L;
private shoppingcart shop = new shoppingcart();

protected void doGet(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
    ServletContext thisContext = getServletContext(); 
    String name = request.getParameter("product");
    Connection cn = DBConnect.getInstance();
    try {
        Statement st = cn.createStatement();
        ResultSet rs = st.executeQuery("select * from inventory where name = '"+name+"' LIMIT 1");
        while(rs.next()){
            Products p = new Products(rs.getString(2),rs.getString(3),rs.getString(4),rs.getFloat(5));
            System.out.println(p.getName()+":"+ p.getPrice());
            shop.ins(p);
            thisContext.setAttribute("shop", shop.getIt());
            for(int i = 0; i< shop.getIt().size(); i++){
                System.out.println(shop.getIt().get(i)+":"+ shop.getIt().get(i).getPrice());
            }
            response.sendRedirect("inventory.jsp?addedto=success");
        }
    } catch (SQLException e) {
        e.printStackTrace();
    }

}


protected void doPost(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {

}

}

产品.java

package com.kd.ecommerce;

public class Products {
private static String name;
private static Float price;
private static String desc;
private static String img;

public Products(String n, String d, String i,Float p){
    name = n;
    price = p;
    desc = d;
    img = i;
}

public String getName(){
    return name;
}

public Float getPrice(){
    return price;
}

public String getImage(){
    return img;
}

public String getDesc(){
    return desc;
}
}

shoppingcart.java

package com.kd.ecommerce;

import java.util.ArrayList;

public class shoppingcart {
private static ArrayList<Products> items;

public shoppingcart(){
    items = new ArrayList<Products>();
}

public void ins(Products p){
    items.add(p);
}

public void remove(Products p){
    items.remove(p);
}

public int getSize(){
    return items.size();
}

public ArrayList<Products> getIt(){
    return items;
}
}

viewcart.jsp

<jsp:include page="menu.jsp"></jsp:include>
<% 
ServletContext sc = getServletConfig().getServletContext();
if(sc.getAttribute("shop") != null){
    ArrayList<Products> it = (ArrayList<Products>)sc.getAttribute("shop");
    out.println(it);
    for(int i = 0; i< it.size(); i++){
        out.println("<span class='price'>"+it.get(i).getPrice()+"</span>");
    }
}else{
    out.println("<span class='empty'>Shopping cart empty</span>");
}
%>

-编辑-

我使用会话的新viewcart.jsp

<% 
if(session.getAttribute("shop") != null){
    shoppingcart sh = (shoppingcart)session.getAttribute("shop");
    ArrayList<Products> pd = sh.getIt();
    for(int i = 0; i< pd.size(); i++){
        out.println("<span class='price'>"+pd.get(i).getPrice()+"</span>");
    }
}else{
    out.println("<span class='empty'>Shopping cart empty</span>");
}
%>

Answer 1

造成您问题的原因是，您已经在Servlet中将shoppingcart设置为实例变量。 这意味着您的Servlet处理的每个HTTP请求都共享同一个购物车 。 要解决此问题，请从类定义代码中移出以下行：

private shoppingcart shop = new shoppingcart();

并将其移至以下行之前的doGet方法中：

shop.ins(p);

然后， doGet方法将包含以下两行：

shoppingcart shop = new shoppingcart();
shop.ins(p);

您甚至可以考虑为shoppingcart类提供一个构造函数，该构造函数将Products对象作为参数并在构造函数中设置项目。 然后，您可以将这两行合并为一，如下所示：

shoppingcart shop = new shoppingcart(p);

在Servlet中将对象定义为实例变量是一个常见的错误。 只要记住，对servlet的每个HTTP调用都将共享实例变量。 如果要存储特定用户的信息，请考虑将其放入会话中。

- - 编辑 - -

事实证明，我实际上没有回答这个问题。 为了使添加的项目保持连续运行，您必须将这些项目保存到一个不会被任何人共享的位置-只有特定用户的数据将保存到该位置。 会议是专门为此目的设计的。 您可以在doGet方法中使用以下会话：

HttpSession session = request.getSession(true);
shoppingcart shop = (shoppingcart)session.getValue("shop");
if(shop == null) {
    shop = new shoppingcart();
    session.putValue("shop", shop);
}

shop.ins(p);

从会话中获取值的行将返回用键“ shop”存储的所有对象。 如果这是用户添加的第一项，则尚未保存任何购物车，因此shop将为null 。 这就是为什么我们要进行空检查； 创建购物车，然后将其与“商店”键相关联，以便用户下次添加商品时，购物车将与“商店”键相关联。

一旦我们有了一个与该特定用户的“商店”键相关联的购物车 ，你应该看到正在向同车对象的项目，如多次提交制成。