Navigation Drawer Activityを利用時に同じアイテムを選択したときfragmentを再生成しないやり方

Question

android studioのデフォルトで用意されているNavigation Drawer Activityを改造してアプリを作っているのですが、navigationViewでHOME→galally→HOMEなどと一度別のfragmentを経由して表示した場合は問題ないのですが、NavigationViewでHOME→HOMEと選び同じフラグメントに遷移するとHomeFragmentに書いたHandler内でのviewの更新がされなくなったり一部挙動がおかしくなってしまいます。

単純な解決方法としては
１．NavigationViewで表示しているアイテムと同じアイテムを選択したら何もしない
２．一度fragmentのインスタンスが生成されていたら再生成しない
上記二つ思いついたのですが、解決方法として正しいのかどうか。
そして実装上どのようにすれば実現できるか知りたいです。
１．の解決方法はデフォルトのテンプレートで各アイテムがfragmentタグでxmlに埋め込まれいるので難しいような気がしています。

補足
動きを見てみましたが
2回目に同じアイテムをタップした場合、新たに生成しようとonCreateが走ったあとすぐに最初に動いていたものを終了しようとonDestroyが走ってしまっています。
これは単純に上記の方法で解決できるのだろうか・・・
１．案で解決できそうですが、表示しているフラグメントと同じアイテム選択時は何も処理しないとかできますか？
静的なfragmentなのでもしかして大幅につくりかえないと駄目でしょうか？

MainActivity.java

import android.Manifest;
import android.content.Intent;
import android.content.pm.PackageManager;
import android.os.Bundle;

import com.google.android.material.floatingactionbutton.FloatingActionButton;
import com.google.android.material.snackbar.Snackbar;

import android.view.MenuItem;
import android.view.View;

import androidx.annotation.NonNull;
import androidx.core.app.ActivityCompat;
import androidx.core.content.ContextCompat;
import androidx.navigation.NavController;
import androidx.navigation.Navigation;
import androidx.navigation.ui.AppBarConfiguration;
import androidx.navigation.ui.NavigationUI;

import com.google.android.material.navigation.NavigationView;

import androidx.drawerlayout.widget.DrawerLayout;

import androidx.appcompat.app.AppCompatActivity;
import androidx.appcompat.widget.Toolbar;

import android.view.Menu;

public class MainActivity extends AppCompatActivity  {

    private AppBarConfiguration mAppBarConfiguration;

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);
        Toolbar toolbar = findViewById(R.id.toolbar);
        setSupportActionBar(toolbar);
        FloatingActionButton fab = findViewById(R.id.fab);
        fab.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {
                Snackbar.make(view, "Replace with your own action", Snackbar.LENGTH_LONG)
                        .setAction("Action", null).show();
            }
        });
        DrawerLayout drawer = findViewById(R.id.drawer_layout);
        NavigationView navigationView = findViewById(R.id.nav_view);

        // Passing each menu ID as a set of Ids because each
        // menu should be considered as top level destinations.
        mAppBarConfiguration = new AppBarConfiguration.Builder(
                R.id.nav_home, R.id.nav_graph, R.id.nav_slideshow,
                R.id.nav_tools, R.id.nav_share, R.id.nav_send)
                .setDrawerLayout(drawer)
                .build();
        NavController navController = Navigation.findNavController(this, R.id.nav_host_fragment);
        NavigationUI.setupActionBarWithNavController(this, navController, mAppBarConfiguration);
        NavigationUI.setupWithNavController(navigationView, navController);
    }

    @Override
    public boolean onCreateOptionsMenu(Menu menu) {
        // Inflate the menu; this adds items to the action bar if it is present.
        getMenuInflater().inflate(R.menu.main, menu);
        return true;
    }

    @Override
    public boolean onSupportNavigateUp() {
        NavController navController = Navigation.findNavController(this, R.id.nav_host_fragment);
        return NavigationUI.navigateUp(navController, mAppBarConfiguration)
                || super.onSupportNavigateUp();
    }

    @Override
    protected void onDestroy()
    {
        super.onDestroy();
    }
}

mobile_navigation.xml

<?xml version="1.0" encoding="utf-8"?>
<navigation 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:id="@+id/mobile_navigation"
    app:startDestination="@+id/nav_home">

    <fragment
        android:id="@+id/nav_home"
        android:name="XXX.ui.home.HomeFragment"
        android:label="@string/menu_home"
        tools:layout="@layout/fragment_home" />

    <fragment
        android:id="@+id/nav_graph"
        android:name="XXX.ui.graph.GraphFragment"
        android:label="@string/menu_graph"
        tools:layout="@layout/fragment_graph" />

    <fragment
        android:id="@+id/nav_slideshow"
        android:name="xxx.ui.slideshow.SlideshowFragment"
        android:label="@string/menu_slideshow"
        tools:layout="@layout/fragment_slideshow" />

    <fragment
        android:id="@+id/nav_tools"
        android:name="xxx.ui.tools.ToolsFragment"
        android:label="@string/menu_tools"
        tools:layout="@layout/fragment_tools" />

    <fragment
        android:id="@+id/nav_share"
        android:name="xxx.ui.share.ShareFragment"
        android:label="@string/menu_share"
        tools:layout="@layout/fragment_share" />

    <fragment
        android:id="@+id/nav_send"
        android:name="xxx.ui.send.SendFragment"
        android:label="@string/menu_send"
        tools:layout="@layout/fragment_send" />
</navigation>

Comments

[kakajika]

NavControllerを使っているということは静的なfragmentでは無いはずですね。

Answer 1

１．NavigationViewで表示しているアイテムと同じアイテムを選択したら何もしない

NavigationUIのソースを見てみましたが、この中にMenuItemの選択チェック処理を挟むのはちょっと厳しそうなので、NavigationViewを拡張してアイテムの再選択時には通知しないクラスを実装するのが良さそうです。関連するIssueでもBottomNavigationViewのような機能（OnNavigationItemReselectedListener）が必要かも？という話になっています。
以下のようなクラスを実装してNavigationViewの代わりに使ってみてください。

public class UnreselectableNavigationView extends NavigationView {
    // コンストラクタ...（略）

    @Override
    public void setNavigationItemSelectedListener(final @Nullable OnNavigationItemSelectedListener listener) {
        if (listener != null) {
            super.setNavigationItemSelectedListener(new OnNavigationItemSelectedListener() {
                @Override
                public boolean onNavigationItemSelected(@NonNull MenuItem item) {
                    // アイテムが選択されていない場合のみ通知
                    if (!item.isChecked()) {
                        return listener.onNavigationItemSelected(item);
                    }
                    return false;
                }
            });
        } else {
            super.setNavigationItemSelectedListener(null);
        }
    }
}

Comments

[luckyclock]

すみません初歩的なところで躓いています。
UnreselectableNavigationViewクラスのコンストラクタがないと怒られてしまいます。
public UnreselectableNavigationView(){

}
there is no default constructor available in・・・

それから
return listener.onNavigationItemSelected(item);の部分も
Variable 'listener' accessed from within inner class, nieeds to be declared final
となります。

[kakajika]

> UnreselectableNavigationViewクラスのコンストラクタがないと怒られてしまいます。

Android Studioでエラーの赤線にカーソルを合わせるとコンストラクタを自動で追加するメニューが出ると思います。そちらで追加してください。

> Variable 'listener' accessed from within inner class, nieeds to be declared final

失礼しました、Java7以前だとこの場合finalが必要ですね。回答を修正しておきましたのでご確認ください。（Java8にすればeffectively finalなので省略できます）

[luckyclock]

新規に作成したクラスを下記に置き換えて実行したところアプリが落ちてしまいます。
// NavigationView navigationView = findViewById(R.id.nav_view);
UnreselectableNavigationView navigationView = findViewById(R.id.nav_view);

logです。キャストできません的なエラーのようです。
java.lang.RuntimeException: Unable to start activity ComponentInfo{XXX}: java.lang.ClassCastException: com.google.android.material.navigation.NavigationView cannot be cast to xxx.UnreselectableNavigationView
at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2778)
at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:2856)
at android.app.ActivityThread.-wrap11(Unknown Source:0)
at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1589)
at android.os.Handler.dispatchMessage(Handler.java:106)
at android.os.Looper.loop(Looper.java:164)
at android.app.ActivityThread.main(ActivityThread.java:6494)
at java.lang.reflect.Method.invoke(Native Method)
at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:438)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:807)
Caused by: java.lang.ClassCastException: com.google.android.material.navigation.NavigationView cannot be cast to xxxx.UnreselectableNavigationView
at xxx.MainActivity.onCreate(MainActivity.java:57)
at android.app.Activity.performCreate(Activity.java:7009)
at android.app.Activity.performCreate(Activity.java:7000)
at android.app.Instrumentation.callActivityOnCreate(Instrumentation.java:1214)
at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2731)
at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:2856) 
at android.app.ActivityThread.-wrap11(Unknown Source:0) 
at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1589) 
at android.os.Handler.dispatchMessage(Handler.java:106) 
at android.os.Looper.loop(Looper.java:164) 
at android.app.ActivityThread.main(ActivityThread.java:6494) 
at java.lang.reflect.Method.invoke(Native Method) 
at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:438) 
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:807)

[kakajika]

カスタムViewを使うのは初めてでしょうか？レイアウトxmlに埋め込んであるNavigationViewを自作クラスに変える必要があります。また、今回はUnreselectableNavigationViewがNavigationViewを継承しているのでJava側の変更は必要ありません。

[luckyclock]

たびたびすみません。
java側は元に戻して
activity_main.xmlの
<com.google.android.material.navigation.NavigationView
android:id="@+id/nav_view"
android:layout_width="wrap_content"
android:layout_height="match_parent"
android:layout_gravity="start"
android:fitsSystemWindows="true"
app:headerLayout="@layout/nav_header_main"
app:menu="@menu/activity_main_drawer" />

を
<XXX.UnreselectableNavigationView
android:id="@+id/nav_view"
android:layout_width="wrap_content"
android:layout_height="match_parent"
android:layout_gravity="start"
android:fitsSystemWindows="true"
app:headerLayout="@layout/nav_header_main"
app:menu="@menu/activity_main_drawer" />
としてみましたがやはりアプリが落ちてしまいます。

[kakajika]

「アプリが落ちてしまいます」だけでは何が起きているのか分からないので、エラーの内容等を載せてください。

[luckyclock]

エラー内容になります。
2020-03-02 09:29:02.617 8283-8283/XXX E/AndroidRuntime: FATAL EXCEPTION: main
Process: XXX, PID: 8283
java.lang.RuntimeException: Unable to start activity ComponentInfo{XXX.MainActivity}: android.view.InflateException: Binary XML file line #16: Binary XML file line #16: Error inflating class XXX.UnreselectableNavigationView
at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2778)
at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:2856)
at android.app.ActivityThread.-wrap11(Unknown Source:0)
at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1589)
at android.os.Handler.dispatchMessage(Handler.java:106)
at android.os.Looper.loop(Looper.java:164)
at android.app.ActivityThread.main(ActivityThread.java:6494)
at java.lang.reflect.Method.invoke(Native Method)
at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:438)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:807)
Caused by: android.view.InflateException: Binary XML file line #16: Binary XML file line #16: Error inflating class XXX.UnreselectableNavigationView
Caused by: android.view.InflateException: Binary XML file line #16: Error inflating class XXX.UnreselectableNavigationView
Caused by: java.lang.NoSuchMethodException: <init> [class android.content.Context, interface android.util.AttributeSet]
at java.lang.Class.getConstructor0(Class.java:2320)
at java.lang.Class.getConstructor(Class.java:1725)
at android.view.LayoutInflater.createView(LayoutInflater.java:615)
at android.view.LayoutInflater.createViewFromTag(LayoutInflater.java:790)
at android.view.LayoutInflater.createViewFromTag(LayoutInflater.java:730)
at android.view.LayoutInflater.rInflate(LayoutInflater.java:863)
at android.view.LayoutInflater.rInflateChildren(LayoutInflater.java:824)
at android.view.LayoutInflater.inflate(LayoutInflater.java:515)
at android.view.LayoutInflater.inflate(LayoutInflater.java:423)
at android.view.LayoutInflater.inflate(LayoutInflater.java:374)
at androidx.appcompat.app.AppCompatDelegateImpl.setContentView(AppCompatDelegateImpl.java:555)
at androidx.appcompat.app.AppCompatActivity.setContentView(AppCompatActivity.java:161)
at XXX.MainActivity.onCreate(MainActivity.java:44)
at android.app.Activity.performCreate(Activity.java:7009)
at android.app.Activity.performCreate(Activity.java:7000)
at android.app.Instrumentation.callActivityOnCreate(Instrumentation.java:1214)
at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2731)
at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:2856)
at android.app.ActivityThread.-wrap11(Unknown Source:0)
at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1589)
at android.os.Handler.dispatchMessage(Handler.java:106)
at android.os.Looper.loop(Looper.java:164)
at android.app.ActivityThread.main(ActivityThread.java:6494)
at java.lang.reflect.Method.invoke(Native Method)
at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:438)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:807)

xmlの16行目の<XXX.UnreselectableNavigationView部分がエラーのようですが、XXX部分はパッケージとしています。

[kakajika]

> Caused by: java.lang.NoSuchMethodException: <init> [class android.content.Context, interface android.util.AttributeSet]
とありますから、必要なコンストラクタが足りていないようです。Android Studioの機能でコンストラクタを追加する際に複数の候補が出ると思いますので、全て選択して追加してください。

[luckyclock]

何度もありがとうございます！
同じアイテムが選択できなくなりうまくいきまいた。