Android Studioで複数ボタンから画面遷移をしたいです。

Question

###前提・実現したいこと
Android Studioを始めたばかりです。
練習として何時に何の業務をしていたか記録するアプリを作ろうとしています。
その第一段階としてメイン画面に複数ボタンを配置して各ボタンごとに別の画面に遷移させたいです。
以下の画面から例えば会議を押したら下図別の画面に遷移して、
それをボタンごとにデータ入力を押したら、データ入力終了の画面に遷移させたいです。

###発生している問題・エラーメッセージ
MainActivityからSubActibityを指定しインテントを作成、値を保持させて
それでボタンを判別しようとしてます。
エラーは吐かれていませんがエミュレータで起動したところ、「○○○ has stopped」と出てアプリが終了してしまいます。

###該当のソースコード
MainActivity.java

import android.content.Intent;
import android.support.v7.app.AppCompatActivity;
import android.os.Bundle;
import android.view.View;
import android.widget.Button;
import android.widget.Toast;


public class MainActivity extends AppCompatActivity{

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

        Button sendkaigi = (Button)findViewById(R.id.kaigi);
        sendkaigi.setOnClickListener(new View.OnClickListener(){
            @Override
            public void onClick(View v){
                //インテントの作成
                Intent intent = new Intent(getApplication(), SubActivity.class);

                intent.putExtra("name","meeting");
                //遷移先の画面起動
                startActivity(intent);
            }
        });
        Button send_datainput = (Button)findViewById(R.id.datainput);
        send_datainput.setOnClickListener(new View.OnClickListener(){
            @Override
            public void onClick(View v){
                Intent intent = new Intent(getApplication(), SubActivity.class);

                intent.putExtra("name1","data_input");

                startActivity(intent);
            }
        });
    }
}

SubActivity.java

import android.content.Intent;
import android.os.Bundle;
import android.support.v7.app.AppCompatActivity;
import android.view.View;
import android.widget.Button;

public class SubActivity extends AppCompatActivity{


    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);

        Intent a = getIntent();
        String name = a.getStringExtra("name");
        String b = "meeting";
        String name1 = a.getStringExtra("name1");
        String c = "data_input";

        if (name.equals(b)) {

            setContentView(R.layout.activity_sub);

            Button kaigi_return = (Button) findViewById(R.id.kaigi_return);

            kaigi_return.setOnClickListener(new View.OnClickListener() {
                @Override
                public void onClick(View view) {
                    finish();
                }


            });


        }
        if (name1.equals(c)){

            setContentView(R.layout.activity_sub_data);

            Button data_return = (Button)findViewById(R.id.data_return);

            data_return.setOnClickListener(new View.OnClickListener(){
                @Override
                public void onClick(View view){
                    finish();
                }

            });

        }
    }
}

###補足情報(言語/FW/ツール等のバージョンなど)
java、Android Studio

Comments

[yag]

has stoppedがいつ出ているのか明確にお願いします。起動直後？会議等のボタンを押下後？ですか？あと、has stoppedは異常終了ですので、エラーログを見逃している可能性が高いです。再度logcatを確認してみてください。

[mi56]

ボタンを押すとhas stoppedと表示されます。logcatに赤字があればエラーということでしょうか

Answer 1

Intent#getStringExtraでセットされていないキーの値を取得しようとすると、nullが返ります。例えばkaigiボタンを押してSubActivityに移行すると、nameには値が入りますが、name1はnullになるでしょう。そのまま

        if (name1.equals(c)){

が実行されると、NullPointerExceptionが飛びますね。それが原因でしょう。

Comments

[mi56]

原因はわかったのですが、どのように改善すれば良いのでしょうか。
教えていただけると助かります。

[keicha_hrs]

キーをnameとname1の2つに分ける必要があるのでしょうか？

kaigiを押したときは
intent.putExtra("name","meeting");

datainputを押したときは
intent.putExtra("name","data_input");

のようにセットして、SubActivityでは"name"から受け取った値で処理を振り分ければいいのではないかと思いますが。

また、SubActivityで振り分け処理をしなくても、Activityを3つ用意して、MainActivityでは押したボタンによって異なるActivityを呼んでもいいような気もします。

[mi56]

nameに統一したところhas stoppedとは表示されなくなりました。

コードには上2つのボタン(会議、データ入力)のみ実装しているのですが、修正したところ会議ボタンは画面遷移して会議終了を押したら初めの画面に戻れたのですが、データ処理では画面遷移はしたのですがデータ終了を押しても画面が戻らなくなってしまいました。
その原因を教えていただけるとありがたいです。

アクティビティを増やした場合にはできたのですが、2つのアクティビティですっきりとさせたいと考え今回アクティビティを2つに制限しました。

[keicha_hrs]

戻らない原因はこのコードの見た目だとわからないですね。activity_sub_data.xmlに記述されたButtonにdata_returnというIDが間違いなく与えられているのでしょうか。それしか考えられないような。

Answer 2

楽な方法ですとレイアウトだけ用意しておいて切り替える方法が単純でわかりやすいです。

        // hogeページへの遷移
        hogeButton = (Button) findViewById(R.id.hogeButton);
        hogeButton.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                setContentView(R.layout.activity_hoge);
            }
        });

        // hugaページへの遷移
        hugaButton = (Button) findViewById(R.id.hugaButton);
        hugaButton.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                setContentView(R.layout.activity_huge);
            }
        });

Comments

[mi56]

上記のコードを参考にしたところ1つ目のボタンしか反応しません。
どのように改善すればよろしいでしょうか。

[hien]

2つとも同じことをしているだけですので1つ目が動くのであれば2つ目も動くはずなのですが...

挙動としてはレイアウトが3つ(Main,hoge,huga)あり、ボタンで切り替えているだけです。