質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.32%

methodがうまく作れない

解決済

回答 3

投稿

  • 評価
  • クリップ 0
  • VIEW 492

kuzurotto

score 403

独自メソッドを作っていますが、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);
    }
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • 退会済みユーザー

    2016/11/18 10:56

    こちらの質問が他のユーザから「問題・課題が含まれていない質問」という指摘を受けました
    teratailでは、漠然とした興味から票を募るような質問や、意見の主張をすることを目的とした投稿は推奨していません。
    「編集」ボタンから編集を行い、質問の意図や解決したい課題を明確に記述していただくと回答が得られやすくなります。

回答 3

+1

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

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

checkベストアンサー

0

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);とも書ける
}
// 必要なだけオーバーロードを定義する
...

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

0

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

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する必要があります。

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2016/11/19 02:40

    ありがとうございます!

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

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

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

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

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

    キャンセル

  • 2016/11/19 02:51

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

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.32%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

同じタグがついた質問を見る