methodがうまく作れない

Question

独自メソッドを作っていますが、ifを使っているせいかうまく動作しません。

何かしらのアクションをした時に
渡したいデータをintentにセットし遷移するための関数です。

第一引数：遷移先アクティビティ,（必須）
第二引数：record_id,（必須）
第三引数：String型の配列,（なくても動くようにしたい）
第四引数：Integer型の配列（なくても動くようにしたい）

どうしたら良いでしょうか？
JAVAはあまりやったことがないので、、、知恵をお貸しください。

    //第一引数：遷移先アクティビティ,第二引数：record_id,第三引数：String型の配列,第四引数：Integer型の配列
    private void changeActivity(Class targetActivity, Integer record_id,ArrayList<String> arrOption1,ArrayList<Integer> arrOption2) {
        Intent intent = new Intent(this, targetActivity.class);
        intent.putExtra("record_id", record_id);
        
        ArrayList<String> arrStr = arrOption1 ;
        if(arrOption1 != null){
            for (int i = 0; i < arrOption1.size(); i++) {
                arrStr.add(arrOption1.get(i);
            }
            for (int i = 0; i < arrStr.size(); i++) {
                intent.putExtra("arrStr", arrStr.get(i));
            }
        }
        
        ArrayList<Integer> arrInt = arrOption2 ;
        if(arrOption2 != null){
            for (int i = 0; i < arrOption2.size(); i++) {
                arrInt.add(arrOption2.get(i);
            }
            for (int i = 0; i < arrInt.size(); i++) {
                intent.putExtra("arrStr", arrInt.get(i));
            }
        }
        startActivity(intent);
    }

Answer 1

うまく動きませんじゃわかりません。
想定している動作と、現状の動作で異なることを具体的に書かないと質問になりません。

リファレンスを読んでいますか？
Intent#putStringArrayListExtraやIntent#putIntegerArrayListExtraを使ってください。
また、キーが重複するとセットした値が上書きされます。

Answer 2

changeActivity(target, id)のように第三パラメーター以降を省略可能にしたいということでしょうか。
他の言語ではメソッドを一つだけ定義していくつかのパラメーターを省略したり名前で指定という言語仕様を持っているものもありますね。javaではそうした仕様がありませんので異なる引数のメソッドとしてオーバーロードするほかは手がありません。定義が若干面倒ですが、こうすることで呼び出しの際に引数を省略した記述が可能です。

// 基本のメソッド(引数の省略なしバージョン)
void ca1(A req1, B req2, ArrayList<String> opt1, ArrayList<Integer> opt2) {
 ...
}
// オーバーロード(opt2を省略可能にしたバージョン)
void ca1(A req1, B req2, ArrayList<String> opt1) {
  ca1(req1, req2, opt1, null);
}
// オーバーロード(opt1, opt2を省略可能にしたバージョン)
void ca1(A req1, B req2) {
  ca1(req1, req2, null); // ca1(req1, req2, null, null);とも書ける
}
// 必要なだけオーバーロードを定義する
...

Answer 3

おそらくですが、この問題の根幹は無限ループではないでしょうか？どういう現象が起きていて困っているのかそれを一切書いてくれないので推測ですが。

ArrayList<String> arrStr = arrOption1 ;

この代入により、変数arrStrと変数arrOption1は同じリストを参照します。これがnullでない場合、

for (int i = 0; i < arrOption1.size(); i++) {
    arrStr.add(arrOption1.get(i));
}

これが実行されます(閉じ括弧が足りなかったので追加)。例えばこれらのリストが["a", "b", "c"]という内容だとすると、

i = 0
arrOption1 -> ["a", "b", "c"] <- arrStr
arrOption1.get(i)で"a"を取得、arrStrに"a"追加
arrOption1 -> ["a", "b", "c", "a"] <- arrStr

i = 1
arrOption1.get(i)で"b"を取得、arrStrに"b"追加
arrOption1 -> ["a", "b", "c", "a", "b"] <- arrStr

という動作になり、以降同じような動作になります。継続条件であるi < arrOption1.size()において、arrOption1.size()が順次大きくなっていくため、いつまでたってもループから抜けることができません(※)。これが「上手く動作しない」の元凶ではないでしょうか？

(※)厳密には、ArrayList#size()はintの最大値(2^31-1)以上のサイズのリストに対して呼び出されるとintの最大値を返すので、iがそこまで到達すれば抜けることができる。
###以前の回答

「なくても動くようにしたい」なら、その引数にnullを渡す方法も考えられますが、その引数なしのメソッドをオーバーロードするという手もあります。
つまり、今あるメソッドはそのままに、

private void changeActivity(Class targetActivity, Integer record_id) {
    changeActivity(targetActivity, record_id, null, null);
}
private void changeActivity(Class targetActivity, Integer record_id, ArrayList<String> arrOption1) {
    changeActivity(targetActivity, record_id, arrOption1, null);
}
private void changeActivity(Class targetActivity, Integer record_id, ArrayList<Integer> arrOption2) {
    changeActivity(targetActivity, record_id, null, arrOption2);
}

というメソッドを加えます。

しかし現状のメソッドには問題があり、リストを回している間、arrStrというキーに結びつくオブジェクトが上書きされ続け、最後に記録したものしか残りません。キーとなる文字列も変化させてputExtraするか、いっそArrayListごとputExtraする必要があります。

Comments

[earnest_gay]

ありがとうございます！

確か読んだ本に似たようなこと書いてました。

public String test(String str) {
return str;
}

public Integer test(Integer num) {
return num;
}

型の自動変換でしたかね、、、

発想としては似たようなところでしょうか。
作ってみます。

[swordone]

他の質問もそうですが、余りに基礎がたりなさすぎます。