1

我正在为一个类做一个 Java 应用程序,需要帮助解决运行应用程序后出现的错误。

该应用程序应该允许用户将商品添加到购物车(商品名称、价格、数量)。用户描述的商品会被添加到购物车数组中,添加成功后会打印出来。

购物车最初的最大容量为 5 件商品,但如果购物车大小 > 5,则会通过 increaseSize 方法增加 + 3。

如下所述,我相信 addToCart 方法不会将用户描述的项目添加到购物车中。这就是为什么它在 toString 方法中是空值的原因。

所以,我基本上需要 addToCart 方法的帮助。如果我错了,请纠正我。谢谢你。

运行应用程序的输出如下:

run:
Enter the name of the item: a
Enter the unit price: 2
Enter the quantity: 2
Exception in thread "main" java.lang.NullPointerException
    at shop.ShoppingCart.toString(ShoppingCart.java:62)
    at java.lang.String.valueOf(String.java:2854)
    at java.io.PrintStream.println(PrintStream.java:821)
    at shop.Shop.main(Shop.java:49)
Java Result: 1

下面是 ShoppingCart 类。

toString 方法不应该产生任何错误,但我认为问题出在 addToCart 方法中。

// **********************************************************************
//   ShoppingCart.java
//
//   Represents a shopping cart as an array of items
// **********************************************************************
import java.text.NumberFormat;

public class ShoppingCart
{

    private Item[] cart;
    private int itemCount;      // total number of items in the cart
    private double totalPrice;  // total price of items in the cart
    private int capacity;       // current cart capacity

    // -----------------------------------------------------------
    //  Creates an empty shopping cart with a capacity of 5 items.
    // -----------------------------------------------------------
    public ShoppingCart()
    {

      capacity = 5;
      cart = new Item[capacity];
      itemCount = 0;
      totalPrice = 0.0;
    }

    // -------------------------------------------------------
    //  Adds an item to the shopping cart.
    // -------------------------------------------------------
    public void addToCart(String itemName, double price, int quantity)
    { 

        Item temp = new Item(itemName, price, quantity);
        totalPrice += (price * quantity);
        itemCount += quantity;
        cart[itemCount] = temp;
        if(itemCount==capacity)
        {
            increaseSize();
        }
    }

    // -------------------------------------------------------
    //  Returns the contents of the cart together with
    //  summary information.
    // -------------------------------------------------------
    public String toString()
    {
      NumberFormat fmt = NumberFormat.getCurrencyInstance();

      String contents = "\nShopping Cart\n";
      contents += "\nItem\t\tUnit Price\tQuantity\tTotal\n";

      for (int i = 0; i < itemCount; i++)
          contents += cart[i].toString() + "\n";

      contents += "\nTotal Price: " + fmt.format(totalPrice);
      contents += "\n";

      return contents;
    }

    // ---------------------------------------------------------
    //  Increases the capacity of the shopping cart by 3
    // ---------------------------------------------------------
    private void increaseSize()
    {
        Item[] temp = new Item[capacity+3];
        for(int i=0; i < capacity; i++)
        {
            temp[i] = cart[i];
        }
        cart = temp; 
        temp = null;
        capacity = cart.length;
    }
}

下面是店主。

ArrayList 目前未使用,但稍后将在更正 toString 错误后使用。

package shop;

// ***************************************************************
//   Shop.java
//
//   Uses the Item class to create items and add them to a shopping
//   cart stored in an ArrayList.
// ***************************************************************

import java.util.ArrayList;
import java.util.Scanner;

public class Shop
{
    public static void main (String[] args)
    {
      ArrayList<Item> cart = new ArrayList<Item>();

      Item item;
      String itemName;
      double itemPrice;
      int quantity;

      Scanner scan = new Scanner(System.in);

      String keepShopping = "y";
      ShoppingCart cart1 = new ShoppingCart();
      do
          {
            System.out.print ("Enter the name of the item: ");
            itemName = scan.next();

            System.out.print ("Enter the unit price: ");
            itemPrice = scan.nextDouble();

            System.out.print ("Enter the quantity: ");
            quantity = scan.nextInt();

            // *** create a new item and add it to the cart
            cart1.addToCart(itemName, itemPrice, quantity);



            // *** print the contents of the cart object using println
            System.out.println(cart1);

            System.out.print ("Continue shopping (y/n)? ");
            keepShopping = scan.next();
          }
      while (keepShopping.equals("y"));

    }
}

以下是物品类别:

package shop;

// ***************************************************************
//   Item.java
//
//   Represents an item in a shopping cart.
// ***************************************************************

import java.text.NumberFormat;

public class Item
{
    private String name;
    private double price;
    private int quantity;

    // -------------------------------------------------------
    //  Create a new item with the given attributes.
    // -------------------------------------------------------
    public Item (String itemName, double itemPrice, int numPurchased)
    {
      name = itemName;
      price = itemPrice;
      quantity = numPurchased;
    }

    // -------------------------------------------------------
    //   Return a string with the information about the item
    // -------------------------------------------------------
    public String toString ()
    {
      NumberFormat fmt = NumberFormat.getCurrencyInstance();

      return (name + "\t" + fmt.format(price) + "\t" + quantity + "\t"
            + fmt.format(price*quantity));
    }

    // -------------------------------------------------
    //   Returns the unit price of the item
    // -------------------------------------------------
    public double getPrice()
    {
      return price;
    }

    // -------------------------------------------------
    //   Returns the name of the item
    // -------------------------------------------------
    public String getName()
    {
      return name;
    }

    // -------------------------------------------------
    //   Returns the quantity of the item
    // -------------------------------------------------
    public int getQuantity()
    {
      return quantity;
    }
} 
4

3 回答 3

2

我相信问题是由以下两行引起的addToCart()

itemCount += quantity;
cart[itemCount] = temp;

除非quantityis 0,否则这意味着cart[0]永远不会包含任何内容。事实上,无论 的值是quantity多少,您都会留下很多空白cart(例如,如果quantity为 2,则为cart[0]cart[1])。

由于 anItem已经包含数量,我相信您的意思是这样做:

cart[itemCount] = temp;
itemCount += 1;

这样第一个项目将被放入cart[0],第二个放入cart[1],依此类推。

另一个小建议:如果您将任何对象与字符串连接,则不需要调用toString()该对象。Java 将自动调用String.valueOf()该对象,这避免了 a ,NullPointerException因为它返回的是 String "null"。在这种情况下,它可能会帮助您调试问题,因为您会注意到输出中出现了空字符串。您必须将代码更改为以下内容:

for (int i = 0; i < itemCount; i++)
    contents += cart[i] + "\n";
于 2013-08-27T18:54:04.190 回答
1
public void addToCart(String itemName, double price, int quantity)
{ 

    Item temp = new Item(itemName, price, quantity);
    totalPrice += (price * quantity);
    itemCount += quantity;
    cart[itemCount] = temp;
    if(itemCount==capacity)
    {
        increaseSize();
    }
}

此代码已损坏。您尝试在索引itemCount处添加到购物车。这会将索引抛出边界异常。基本上,您将购物车的大小增加到晚。

这也会导致阵列中出现很多空位。您不会在您的 arra 的下一个位置添加您的下一个项目,但您会跳到前面的数量位置。这会导致数组中的某些值为null。这最有可能导致toString ()中的NullPointerExceütion

关于如何实现这个自我成长列表。您可能想看看 JDK 附带的类 ArrayList。

我还想指出其他几点:

  • 谷歌javaDoc并使用它而不是您自己的自定义注释
  • 用for -each-loop替换toString()中的for-loop
于 2013-08-27T19:09:33.393 回答
0

问题在于您的 addToCart 方法。如果您键入此代码:

ShoppingCart shopcart= new ShoppingCart();
shopcart.addToCart("foo", 3, 2);

购物车将具有以下属性:

totalPrice=6
itemCount = 2;
cart = { null, null, foo, null, null};

问题是 addToCart 只修改数组的“itemcount”元素,cart而不考虑添加的元素数量。此外,cart[0] 将永远保持为空。您应该替换这些行 itemCount += quantity; 购物车[itemCount] = 温度;

经过

for ( int i =0 ; i<quantity;i++)
{
     cart[itemCount+i] = temp;
} 
itemCount += quantity;
于 2013-08-27T18:58:12.677 回答