独自リスナークラスからのintentによる画面遷移の仕方

Question

MainActivityからNextActivityにintentにより画面遷移させようとしています。

main_activity.xmlはbuttonというid名のボタンを保持しています。
mainActivityのbuttonの状態を見るButtonListenerクラスを作成し、これをsetOnClickListnerしています。
この時MainActivityのgetApplicationContext()をintentのコンストラクタに渡し、startActivity(intent)しているのですが、startActivityでNullPointExceptionで止まってしまいます。

外のクラスからintentで画面遷移させることはできないのでしょうか？

（エラーログ）
NullPointerException: Attempt to invoke virtual method 'android.app.ActivityThread$ApplicationThread android.app.ActivityThread.getApplicationThread()' on a null object reference
at android.app.Activity.startActivityForResult(Activity.java:4488)
at android.support.v4.app.FragmentActivity.startActivityForResult(FragmentActivity.java:767)
at android.app.Activity.startActivityForResult(Activity.java:4445)

public class MainActivity extends AppCompatActivity {
    private static Context context;
    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);
        context = getApplicationContext();
        View button = findViewById(R.id.button);

        ButtonListener listener = new ButtonListener();
        button.setOnClickListener(listener);
    }

    public static Context getAppContext() {
        return context;
    }
}

public class ButtonListener extends AppCompatActivity implements View.OnClickListener {
    @Override
    public void onClick(View view){
        View v = view.getRootView();
        EditText editName = (EditText)v.findViewById(R.id.editTextName);
        String strName = editName.getText().toString();
        EditText editEmail = (EditText)v.findViewById(R.id.editTextAddress);
        String strEmail = editEmail.getText().toString();
        Context context = MainActivity.getAppContext();
        Intent intent = new Intent(context, NextActivity.class);
        intent.putExtra("name",strName);
        intent.putExtra("email",strEmail);
        startActivity(intent);
    }
}

public class NextActivity extends AppCompatActivity {
    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_next);
        Intent intent = getIntent();
        String name = intent.getStringExtra("name");
        String email = intent.getStringExtra("email");
        if(name != null && ! name.isEmpty() && email != null && ! email.isEmpty()){
            ((TextView)findViewById(R.id.nameTextView)).setText(name);
            ((TextView)findViewById(R.id.emailTextView)).setText(email);
        }
    }
}

Answer 1

ButtonListenerはAppCompatActivityから派生せず、コンストラクタでMainActivityからContextを受け取ってください。(AppCompatActivityの派生クラスならthisを渡せばいいです)

public class MainActivity extends AppCompatActivity {
    @Override
    protected void onCreate(Bundle savedInstanceState) {
        ～
        ButtonListener listener = new ButtonListener(this); // this = Context
        button.setOnClickListener(listener);
    }
}

public class ButtonListener implements View.OnClickListener {
    private Context _context;
    ButtonListener(Context context)
    {
        _context = context;
    }
    @Override
    public void onClick(View view){
        ～
        _context.startActivity(intent);
    }
}

Comments

[konpeit]

ご回答ありがとうございます。
正常に遷移させることができました！
しかしなぜコンストラクタで受け取らないと正常動作しないのでしょうか？
メインアクティビティのメソッドから別途contextを渡しても同じことだと思ったのですが、、。

[toki_td]

いえ、元の例でも context.startActivity とすれば恐らく動くはずです。
元のはなんでクラッシュしてたかというと、ActivityはstartActivity等を通してシステムによって初期化して起動してもらわなければ正しく動きません。
ButtonClickListenerは画面を表すクラスではなく直接new ButtonClickListenerと生成しており、正しくActivityとして動くことができません。そもそもAppCompatActivityを継承している構造がおかしいです。

ですので、ButtonClickListenerはAppCompatActivityを継承しないほうが正しく、その代わりに何らかの手段でstartActivityを呼ぶためにContextを入手しないといけないわけですが、コンストラクタという初期化のタイミングがあるんだったらそこで渡すのが順当、ということです。
そしたらButtonClickListenerはMainActivityへの依存を排除できますね。

これ以外にも、内部クラスとか匿名クラスを使えばContextを渡す必要もなくなります。

[konpeit]

理解できました。
大変ありがとうございました。