0 
Create an application containing an array that stores 20 double values, 
such as 2.34, 7.89, 1.34, and so on. The application should:
     *      Display the sum of all the array elements
     *      Display the average of all the array elements
     *      Display the largest value of all the array elements

これは Java クラスの割り当てです。私の教授が数値リテラルを使用したと言ったので、私はこれで悪い成績を取りました。私は 40 点満点中 28 点を失いました。ソリューションの設計が下手だったのでしょうか。彼の正確なコメント:

 "The program you submitted uses numeric literals in place of the array’s length. 
 This cause several runtime errors when I change the size of your array and tested the code."

そして私の解決策：

import java.util.Random; 
import java.util.Arrays;

public class MyArray{ 
  public static double[] doubles;

    public static void main(String args[]) {
       MyArray.createDoublesArray();
       MyArray.displayDoublesArray();
       MyArray.displaySum();
       MyArray.displayAverage();
       MyArray.displayTheLargestValue();
    } 

    /*Fill up the double Array class member*/
    public static void createDoublesArray(){
        doubles = new double[20];
        //Create Random object 
        Random r=new Random(); 
        double rangeMin = 1, rangeMax = 9; 
        //Generate random double number - 20 times within the range of 2 to 9
        for(int i=0;i<20;i++) { 
            //Generate random double numbers and round them to the two decimal places
            double randomdouble = Math.round((rangeMin + (rangeMax - rangeMin) * r.nextDouble())*100.0)/100.0; 
            doubles[i] = randomdouble;            
        }         
    }    

    /*Display the double Array*/
    public static void displayDoublesArray(){
        String delimiter;
        Arrays.sort(doubles);
        System.out.println("The double array: ");
        // iterate through all the array elements
        System.out.print("{");
        for(int i=0;i<20;i++) { 
           if(i < 19){
               delimiter = ", ";
           }
           else{
               delimiter = "}";
           }
           System.out.print(doubles[i]+ delimiter);      
        } 
        System.out.println("\n");
    }

    /*Displays the sum of the double array.*/
    public static void displaySum() {
        //initialize the sum with 0
        double sum = 0.0;

        // iterate through all the array elements
        for (int i = 0; i < doubles.length; i++) {
            // add up each element to the sum variable
            sum += doubles[i];
        }

        // display the sum
        System.out.println("The sum is: " + Math.round(sum*100.0)/100.0 + "\n");
    }

    /*Displays the average of the double array.*/
    public static void displayAverage() {
        // initialize the sum with 0
        double sum = 0.0;

        // iterate through all the array elements
        for (int i = 0; i < doubles.length; i++) {
            sum += doubles[i];
        }
        // display the average by dividing the sum to the length of the array
        System.out.println("The average is: " + Math.round((sum / doubles.length)*100.0)/100.0 + "\n");
    }

    /*Displays the largest value from the double array */
    public static void displayTheLargestValue() {
        //initialize the largest value with the first element
        double largestValue = doubles[0];

        //iterate through the remaining elements (ignore the first)
        for (int i = 1; i < doubles.length; i++) {
            // check if "i" element is greater then the current largest value
            if (doubles[i] > largestValue) {
                largestValue = doubles[i];
            }
        }
        // display the largest value
        System.out.println("The largest value is: " + largestValue);
    }
}
4

3 に答える 3

3

次のようなものではなく、

  for(int i=0;i<20;i++)

length彼はあなたにその物件を使ってほしかった

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

さらに、ここで

    //initialize the largest value with the first element
    double largestValue = doubles[0];

doubles空ではないと仮定しています。空の場合、例外がスローされます

于 2013-10-23T20:54:14.877 に答える
2

このコードを簡単に維持できるようにするために、まず最初に作成します。

private final static int SIZE = 20;

createDoublesArray :

 public static void createDoublesArray(){
    doubles = new double[SIZE];
    //Create Random object 
    Random r=new Random(); 
    double rangeMin = 1, rangeMax = 9; 
    //Generate random double number - 20 times within the range of 2 to 9

    double randomdouble = 0.0;  // <- we don't want to initiate double in 'for' loop

    for(int i=0;i<SIZE;i++) { 
        //Generate random double numbers and round them to the two decimal places
        randomdouble = Math.round((rangeMin + (rangeMax - rangeMin) * r.nextDouble())*100.0)/100.0; 
        doubles[i] = randomdouble;            
    }         
}

ディスプレイダブル配列:

    /*Display the double Array*/
       public static void displayDoublesArray(){
        String delimiter;
        Arrays.sort(doubles);
        System.out.println("The double array: ");
        // iterate through all the array elements

        StringBuilder buff = new StringBuilder(); // use buffer 

        buff.append("{");
        for(int i=0;i<SIZE;i++) { 
           if(i < SIZE-1){
               delimiter = ", ";
           }
           else{
               delimiter = "}";
           }
           buff.append(doubles[i]+ delimiter);                  
        } 

        buff.append("\n");

        System.out.println(buff.toString());
    }

私たちのコードはもう少し一般的なようで、SIZE実際にコードを変更せずに (必要に応じて) 変更できます。

于 2013-10-23T21:02:46.140 に答える
1

私はマキシム・ショースティンに同意します...

追加するコメントは 1 つだけです

1) 常に double を使用する必要はありません。たとえば。

double rangeMin = 1, rangeMax = 9; //can be replaced by int.

ref: プリミティブ データ型

于 2013-10-23T21:14:28.297 に答える