listviewの1行をタップして次の画面に遷移したいが、 呼び出したメゾットがnullと言われる。

Question

listviewの1行をタップして、次の画面に遷移したいのですが、
取得したメソッドがnullと言われ、クラッシュし、画面遷移できません。

どなたか原因をご教授いただけると嬉しいです。

以下、エラー文です。

2020-10-20 00:45:10.522 24959-24959/com.websarva.wing.android.trylistviewbydatebinding E/AndroidRuntime: FATAL EXCEPTION: main
    Process: com.websarva.wing.android.trylistviewbydatebinding, PID: 24959
    java.lang.NullPointerException: Attempt to invoke virtual method 'void com.websarva.wing.android.trylistviewbydatebinding.MainActivity.onGoEditRecord()' on a null object reference

以下、コードです。
↓activity_main.xml

<?xml version="1.0" encoding="utf-8"?>

<layout>
    <data>
        <variable
            name="ViewModel"
            type="com.websarva.wing.android.trylistviewadaptermvvm.MainViewModel" />
    </data>
    <LinearLayout
        android:id="@+id/container"
        xmlns:android="http://schemas.android.com/apk/res/android"
        xmlns:tools="http://schemas.android.com/tools"
        android:layout_width="match_parent"
        android:layout_height="match_parent"
        tools:context=".MainActivity">


        <ListView
            android:layout_width="wrap_content"
            android:layout_height="wrap_content"
            android:id="@+id/listView"/>


    </LinearLayout>
</layout>

↓foodrow.xml

<?xml version="1.0" encoding="utf-8"?>
<layout>
    <data>
        <variable
            name="ViewModel"
            type="com.websarva.wing.android.trylistviewadaptermvvm.MainViewModel" />
    </data>
    <LinearLayout
        xmlns:android="http://schemas.android.com/apk/res/android"
        android:orientation="vertical"
        android:layout_width="match_parent"
        android:layout_height="match_parent">

        <LinearLayout
            android:orientation="horizontal"
            android:layout_width="match_parent"
            android:layout_height="match_parent"
            android:onClick="@{()->ViewModel.onClicked()}">

            <TextView
                android:layout_width="wrap_content"
                android:layout_height="wrap_content"
                android:text="@{ViewModel.header ,default = v3月14日}"
                android:id="@+id/name"
                android:textSize="30dp"
                android:layout_weight="2" />

            <TextView
                android:layout_width="wrap_content"
                android:layout_height="wrap_content"
                android:text="@{ViewModel.desc ,default = v3月14日}"
                android:id="@+id/price"
                android:textSize="30dp"
                android:layout_weight="1" />
        </LinearLayout>

    </LinearLayout>
</layout>

↓MainActivity.java

package com.websarva.wing.android.trylistviewadaptermvvm;

import androidx.appcompat.app.AppCompatActivity;
import androidx.databinding.DataBindingUtil;

import android.content.Intent;
import android.os.Bundle;

import com.websarva.wing.android.trylistviewadaptermvvm.databinding.ActivityMainBinding;
import com.websarva.wing.android.trylistviewadaptermvvm.databinding.ActivityMainBindingImpl;

import java.util.ArrayList;

public class MainActivity extends AppCompatActivity implements Navigator {
    private ActivityMainBinding activityMainBinding;
    private MainViewModel mainViewModel;
    private ArrayList<MainViewModel> mainViewModelArrayList;
    private MyAdapter myAdapter;


    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        //setContentView(R.layout.activity_main);
        activityMainBinding = DataBindingUtil.setContentView(this, R.layout.activity_main);
        mainViewModel= new MainViewModel();
        mainViewModel.setNavigator(this);
        mainViewModelArrayList = mainViewModel.getArrayList();
        myAdapter = new MyAdapter(this, mainViewModelArrayList);
        activityMainBinding.listView.setAdapter(myAdapter);
        //mainFragment.setViewModel(mainViewModel);
    }

    @Override
    public void onGoEditRecord() {
        Intent intent = new Intent(this, Bottom.class);
        startActivity(intent);
    }
}

↓MainViewModel.java

package com.websarva.wing.android.trylistviewadaptermvvm;

import androidx.lifecycle.MutableLiveData;

import java.util.ArrayList;

public class MainViewModel {

    public String header;
    public String desc;
    public Navigator mNavigator;

    public MainViewModel(Model model){
        this.header = model.header;
        this.desc = model.desc;

    }

    public MainViewModel() {
    }


    public void setNavigator(Navigator navigator) {
        this.mNavigator = navigator;

    }

    public String getHeader() {
        return header;
    }

    public void setHeader(String header) {
        this.header = header;
    }

    public String getDesc() {
        return desc;
    }

    public void setDesc(String desc) {
        this.desc = desc;
    }

    public ArrayList<MainViewModel> getArrayList(){
        ArrayList<MainViewModel> modelArrayList = new ArrayList<>();
        MainViewModel viewModel1 = new MainViewModel(new Model("りんご","100"));
        MainViewModel viewModel2 = new MainViewModel(new Model("みかん","100"));
        MainViewModel viewModel3 = new MainViewModel(new Model("いちご","100"));
        MainViewModel viewModel4 = new MainViewModel(new Model("スイカ","100"));

        modelArrayList.add(viewModel1);
        modelArrayList.add(viewModel2);
        modelArrayList.add(viewModel3);
        modelArrayList.add(viewModel4);

        return modelArrayList;

    }

    public void onClicked(){
        System.out.println(mNavigator);
        mNavigator.onGoEditRecord();
    }
}

↓MyAdapter.java

package com.websarva.wing.android.trylistviewadaptermvvm;

import android.content.Context;
import android.view.LayoutInflater;
import android.view.View;
import android.view.ViewGroup;
import android.widget.ArrayAdapter;

import androidx.annotation.NonNull;
import androidx.annotation.Nullable;
import androidx.databinding.DataBindingUtil;

import com.websarva.wing.android.trylistviewadaptermvvm.databinding.FoodrowBinding;

import java.util.ArrayList;

public class MyAdapter extends ArrayAdapter<MainViewModel> {
    private ArrayList<MainViewModel> modelArrayList;
    private Context context1;

    public MyAdapter(@NonNull Context context, ArrayList<MainViewModel> modelArrayList){
        super(context, R.layout.foodrow,modelArrayList);
        this.context1 = context;
        this.modelArrayList = modelArrayList;
    }

    @NonNull
    @Override
    public View getView(int position, @Nullable View convertView, @NonNull ViewGroup parent) {

        LayoutInflater layoutInflater = LayoutInflater.from(parent.getContext());
        FoodrowBinding foodrowBinding = DataBindingUtil.inflate(layoutInflater,R.layout.foodrow,parent,false);
        foodrowBinding.setViewModel(modelArrayList.get(position));

        return foodrowBinding.getRoot();
    }
}

↓Navigator.java

package com.websarva.wing.android.trylistviewadaptermvvm;

public interface Navigator {
        void onGoEditRecord();

}

↓Model.java

package com.websarva.wing.android.trylistviewadaptermvvm;

public class Model {
    public String header;
    public String desc;

    public Model(String header, String desc) {
        this.header = header;
        this.desc = desc;
    }
}

Comments

[pg0084]

ListViewに値をセットしているのはMainFragmentでしょうか？
今載せていただいているコードで判断難しいため、ListView使用しているコードを見せていただくことはできますか？

[riria]

ご質問ありがとうございます。
全てのコードをのせてました。

Listviewにセットしているのは、viewmodelです。
前回載せた時から中身をいじっているので、変更してしまっている箇所がいくつかありますが、
依然として、同じエラーが出ています。

自分で調べているうちに、MainViewModel.java の以下の部分でmNavigatorがnullになっているため、本エラーが起きているようです。
public void onClicked(){
System.out.println(mNavigator);
mNavigator.onGoEditRecord();
}

MainViewModel.java のsetNavigatorでは、mNavigatorはnullではないので、
どこか途中で、mNavigatorがnullに初期化されているような気がしています。

原因・解決方法をご存知でしたら、教えて頂けると、幸いです。

[pg0084]

activity_main.xml に list_viewまたはlistViewがないようなのですが、activityMainBinding.listViewはどこをみてるんでしょうか？

[riria]

すいません、転載ミスをしていまして、今は修正済です。
activity_main.xml の中にlistviewを用意しております。

Answer 1

MainViewModel.java のsetNavigatorでは、mNavigatorはnullではないので、

どこか途中で、mNavigatorがnullに初期化されているような気がしています。

結論としてはgetArrayListでnew ViewModel()をしてArrayListを作ってる際に、各ViewModelにsetNavigatorをしていないのでnavigatorが存在していないのだと思います。

ちょっと基礎的なことから言うと、VIewModelはList単位ではなく、画面単位で持つことをオススメします。
MainViewModel内でMainViewModelのリストを作るのはそもそもあまり好ましくないため、MainViewModelの複製をMainActivityでやるか、以下のようにMainViewModelをAdapterに渡してそこArrayList<Model>
を作成して渡すと言うのが良いかと思います。

どうしてもitemのclickイベントをviewModelに紐付けたい場合は以下のような形式が良いのかと思います。
脳内コンパイルで申し訳ないですが、お試しください。

positionが必要な場合はxmlのonClickにやModelの値をつけてあげて下さい。

MainViewModel.java

public ArrayList<Model> modelArrayList;

    public MainViewModel(Model model){
        this.header = model.header;
        this.desc = model.desc;

        setArrayList();
    }

    private ArrayList<Model> setArrayList(){
        ArrayList<Model> modelArrayList = new ArrayList<>();
        modelArrayList.add(new Model("りんご","100"));
        modelArrayList.add(new Model("みかん","100"));
        modelArrayList.add(new Model("いちご","100"));
        modelArrayList.add(new Model("スイカ","100"));

        this.modelArrayList = modelArrayList;
    }

MyAdapter.java

public class MyAdapter extends ArrayAdapter<MainViewModel> {
    private MainViewModel mainViewModel;
    private Context context1;

    public MyAdapter(@NonNull Context context, MainViewModel viewModel){
        super(context, R.layout.foodrow,modelArrayList);
        this.context1 = context;
        this.mainViewModel = viewModel;
    }

    @NonNull
    @Override
    public View getView(int position, @Nullable View convertView, @NonNull ViewGroup parent) {

        LayoutInflater layoutInflater = LayoutInflater.from(parent.getContext());
        FoodrowBinding foodrowBinding = DataBindingUtil.inflate(layoutInflater,R.layout.foodrow,parent,false);
        foodrowBinding.setViewModel(mainViewModel);
        foodrowBinding.setPosition(position);

        return foodrowBinding.getRoot();
    }
}

foodrow.xml

<?xml version="1.0" encoding="utf-8"?>
<layout>
    <data>
        <variable
            name="ViewModel"
            type="com.websarva.wing.android.trylistviewadaptermvvm.MainViewModel" />

        <variable
                name="position"
                type="Integer" />
    </data>
    <LinearLayout
        xmlns:android="http://schemas.android.com/apk/res/android"
        android:orientation="vertical"
        android:layout_width="match_parent"
        android:layout_height="match_parent">

        <LinearLayout
            android:orientation="horizontal"
            android:layout_width="match_parent"
            android:layout_height="match_parent"
            android:onClick="@{()->ViewModel.onClicked()}">

            <TextView
                android:layout_width="wrap_content"
                android:layout_height="wrap_content"
                android:text="@{ViewModel.modelArrayList[position].header ,default = v3月14日}"
                android:id="@+id/name"
                android:textSize="30dp"
                android:layout_weight="2" />

            <TextView
                android:layout_width="wrap_content"
                android:layout_height="wrap_content"
                android:text="@{ViewModel.desc ,default = v3月14日}"
                android:id="@+id/price"
                android:textSize="30dp"
                android:layout_weight="1" />
        </LinearLayout>

    </LinearLayout>
</layout>

Comments

[riria]

丁寧な回答ありがとうございます。
MainViewModelをAdapterに渡すやり方でうまく行きました。