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

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

新規登録して質問してみよう
ただいま回答率
85.35%
Java

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

アルゴリズム

アルゴリズムとは、定められた目的を達成するために、プログラムの理論的な動作を定義するものです。

コードレビュー

コードレビューは、ソフトウェア開発の一工程で、 ソースコードの検査を行い、開発工程で見過ごされた誤りを検出する事で、 ソフトウェア品質を高めるためのものです。

Q&A

解決済

1回答

740閲覧

二分探索法がうまくできません!

ooaai

総合スコア17

Java

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

アルゴリズム

アルゴリズムとは、定められた目的を達成するために、プログラムの理論的な動作を定義するものです。

コードレビュー

コードレビューは、ソフトウェア開発の一工程で、 ソースコードの検査を行い、開発工程で見過ごされた誤りを検出する事で、 ソフトウェア品質を高めるためのものです。

0グッド

0クリップ

投稿2021/04/11 07:38

編集2021/04/11 07:41

アルゴリズムの勉強をしています。二分探索法というものの理解をしようと思い自分で記述してみたのですが意図した出力結果が出ません。

調べてみるとfor文を使っている方がいたので同じように
for(head<=tail){...}
というような記述方法を試してみたのですが無限ループになってしまいます。

class dot{ public static void main(String[] args){ int[] ary = {11, 13, 17, 19, 23, 29, 31}; int head = 0; int tail = ary.length; for(int i = 0; i < ary.length; i++){ int center = (head + tail) / 2; //3 if(ary[center] == 17){ System.out.println((center + 1)+"番目が一致"); }else{ if(ary[center] < 17){ head = center + 1; }else if(ary[center] > 17){ tail = center -1; } } } } }

出力結果↓

3番目が一致 3番目が一致 3番目が一致 3番目が一致 3番目が一致

条件分岐をしているはずなのに毎回「3番目が一致」と表示されるのはなぜなのでしょうか
アドバイスお願いします!

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

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答1

0

ベストアンサー

目的の値を見つけたとき、プリントのあとでただちにfor-loopから抜けていないから。

Java

1class dot{ 2 3 static int find(int[] ary, int target) { 4 int head = 0; 5 int tail = ary.length-1; 6 7 while ( head <= tail ) { 8 int center = (head + tail) / 2; //3 9 if(ary[center] == target){ 10 return center; 11 }else{ 12 if(ary[center] < target){ 13 head = center + 1; 14 }else if(ary[center] > target){ 15 tail = center -1; 16 } 17 } 18 } 19 return -1; 20 } 21 22 public static void main(String[] args){ 23 24 int[] ary = {11, 13, 17, 19, 23, 29, 31}; 25 int[] targets = { 9, 17, 23, 31, 35 }; 26 for ( int target : targets ) { 27 int center = find(ary, target); 28 if ( center >= 0 ) { 29 System.out.printf("%d : %d 番目が一致\n", target, center+1); 30 } else { 31 System.out.printf("%d : ごめんみつからへんわ\n", target); 32 } 33 } 34 35 } 36}

投稿2021/04/11 07:52

episteme

総合スコア16612

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

ooaai

2021/04/11 09:41

見落としてました... メソッドにするといいんですね!ありがとうございます
episteme

2021/04/11 11:06 編集

> メソッドにするといいんですね! メソッドにすることが僕の回答ではないぞ。 メソッドにしたのは 入力/処理/出力 をきっちり分離したいがため。
ooaai

2021/04/12 08:54

もちろんbreakがなかったがためにループから抜けられていなかったということは理解しています 自分の力だと上手にメソッドにまとめることができなかったので参考になりました!という意味です><
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.35%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問