Пожалуйста, помогите Пытаясь разобраться массив Java

Question

public class selectionsorter 
    { 
public selectionsorter(int[] x) 
{ 
    theArray=x; 
} 
public void sort() 
{ 
    for(int i=0; i<theArray.length-1;i++) 
    { 
     start=i; 
     findMinPos(); 
    } 
} 
public void findMinPos() 
{ 
    int minpos=start; 
    for(int i=0;i<theArray.length;i++) 
    { 
     if(i>start) 
     { 
     if(theArray[i]<theArray[start]) 
     { 
     start=i; 
     } 
     } 
    } 
    swap(); 
} 
public void swap() 
{ 
    temp=theArray[start]; 
    theArray[start]=theArray[minpos]; 
    theArray[minpos]=temp; 
} 
private int[] theArray; 
private int minpos; 
private int start; 
private int temp; 
    } 

Tester Файл

public class selectionsortertester 
    { 
public static void main(String[] args) 
{ 
    int[] x ={3,7,5,6,9,2}; 
    selectionsorter y=new selectionsorter(x); 

    y.sort(); 
    for(int i=0; i<x.length;i++) 
    System.out.print(x[i]+" "); 
} 
    } 

Я хочу, чтобы отсортировать массив его от низшего к высшему, и это делает первый номер, а выход «2 7 5 6 9 3" Пожалуйста, помогите и спасибо кто-нибудь знает, почему он это делает, и как я могу это исправить, спасибо

Answer 1

Вы можете сделать это таким образом

Arrays.sort(x); 

См. Документы here.

Answer 2

Проблема в том, что вы сохранили все переменные как Class attributes вместо локальных переменных в методах, и все ваши методы изменяют переменные класса.

Это неверно.
Удалите их из класса

private int minpos; 
private int start; 
private int temp; 

Теперь передать значения в качестве аргументов метода:

public class SelectionSorter { 
    public SelectionSorter(int[] x) { 
     theArray = x; 
    } 

    public void sort() { 
     for (int i = 0; i < theArray.length - 1; i++) { 
      findMinPos(i); 
     } 
    } 

    public void findMinPos(int start) { 
     int minpos = start; 
     for (int i = start + 1; i < theArray.length; i++) { 
      if (theArray[i] < theArray[start]) { 
       minpos = i; 
      } 
     } 
     if (minpos != start) { 
      swap(minpos,start); 
     } 
    } 

    public void swap(int minpos, int start) { 
     int temp = theArray[minpos]; 
     theArray[minpos] = theArray[start]; 
     theArray[start] = temp; 
    } 

    private int[] theArray; 
} 

См this SO question к learm больше о Атрибуты класса