Старайтесь как можно раньше начать использовать ООП вместо процедурной парадигмы. Выбирайтесь из метода main(). Главная задача - читаемость кода! Для этого для начала помните всего несколько простых правил: 1)осмысленное именование переменных, соблюдение конвенции об именовании; 2)чтение множества простейших (коротких) методов на много понятнее, чем чтение одного большого метода (принцип "разделяй и властвуй") хотя бы просто потому, что у метода есть осмысленное имя , что в сочетании с простейшим кодом дает быстрое понимание о том, как он работает и что он делает; 3) статика в большей части случаев - зло (почему - поймете со временем); 4)что бы вам не говорили, код написанный в ООП является самым читаемым и простым для понимания.
Я сознательно не пытался сделать класс универсальным и использовать интерфейсы, чтобы не запутывать вас лишний раз.
Предлагаю дополнительно разобрать такое решение. В нашей задаче нам необходимо выполнять над целочисленным массивом какие-либо дополнительные операции (которых нет из коробки у массива). Тогда давайте сделаем класс-обертку для массива, который будет хранить массив,а методы нашего класса будут выполнять все необходимые задачи. Добавим 2 конструктора : первый будет принимать готовый массив, а второй - параметры для того, чтобы сгенерировать массив с рандомными числами. Мы знаем нашу задачу, поэтому найти индексы максимального и минимального значение мы можем сразу (сами значения нам хранить не надо, ведь у массивов временная сложность доступа элементу по индексу О(1)). теперь вся информация, необходимая нам, подготовлена. Остается добавит методы для получения индексов и значений макс. и мин. элементов, а также метод , меняющий их местами, и метод вывода массива в консоль. Вот и все. Теперь вы можете использовать класс-обретку неограниченное количество раз, создавая экземпляры класса, мало того, вы можете добавлять новые методы, оптимизировать их выполнение (например, замена местами максимального и минимального элементов в нашей реализации больше не требует итераций, а их поиск при инициализации выполняется в один проход, вместе с тем вы можете использовать ленивую инициализацию, выполняя поиск макс. и мин. элементов при обращении к соответствующим методам и т.д.). Как видите, даже в таком простом примере можно получит ряд преимуществ при использовании такого подхода. Можете задавать вопросы, если они появятся.
public class IntArrayWrapper {
private final int [] array;
private int minIndex, maxIndex;
public IntArrayWrapper(int [] array) {
this.array = array;
findMinMaxIndex();
}
public IntArrayWrapper(int size, int minr, int maxr) {
this(new java.util.Random().ints(size, minr, maxr).toArray());
}
private void findMinMaxIndex() {
for (int i = 0; i < array.length; i++) {
if (array[i] < array[minIndex]) minIndex = i;
if (array[i] > array[maxIndex]) maxIndex = i;
}
}
public void replaceMaxMin() {
int m = array[maxIndex];
array[maxIndex] = array[minIndex];
array[minIndex] = m;
m = minIndex;
minIndex = maxIndex;
maxIndex = m;
}
public int getMinIndex() {
return minIndex;
}
public int getMaxIndex() {
return maxIndex;
}
public int getMinValue() {
return array[minIndex];
}
public int getMaxValue() {
return array[maxIndex];
}
@Override
public String toString() {
return java.util.Arrays.toString(array);
}
}
public class task_4_4 {
public static void main(String[] args) {
IntArrayWrapper wrapper = new IntArrayWrapper(20, -20, 20);
System.out.println(wrapper.toString());
System.out.println("Наименьший элемент: " + wrapper.getMinValue() + " его индекс: " + wrapper.getMinIndex());
System.out.println("Максимальный элемент: " + wrapper.getMaxValue() + " его индекс: " + wrapper.getMaxIndex());
wrapper.replaceMaxMin();
System.out.println("************************************************");
System.out.println(wrapper.toString());
System.out.println("Наименьший элемент: " + wrapper.getMinValue() + " его индекс: " + wrapper.getMinIndex());
System.out.println("Максимальный элемент: " + wrapper.getMaxValue() + " его индекс: " + wrapper.getMaxIndex());
}
}