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

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

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

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

Q&A

解決済

2回答

470閲覧

メソッドを用いた配列の問題

PiROt

総合スコア1

Java

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

0グッド

0クリップ

投稿2020/08/02 09:19

前提・実現したいこと

配列の中身を小さい順に並び変えるメソッドを作ろうとしています。
(配列の最小値を求めるメソッドを用いて)
方法としては、配列の最小値を取り出して、新しい配列の最初の要素に代入します。そのあとその配列の最小値があった部分にintの型で表せる最大の数を入れてから、また最小値を求めていくという繰り返しを用いていこうと考えています。

発生している問題・エラーメッセージ

出力した配列において、どうしても配列のすべての値が元の配列の最小値のみで構成されてしまいます。

該当のソースコード

Java

1// 配列の並べ替えを行うメソッド 2import java.util.Scanner; 3 4class Hainara { 5 6 static int c = 2147483647; 7 static int d = 0; 8 9// 配列の最小値を取り出すメソッド 10 static int Haisyou(int[] x) { 11 for(int i = 0;i < x.length; i++) { 12 if(c > x[i]) c = x[i]; 13 } 14 return c; 15 } 16 17// 配列の並べ替えを行うメソッド 18 static int[] Hainara(int[] x) { 19 int [] b = new int[x.length]; 20 for(int i = 0;i < x.length; i++) { 21 b[i] = Haisyou(x); 22 for(int j = 0;j < x.length; j++) { 23 if(b[i] == x[j]) { 24 x[j] = 2147483647; break; 25 } 26 } 27 } 28 return b; 29 } 30 31 32 public static void main(String[] args) { 33 Scanner stdIn = new Scanner(System.in); 34 35 System.out.print("要素数:"); 36 int n = stdIn.nextInt(); 37 int [] a = new int[n]; 38 39 for(int i = 0;i < a.length; i++){ 40 System.out.print("a["+i+"]:"); 41 a[i] = stdIn.nextInt(); 42 } 43 44 int [] e = Hainara(a); 45 46 System.out.println("[配列]"); 47 48 for(int i = 0;i < a.length; i++){ 49 System.out.print(e[i]+" "); 50 } 51 } 52}

試したこと

配列のどの場所に最小値が位置しているのかがこの出力結果に影響すると思ったので、様々な場合の配列で出力結果を調査してみました。ですが、どれも結果は変わりませんでした。

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

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

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

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

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

dodox86

2020/08/02 10:13

Eclipseなどのデバッガーは使っていないのでしょうか。もし初心なのであれば、書いたコードと実行結果だけで原因を突きとめるのは(不可能ではないものの)困難です。ちなみに本質問の問題の原因は、static int c; で一番最初の検索で一番小さい値を保存したあと、ずっとそれが使われ続ける(Haisyouメソッドでそれが返される)せいです。これで分かりますか?
PiROt

2020/08/02 10:46

理解できました!ありがとうございます。デバッガーについても今後のために調べておきたいと思います。 もしよろしければですが、一番小さい値を更新する方法はご存知でしょうか?
guest

回答2

0

ベストアンサー

c を Haisyouメソッドのローカル変数にすればよいでしょう。

diff

1 static int Haisyou(int[] x) { 2- for(int i = 0;i < x.length; i++) { 3+ int c = x[0]; 4+ for(int i = 1; i < x.length; i++) { 5 if(c > x[i]) c = x[i]; 6 }

static int c;static int d; も使っていないので削除してよいでしょう。

投稿2020/08/03 16:57

編集2020/08/03 18:35
kazuma-s

総合スコア8224

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

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

PiROt

2020/08/04 05:11

ありがとうございます!
guest

0

Haishouメソッドを1回実行すると、cが配列最小値と同じ値になります。これはstatic変数なので、2回目以降の呼び出しの際にも保持されます。
2回目以降、配列の値はcの値以上のものしかないため、cは更新されず、全てcの値を返すことになります。

投稿2020/08/02 11:26

swordone

総合スコア20651

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

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

PiROt

2020/08/02 11:45

ありがとうございます! もしよろしければですが、保持されないように呼び出すことは可能でしょうか?
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.46%

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

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

質問する

関連した質問