logcatにNullPointerExceptionと、エラーが発生してしまった。

Question

実現したいこと

logcatのエラーをなくして正常な動作に戻したい。

発生している問題・分からないこと

ListViewのtextを押して画面変更しようとすると、logcatに以下のように提示されます。
java.lang.NullPointerException: Attempt to invoke virtual method 'void android.widget.TextView.setText(java.lang.CharSequence)' on a null object reference
atcom.websarva.wings.android.fragmentsample.MenuThanksFragment.onViewCreated(MenuThanksFragment.java:32)

エラーメッセージ

 Process: com.websarva.wings.android.fragmentsample, PID: 16880
 java.lang.NullPointerException: Attempt to invoke virtual method 'void android.widget.TextView.setText(java.lang.CharSequence)' on a null object reference
at com.websarva.wings.android.fragmentsample.MenuThanksFragment.onViewCreated(MenuThanksFragment.java:32)
at androidx.fragment.app.Fragment.performViewCreated(Fragment.java:2987)
at androidx.fragment.app.FragmentStateManager.createView(FragmentStateManager.java:546)
at androidx.fragment.app.FragmentStateManager.moveToExpectedState(FragmentStateManager.java:282)
at androidx.fragment.app.FragmentManager.executeOpsTogether(FragmentManager.java:2189)
at androidx.fragment.app.FragmentManager.removeRedundantOperationsAndExecute(FragmentManager.java:2106)
at androidx.fragment.app.FragmentManager.execPendingActions(FragmentManager.java:2002)
at androidx.fragment.app.FragmentManager$5.run(FragmentManager.java:524)
at android.os.Handler.handleCallback(Handler.java:942)
at android.os.Handler.dispatchMessage(Handler.java:99)
at android.os.Looper.loopOnce(Looper.java:201)
at android.os.Looper.loop(Looper.java:288)
at android.app.ActivityThread.main(ActivityThread.java:7872)
at java.lang.reflect.Method.invoke(Native Method)
at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:548)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:936)

該当のソースコード

package com.websarva.wings.android.fragmentsample;

import android.app.Activity;
import android.os.Bundle;

import androidx.annotation.NonNull;
import androidx.annotation.Nullable;
import androidx.fragment.app.Fragment;
import androidx.fragment.app.FragmentManager;
import androidx.fragment.app.FragmentTransaction;

import android.view.View;
import android.widget.AdapterView;
import android.widget.ListView;
import android.widget.SimpleAdapter;

import java.util.ArrayList;
import java.util.HashMap;
import java.util.List;
import java.util.Map;


public class MenuListFragment extends Fragment {
    public MenuListFragment() {
        super(R.layout.fragment_menu_list);
    }

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

        ListView lvMenu = view.findViewById(R.id.lvMenu);
        List<Map<String, String>> menuList = new ArrayList<>();
        Map<String, String> menu = new HashMap<>();
        menu.put("name", "唐揚げ定食");
        menu.put("price", "800円");
        menuList.add(menu);

        menu = new HashMap<>();
        menu.put("name", "ハンバーグ定食");
        menu.put("price", "850円");
        menuList.add(menu);

        menu = new HashMap<>();
        menu.put("name", "刺身定食");
        menu.put("price", "1200円");
        menuList.add(menu);

        menu = new HashMap<>();
        menu.put("name", "お好み焼き");
        menu.put("price", "700円");
        menuList.add(menu);

        Activity parentActivity = getActivity();
        String[] from = {"name", "price"};
        int[] to = {android.R.id.text1, android.R.id.text2};
        SimpleAdapter adapter = new SimpleAdapter(parentActivity, menuList,
                android.R.layout.simple_list_item_2, from, to);
        lvMenu.setAdapter(adapter);

        lvMenu.setOnItemClickListener(new ListItemClickListener());
    }

    private class ListItemClickListener implements AdapterView.OnItemClickListener {
        @Override
        public void onItemClick(AdapterView<?> parent, View view, int position, long id){
            Map<String,String> item = (Map<String, String>) parent.getItemAtPosition(position);
            String menuName = item.get("name");
            String menuPrice = item.get("price");

            Bundle bundle = new Bundle();
            bundle.putString("menuName",menuName);
            bundle.putString("menuPrice",menuPrice);

            FragmentManager manager = getParentFragmentManager();
            FragmentTransaction transaction = manager.beginTransaction();
            transaction.setReorderingAllowed(true);
            transaction.addToBackStack("Only List");
            transaction.replace(R.id.fragmentMainContainer, MenuThanksFragment.class, bundle);
            transaction.commit();
        }
    }
}

package com.websarva.wings.android.fragmentsample;



import android.os.Bundle;
import android.view.View;
import android.widget.Button;
import android.widget.TextView;

import androidx.annotation.NonNull;
import androidx.annotation.Nullable;
import androidx.fragment.app.Fragment;
import androidx.fragment.app.FragmentManager;

public class MenuThanksFragment extends Fragment {
    public MenuThanksFragment(){
        super(R.layout.fragment_menu_list);
    }

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

        Bundle extras = getArguments();
        String menuName = "";
        String menuPrice = "";
        if (extras != null){
            menuName = extras.getString("menuName");
            menuPrice = extras.getString("menuPrice");
        }
        TextView tvMenuName = view.findViewById(R.id.tvMenuName);
        TextView tvMenuPrice = view.findViewById(R.id.tvMenuPrice);
        tvMenuName.setText(menuName);
        tvMenuPrice.setText(menuPrice);

        Button btBackButton = view.findViewById(R.id.btThxBack);
        btBackButton.setOnClickListener(new ButtonClickListener());
    }

    private class ButtonClickListener implements View.OnClickListener{
        @Override
        public void onClick(View view){
            FragmentManager manager = getParentFragmentManager();
            manager.popBackStack();
        }
    }
}

<?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=".MainActivity">

    <TextView
        android:id="@+id/tvThxTitle"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:layout_marginStart="8dp"
        android:layout_marginTop="8dp"
        android:layout_marginEnd="8dp"
        android:text="@string/tv_thx_title"
        android:textSize="24dp"
        app:layout_constraintEnd_toEndOf="parent"
        app:layout_constraintStart_toStartOf="parent"
        app:layout_constraintTop_toTopOf="parent" />

    <TextView
        android:id="@+id/tvThxDesc"
        android:layout_width="0dp"
        android:layout_height="wrap_content"
        android:layout_marginStart="8dp"
        android:layout_marginTop="8dp"
        android:layout_marginEnd="8dp"
        android:text="@string/tv_thx_desc"
        app:layout_constraintEnd_toEndOf="parent"
        app:layout_constraintHorizontal_bias="0.0"
        app:layout_constraintStart_toStartOf="parent"
        app:layout_constraintTop_toBottomOf="@+id/tvThxTitle" />

    <TextView
        android:id="@+id/tvMenuName"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:layout_marginStart="8dp"
        android:layout_marginTop="8dp"
        app:layout_constraintEnd_toEndOf="parent"
        app:layout_constraintHorizontal_bias="0.0"
        app:layout_constraintStart_toStartOf="parent"
        app:layout_constraintTop_toBottomOf="@+id/tvThxDesc" />

    <Button
        android:id="@+id/btThxBack"
        android:layout_width="0dp"
        android:layout_height="wrap_content"
        android:layout_marginStart="8dp"
        android:layout_marginEnd="8dp"
        android:text="@string/bt_thx_back"
        app:layout_constraintEnd_toEndOf="parent"
        app:layout_constraintHorizontal_bias="0.0"
        app:layout_constraintStart_toStartOf="parent"
        app:layout_constraintTop_toBottomOf="@+id/tvMenuName" />

    <TextView
        android:id="@+id/tvMenuPrice"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:layout_marginEnd="8dp"
        app:layout_constraintEnd_toEndOf="parent"
        app:layout_constraintTop_toBottomOf="@+id/tvThxDesc" />

</androidx.constraintlayout.widget.ConstraintLayout>

<?xml version="1.0" encoding="utf-8"?>
<ListView
    xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:tools="http://schemas.android.com/tools"
    android:id="@+id/lvMenu"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    tools:context=".MenuListFragment">

    <!-- TODO: Update blank fragment layout -->

</ListView>

試したこと・調べたこと

• teratailやGoogle等で検索した
• ソースコードを自分なりに変更した
• 知人に聞いた
• その他

上記の詳細・結果

あまり情報を落としこめなかった。

補足

特になし

Comments

[jimbe]

エラーメッセージがコードのマークダウンから外れているようです。
また、ソースコードにもなぜ"特になし"というのがあるのでしょうか。

なお、コードのマークダウンで error とかファイル名らしきものを書かれている個所は java とか xml とかの"コードの言語名"を書く所です。
ファイル名はマークダウンの外に書いてください。
そして、本件は android studio で発生しているのでは無く android アプリで発生しているものです。android studio は単にそれを使って作っているというだけですので、タグには android を入れ android studio は入れないほうが良いと思います。

[reo4949]

コメントありがとうございます。質問を修正したのでご確認よろしくお願いいたします。

[jimbe]

編集ありがとうございます。
言語名の所にまだファイル名を書かれているようですが…。
回答もご確認ください。

[reo4949]

ご指摘ありがとうございます。質問を修正したのでご確認よろしくお願いいたします。

Answer 1

    public MenuThanksFragment(){
        super(R.layout.fragment_menu_list);
    }

fragment_menu_thanks でなく fragment_menu_list になっています。
↓のようになるのではないでしょうか。

    public MenuThanksFragment(){
        super(R.layout.fragment_menu_thanks);
    }

Comments

[reo4949]

すみません、追記した　fragment_menu_list.xml　を指定しています。
紛らわしくて申し訳ないです。

[jimbe]

恐らくそこが勘違いしてる所だと思います。
fragment の ViewCreated の引数として渡される view を使っての
view.findViewById(R.id.tvMenuName);
で得られるのは、コンストラクタに指定した fragment_menu_list に定義されているものだけです。
ですから他の xml に定義されている View は得られません。

普通は fragment とレイアウト xml ファイルとは 1:1 で対応させ同じ名前を使用します。
MenuThanksFragment ならレイアウトは fragment_menu_thanks なのではないでしょうか。

[reo4949]

指摘されたレイアウトの部分を直したら、エラーが解決しました！
ありがとうございます！