fragmentにて戻るボタンを押すと表示されるはずが、何も表示されない。

Question

MainActivity→Fragment1→Fragment2
の順で表示されるアプリを作成しております。
戻るボタンを押すことで、逆順に戻るようにしたいのですが、
Fragment2で戻るボタンを押すとFragment1について何も表示されなくなってしまいます。
デバックしてみたところ、戻るボタンを押した際に、MainActivityのonBackPressedメソッドが呼ばれていることは確認できていますので、Fragment2→Fragment1に遷移はできていると思われます。そのため、戻ってきた際にFragment1#onCreateViewメソッドが呼ばれていないということなのでしょうか。
原因について、わかる方がいらっしゃれば教えていただきたいです。
よろしくお願いします。

import android.support.v4.app.FragmentManager;
import android.support.v4.app.FragmentTransaction;
import android.support.v7.app.AppCompatActivity;
import android.os.Bundle;

public class MainActivity extends AppCompatActivity {

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);

        if(savedInstanceState == null){
            FragmentManager fragmentManager = getSupportFragmentManager();
            FragmentTransaction fragmentTransaction = fragmentManager.beginTransaction();
            fragmentTransaction.addToBackStack(null);
            fragmentTransaction.replace(R.id.container, new Fragment1());
            fragmentTransaction.commit();
        }
    }

    @Override
    public void onBackPressed() {
        FragmentManager fragmentManager = getSupportFragmentManager();
        if(fragmentManager != null) {
            fragmentManager.popBackStack();
        }
        super.onBackPressed();
    }
}

import android.os.Bundle;
import android.support.v4.app.Fragment;
import android.support.v7.widget.LinearLayoutManager;
import android.support.v7.widget.RecyclerView;
import android.view.LayoutInflater;
import android.view.View;
import android.view.ViewGroup;
import android.widget.Button;

import java.util.ArrayList;


public class Fragment1 extends Fragment {

    private RecyclerView mRecyclerView;
    private RecyclerView.Adapter mAdapter;
    private RecyclerView.LayoutManager mLayoutManager;
    public static int click_pos;

    ArrayList<UserDataBean> userDataBeanList = new ArrayList<UserDataBean>();


    public Fragment1() {
        // Required empty public constructor
    }

    @Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);

    }

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

        View view =  inflater.inflate(R.layout.fragment_friend_list, container, false);

        mRecyclerView = view.findViewById(R.id.my_recycler_view);

        mRecyclerView.setHasFixedSize(true);

        mLayoutManager = new LinearLayoutManager(getActivity());

        mRecyclerView.setLayoutManager(mLayoutManager);

        UserDataBean userDataBean = ・・・
        userDataBeanList.add(userDataBean);
　　　　　・・・・
        userDataBeanList.add(userDataBean);

        mAdapter = new Adapter1(userDataBeanList,getActivity());
        mRecyclerView.setAdapter(mAdapter);

        return view;
    }
}

import android.content.Context;
import android.support.v4.app.FragmentManager;
import android.support.v4.app.FragmentTransaction;
import android.support.v7.widget.RecyclerView;
import android.view.LayoutInflater;
import android.view.View;
import android.view.ViewGroup;
import android.widget.ImageView;
import android.widget.TextView;
import android.widget.Toast;

import java.util.ArrayList;


public class Adapter1 extends RecyclerView.Adapter<Adapter1.ViewHolder> implements View.OnClickListener {

    private ArrayList<UserDataBean> userDataBeanList;
    private Context context;

    public Adapter1(ArrayList<UserDataBean> userDataBeanList, Context context) {
        this.userDataBeanList = userDataBeanList;
        this.context = context;
    }


    static class ViewHolder extends RecyclerView.ViewHolder {
        ImageView imageView;
        TextView textView;
        TextView messageView;

        ViewHolder(View v) {
            super(v);
            imageView = v.findViewById(R.id.image_view);
            textView = v.findViewById(R.id.text_view);
            messageView = v.findViewById(R.id.email_view);
        }
    }

    @Override
    public ViewHolder onCreateViewHolder(ViewGroup parent, int viewType) {
        View view = LayoutInflater.from(parent.getContext())
                .inflate(R.layout.list_items, parent, false);

        return new ViewHolder(view);

    }

    @Override
    public void onBindViewHolder(final ViewHolder holder, int position) {
        holder.imageView.setImageResource(userDataBeanList.get(position).getPhotos());
        holder.textView.setText(userDataBeanList.get(position).getUserName());
        holder.messageView.setText(userDataBeanList.get(position).getMessage());

        holder.imageView.setOnClickListener(this);
        holder.textView.setOnClickListener(this);
        holder.messageView.setOnClickListener(this);
    }

    @Override
    public int getItemCount() {
        return userDataBeanList.size();
    }

    @Override
    public void onClick(View v) {
        FragmentManager fragmentManager = ((MainActivity)context).getSupportFragmentManager();
        FragmentTransaction fragmentTransaction = fragmentManager.beginTransaction();
        fragmentTransaction.addToBackStack(null);
        fragmentTransaction.replace(R.id.container, new TalkFragment());
        fragmentTransaction.commit();
    }
}

Answer 1

おそらく、onBackPressed()の実装に問題があります。
FragmentActivityのonBackPressed()内ではgetSupportFragmentManager().popBackStack()に当たる処理が呼ばれるので、現在の実装だと2重にpopBackStack()が呼ばれるようになってしまっています。

ですので、onBackPressed()の実装を削除するか、バックスタックに関する処理を自前で行う場合は以下のようにsuper.onBackPressed()が呼ばれないようにする必要があります。

@Override
public void onBackPressed() {
    FragmentManager fragmentManager = getSupportFragmentManager();
    if(fragmentManager != null && fragmentManager.popBackStackImmediate()) { // バックスタックがある場合は1つ戻る
        return; // super.onBackPressed()が呼ばれないようにreturn
    }
    super.onBackPressed();
}

Comments

[m_s]

onBackPressedを削除するとうまくいきました。ありがとうございます。
「FragmentActivityのonBackPressed()内ではgetSupportFragmentManager().popBackStack()に当たる処理が呼ばれるので、現在の実装だと2重にpopBackStack()が呼ばれるようになってしまっています。」
つまり、onBackPressedをオーバーライドしなくても、デフォルトで既にpopBackStackが実装されている(実行される)→戻るボタンを押せばpopBackStackが呼ばれるということでしょうか。

[kakajika]

はい、特殊な処理が必要ないのであれば自分で実装しなくても呼ばれます。Androidのソースコードを辿ってみると分かると思います。

[m_s]

ありがとうございます。

Answer 2

    fragmentTransaction.replace(R.id.container, new TalkFragment());

replaceしてるから、fragment1を消して追加しちゃってるので、addにしてください。

Comments

[kakajika]

バックスタックに積んでるのでreplaceでも問題ないはずですよ。むしろaddだとレイアウトが重なってしまって良くないです。