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

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

ただいまの
回答率

90.45%

  • Java

    16212questions

    Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

  • Android Studio

    4372questions

    Android Studioは、 Google社によって開発された、 Androidのネイティブアプリケーション開発に特化した統合開発ツールです。

可変引数すべてを比較したい

解決済

回答 3

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 278
退会済みユーザー

退会済みユーザー

実現したいコード(※書きたいコードではない)

  public void answer(int num1,int num2,int num3) {
        //3つならまだしも100とかだと大変
        if (num1 == 1 || num2 == 1 || num3 == 1) {

            ANSWER = true;
        }
       //すべて&&でつなげることが重要
        if (num1 == 0 && num2 == 0 && num3 == 0) {
            ANSWER = false;
        }


    }


可変引数を使ったやり方(試したこと)

private boolean ANSWER;
   public void answer( int... nums) {

        for (int i = 0; i < nums.length; i++) {
            for (int j = 0; j < nums.length; j++) {
                /*
                * 問題の部分での重複を防ぐ
                * num[0]==0&&num[0]==0等では機能しないため
                *
                * */
                if (i == j && j != nums.length - 1) {
                    continue;
                }
                if (i == j && j == nums.length - 1) {
                    break;
                }
                /*--------------------------------*/

                while (nums[i] == 1) {

                    ANSWER= true;
                    break;
                }

 /*
    * 問題の部分
    * この書き方だと最後尾しか判定されない
    * ここで「可変引数内すべてが0である場合は」としたい*/
                while (nums[i] == 0 && nums[j] == 0) {

                    ANSWER = false;

                    break;
                }


            }
        }

    }


といった感じです
for分は繰り返しとはいえ、判定するのは一往復一数字のみですよね?
そこで、その判定中の一点の数字以外全ても引き合いに出したいのですが何かいい方法はありますか?

 追記

説明が下手くそで、すみません。。。

目的はみなさんの言う通り
1が一つでも含まれていたらANSWERをtrue、 全てが0ならANSWERをfalse
です。

上記のコードはそのクラスでは動かしませんインスタンス生成で違うクラス使います
このクラスでしてほしいことはANSWERがtrueかfalseかのみです。
そして、使われた場合numsに入れ込むのもまた違うクラスからとってきたゲッターです。
そのゲッターは変数なので排他的関係にはなりません。
(ちなみに、それぞれ違うタイミングでに0~2に変動)

//上記クラスをAクラスとし
//使うクラスをC、ゲッターはBとするなら


A.answer(B.num,B2.num,B3.num);


ちなみにお時間とらして恐縮なのですが、
私のアホみたいなコードだとなぜ最後尾しか判定されないのでしょうか?
私のイメージだとiもjもずっとグルグル回ってるイメージなのですが、、、
すみません答えだけでなく、なぜ間違ったのかも理解したくて。

  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

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

  • unz.hori

    2018/03/04 20:08

    処理の仕様がよくわかりません。簡潔に説明を質問に追記してください。

    キャンセル

  • asahina1979

    2018/03/04 20:11

    すべて同じ値であったらならば HashSet に入れなおして SIZEが1かどうかじゃいかんの?

    キャンセル

  • umyu

    2018/03/04 20:28

    質問文の比較条件は排他的な関係では? 配列のいずれかに1が入っている or 配列が全て0

    キャンセル

  • 退会済みユーザー

    退会済みユーザー

    2018/03/04 22:02

    Hashsetはよく知らないのですが調べた感じだとそれでもいける気がするような、わからないような、、

    キャンセル

回答 3

checkベストアンサー

+2

要するに配列に対して次の二つが実現できればいいのですね?

  • 1が一つでも含まれるかのチェック
  • 全て0であるかのチェック

愚直に書くならこんな感じになるんじゃないでしょうか。

static private boolean includeOne(int[] nums) {
    for(int i = 0; i < nums.length; ++i) {
        if(nums[i] == 1) return true;
    }
    return false;
}
static private boolean allZero(int[] nums) {
    for(int i = 0; i < nums.length; ++i) {
        if(nums[i] != 0) return false;
    }
    return true;
}

public void answer(int... nums) {
    if(includeOne(nums)) ANSWER = true;
    if(allZero(nums)) ANSWER = false;

    ...
}

実際には要件に応じてもうちょっと柔軟なコードにしますが。

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/03/04 22:07

    これだ!!って思ったんですが
    質問の追記に書いた通りメゾットとして使うので、int[] numsのところに外部から入れ込みたいのですが、
    これだとできませんよね?
    あと、仮に百個入れるとしたら、includeOneもallZeroも合わせると二百個になるんですがここは、一つにはできませんか?

    キャンセル

  • 2018/03/04 22:26

    メソッドanswerだけ呼び出せれば問題ないはずですが...
    includeOneが100回呼び出されるときは、answerが100回呼び出されたときです。
    与える変数の長さや出どころには依存しないかと。

    キャンセル

  • 2018/03/04 23:50

    あ!全然これでいけました
    勘違いしてました。answerだけ呼び出しでいけますね。すみません。
    御足労おかけしました。

    キャンセル

+1

どれかが1ならtrue というならこうなるかと。

   public void answer( int... nums) 
   {
        ANSWER= false;

        for (int i = 0; i < nums.length; i++) {
            if(nums[i]==1){
               ANSWER= true;
               break;
            }
        }


       // どれかが0でないならtrue
        ANSWER= false;
        for (int i = 0; i < nums.length; i++) {
            if(nums[i]!=0){
               ANSWER= true;
               break;
            }
        }

    }

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/03/04 22:10

    回答ありがとうございます。
    どうやら、LouiS0616さんの回答が私の求めているものに近いようです。

    キャンセル

0

私のアホみたいなコードだとなぜ最後尾しか判定されないのでしょうか?

nums[i] == 1が成立してもnums[i] == 0 && nums[j] == 0が成立しても、forのループは最後まで回り続けます。
その間、各条件を満たすたびにANSWERの値は上書きされます。
結果、最後に上書きした値だけが残ります。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/03/05 00:09

    なるほど。
    一応ぐるぐる回ってはくれているんですね。
    他のところで使って気づかなかったのは途中で処理自体を強制終了させていたから、途中結果が辛うじて返ってきたわけですか。
    つまり処理自体に影響力がない場合、一通り最後までやってその都度結果を返し、最終的に最新の情報だけ残る、と
    分かりました。ありがとうございます。

    キャンセル

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

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

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

  • Java

    16212questions

    Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

  • Android Studio

    4372questions

    Android Studioは、 Google社によって開発された、 Androidのネイティブアプリケーション開発に特化した統合開発ツールです。