フラグメント内のfindViewByIdを宣言した後のjava.lang.NullpointerException

Question

findViewByIdを使用しようとしていますが、警告が表示されます。私はserveralがこの警告を解決するために使用されたさまざまな方法を見てきましたが、自分のコードに関して正しいかどうかわかりません。この文脈でこの警告を削除するのが正しい方法を知っていますか？

メソッドの呼び出しが 'findViewById' 'のjava.lang.NullPointerException'

を生成することがPage1Fragment.java

public class Page1Fragment extends android.support.v4.app.Fragment { 

    boolean squareState; 

    public Page1Fragment() { 

    } 

    @Override 
    public View onCreateView(LayoutInflater inflater, @Nullable ViewGroup container, @Nullable Bundle savedInstanceState) { 

     return inflater.inflate(R.layout.fragment_page1, container, false); 
    } 

    @Override 
    public void onResume(){ 
     super.onResume(); 
     loadPreferences(); 
     displaySettings(); 
    } 

    public void loadPreferences(){ 
     SharedPreferences pref = this.getActivity().getSharedPreferences("settings", AppCompatActivity.MODE_PRIVATE); 
     squareState = pref.getBoolean("square_state", true); 
    } 

    public void displaySettings() { 
     if (squareState) { 
      getView().findViewById(R.id.blue_square).setVisibility(View.VISIBLE); 
     } else { 
      getView().findViewById(R.id.blue_square).setVisibility(View.GONE); 
     } 
    } 
} 

Answer 1

私はあなたが糸くずの警告の話をしていると仮定します。

フラグメントが存在するライフサイクルのどの部分に応じて、まだonCreateViewメソッドが呼び出されていてもいなくてもかまいません。

フラグメントのビューがまだ作成されていない場合（onCreateViewがまだ呼び出されていない場合）、getView（）を呼び出すとnullが返されます。

したがって、あなたはとてもようにnullをチェックする必要があります：

View contentView = getView(); 
if (contentView != null) { 
    contentView.findViewById(R.id.blue_square).setVisibility(View.VISIBLE); 
} 

それは断片が活動に取り付けられており、そのビューが作成されたときを知ることは必ずしも容易ではありません。たとえば、AsyncTaskを起動してバックグラウンドスレッドのデータを取得してからUIを更新すると、ユーザはフラグメントからナビゲートしてしまい、getView（）はnullを返します。

フラグメントライフサイクルの詳細については、developer.android.comのFragmentのドキュメントをご覧ください。

Answer 2

onCreateViewの後でonDestroyViewの後にgetView（）メソッドを呼び出している場合は、これらの警告を無視できます。 このメソッドは、この2つのコールバックの外側にnullを返します。

繰り返し確認することを避けるためのきれいな方法は、rootViewをdislplaySettingsメソッドへの参照として渡すことです。

@Override 
public void onResume() { 
    displaySettings(getView()); // You're safe here! 
} 

フラグメントのライフサイクルdocsを参照してください：次のようにあなたがonResume内で使用するときは、なし得るでしょう

public void displaySettings(View rootView) { 
    if (squareState) { 
     rootView.findViewById(R.id.blue_square).setVisibility(View.VISIBLE); 
    } else { 
     rootView.findViewById(R.id.blue_square).setVisibility(View.GONE); 
    } 
} 

は文句を言います。

ボーナス

はフラグメントがであるライフサイクルのどの部分に応じて、いくつかのコードの重複

public void displaySettings(View rootView) { 
    rootView.findViewById(R.id.blue_square).setVisibility(squareState ? View.VISIBLE : View.GONE); 
}