Поменять местами максимальный и минимальный элемент

@T-Tim · Регистрация: 24.03.2015

Студворк — интернет-сервис помощи студентам

Поменять местами максимальный и минимальный элемент в одномерном массиве
Я не много запутался, помогите в строке 25-27. Ибо выводит точно такой же массив

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
  private void button2_Click(object sender, EventArgs e)
        {
                        int  n;
            try
            {
                string[] b = richTextBox2.Lines;
                n = b.Length;
                int i;
                int[] mass = new int[n];
                int min = 0;
                int max1 = 0;
                //заполнение данных
                for (i = 0; i < n; i++)
                    {
                        mass[i] = Convert.ToInt32(b[i]);
                    }
                //проверка
                for (i = 0; i < n; i++)
                {
                    if (min >mass[i])
                        min = mass[i];
                    if (max1 < mass[i])
                        max1 = mass[i];
                }
                int tmp = mass[min];
                mass[min] = mass[max1];
                mass[max1] = tmp;
               
                //вывод 
                for (i = 0; i < mass.Length; i++)
                    listBox1.Items.Add(mass[i]);
            }
            catch (FormatException) { textBox1.Text = "Неверный формат"; }
            catch (IndexOutOfRangeException) { textBox1.Text = "Индекс вне границ"; }
 
        }

@MVK · 25.05.2016, 20:08

Может быть так будет проще:

C#
1
2
3
4
5
6
7
8
9
int[] fr = { 1, 2, 3, 4, 5, 6 };
            int mm = fr[0]; int imm = 0;
            int mmm = fr[0]; int immm = 0;
            for (int ii=1; ii<fr.Length;ii++)
            {
                if (fr[ii] < mm) { mm = fr[ii]; imm = ii; continue; }
                if (fr[ii] > mmm){mmm= fr[ii]; immm = ii;}
            }
            fr[imm] = mmm; fr[immm] = mm;

@T-Tim · 26.05.2016, 08:23 **[ТС]**

Последний элемент на ноль заменяет

@MVK · 26.05.2016, 08:53

Это как??

@T-Tim · 26.05.2016, 15:01 **[ТС]**

1,2,3. При нажатии на кнопку 3 заменяется 0. Перестановка не происходит. Может быть я что то в коде напутал?

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
private void button2_Click(object sender, EventArgs e)
        {
                        int  n;
            try
            {
                string[] b = richTextBox2.Lines;
                n = b.Length;
                int i;
                int[] mass = new int[n];
                int min = mass[0];
                int max1 = mass[0];
                int imm = 0;
                int immm = 0;
                //заполнение данных
                for (i = 0; i < n; i++)
                    {
                        mass[i] = Convert.ToInt32(b[i]);
                    }
                //проверка
                for (i = 1; i<mass.Length; i++)
                {
                    if (mass[i] < min)
                        min = mass[i];
                        imm = i;
            
                    if (mass[i] > max1)
                        max1 = mass[i];
                    immm = i;
                }
 
                mass[imm] = max1; mass[immm] = min;
 
              /*  int tmp = mass[min];
                mass[min] = mass[max1];
                mass[max1] = tmp;*/
               
                //вывод 
                for (i = 0; i < mass.Length; i++)
                    listBox1.Items.Add(mass[i]);
            }
            catch (FormatException) { textBox1.Text = "Неверный формат"; }
            catch (IndexOutOfRangeException) { textBox1.Text = "Индекс вне границ"; }
 
        }

@MVK · 26.05.2016, 17:55

А скобочки нужно ставить:

C#
1
2
3
4
5
6
7
8
9
10
if (mass[i] < min)
{
                        min = mass[i];
                        imm = i;
            }
                    if (mass[i] > max1)
{
                        max1 = mass[i];
                    immm = i;
}

@T-Tim · 27.05.2016, 14:43 **[ТС]**

Теперь перестановка происходит, но последний элемент на 0 всё равно заменяется

@MVK · 27.05.2016, 16:12

Выложи код. Посмотрим

@T-Tim · 27.05.2016, 21:05 **[ТС]**

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
  private void button2_Click(object sender, EventArgs e)
        {
                        int  n;
            try
            {
                string[] b = richTextBox2.Lines;
                n = b.Length;
                int i;
                int[] mass = new int[n];
                int min = mass[0];
                int max1 = mass[0];
                int imm = 0;
                int immm = 0;
                //заполнение данных
                for (i = 0; i < n; i++)
                    {
                        mass[i] = Convert.ToInt32(b[i]);
                    }
                //проверка
                for (i = 1; i<mass.Length; i++)
                {
                    if (mass[i] < min)
                    {
                        min = mass[i];
                        imm = i;}
                
                    if (mass[i] > max1)
                    {
                        max1 = mass[i];
                    immm = i;}
                }
 
                mass[imm] = max1; mass[immm] = min;
 
               
                //вывод 
                for (i = 0; i < mass.Length; i++)
                    listBox1.Items.Add(mass[i]);
            }
            catch (FormatException) { textBox1.Text = "Неверный формат"; }
            catch (IndexOutOfRangeException) { textBox1.Text = "Индекс вне границ"; }
 
        }

@chronicler · 27.05.2016, 21:43

T-Tim,

Ошибка, потому что

C#
1
2
                int min = mass[0];
                int max1 = mass[0];

Нужно присваивать после отработки цикла:

C#
1
for (i = 0; i < n; i++)

иначе, если в массиве нет нулей, min будет все время == 0

вот так примерно нужно сделать:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
                string[] b = richTextBox2.Lines;
                n = b.Length;
                int i;
                int[] mass = new int[n];
 
                int imm = 0;
                int immm = 0;
 
                //заполнение данных
                for (i = 0; i < n; i++)
                {
                    mass[i] = Convert.ToInt32(b[i]);
                }
 
                int min = mass[0];
                int max1 = mass[0];
 
                //проверка
                for (i = 0; i < mass.Length; i++)
                {
                    if (mass[i] < min)
                    {
                        min = mass[i];
                        imm = i;
                    }
 
                    if (mass[i] > max1)
                    {
                        max1 = mass[i];
                        immm = i;
                    }
                }
 
                mass[imm] = max1;
                mass[immm] = min;

@MVK · 27.05.2016, 22:54

Я изменил три строки в Вашем коде: все работает.

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
int n = 10;
            int i;
            int[] mass = new int[n];
 
            int imm = 0;
            int immm = 0;
 
            //заполнение данных
            for (i = 0; i < n; i++)
            {
                mass[i] = i;
            }
 
            int min = mass[0];
            int max1 = mass[0];
 
            //проверка
            for (i = 1; i < mass.Length; i++)
            {
                if (mass[i] < min)
                {
                    min = mass[i];
                    imm = i;
                }
 
                if (mass[i] > max1)
                {
                    max1 = mass[i];
                    immm = i;
                }
            }
 
            mass[imm] = max1;
            mass[immm] = min;

Можно, конечно, цикл начинать с нулевого элемента, но первая итерация будет лишней. Минимум и ноль никак не связаны. Кстати, что Вы вводите в richTextBox?

@T-Tim · 28.05.2016, 15:34 **[ТС]**

Спасибо за помощь.
в richTextBox я ввожу элементы массива

Новые блоги и статьи Все статьи Все блоги /
Символьное дифференцирование igorrr37 13.02.2026 / * Программа принимает математическое выражение в виде строки и выдаёт его производную в виде строки и вычисляет значение производной при заданном х Логарифм записывается как: (x-2)log(x^2+2) -. . .	Камера Toupcam IUA500KMA Eddy_Em 12.02.2026 Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .	И ясному Солнцу zbw 12.02.2026 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.	«Знание-Сила» zbw 12.02.2026 «Знание-Сила» «Время-Деньги» «Деньги -Пуля»
SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 12.02.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 11.02.2026 Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL3_image 8Observer8 10.02.2026 Содержание блога Библиотека SDL3_image содержит инструменты для расширенной работы с изображениями. Пошагово создадим проект для загрузки изображения формата PNG с альфа-каналом (с прозрачным. . .	Установка Qt-версии Lazarus IDE в Debian Trixie Xfce volvo 10.02.2026 В общем, достали меня глюки IDE Лазаруса, собранной с использованием набора виджетов Gtk2 (конкретно: если набирать текст в редакторе и вызвать подсказку через Ctrl+Space, то после закрытия окошка. . .

@T-Tim 1 / 1 / 0 Регистрация: 24.03.2015 Сообщений: 206
	26.05.2016, 08:23 [ТС]
	Последний элемент на ноль заменяет 0

@MVK 160 / 148 / 92 Регистрация: 30.03.2011 Сообщений: 569
	26.05.2016, 08:53
	Это как?? 0

@T-Tim 1 / 1 / 0 Регистрация: 24.03.2015 Сообщений: 206
	27.05.2016, 14:43 [ТС]
	Теперь перестановка происходит, но последний элемент на 0 всё равно заменяется 0

@MVK 160 / 148 / 92 Регистрация: 30.03.2011 Сообщений: 569
	27.05.2016, 16:12
	Выложи код. Посмотрим 0

@T-Tim 1 / 1 / 0 Регистрация: 24.03.2015 Сообщений: 206
	28.05.2016, 15:34 [ТС]
	Спасибо за помощь. в richTextBox я ввожу элементы массива 0