配列、最大値、最小値

Question

実数型の配列が与えられてメソッドfindRangeを作成し、全ての要素の値に対して以下の計算を行い，それらの値の最大値と最小値を求め表示しなさい。
要素の値をxとするとき，|x|3-|x|+3
とゆうものでプログラムを作ったのですが、
absを使わないとすればどーしたらいいですか？
min.maxの初期値は0でいいのか？
適切な変数名はなにか？
教えて欲しいです。よろしくお願いします！
public static void findRange(double[] array) {

double min = 0;

double max = 0;

for (int i = 0; i < array.length; i++) {

    double a = Math.abs(array[i]);

    double b = a * a * a - a + 3;

    if (i == 0) {

        max = b;

        min = b;

    }

    if (max < b) {

        max = b;

    } else if (min > b) {

        min = b;

    }

}

System.out.print("最大値:" + max + "最小値:" + min);

}

Comments

[kei344]

質問文のコードはコードブロックで囲んでいただけませんか？ ```（バッククオート3つ）で囲み、前後に改行をいれるか、コードを選択して「<code>」ボタンを押すとコードブロックになります。また、質問タグに「JAVA」など言語名を指定してください。

Answer 1

少し難しいかも知れませんが、LouiS0616さんが言及されてない点だけコメントします。

min,maxの初期値

初期値がなんでもよいのはLouiS0616さんのコメント通りですが、初期化は必須です。

int min;
for (int i = 0; i < array.length; i++) {
  ...
}
System.out.println(..., min);

このようなコードを書くとJavaコンパイラーは、

for文の中身が一度も実行されないことが有り得る
＝＞minに何も設定されないケースが有り得る
＝＞にもかかわらずprintlnメソッドの引数で参照している
＝＞それはプログラムの誤り

と見做しコンパイルエラーとなります。

それはさておき、使わないのに無意味な値を初期化するのは面白くないと考える人もいます。そこで、「forループの内部で最初の要素だったらmin,maxを初期化する」のだから「最初から先頭要素で初期化すればいいではないか」という考えも出てきます。そうすると、ループは配列の2番目から始めればよいことになりますね。

int min = array[0];
int max = array[0];
for (int i = 1; i < array.length; i++) {
  if (array[i] < min)
    min = array[i];
  else if (array[i] > max)
    max = array[i];
}
System.out.println(..., min, max);

しかし、配列の要素数が0の場合min, maxの初期化を無条件に行うと存在しない配列要素をアクセスしようとしているという意味でArrayIndexOutOfBoundsExceptionが起こってしまいます(要するに実行時エラー)。それだと困る！という場合は、メソッドの先頭で「要素数が0なら何もせずに処理を終了する」といった配慮をします。

こうした細々とした点を考えるのは最初は混乱するかも知れません。分かりにくいと感じたら無理するよりは自分にとって分かり易いと感じる実装をしておくのもよい考えだと思います。

Comments

[LouiS0616]

コンパイルエラーになりえる点は見逃していました。ありがとうございます。

[KSwordOfHaste]

自分もよくやるのです・・・

Answer 2

absを使わないとすればどーしたらいいですか？

なぜ使ってはいけないのかわかりませんが。
要素が負の数であったら-1をかけるようにすればよいです。

min.maxの初期値は0でいいのか？

どうせ配列の最初の要素を直後に代入するので、初期値はなんでもいいです。
プログラム冒頭で初期化する必要もないです。
追記: 上記一文はちょっと不正確な記述です。KSwordOfHasteさんの回答をご覧ください。

適切な変数名はなにか？

aとかbとかの命名が微妙ですよね。
ご提示の計算を行う静的メソッドを定義すれば、これらを隠蔽できます。

---

質問をする際は、言語を明確にし、適切なタグをつけてください。
この回答はJavaを前提に書いていますが、あっているのでしょうか。

また、teratailにはコードを見やすく表示する機能があります。
編集画面を開き、コードを選択した状態で<code>ボタンを押してください。
そうすると、次のように見やすく表示されます。

class Main {
    public static void main(String[] args) {
        System.out.println("Hello world!");
    }
}

Comments

[退会済みユーザー]

ありがとうございます。
これから気をつけます。

あともう一つ質問なんですが静的メソッドとはなんですか？
教えてください。
何度も質問すみません。

[LouiS0616]

インスタンスの状態に依存しないメソッドのことです。
例えばjava.lang.Math#powなどのように、処理に必要な演算が引数だけで完結し、同じ引数に対して同じ戻り値を返せばいいようなメソッドは静的に出来ます。
Javaでは、staticキーワードをメソッドに与えることによって実現できます。
(一番よく見る静的メソッドは間違いなくmainです。それを真似すればよいです。)