空指針，對我沒有意義嗎？

Question

我目前正在一個程序和任何時候我打電話產品[1]沒有空指針錯誤，但當我打電話產品[0]或產品[2]我得到一個空指針錯誤。但是，我仍然得到2個不同的輸出，就像陣列中有[0]和1或1和2一樣。這裏是我的代碼

FileReader file = new FileReader(location); 
    BufferedReader reader = new BufferedReader(file); 

    int numberOfLines = readLines(); 
    String [] data = new String[numberOfLines]; 
    Products = new Product[numberOfLines]; 
    calc = new Calculator(); 

    int prod_count = 0; 
    for(int i = 0; i < numberOfLines; i++) 
    { 
     data = reader.readLine().split("(?<=\\d)\\s+|\\s+at\\s+"); 
     if(data[i].contains("input")) 
     { 
      continue; 
     } 
     Products[prod_count] = new Product(); 
     Products[prod_count].setName(data[1]); 
     System.out.println(Products[prod_count].getName()); 
     BigDecimal price = new BigDecimal(data[2]); 
     Products[prod_count].setPrice(price); 


     for(String dataSt : data) 
     { 

      if(dataSt.toLowerCase().contains("imported")) 
     { 
       Products[prod_count].setImported(true); 
     } 
      else{ 
       Products[prod_count].setImported(false); 
      } 

     } 



     calc.calculateTax(Products[prod_count]);  
     calc.calculateItemTotal(Products[prod_count]); 
     prod_count++; 

這是輸出：

imported box of chocolates 
1.50 
11.50 
imported bottle of perfume 
7.12 
54.62 

這版畫作品System.out.println(Products[1].getProductTotal());

這將成爲一個空指針System.out.println(Products[2].getProductTotal());

這也成爲一個空指針System.out.println(Products[0].getProductTotal());

Answer 1

您正在跳過包含「輸入」的行。

if(data[i].contains("input")) { 
    continue;   // Products[i] will be null 
} 

也許這將是更好地使products一個ArrayList，只有有意義的行添加到它。

products也應該以小寫字母開頭以遵循Java約定。類型以大寫字母開始，參數&變量以小寫字母開頭。並非所有的Java編碼規範都是完美的 - 但這個非常有用。

代碼的結構很好，但數組並不是一個非常靈活的類型來從程序邏輯構建（因爲長度必須預先確定，跳過要求您跟蹤索引，並且它不能跟蹤你建立它的大小）。

通常你應該建立List（ArrayList）。 Map（HashMap，LinkedHashMap，TreeMap）和Set（HashSet）也是有用的。

---

第二個錯誤：波希米亞說：在data[]你弄得所有行的列表的概念，並data[]正在分析從單線/分的標記。

「數據」通常是一個沒有意義的術語。使用有意義的術語/名稱&您的程序不太可能在其中存在錯誤。

您應該只使用tokens作爲行標記，而不是在需要之前在外部/之前聲明它，而不是嘗試按行索引它 - 因爲很簡單，應該絕對不需要。

for(int i = 0; i < numberOfLines; i++) { 
    // we shouldn't need data[] for all lines, and we weren't using it as such. 
    String line = reader.readLine(); 
    String[] tokens = line.split("(?<=\\d)\\s+|\\s+at\\s+"); 
    // 
    if (tokens[0].equals("input")) {  // unclear which you actually mean. 
    /* if (line.contains("input")) { */  
     continue; 
    } 

當您爲某個問題提供樣例輸入時，請將其編輯到問題的主體中，以使其可讀。把它放在評論中，不能正確閱讀的地方，只是在浪費試圖幫助你的人的時間。

Answer 2

錯誤警告：您將要覆蓋data：

String [] data = new String[numberOfLines]; 

然後在循環：

data = reader.readLine().split("(?<=\\d)\\s+|\\s+at\\s+"); 

因此，誰知道它是多麼大的 - 取決於分割的成功 - 但你的代碼依靠它是numberOfLines長。

Answer 3

您需要爲行號和新產品對象使用不同的索引。如果你有20行，但其中5個是「輸入」，那麼你只有15個新產品對象。

例如：

int prod_count = 0; 

for (int i = 0; i < numberOfLines; i++) 
{ 
     data = reader.readLine().split("(?<=\\d)\\s+|\\s+at\\s+"); 
     if (data[i].contains("input")) 
     { 
      continue; 
     } 
     Products[prod_count] = new Product(); 
     Products[prod_count].setName(data[1]); 
     // etc. 
     prod_count++; // last thing to do 
}