配列をコレクションのarrayListで書き替えたいです。

Question

前提・実現したいこと

ランダムな整数を５つインプットさせ、その番号を大きい順に並べ替えるプログラムです。

模範解答は、配列をつかっているのですが、これをArrayListを使って書きかえるのが目標です。
ArrayListを引数としてメソッドにいれているのですが、その部分がエラーになっています。
ウェブサイトなどをみてArrayListを引数にいれているコードをみたのですが、同じように使っていると思います。

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

MacBook:src tomifukumoto$ javac Array.java
Array.java:21: エラー: <identifier>がありません
public static void printArrayList(ArrayList<Integer>, array){
^
Array.java:21: エラー: <identifier>がありません
public static void printArrayList(ArrayList<Integer>, array){
^
Array.java:26: エラー: <identifier>がありません
public static ArrayList<Integer> sortIntegers(ArrayList<Integer>, array){
^
Array.java:26: エラー: <identifier>がありません
public static ArrayList<Integer> sortIntegers(ArrayList<Integer>, array){
^
エラー4個

#［自分コード］

import java.util.ArrayList;
import java.util.Scanner;

public class Main {
Scanner scanner = new Scanner(System.in);
    public static void main(String[] args) {
        ArrayList<Integer> numbers= getIntegers(5);
        ArrayList<Integer> sorted=sortIntegers(numbers);
        printArrayList(sorted);
    }

        public static ArrayList<Integer> getIntegers(int capacity){
           ArrayList<Integer> numbers= new ArrayList<>();
           system.out.println("Type in" + capacity + "numbers" );
               for(int i=0; i<capacity-1 ; i++){
               numbers.set(i)=scanner.nextInt();
               }
           return numbers;
        } 

        public static void printArrayList(ArrayList<Integer>, array){
           for(int i=0; i<=array.size(); i++){
               System.out.println(array.get(i));
           }
        }
        public static ArrayList<Integer> sortIntegers(ArrayList<Integer>, array){
          ArrayList<Integer> copyNumbers=new ArrayList<Integer>(array);
          //ディープコピーをした。
          boolean flag=true;
          int temp;
            while(flag){
                 flag=false;
                for(i=0; i<=copyNumbers.size(); i++){
                        if(copyNumbers.get(i)<copyNumbers.get(i+1)){
                         temp=copyNumbers.get(i);
                         copyNumbers.set(i)=copyNumbers.set(i+1);
                         copyNumbers.set(i+1)=temp;
                         flag=true;
                        }
                }
            }
            return copyNumbers;
        }
}

#模範回答コード

package com.timbuchalka;

import java.util.Arrays;
import java.util.Scanner;

public class Main {
    private static Scanner scanner = new Scanner(System.in);

    public static void main(String[] args) {
	    // Create a program using arrays that sorts a list of integers in descending order.
        // Descending order is highest value to lowest.
        // In other words if the array had the values in it 106, 26, 81, 5, 15 your program should
        // ultimately have an array with 106,81,26, 15, 5 in it.
        // Set up the program so that the numbers to sort are read in from the keyboard.
        // Implement the following methods - getIntegers, printArray, and sortIntegers
        // getIntegers returns an array of entered integers from keyboard
        // printArray prints out the contents of the array
        // and sortIntegers should sort the array and return a new array containing the sorted numbers
        // you will have to figure out how to copy the array elements from the passed array into a new
        // array and sort them and return the new sorted array.

        int[] myIntegers = getIntegers(5);
        int[] sorted = sortIntegers(myIntegers);
        printArray(sorted);

    }

    public static int[] getIntegers(int capacity) {
        int[] array = new int[capacity];
        System.out.println("Enter " + capacity +" integer values:\r");
        for(int i=0; i<array.length; i++) {
            array[i] = scanner.nextInt();
        }
        return array;
    }

    public static void printArray(int[] array) {
        for(int i=0; i<array.length; i++) {
            System.out.println("Element " + i + " contents " + array[i]);
        }
    }

    public static int[] sortIntegers(int[] array) {
//        int[] sortedArray = new int[array.length];
//        for(int i=0; i<array.length; i++) {
//            sortedArray[i] = array[i];
//        }
        int[] sortedArray = Arrays.copyOf(array, array.length);

        boolean flag = true;
        int temp;
        while(flag) {
            flag = false;
            // element 0     160
            // element 1     50
            // element 2     40

            for(int i=0; i<sortedArray.length-1; i++) {
                if(sortedArray[i] < sortedArray[i+1]) {
                    temp = sortedArray[i];
                    sortedArray[i] = sortedArray[i+1];
                    sortedArray[i+1] = temp;
                    flag = true;
                }
            }
        }

        return sortedArray;
    }
}

試したこと

arraylistを引数としてメソッドに組み込むところでエラーになっているようなので、arraylistをメソッドにいれているコードを閲覧してみました。
同じような組み込み方で組み込んでいました。

Comments

[javahack]

ネットで見つけた"模範解答"なんか無駄に張らずに自分で書いたコードを質問文に張り付けてください。

タイトルのarrayListがjava.util.ArrayListクラスを指しているなら正しい綴りで書いてください。

[BeatStar]

プログラミングは『こう書けばいい』ではありません。

またエラーメッセージぐらいは読みましょう。

エラーメッセージは怒声でも罵倒でも罵声でもハラスメントでもなく、
コンパイラ等からのメッセージです。

[m.ts10806]

以前得られたアドバイスを何も参考にせず活かさずわが道いくこと決めたなら質問してもアドバイス得られなくなっていくだけかと。

[dodox86]

> ウェブサイトなどをみてArrayListを引数にいれているコードをみたのですが、同じように使っていると思います。

ですから「ように」じゃダメです。ちゃんと内容を理解して消化しないと。見よう見真似でいきなり書き換えるようとするのではなく、ArrayListを使った簡単なコードを自分で書いて試すものです。

[m.ts10806]

可能なら俺が考えた俺の質問テンプレート
じゃなくteratailの質問テンプレート使ってほしいですね。
自分のテンプレートなのに埋まってない項目そのまま出してちゃ読みづらいだけ。項目も多すぎて要件の把握がしづらいですし、ネットで探してきたコードをそのまま提示って、正直文字数の無駄です。

[m.ts10806]

真面目なのはいいことですが、力を入れるべきところを間違えたら単なる徒労。やったわりに得られるものがない。

[swordone]

> 2021年５月中旬から独学でJavaを学習しはじめました。
progateを３巡、ドットインストール１巡、スッキリJavaの本を８割読みました。

何を何巡したかは関係ない。スッキリJava8割読んでも、最初の1割も理解できてない。「読む」とか「勉強する」とかの意味を勘違いしていますね。

[talabagani]

テンプレートを使って書き直してみました。

[m.ts10806]

あまりちゃんとコメント読まれないんですね。
同じような投稿繰り返してることに気づいてください。
投稿が同じ以上、応えもまた同じですよ。

[m.ts10806]

swordoneさん
「〇時間調べました」「もう〇日も詰まってます」というのと同じですね。
見当違いなやり方だと何年やっても同じ。何周しても同じ。
そこに本人が気づいてないだけ。やった気になってるだけで、時間浪費しているだけということに気づかない。
コード書くことだけが目的になってる人にありがち。

Answer 1

今回のエラーメッセージを読むと問題は2つあるようです。

1つは単純なミス(基礎がわかっていれば回避できるもの)で、

もう一つは言語的な制限のようです。

1つ目のエラーを見てみると、

Array.java:21: エラー: <identifier>がありません
public static void printArrayList(ArrayList<Integer>, array){
^

ですね。

まずはArray.javaの21行目を見てください。
(なんか、クラス名が違うようだけどPaiza IOとかで提示したかなんかだと仮定)

よーく見てください。

引数が

ArrayList<Integer>, array

となっています。

プリミティブ型(intとか)だと

int, a

のような状態です。

引数はJavaの場合、コンマ(,)で区切ります。

つまり『変数名』に相当する部分(仮引数名)が無いと解釈されているのです。

基礎がわかっているなら対処できますね。

二個目のエラーも同じ。
(場所が違うが)

Comments

[talabagani]

本当ですね。人の思い込みってすごい。カンマが必要だと思い込んでました。