TextViewのsetTextで起こるエラー（メソッドでActivityをいじるのはよくないのか）

Question

前提・実現したいこと

TextViewに取得したBluetoothDeviceの名前を表示させたいです
サブクラスでデバイスを取得してデバイス名をメインクラスで表示させようとしています

アクティビティをセットしてあるクラスです
メソッドでいじるのはよくないのでしょうか？

発生している問題・エラーメッセージ

E/AndroidRuntime: FATAL EXCEPTION: main
    Process: com.example.esp32ble, PID: 29278
    java.lang.NullPointerException: Attempt to invoke virtual method 'void android.widget.TextView.setText(java.lang.CharSequence)' on a null object reference
        at com.example.esp32ble.MainActivity.setTextName(MainActivity.java:174)
        at com.example.esp32ble.SubsidyBle.scanAndConnect(SubsidyBle.java:121)
        at com.example.esp32ble.MainActivity.chartChange(MainActivity.java:160)
        at com.example.esp32ble.-$$Lambda$TW4Z-_TbHlSaQ3_9pMVQ_fCEm0w.onClick(Unknown Source:2)
        at android.view.View.performClick(View.java:7217)
        at android.view.View.performClickInternal(View.java:7191)
        at android.view.View.access$3500(View.java:828)
        at android.view.View$PerformClick.run(View.java:27679)
        at android.os.Handler.handleCallback(Handler.java:900)
        at android.os.Handler.dispatchMessage(Handler.java:103)
        at android.os.Looper.loop(Looper.java:219)
        at android.app.ActivityThread.main(ActivityThread.java:8349)
        at java.lang.reflect.Method.invoke(Native Method)
        at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:513)
        at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1055)

該当のソースコード

public class MainActivity extends AppCompatActivity implements MenuItem.OnMenuItemClickListener {

    private int i = 0;
    private LineChart chart;
    private boolean connectState = false;
    private Button changeButton;
    private TextView nameText;
    private final String NOW_CONNECT = "接続を終了";
    private final String NOW_DISCONNECT = "接続を開始";

    private BluetoothManager bluetoothManager;
    private BluetoothAdapter bluetoothAdapter;
    private final int duration = Toast.LENGTH_SHORT;
    private SubsidyBle subBle;

    private BtDialogFragment dialog;
    private Intent intent;

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

        Toolbar Toolbar = findViewById(R.id.main_toolbar);
        setSupportActionBar(Toolbar);

        changeButton = findViewById(R.id.chart_change);
        changeButton.setOnClickListener(this::chartChange);
        nameText = findViewById(R.id.device_name_text);
        
        lineChart();
        intent = new Intent();
        subBle = new SubsidyBle();

        //BLEの設定
        if( !getPackageManager().hasSystemFeature( PackageManager.FEATURE_BLUETOOTH_LE ) ) {
            Log.d("BtActivity","Bluetooth is not supported");
            finish();    // アプリ終了宣言
            return;
        }
        // Bluetoothアダプタの取得
        bluetoothManager = (BluetoothManager)getSystemService(Context.BLUETOOTH_SERVICE);
        bluetoothAdapter = bluetoothManager.getAdapter();
        if( null == bluetoothAdapter ) {
            // Android端末がBluetoothをサポートしていない
            Log.d("BtActivity","Bluetooth is not supported");
            finish();    // アプリ終了宣言
            return;
        }
        bluetoothAdapter = BluetoothAdapter.getDefaultAdapter();

        dialog = new BtDialogFragment();
        dialog.show(getSupportFragmentManager(),"my_dialog");
    }

~~~~~~~~~~~~~~~

    //subClassで検出したデバイスの名前を受け取って表示
    public void setTextName(String deviceName){
        Log.d("kiteru?",deviceName);
        nameText.setText(deviceName);
    }

private MainActivity m = new MainActivity();

//ペアデバイスを取得してサーバーに接続する
    public void scanAndConnect(Context context,BluetoothAdapter bluetoothAdapter){
        pairedDevices = bluetoothAdapter.getBondedDevices();
        if (pairedDevices.size() > 0) {
            for (BluetoothDevice device : pairedDevices) {
                Log.d("pairedDevice","Name:"+device.getName()+"Address:"+device.getAddress());
                Toast.makeText(context,"接続を開始します",Toast.LENGTH_SHORT).show();
                device = bluetoothAdapter.getRemoteDevice(device.getAddress());
                bluetoothGatt = device.connectGatt(context, false, bluetoothGattCallback);
                m.setTextName("" + device.getName());
            }
        }
        if (bluetoothGatt == null) {
            Toast.makeText(context, "ペアリングされたデバイスが存在しません", Toast.LENGTH_SHORT).show();
        }
        Toast.makeText(context,"Bluetooth通信に異常が発生しています",Toast.LENGTH_LONG).show();
    }

試したこと

onCreateにfindViewByIdを書いてみた

補足情報（FW/ツールのバージョンなど）

Log.dで取得した文字列は確認できる
ViewのIdはあってる

android studio
java

Comments

[jimbe]

>onCreateにfindViewByIdを書いてみた
どのように考えられてその試験をし、どのように「書いてみた」のでしょうか。その結果何が起き、何が分かったのでしょう。

[iruma]

もしかしたら、メソッドに書いているのが間違ってるのではないのかと思ったからです。

今試したところ、その検証はあっていました
ただ、今回の本題のsetTextでエラーが発生します

[jimbe]

onCreate に nameText = findViewById(R.id.device_name_text); を移動した形にされたのですね。
それで「その検証はあっていました」と言われるのは、移動により例外は無くなって想定通りの動作をしたということでしょうか。
そして、移動したら動作した(=移動しないと動作しない)のは何故か？というご質問ということで宜しいですか？

[iruma]

おっしゃる通り例外が発生しなくなったということです。
しかし、今回の質問内容としてはfindViewByIdを記述する場所ではなく、setTextでエラーがどうして発生するのかについて知りたいです。お願いします。

[jimbe]

例外が発生しなくなった・・・とのことですが、「発生している問題・エラーメッセージ」を更新されて(更新前とは少し違いますが) NullPointerException 例外が発生しているように見受けられますけども・・・どういうことでしょう？

回答ぽくなりますが、
> findViewById を記述する場所ではなく、setTextでエラーがどうして発生するのか
という点に置きましては、各メソッドは個別に存在・動作するわけではなく、それぞれが内部で複雑に関係しているから…となります。実際にどう関係してエラーとなるのかは、細かく言えば状況によりますし、ざっくり言えば各メソッドの仕様・使い方通りに使えていないから…となります。

ちなみに、 SubsidyBle クラスの scanAndConnect メソッドからどのように setTextName を呼び出しているのか、scanAndConnect メソッドをご提示願えますか？

[iruma]

返信に不備があり申し訳ありません。
正しくは、また別のエラーが発生したということになります

サブクラスのコードを追加しました。よろしくお願いします

[jimbe]

ありがとうございます。

Answer 1

本件の一連の例外の原因は、SubsidyBle クラス内 scanAndConnect メソッドで行われている

m.setTextName("" + device.getName());

の m が、同じく SubsidyBle クラス内で

private MainActivity m = new MainActivity();

として宣言・生成されている為と思われます。

よくある勘違いですが、アプリを実行した時にシステムによって生成・表示された MainActivity オブジェクトと、この SubsidyBle クラス内で new MainActivity() として生成された MainActivity オブジェクトは「同じ構造の別のモノ」です。
そして、システムによって生成・表示された方の MainActivity オブジェクトにはシステムによって様々な設定が行われ、例えば findViewById によって View が取得できるようになっていますが、new MainActivity() として生成された MainActivity オブジェクトには設定が行われておらず、findViewById の内部で例外が発生します。

setTextName メソッド内に findViewById を書いていた時はその例外が発生しており、また findViewById を onCreate に移動した際には SubsidyBle クラス内で new MainActivity() しても onCreate は呼ばれない為、private TextView nameText が null のままとなっていて setText で例外が発生していると考えられます。

この対応としてましては、 SubsidyBle の生成時に MainActivity をコンストラクタのパラメータとして受け取り、 m に代入しておくのが簡単かと思います。

class SubsidyBle {
  private MainActivity m;
  public SubsidyBle(MainActivity main) {
    m = main;
  }
  :
}

Comments

[iruma]

ありがとうございます。

エラーを修正することができました！

[iruma]

以前までは、MainActivityという構造のMainActivityとは別のものが生成されていて、それを編集しようとしていたからエラーが発生していた。なので、サブクラスのコンストラクタで生成済みのMainActivityを取得することで生成されたものに対して編集することができるようになった、という認識で問題ないでしょうか。

[jimbe]

問題無いかと思います。
そうなれば、findViewById を元のように setTextName 内で行っても動作するのではないでしょうか。
もっとも、findViewById の結果のように「一度取得すれば変わることが無い」ようなものは、 onCreate で行ったほうが良いようには思いますが ^^;

[jimbe]

失礼、ご質問とは関係ない蛇足になりますが、このような具体的なコードに対してのお話の場合、「サブクラス」「メインクラス」というキーワードでは(特に初見時は)分かり難くなりますので、具体的なクラス名を書かれたほうが良いですね。