配列をディープコピーをして配列を昇順に並び替えたい

Question

出したい実行結果

並び替え前：{ 80 30 60 10 40 70 50 20 }
並び替え後：{ 10 20 30 40 50 60 70 80 }

現状のコード

public int sort(int[] num){
		int [] sortedNum =new int [num.length] ;
		
		for (int i = 0; i<sortedNum.length;i++){
			int min =0;
			int  minIdx = i;
			
			for(int j = j+1; j<sortedNum.length; j++) {
				if(sortedNum[j]<min[minIdx]){
					minIdx=j;
				}
			}
			
			int temp = sortedNum[i];
			sortedNum[i] = sortedNum[minIdx];
			sortedNum[minIdx]=temp;
				
	}
		
	return sort;
	}

public class Main {

    public static void main(String[] args) {
        int[] num = {80, 30, 60, 10, 40, 70, 50, 20};
        MyArrays arrays = new MyArrays();
        int[] sortedNum = arrays.sort(num);
        
        System.out.print("before：{ ");
        for (int i = 0; i < num.length; i++) {
            System.out.print(num[i] + " ");
        }
        System.out.println("}");
        
        System.out.print("after：{ ");
        for (int i = 0; i < sortedNum.length; i++) {
            System.out.print(sortedNum[i] + " ");
        }
        System.out.println("}");
    }
}

現状のコンパイル結果

Toi18Main.java:6: エラー: 不適合な型: intをint[]に変換できません:
int[] sortedNum = arrays.sort(num);
^
.\MyArrays.java:50: エラー: 配列が要求されましたが、intが見つかりました
if(sortedNum[j]<min[minIdx]){
^
.\MyArrays.java:61: エラー: シンボルを見つけられません
return sort;
^
シンボル: 変数 sort
場所: クラス MyArrays
エラー3個

条件

• Mainクラスは変えない
• パッケージ、インポートは使わない

わかっていること

ディープコピーをすること

やってみたこと

https://www.delftstack.com/ja/howto/java/java-deep-copy-array/
https://www.sejuku.net/blog/14067
https://www.javadrive.jp/start/array/index2.html

添付サイトを参考にしたコードの修正

Comments

[xebme]

int[]はプリミティヴ型の配列なのでディープコピーする必要はありません。ディープコピーをどのように理解しているのか説明してください。
↑これは私が間違いました。配列本体は参照がコピーされるのでシャローコピーになります。要素もコピーするのがディープコピーでした。
要素が参照型の配列は、要素自体のディープコピーが必要。こちらのことを考えていました。

[janetto]

解決しましたありがとうございました。

Answer 1

public class MyArrays {
    public int[] sort(int[] num){
		int [] sortedNum =new int [num.length] ;
		
		for (int i = 0; i < num.length;i++){
		    int minIdx = 0;
		    for (int j = i - 1; j >= 0; j--) {
		        if (sortedNum[j] < num[i]) {
		            minIdx = j+1;
		            break;
		        }
	            sortedNum[j+1] = sortedNum[j];
		    }
		    sortedNum[minIdx] = num[i];
		}
		
    	return sortedNum;
	}
}

Comments

[janetto]

解決しましたありがとうございました。

Answer 2

Toi18Main.java:6: エラー: 不適合な型: intをint[]に変換できません:
int[] sortedNum = arrays.sort(num);
^

->int[] じゃなく、int 返してます。↓

public int sort(int[] num){

－－－

.\MyArrays.java:50: エラー: 配列が要求されましたが、intが見つかりました
if(sortedNum[j]<min[minIdx]){
^

-> minは配列じゃないですよ。↓ホラ。

int min =0;

－－－

.\MyArrays.java:61: エラー: シンボルを見つけられません
return sort;
^
シンボル: 変数 sort

-> sortって変数じゃなくてメソッド名じゃないですか？

エラーの箇所やエラー内容、よく見ると原因書いてあります。
変数名はわかりやすく、他と混同しないような名前にしたほうがバグ少なくなるかと思いますよ。

Comments

[janetto]

解決しましたありがとうございました。