1

我从这段代码中得到一个空异常错误，我不确定是什么原因造成的。数组 itemcatalog 已填充 i = 0 到 8。我是 java 新手，因此将不胜感激。错误消息指向 while 语句的行。谢谢

public class ItemCatalog {

   private static ItemCatalog instance = new ItemCatalog(); 
   private Item itemCatalog[] = new Item[9];

   private ItemCatalog(){
   };

   public static synchronized ItemCatalog getInstance() {
   return instance;
   }

   public void populateCatalog()
   {  
      itemCatalog[0] = new Item("bb","Baked Beans",new BigDecimal("0.35"));
      itemCatalog[1] = new Item("cf","Cornflakes",new BigDecimal("1.00"));
      itemCatalog[2] = new Item("s0","Sugar",new BigDecimal("0.50"));
      itemCatalog[3] = new Item("tb","Tea Bags",new BigDecimal("1.15"));
      itemCatalog[4] = new Item("ic","Instant Coffee",new BigDecimal("2.50"));
      itemCatalog[5] = new Item("b0","Bread",new BigDecimal("0.50"));
      itemCatalog[6] = new Item("s0","Sausages",new BigDecimal("1.30"));
      itemCatalog[7] = new Item("e0","Eggs",new BigDecimal("0.75"));
      itemCatalog[8] = new Item("m0","Milk",new BigDecimal("0.65"));
   }

   public BigDecimal getPrice(String itemCode)
   {  

      int i = 0;
      while (!itemCode.equals(itemCatalog[i].getItemCode()))
      {
        i++;
      }
      BigDecimal itemPrice = itemCatalog[i].getItemprice();
   return itemPrice;
   }
}

我解决了这个问题。我在给出空异常错误的主类中填充目录。我改为在 jframe 中实例化它并且它可以工作。下面的代码解决了这个问题，但这是填充目录的最佳位置吗？

private void saleButtonActionPerformed(java.awt.event.ActionEvent evt) {                                           

        String itemCode = this.itemCodeinput.getText();
        int itemQuantity =Integer.parseInt(this.itemQuantityinput.getText());

        ItemCatalog catalog = ItemCatalog.getInstance();

        catalog.populateCatalog(); 
        BigDecimal price = catalog.getPrice(itemCode);
        itemCostoutput.setText(price.toString());
}
4

3 回答 3

2

如果您的 itemCode 与 itemCatalog 中的任何条目都不匹配，那么最终

  while (!itemCode.equals(itemCatalog[i].getItemCode()))
  {
      i++;
  }

将 i 增加到 11，在这种情况下 itemCatalog[11] 为空或超出范围。

如果添加，您应该使用 for 循环遍历 itemCatalog：

for (int i = 0; i < itemCatalog.length; i++) {
    if (itemCode.equals(itemCatalog[i].getItemCode()) {
        return (BigDecimal) itemCatalog[i].getItemprice();
    }
}

return null // you can change this from null to a flag 
            // value for not finding the item.
于 2012-10-17T20:13:24.760 回答
0

你如何结束你的循环？似乎循环将继续进行，直到 i 是 10。那么你将超过限制。

除非这是必须使用数组的 uni 分配，否则我还建议使用地图，而不是数组。这样，无论您的收藏有 100,000 个条目还是 10 个条目，您的查找都将是同一时间。

您还将降低 NPE 或 ArrayOutOfBounds 异常的风险

请参阅http://docs.oracle.com/javase/1.4.2/docs/api/java/util/HashMap.html

添加对象时，使用项目代码作为键。然后通过键查找。

使用地图的代价是增加了内存使用量。

于 2012-10-17T20:14:52.153 回答
0

从评论中，很明显设计不合理。

这是一个可能的解决方案：

public BigDecimal getPrice(String itemCode) {  
      for (int i=0; i<itemCatalog.length; i++) { // not going outside the array
          if (itemCatalog[i].getItemCode().equals(itemCode)) { // inversing the test to avoid npe if itemCode is null
             return itemCatalog[i].getItemprice();
          }
      }
      return null; // default value
}

这假设您的数组正确填充了具有 itemCode 的 itemCatalogs。

于 2012-10-17T20:15:02.203 回答