0

Нужно поменять максимальное и минимальное значение местами. не понимаю в чём ошибка. Должен выводить "5 2 3 4 1" а выводит "1"

public int swapArrayMaxAndMin(int array[])
    {   
        int max = 0; int min = 0;

        for(int i = 0; i < array.length; i++)
        {
            if(array[max] < array[min])
            {
                max = i;
            }
            if(array[min] > array[max])
            {
                min = i;
            }
        }

        int tmp = array[max];
        array[max] = array[min];
        array[min] = tmp;

        return tmp;
    }

    public void showArray()
    {
        int array[] = new int[] {1, 2, 3, 4, 5};
        
        for(int i = 0; i < array.length; i++)
        {
            System.out.print(array[i] + " ");
        }

        System.out.println(swapArrayMaxAndMin(array));

    }
2
  • Перечитайте свой код и подумайте, что в нём не так, раз он выводит изначальный массив, а потом ещё одно число. И ещё подумайте, когда это будет true array[max] < array[min]?
    – timur
    Commented 14 окт. 2020 в 18:40
  • Хорошо, спасибо! Commented 14 окт. 2020 в 18:41

1 ответ 1

1
int tmp = array[max];
array[max] = array[min];
array[min] = tmp;

return tmp;

место tmp надо возвращать array


if(array[max] < array[min])
            {
                max = i;
            }
            if(array[min] > array[max])
            {
                min = i;
            }

надо поменять на это:

if (array[i] < array[min]){min = i;)
if (array[i] > array[max]){max = i;)
1
  • Спасибо большое, уже решил задачу! Но всё равно очень благодарен за помощь! Commented 14 окт. 2020 в 19:19

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.