这个方法有什么问题-（在Jlist中添加元素）？

Question

我目前正在创建一个购物车模拟程序。主GUI包含两个列表，一个是产品列表或库存。（存储在. dat文件中的产品在启动时自动加载）另一个是空白的，是对我的购物篮进行建模。这个想法是能够将我的库存中的物品扫描到结账篮中。当这发生时，我想要一个我创建的文本字段，以动态更新篮子中所有物品的成本。

下面是我的扫描按钮的方法，它应该执行上述操作：

 public void actionPerformed(ActionEvent evt) {
      //Get the newly added list values.
      JList list = productList.getSelectedValuesList();
      double totalAddedValue = 0.0;
      double oldCartValue    = 0.0;

  //Iterate to get the price of the new items.
  for (int i = 0; i < list.getModel().getSize(); i++) {
    CartItem item = (CartItem) list.getModel().getElementAt(i);
    totalAddedValue += Double.ParseDouble(item.getPrice());
  }

  //Set total price value as an addition to cart total field.

  //cartTotalField must be accessible here.
  string cartFieldText = cartTotalField.getText();

  //Check that cartTextField already contains a value.
  if(cartTextField != null && !cartTextField.isEmpty())
  {
    oldCartValue = Double.parseDouble(cartFieldText);
  }

  cartTotalField.setText(String.valueOf(oldCartValue  + totalAddedValue));
  checkoutBasket.addElement(list);
}

然而，目前扫描该项目会将其添加到列表中，但总的来说会打印奇怪的结果。（每个项目增加5.5，而不考虑其实际值**）。它还将在项目名称下打印一行，例如javax。摆动JList[，0,0344x326，布局=java.awt.BorderLa….

下面是我的CartItem和ItemList的类，如果它们可能会有所帮助的话。谢谢你。

-购物车项目

import java.io.*;

public class CartItem implements Serializable {

    private String barcodeNo;
    private String itemName;
    private String price;

    public CartItem() {

    }

    public CartItem (String barno, String in, String cost) {
            barcodeNo = barno;
            itemName = in;
            price = cost;
    }

    public String getBarcodeNo(){
        return barcodeNo;
    }

    public String getItemName(){
        return itemName;
    }

    public void setitemName(String itemName){
            this.itemName = itemName;
        }

    public String getPrice(){
        return price;
    }


    public String toString(){
        return barcodeNo + ": " + itemName + ", " + price;
    }

    public Object getID() {
        // TODO Auto-generated method stub
        return null;
    }
}

-项目列表

import javax.swing.DefaultListModel;

public class ItemList extends DefaultListModel {
    public ItemList(){
        super();
    }
public void addCartItem(String barcodeNo, String itemName, String price){
    super.addElement(new CartItem(barcodeNo, itemName, price));
}

public CartItem findItemByName(String name){
    CartItem temp;
    int indexLocation = -1;
    for (int i = 0; i < super.size(); i++) {
        temp = (CartItem)super.elementAt(i);
        if (temp.getItemName().equals(name)){
            indexLocation = i;
            break;
        }
    }

    if (indexLocation == -1) {
        return null;
    } else {
        return (CartItem)super.elementAt(indexLocation);
    }
}

public CartItem findItemByBarcode(String id){
    CartItem temp;
    int indexLocation = -1;
    for (int i = 0; i < super.size(); i++) {
        temp = (CartItem)super.elementAt(i);
        if (temp.getBarcodeNo().equals(id)){
            indexLocation = i;
            break;
        }
    }

    if (indexLocation == -1) {
        return null;
    } else {
        return (CartItem)super.elementAt(indexLocation);
    }        
}

public void removeItem(String id){
    CartItem empToGo = this.findItemByBarcode(id);
    super.removeElement(empToGo);
}

}

Answer 1

您将JList本身添加到签出篮中，这没有意义：

checkoutBasket.addElement(list);

这是javax。摆动JList[，0,0344x326，layout=java.awt.BorderLa…显示有东西正在显示JList的表示形式，很可能是您的结帐篮。

关于

每个项目增加5.5，而不考虑其实际值

我认为您当前的代码没有显示发生这种情况的原因，您可能希望创建并发布mcve。

其他想法：

• 不要使用字符串来表示价格，而是考虑一个数字字段，如果您想精确计算货币，可以使用BigDecimal
• 你看起来把你的观点和你的模型混合得太多了。尽量将它们分开，也就是说，您应该有更多的非GUI类，包括一个表示购物篮的类和任何其他名词