2015-05-09 59 views
4

我收到用戶的崩潰報告,似乎是不可能的。 stacktrace表明一個對象爲null,並且他得到了一個nullpointerException。在完全創建對象之前可以調用對象方法嗎?

(這裏是行,如果你想看到)

public class City extends Unit { 
    private ArrayList<SolderType> Queue = new ArrayList<SolderType>(); 

    public float getPrecentCompleted() 
    { 
     if(Queue.isEmpty()) 
     { 
      return 0f; 
     } 
     //More code that is not relevent 
    } 
} 

它不會留下太大的解釋隊列可能是空,但隊列是在我的代碼,一個地方只能創建,這是在構造函數中。所以我不明白它是如何可以爲空。該對象在多於一個線程上共享,並且始終創建新對象。但隊列點只能在創建對象時設置。所以我不明白這是可能的。一個線程可以調用一個對象方法,而另一個線程可以創建對象,但沒有完成?

編輯增加了一些可能與問題相關的代碼。

+1

是的,這是可能的,除非你添加了同步來防止它。 –

+0

也許吧。但也許它應該是最終的或不穩定的。而且由於它只在創作時分配,所以應該是最終的。 –

+2

很高興看到構造函數體以及完整的異常堆棧跟蹤。 –

回答

2

該評論給出了正確的答案:是的,這是可能的,除非你添加了同步來阻止它。執行的操作只有至極限定兩個線程之間的順序如下(見https://docs.oracle.com/javase/specs/jls/se7/html/jls-17.html#jls-17.4.4):

  1. 上監視器M的解鎖動作同步-與所有後續鎖米 操作(其中,「隨後的」根據定義 同步順序)。
  2. 寫入易失性變量v(§8.3.1。4)與任何線程(其中「後續」根據同步順序被定義爲 )的後續v讀取同步。
  3. 啓動一個線程的動作同步,與 的第一項行動,開始線程。
  4. 默認值(零,false或null)到每個 變量的寫入同步,在每個線程的第一個動作。雖然在分配包含變量的對象之前向變量寫入默認值似乎有點奇怪,但從概念上講,每個對象都是在程序開始時以其默認初始化值創建的。
  5. 在一個線程T1的最後動作同步-與 任何行動,其檢測T1已經終止另一個線程T2。
  6. 如果線程T1中斷線程T2,由T1 中斷同步-與任何點處的任何其他線程(包括T2) 確定T2已經被中斷(由具有 InterruptedException的拋出或通過調用Thread.interrupted或 Thread.isInterrupted)。

沒有這樣的操作,你會看到其他線程中的對象處於未定義狀態。

上的方式來解決空指針異常是使用最後一個字段(見https://docs.oracle.com/javase/specs/jls/se7/html/jls-17.html#jls-17.5)。但是你也必須同步ArrayList。我建議使用java.util.concurrent包中的一個隊列。

2

這種情況的最常見的原因是從調用父類的構造一個覆蓋的方法是這樣的:

import java.util.ArrayList; 


public class NPEInheritance { 
    static class Parent { 
     Parent() { 
      validate(); 
     } 

     void validate() {} 
    } 

    static class Child extends Parent { 
     private ArrayList<Object> Queue; 

     Child() { 
      Queue = new ArrayList<>(); 
     } 

     @Override 
     void validate() { 
      if(Queue.isEmpty()) { 
       System.out.println("Queue is empty"); 
      } 
     } 
    } 

    public static void main(String[] args) { 
     new Child(); 
    } 
} 

當你運行這段代碼,你會看到這樣的「不可能NullPointerException」。正如你所看到的,這裏Parent構造函數調用在Child類中被覆蓋的方法,並且覆蓋方法使用未被初始化的字段,因爲Child構造函數仍未執行。

+0

感謝您的提示,但這不是問題,因爲它是在聲明的位置創建的,並且包含在所有構造函數中。 – Frozendragon

0

我認爲當你創建子對象時,它首先要做的就是初始化父對象(因爲它的擴展),當你的父對象試圖驗證它的對象時,你會得到NullPointer。 我覺得你需要把你的Queue對象放入父類,就像這樣。

import java.util.ArrayList; 

public class NPEInheritance { 
    static class Parent { 

     protected ArrayList<Object> Queue; 

     Parent() { 
      Queue = new ArrayList<>(); 
      validate(); 
     } 

     void validate() {} 
    } 

    static class Child extends Parent { 

     Child() { 
     } 

     @Override 
     void validate() { 
      if(Queue.isEmpty()) { 
       System.out.println("Queue is empty"); 
      } 
     } 
    } 

    public static void main(String[] args) { 
     new Child(); 
    } 
} 
0

特殊情況需要提防出現時,A類的靜態初始化創建或引用B類(可能是靜態的)情況下,和B的靜態初始化對A.

類似的依賴性顯然,在這種情況下,兩個班都不能完全初始化。

Java通過允許其中一個類返回未初始化的值來靜靜地打破死鎖。因此,如果沒有警告,您可能會看到「不可能」的零或空值。由於這個依賴循環可能會經過其他幾個類,因此即使知道存在這種風險,追查也是非常痛苦的。

修復通常是將一些或所有涉及到的靜態對象重構爲第三個類,從而使依賴關係圖再次成爲樹。一旦你發現問題,簡單明瞭。當然,更好的做法是首先避免造成問題。

相關問題