常にタブを表示させる。

Question

前提

先ほど以下のURLの質問をしたものです。
https://teratail.com/questions/5ljzf8nuh7x07v#reply-a2lvevxofy062w
画面遷移を行うプログラムにおいて、明らかに分からない点がありましたので質問させてください。
自分の作成しているプログラムはandroidにおいてタブをフッダーにして操作するものです。しかし先ほどのやり方でした場合、遷移までは成功したのですが、Page1Fragmentの中の内容が処理するごとに増えていき、画面が多くなるにつれて見にくいコードになってしまいます。それを防ぐためにクラスを分けたいです(可読性を上げるため)。
しかしクラスの数は増やしたくありません。1画面1つに抑えたいです(沢山画面を作る予定のため)

実現したいこと

常にタブを表示させる。

該当のソースコード

import androidx.annotation.NonNull;
import androidx.appcompat.app.AppCompatActivity;
import androidx.fragment.app.*;
import androidx.viewpager2.adapter.FragmentStateAdapter;
import androidx.viewpager2.widget.ViewPager2;

import android.os.Bundle;

import com.google.android.material.tabs.TabLayout;
import com.google.android.material.tabs.TabLayoutMediator;

public class MainActivity extends AppCompatActivity {
    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);

        ViewPager2 pager = findViewById(R.id.pager);
        TabLayout tabs = findViewById(R.id.tab);

        String[] hposition = {"てすと１","てすと２"};
        pager.setAdapter(new PageAdapter(this));
        new TabLayoutMediator(tabs, pager, (tab, position) -> tab.setText(hposition[position])).attach();
    }
    
    private static class PageAdapter extends FragmentStateAdapter {
        public PageAdapter(FragmentActivity activity) {
            super(activity);
        }

        @NonNull
        @Override
        public Fragment createFragment(int position) {
            Fragment fragment = null;
            if (position == 0) {
                fragment = new Page1Fragment(R.layout.activity_page1_fragment);
            } else if (position == 1) {
                fragment = new Page2Fragment();
            }
            return fragment;
        }

        @Override
        public int getItemCount() {
            return 2;
        }
    }
}

import androidx.annotation.NonNull;
import androidx.annotation.Nullable;
import androidx.appcompat.app.AppCompatActivity;
import androidx.fragment.app.Fragment;

import android.content.Intent;
import android.os.Bundle;
import android.view.View;
import android.widget.Button;

public class Page1Fragment extends Fragment {

    public Page1Fragment(int layout) {
        super(layout);
    }

    @Override
    public void onViewCreated(@NonNull View view, @Nullable Bundle savedInstanceState) {
        super.onViewCreated(view, savedInstanceState);

        Button button1 = view.findViewById(R.id.button1);
        if(button1 != null){
            button1.setOnClickListener(v -> {
                startActivity(new Intent(getActivity(), SampleActivity.class));
            });
        }

        Button back = view.findViewById(R.id.back);
        if(back != null){
            back.setOnClickListener(v -> {
                getActivity().finish();
            });
        }
    }
}

import androidx.annotation.NonNull;
import androidx.annotation.Nullable;
import androidx.appcompat.app.AppCompatActivity;
import androidx.fragment.app.Fragment;

import android.content.Intent;
import android.os.Bundle;
import android.view.View;
import android.widget.Button;

public class Page2Fragment extends Fragment {
    public Page2Fragment() {
        super(R.layout.activity_page2_fragment);
    }

    @Override
    public void onViewCreated(@NonNull View view, @Nullable Bundle savedInstanceState) {
        super.onViewCreated(view, savedInstanceState);

        Button button2 = view.findViewById(R.id.button2);
        button2.setOnClickListener(v -> {
            startActivity(new Intent(getActivity(), SampleActivity.class));
        });
    }
}

import androidx.annotation.NonNull;
import androidx.appcompat.app.AppCompatActivity;
import androidx.fragment.app.Fragment;
import androidx.fragment.app.FragmentActivity;
import androidx.viewpager2.adapter.FragmentStateAdapter;
import androidx.viewpager2.widget.ViewPager2;

import android.os.Bundle;

import com.google.android.material.tabs.TabLayout;
import com.google.android.material.tabs.TabLayoutMediator;

public class SampleActivity extends AppCompatActivity {

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

        ViewPager2 pager = findViewById(R.id.pager); 
        TabLayout tabs = findViewById(R.id.tab);

        String[] hposition = {"てすと１","てすと２"};
        pager.setAdapter(new PageAdapter(this));
        new TabLayoutMediator(tabs, pager, (tab, position) -> tab.setText(hposition[position])).attach();
    }

    private static class PageAdapter extends FragmentStateAdapter {
        public PageAdapter(FragmentActivity activity) {
            super(activity);
        }

        @NonNull
        @Override
        public Fragment createFragment(int position) {
            Fragment fragment = null;
            if (position == 0) {
                fragment = new Page1Fragment(R.layout.activity_sample);
            } else if (position == 1) {
                fragment = new Page2Fragment();
            }
            return fragment;
        }

        @Override
        public int getItemCount() {
            return 2;
        }
    }
}

<?xml version="1.0" encoding="utf-8"?>
<androidx.constraintlayout.widget.ConstraintLayout
    xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:app="http://schemas.android.com/apk/res-auto"
    xmlns:tools="http://schemas.android.com/tools"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    android:gravity="center"
    android:orientation="vertical"
    android:padding="20dp"
    tools:context=".MainActivity">

    <androidx.viewpager2.widget.ViewPager2
        android:id="@+id/pager"
        android:layout_width="0dp"
        android:layout_height="0dp"
        android:layout_weight="1"
        app:layout_constraintBottom_toTopOf="@+id/tab"
        app:layout_constraintEnd_toEndOf="parent"
        app:layout_constraintStart_toStartOf="parent"
        app:layout_constraintTop_toTopOf="parent" />

    <com.google.android.material.tabs.TabLayout
        android:id="@+id/tab"
        android:layout_width="match_parent"
        android:layout_height="0dp"
        app:layout_constraintBottom_toBottomOf="parent"
        app:layout_constraintEnd_toEndOf="parent"
        app:layout_constraintStart_toStartOf="parent"
        app:tabSelectedTextColor="@color/teal_200" />
</androidx.constraintlayout.widget.ConstraintLayout>

<?xml version="1.0" encoding="utf-8"?>
<androidx.constraintlayout.widget.ConstraintLayout xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:app="http://schemas.android.com/apk/res-auto"
    xmlns:tools="http://schemas.android.com/tools"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    tools:context=".Page1Fragment">


    <Button
        android:id="@+id/button1"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:text="ボタン１"
        app:layout_constraintBottom_toBottomOf="parent"
        app:layout_constraintEnd_toEndOf="parent"
        app:layout_constraintStart_toStartOf="parent"
        app:layout_constraintTop_toTopOf="parent" />
</androidx.constraintlayout.widget.ConstraintLayout>

<?xml version="1.0" encoding="utf-8"?>
<androidx.constraintlayout.widget.ConstraintLayout xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:app="http://schemas.android.com/apk/res-auto"
    xmlns:tools="http://schemas.android.com/tools"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    tools:context=".Page2Fragment">

    <Button
        android:id="@+id/button2"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:text="ボタン２"
        app:layout_constraintBottom_toBottomOf="parent"
        app:layout_constraintEnd_toEndOf="parent"
        app:layout_constraintStart_toStartOf="parent"
        app:layout_constraintTop_toTopOf="parent" />
</androidx.constraintlayout.widget.ConstraintLayout>

<?xml version="1.0" encoding="utf-8"?>
<androidx.constraintlayout.widget.ConstraintLayout xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:app="http://schemas.android.com/apk/res-auto"
    xmlns:tools="http://schemas.android.com/tools"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    tools:context=".SampleActivity">

    <Button
        android:id="@+id/back"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:text="戻る"
        app:layout_constraintBottom_toBottomOf="parent"
        app:layout_constraintEnd_toEndOf="parent"
        app:layout_constraintStart_toStartOf="parent"
        app:layout_constraintTop_toTopOf="parent" />
</androidx.constraintlayout.widget.ConstraintLayout>

試したこと

クラス分けをして、フラグメントを分ければ、処理の部分を分ける羽目になってしまいます。
例えば
Page1Fragmentから別のSampleActivityに飛ばしたとして、sampleの処理は別のフラグメント(SampleFragment)に飛ばすことになると思います。(遷移先でもタブを操作したいため)
そうなってくるとファイル数が1画面で２つ必要となってきて整理が大変になるため、実用的ではありません。

Comments

[jimbe]

まず、フラグメントのコンストラクタにパラメータを付けてはいけません。
フラグメントはシステムによって再生成される場合があり、その時はパラメータの無いコンストラクタが呼ばれます。その際にパラメータの無いコンストラクタが無いとアプリが落ちます。
パラメータが必要でしたら、setArgument(Bundle) で設定してフラグメント内で getArgument() する処理が必要です。(そしてその手間を隠す為に newInstance メソッド等を作ることになります。)

[jimbe]

「可読性を上げるため」にクラスを分けるのに「クラスの数は増やしたくありません」というのは二律背反です。
複数個所で行っている処理を纏めるというのはわかりますが、それにしては質問のコードは何もしていません。
何が分ける必要があって何が共通化出来るのかをコードから読み取るとすれば、そもそもアクティビティを分ける必要が無いです。

フラグメントから直接 startActivity したり 親アクティビティを finish したりはしない方が良いと思います。
画面遷移をあちこちで行うと、それこそ可読性が下がる行為ではないでしょうか。

[jimbe]

ちなみに、 フラグメントの中にもフラグメントを置けますので、 ViewPager で管理する領域に親フラグメントを配置し、その中に子フラグメントを入れることもできます。
単にタブの中のフラグメントを別のものにするのであれば、 PageAdapter の createFragment で該当タブに設置するフラグメントを別のものにすることも出来るでしょう。

[syosinsya109]

回答ありがとうございます。
自分はmainみたいな切り替えるactivityを遷移するごとに行わなければいけません。なのでmainを再利用するには、フラグメントで処理を分ける事しか思いつかなかったので、フラグメントのコンストラクタに引数を置いてしまいました。

確かにあちこちで遷移を行うと可読性が下がりうると思います。しかしその仕様しか思いつきませんでした。。。遷移用のactivityなどを作った方がいいということでしょうか。

ViewPager で管理する領域に親フラグメントを配置し、その中に子フラグメントを入れるということは、Page1Fragmentに他のページで使うフラグメントを書くということでしょうか。コードが長くならないでしょうか。。それが心配です。。

[jimbe]

> 自分はmainみたいな切り替えるactivityを遷移するごとに行わなければいけません。

「アクティビティにタブがあり、それはアプリ実行中は常に有って、そのタブの中のフラグメントでボタンを押したら、そのタブ内のフラグメントが別のフラグメントに変わる」ということの為にアクティビティから作り直すのは、画面遷移としては無駄が多過ぎると思います。

> Page1Fragmentに他のページで使うフラグメントを書くということでしょうか

ほんの僅かの違いでしか無ければそれでも良いかもしれませんし、大分違うのであれば別のフラグメントとして作られればよいでしょう。大分違うと思っていても、違いを吸収するようば構造・技術を使えば、流用出来るかもしれません。前の質問で状況に依ると申し上げたのはそういうことです。

コード量は、少なくとも質問からは本来やろうとされていることの片鱗も分かりませんので、何をするのにどれだけのコード量になるのを心配されているのかも分かりません。
別にコードが数10万行になったところでコンピュータからはみ出してくるわけでもありませんので、3画面程度を実際に書いてみてからどうなるか心配してみては如何でしょうか。

Answer 1

前の質問の回答のコードから一部流用となりますが、 "てすと1" タブ内で2つのフラグメント (Page1Fragment と Page11Fragment ) がボタン操作で入れ替わるようにしてみました。
activity_main.xml 、 Page2Fragment とそのレイアウトは変わらずなので省略です。

MainActivity.java

import android.os.Bundle;

import androidx.annotation.NonNull;
import androidx.appcompat.app.AppCompatActivity;
import androidx.fragment.app.*;
import androidx.lifecycle.ViewModelProvider;
import androidx.viewpager2.adapter.FragmentStateAdapter;
import androidx.viewpager2.widget.ViewPager2;

import com.google.android.material.tabs.*;

import java.util.*;
import java.util.function.Supplier;

public class MainActivity extends AppCompatActivity {
  @Override
  protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity_main);

    ViewPager2 pager = findViewById(R.id.pager);
    TabLayout tabs = findViewById(R.id.tab);

    TabPageAdapter adapter = new TabPageAdapter(this)
            .addTab("てすと１", Page1Fragment::new)
            .addTab("てすと２", Page2Fragment::new);

    pager.setAdapter(adapter);
    new TabLayoutMediator(tabs, pager, adapter).attach();

    MainViewModel viewModel = new ViewModelProvider(this).get(MainViewModel.class);
    viewModel.getPage1Value().observe(this, text -> {
      adapter.changeSupplier("てすと１", Page11Fragment::new);
    });
    viewModel.getPage11Value().observe(this, b -> {
      adapter.changeSupplier("てすと１", Page1Fragment::new);
    });
  }

  private static class TabPageAdapter extends FragmentStateAdapter implements TabLayoutMediator.TabConfigurationStrategy {
    private static class TabInfo {
      final String text; //タブのテキスト
      final Supplier<Fragment> supplier; //フラグメント生成器
      TabInfo(String text, Supplier<Fragment> supplier) {
        this.text = text;
        this.supplier = supplier;
      }
    }
    private List<TabInfo> tabInfoList = new ArrayList<>();

    public TabPageAdapter(FragmentActivity activity) {
      super(activity);
    }

    TabPageAdapter addTab(@NonNull String tabText, @NonNull Supplier<Fragment> supplier) {
      int position = tabInfoList.size();
      tabInfoList.add(new TabInfo(tabText, supplier));
      notifyItemInserted(position);
      return this;
    }

    void changeSupplier(@NonNull String tabText, @NonNull Supplier<Fragment> newSupplier) {
      for(int i=0; i<tabInfoList.size(); i++) {
        if(tabInfoList.get(i).text.equals(tabText)) {
          tabInfoList.set(i, new TabInfo(tabText, newSupplier));
          notifyItemChanged(i);
          break;
        }
      }
    }

    @NonNull
    @Override
    public Fragment createFragment(int position) {
      return tabInfoList.get(position).supplier.get();
    }

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

    @Override
    public long getItemId(int position) {
      return tabInfoList.get(position).hashCode();
    }

    @Override
    public boolean containsItem(long itemId) {
      for(TabInfo t : tabInfoList) if(t.hashCode() == itemId) return true;
      return false;
    }

    @Override
    public void onConfigureTab(@NonNull TabLayout.Tab tab, int position) {
      tab.setText(tabInfoList.get(position).text);
    }
  }
}

MainViewModel.java

import androidx.lifecycle.*;

public class MainViewModel extends ViewModel {
  private MutableLiveData<String> page1ValueLiveData = new MutableLiveData<>("");
  LiveData<String> getPage1Value() { return page1ValueLiveData; }
  void setPage1Value(String value) { page1ValueLiveData.setValue(value); }

  private MutableLiveData<Boolean> page11ValueLiveData = new MutableLiveData<>(false);
  LiveData<Boolean> getPage11Value() { return page11ValueLiveData; }
  void setPage11Value(boolean value) { page11ValueLiveData.setValue(value); }
}

Page1Fragment.java

import android.os.Bundle;
import android.view.View;
import android.widget.*;

import androidx.annotation.*;
import androidx.fragment.app.Fragment;
import androidx.lifecycle.ViewModelProvider;

public class Page1Fragment extends Fragment {
  public Page1Fragment() {
    super(R.layout.fragment_page1);
  }

  @Override
  public void onViewCreated(@NonNull View view, @Nullable Bundle savedInstanceState) {
    super.onViewCreated(view, savedInstanceState);

    MainViewModel viewModel = new ViewModelProvider(requireActivity()).get(MainViewModel.class);

    EditText editText = view.findViewById(R.id.edittext);

    Button setButton = view.findViewById(R.id.set_button);
    setButton.setOnClickListener(v -> {
      viewModel.setPage1Value(editText.getText().toString());
    });
  }
}

res/layout/fragment_page1.xml

<?xml version="1.0" encoding="utf-8"?>
<androidx.constraintlayout.widget.ConstraintLayout
    xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:app="http://schemas.android.com/apk/res-auto"
    xmlns:tools="http://schemas.android.com/tools"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    tools:context=".Page1Fragment">

  <EditText
      android:id="@+id/edittext"
      android:layout_width="match_parent"
      android:layout_height="wrap_content"
      android:textSize="30dp"
      android:inputType="text"
      android:hint="text"
      app:layout_constraintBottom_toBottomOf="parent"
      app:layout_constraintEnd_toEndOf="parent"
      app:layout_constraintStart_toStartOf="parent"
      app:layout_constraintTop_toTopOf="parent" />
  <Button
      android:id="@+id/set_button"
      android:layout_width="wrap_content"
      android:layout_height="wrap_content"
      android:text="セット"
      app:layout_constraintBottom_toBottomOf="parent"
      app:layout_constraintEnd_toEndOf="parent"
      app:layout_constraintStart_toStartOf="parent"
      app:layout_constraintTop_toBottomOf="@id/edittext" />
</androidx.constraintlayout.widget.ConstraintLayout>

Page11Fragment.java

import android.os.Bundle;
import android.view.View;
import android.widget.*;

import androidx.annotation.*;
import androidx.fragment.app.Fragment;
import androidx.lifecycle.ViewModelProvider;

public class Page11Fragment extends Fragment {
  public Page11Fragment() {
    super(R.layout.fragment_page11);
  }

  @Override
  public void onViewCreated(@NonNull View view, @Nullable Bundle savedInstanceState) {
    super.onViewCreated(view, savedInstanceState);

    MainViewModel viewModel = new ViewModelProvider(requireActivity()).get(MainViewModel.class);

    TextView textView = view.findViewById(R.id.textview);
    viewModel.getPage1Value().observe(getViewLifecycleOwner(), text -> {
      textView.setText(text);
    });

    Button returnButton = view.findViewById(R.id.return_button);
    returnButton.setOnClickListener(v -> {
      viewModel.setPage11Value(true); //値は使っていないので何でも良い
    });
  }
}

res/layout/fragment_page11.xml

<?xml version="1.0" encoding="utf-8"?>
<androidx.constraintlayout.widget.ConstraintLayout
    xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:app="http://schemas.android.com/apk/res-auto"
    xmlns:tools="http://schemas.android.com/tools"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    android:background="#ffeeff"
    tools:context=".Page11Fragment">

  <Button
      android:id="@+id/return_button"
      android:layout_width="wrap_content"
      android:layout_height="wrap_content"
      android:text="戻る"
      app:layout_constraintBottom_toTopOf="@id/textview"
      app:layout_constraintEnd_toEndOf="parent"
      app:layout_constraintStart_toStartOf="parent"
      app:layout_constraintTop_toTopOf="parent" />
  <TextView
      android:id="@+id/textview"
      android:layout_width="match_parent"
      android:layout_height="wrap_content"
      android:textSize="30dp"
      app:layout_constraintBottom_toBottomOf="parent"
      app:layout_constraintEnd_toEndOf="parent"
      app:layout_constraintStart_toStartOf="parent"
      app:layout_constraintTop_toTopOf="parent" />
</androidx.constraintlayout.widget.ConstraintLayout>