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

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

ただいまの
回答率

90.51%

  • Java

    15807questions

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

配列を使った数値処理

解決済

回答 4

投稿

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

baio64

score 1

現在javaの課題に取り組んでいます。内容は配列を用意し最小値、最大値、平均を求め、後に昇順にソートするというプログラムです。実行結果は正常ですが、もっと効率が出来ないか考えています。
import java.io.*;
import java.util.Arrays;
class Stat1
{
static void disp_all_data( int[] adata )
{
System.out.print("[ ");
for( int i = 0; i < adata.length-1; i++ )
System.out.print( adata[i] + ", ");
System.out.print( adata[adata.length-1] );
System.out.println(" ]");
}
// メソッドを使って平均値を求める
/*static float average( int[] adata )
{
return average;
}*/
public static void main(String[] args) throws IOException
{
int datanum;    
int[] data;    
int min, max;    
float sum, ave;    
BufferedReader br = new BufferedReader(new InputStreamReader(System.in));
System.out.println("処理するデータの個数を入力して下さい。");
datanum = Integer.parseInt(br.readLine());
System.out.println( datanum +"個のデータを入力して下さい。");
data= new int [datanum];
sum=0;
for(int i=0;i<data.length;i++)
{
data[i]=Integer.parseInt(br.readLine());
sum=sum+data[i];
}
System.out.println("入力された配列データ:");
disp_all_data(data);
min=data[0];
max=data[0];
ave=0;
for(int i=0;i<data.length;i++)
{
if(max<data[i])
{
max=data[i];
}
if(min>data[i])
{
min=data[i];
}
}
ave=sum/datanum;
System.out.println("最小値:"+ min +"、最大値:"+ max +"、平均値:"+ ave);
Arrays.sort(data);
System.out.println("ソート後の配列データ:");
disp_all_data(data);
}
}

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

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

  • mts10806

    2017/05/28 21:01

    コード部分は選択して</>ボタンを利用してコード記法にしてください。

    キャンセル

  • swordone

    2017/05/28 21:48 編集

    だいぶ前からコードマークダウンのボタン変わってるのよ 
    質問編集画面でコードを反転選択し、画面上部の<code>を押してください。

    キャンセル

  • mts10806

    2017/05/29 08:07

    あ本当ですね。質問上げたことなかったので回答と同じに思ってました。(なんとなく伝わればいいな・・・)

    キャンセル

回答 4

checkベストアンサー

+3

課題ということでStreamなどの「既にある機能を使うのではなく、自分でアルゴリズムを実装する練習をする」のが出題者の意図と捉えてコメントします。短いコードなのでいずれも細かい指摘です。

効率化

  • ループ開始位置
    min, maxをdata[0]で初期化しています。さてループの開始インデックスは0にしてますがそれは妥当でしょうか?

  • 最小と最大
    if文を常に2回実行しています。でも「最大値より大きい」ならそれは「最小値にはなり得ません」よね?

  • 合計値(sum)の型
    floatにする合理的な理由がありません。集計が遅くなるだけです。int(あるいはオーバーフローを気にするならlong)が妥当と思います。

  • 平均値(ave)の初期化
    aveに0を代入していますが無意味です。これについては別の指摘もあります(後述※)。

その他の指摘

  • 変数の宣言位置(※)
    mainメソッドの先頭で変数を宣言していますが、よい習慣とは言えません。使わない変数を必要のない時点で宣言することは、変数のスコープを無暗に広くするだけでよいこととは言えないと思います。先に指摘したaveへの無駄な初期値設定ですが、aveを最初に計算する時点で宣言する習慣があればこのような無駄なコードを書くこともなくなると思います。

  • 機能の分離
    takasima20さんが既に指摘されてますが、mainメソッドで「データの入力」「計算」「表示」を全て行っていますね。それでもこの課題の回答としてはよいかも知れません。ただ、今後もっと大きなプログラムを書く場合に「様々な機能をごちゃっと一つのメソッドに押し込むこと」はイマイチであることに気づかれるだろうと思います。例えばmainメソッドにある計算部分ですが、これを複数個所で用いたくなった場合、データの入力や表示の論理と分離していた方が都合がよいですね。それら個々の「最小限の機能の単位」でメソッドに分割して実装すれば、mainメソッドは次のようにスッキリしますし、プログラムの発展性(機能追加のしやすさ)もよくなると思います。

public static void main(String[] args) {
  int[] data = inputData();
  sort(data);
  Statistics statistics = calculateStatistics(data);
  outputData(data);
}
  • コーディングスタイル
    非常に細かな点で恐縮ですが、スペースを空けていたり開けてなかったりスタイルに乱れを感じます。「そんな細かなことを一々気にしなくても・・・」と思われるかも知れませんが、多分経験を積んだプログラマーのコードを意識して調べてみると「統一されたスタイルで整然と書かれている」ことに気づくと思います。多分ある程度経験を積んだプログラマーは「ほとんど神経を使わず自然にそう書いている」と思います。

ついでですが、人に見せるコードとして字下げされてないコードはNGと言わざるを得ません。質問コメントでも指摘されてますね。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/05/29 16:05

    回答ありがとうございます。授業では気づけない部分と細かい指摘について初心者の私にも理解しやすかったのでベストアンサーにさせていただきました。「字下げ」の部分は申し訳ありません、軽率でした。以後気を付けます。

    キャンセル

+1

どうせソートするなら先にソートすればいいのでは?
ソート後最初の要素が最小、最後の要素が最大になっています。

実はStreamで最大最小平均まとめて出す方法があったり

// 数値を半角スペース区切りで入力されていること前提
int[] array = Arrays.stream(br.readLine().split(" "))
                  .mapToInt(Integer::parseInt).sorted().toArray();
IntSummaryStatistics iss = Arrays.stream(array).summaryStatistics();
System.out.println("最小値:"+ iss.getMin() +"、最大値:"+ iss.getMax() +"、平均値:"+ iss.getAverage()); 

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

+1

もっと効率が出来ないか考えています

"効率化できないか" の誤記だとして、提示されたプログラムは手動でデータを入力する格好なので、効率化する意味はあんまりありません。

あとは Java の Stream API に sum も max も average も全部実装済みなので使えばいいと思います。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

0

転記ミスかもしれませんが、disp_all_data() 定義の閉じ括弧
抜けてないスか?

で、何を求めてらっしゃるのかよく分かりませんが、
とりあえず、機能分割して関数に分けると
少しは(ぱっと見て)分かりやすくなると思います。

あとは、if で比較するんでなく Math.max() 使うとか?

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

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

  • Java

    15807questions

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