Fragmentの遷移ができません。

Question

初歩的な質問で申し訳ありませんが、Fragmentの遷移について困っているので助けてください。
Fragmentの遷移は、アプリ内の別のアクションではできていたので、
それと同じように作っていたつもりだったのですがなぜか失敗してしまいました。
具体的な解決法でなくてもよいのでアドバイス頂けると助かります。

やりたいこと

FragmentAでボタンをクリックしたとき、FragmentBに移動したい

発生している問題

下記のコードをFragmentAに記述したら、FragmentAが表示されるときにアプリが落ちてしまうようになった。（それまでは普通に表示できていた）

        //FragmentBへ移動
        fab.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {
                transaction.addToBackStack("");
		transaction.replace(R.id.container, FragmentB.newInstance());
		transaction.commit();
            }
        });

該当のソースコード

・Fragment A

public class FragmentA extends Fragment {
    private FloatingActionButton fab;
    private FragmentManager manager = getFragmentManager();
    private FragmentTransaction transaction = manager.beginTransaction();

    public static FragmentA newInstance(){
        FragmentA f = new FragmentA();
        Bundle args = new Bundle();
        f.setArguments(args);
        return f;
    }

    @Override
    public View onCreateView(LayoutInflater inflater, ViewGroup container,
                             Bundle savedInstanceState) {
        View view = inflater.inflate(R.layout.fragment_a, container, false);
        fab = view.findViewById(R.id.fab);
        return view;
    }

    @Override
    public void onStart() {
        super.onStart();

        //FragmentBへ移動
        fab.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {
                transaction.addToBackStack("");
                transaction.replace(R.id.container, FragmentB.newInstance());
                transaction.commit();
            }
        });
    }
}

・FragmentB

public class FragmentB extends Fragment {

    public static FragmentB newInstance() {
        FragmentB f = new FragmentB();
        Bundle args = new Bundle();
        f.setArguments(args);
        return f;
    }

    @Override
    public View onCreateView(LayoutInflater inflater, ViewGroup container,
                             Bundle savedInstanceState) {
        View view = inflater.inflate(R.layout.fragment_b, container, false);
        return view;
    }
}

試したこと

Activityに遷移の処理をするメソッドを作って、そのメソッドをFragmentAから呼び出すようにした。
結果、FragmentAは問題なく表示できるようになったが、今度はボタンをクリックした時点で落ちてしまうようになった。

//Fragmentを表示させているMainActivity

FragmentManager manager = getSupportFragmentManager();
FragmentTransaction transaction = manager.beginTransaction();

public void replaceFragmentB() {
    FragmentB fragment = FragmentB.newInstance();
    transaction.addToBackStack("");
    transaction.replace(R.id.container, fragment);
    transaction.commit();
}

//FragmentA

fab.setOnClickListener(new View.OnClickListener() {
    @Override
    public void onClick(View view) {
        MainActivity activity = (MainActivity) getActivity();
        activity.replaceFragmentB();
    }
});

Comments

[退会済みユーザー]

どういうエラーが出ますか？

[kashikoma]

ビルドでは何もエラーは出ず、実機でアプリを開いてからFragmentAを開くと落ちてしまいます。
ログのとり方がよく分からないのですが、ログを取った方がいいんでしょうか？

[退会済みユーザー]

まあ、Android Studioで見れますしねえ…
取ったほうがいい、と言われれば、それを当たり前にして頂かないと。

[kashikoma]

エラーが出なかったらなんとも言えないという感じでしょうか？

[jimbe]

何が起きているのか分からないのでは, デバッグも進まないと思います.
android studio から実機で実行させ, logcat 画面でログを見れないのでしょうか.

Answer 1

private FragmentManager manager = getFragmentManager();
private FragmentTransaction transaction = manager.beginTransaction();

この辺りは問題ありそうに思います.
これを消して

  FragmentB fragment = FragmentB.newInstance();
  FragmentTransaction transaction = getFragmentManager().beginTransaction(); //追加
  transaction.addToBackStack("");

とされては如何でしょうか.

Comments

[kashikoma]

返信が遅れてしまい申し訳ありませんでした。
教えて頂いたようにしてみたらちゃんと遷移できるようになりました。
privateつけてたのが良くなかったのでしょうか・・
あとgetFragmentManager().beginTransaction();が1行で書けるの知らなかったのでありがたいです。
ご回答いただきありがとうございました。

[jimbe]

> privateつけてたのが良くなかったのでしょうか
private ではなく, クラスのフィールドにしていたのが問題です.
getFragmentManager() のドキュメントに書いてあったように思いますが, このメソッドはフラグメントがアクティビティに設定されるまでは null を返します. フィールドはインスタンスの生成時に初期化されますから, その時点ではこのメソッドは null を返すことになります.
また, beginTransaction() は「あらかじめやっておく」ようなものではありません. フラグメントを操作する都度, beginTransaction ～ commt をセットで行うようにしてください.

> getFragmentManager().beginTransaction();が1行で書けるの知らなかった
getFragmentManager() が有効なオブジェクトを返せば, そのメソッドが使用できるのが普通です.
この場合は FragmentManager オブジェクト自体は不要ですのでこのように出来ます. また, もし有効なオブジェクトを返されない場合があればここで例外が発生しバグとなりますので, 一度変数に入れてチェックする必要があります.