Сортируются все элементы, кроме первого

@takhvatulin · Регистрация: 04.04.2014

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

Всем привет!
Пишу сортировку, но где то закралась ошибка:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
int A[5];   
A[0] = 6;
A[1] = 7;
A[2] = 3;
A[3] = 10;
A[4] = 5;
int q;
for(int i = 0; i < 5; i++)
{
    if(A[i] > A[i+1])
    {
        q = A[i];
        A[i] = A[i+1];
        A[i+1] = q;
        i = 0;
    }
}

первый элемент остается "6", остальное сортируется нормально

Ilot · 11.07.2014, 13:47

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
int A[5];
A[0] = 6;
A[1] = 7;
A[2] = 3;
A[3] = 10;
A[4] = 5;
int q;
 
for(int j = 0; j < 4; ++j)
{
    for(int i = 0; i < 4 - j; ++i)
        if(A[i] > A[i+1])
        {
            q = A[i];
            A[i] = A[i+1];
            A[i+1] = q;
            i = 0;
        }
}

@castaway · 11.07.2014, 13:47

Если i = 4, тогда на что ссылается A[i + 1] ?

@takhvatulin · 11.07.2014, 13:48 **[ТС]**

Сообщение от castaway

Если i = 4, тогда на что ссылается A[i + 1] ?

Это я только что подправил, но это не имеет отношения к проблеме с первым элементом

@pesenka1 · 11.07.2014, 13:49

http://ru.wikibooks.org/wiki/%... 0%BE%D0%BC

@takhvatulin · 11.07.2014, 13:50 **[ТС]**

pesenka1, не надо мне статей, я сам хочу понять логику и написать. Я написал код, он должен работать, судя по моей логике, но почему то первый элемент не сортируется, видимо из за какой то невнимательной ошибке, которую я в упор не вижу(

@Supernatural · 11.07.2014, 13:53

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
void main()
{
int A[5];
A[0] = 6;
A[1] = 7;
A[2] = 3;
A[3] = 10;
A[4] = 5;
int q;
for(int i = 0; i < 5; i++)
    for(int j = 0; j < 5 - i - 1; j++)
{
    if(A[i] > A[i+1])
    {
        q = A[i];
        A[i] = A[i+1];
        A[i+1] = q;
        i = 0;
    }
}
for(int i = 0; i < 5; i++)
cout<<A[i]<<" ";
system("pause");
}

@takhvatulin · 11.07.2014, 13:54 **[ТС]**

Supernatural, а в чем моя ошибка заключается?

@Supernatural · 11.07.2014, 13:57

Сообщение от takhvatulin

а в чем моя ошибка заключается?

добавил внутренний цикл, этот метод пузырька вроде
он описан на википедии

@takhvatulin · 11.07.2014, 13:59 **[ТС]**

Supernatural, да елки палки, не интересует меня другой метод, конкретно в моем методе в чем ошибка?
P.S. если б хотел тупо сортировать, а не научится сортировать, взял бы готовый код и не парился.

@castaway · 11.07.2014, 14:01

А что ты там подправил ты покажешь?

@takhvatulin · 11.07.2014, 14:04 **[ТС]**

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
int A[5];   
A[0] = 6;
A[1] = 7;
A[2] = 3;
A[3] = 10;
A[4] = 1;
int q;
for(int i = 0; i < 4; i++)
{
    if(A[i] > A[i+1])
    {
        q = A[i];
        A[i] = A[i+1];
        A[i+1] = q;
        i = 0;
    }
}

последний вариант мой такой

@Supernatural · 11.07.2014, 14:06

случай такой, при каждом проходе цикла количество участвующих элементов при обмене уменьшается, у тебя только один цикл, то бишь только один раз будет обмен.

@takhvatulin · 11.07.2014, 14:13 **[ТС]**

Сообщение от Supernatural

один раз будет обмен.

при каждом обмене я обнуляю счетчик:

C++
1
i = 0;

После этого я заново с нулевого элемента начинаю проверять, если снова встречается случай когда нужно сделать обмен, я обмениваю элементы и снова обнуляю "i"

Добавлено через 36 секунд
но почему то сравнение нулевого и первого элемента игнорится...

Добавлено через 48 секунд

Сообщение от Supernatural

один раз будет обмен.

если бы обмен у меня был один раз, то единица с последнего места не переместилась бы на второе

Добавлено через 3 минуты
Сделал вот так:

C++
1
2
3
4
5
6
int A[5];   
A[0] = 11;
A[1] = 7;
A[2] = 3;
A[3] = 10;
A[4] = 1;

теперь на первом месте 7, на последнем 11....

@castaway · 11.07.2014, 14:13

Не вижу проблемы. Покажи весь код с выводом и т.п.

@takhvatulin · 11.07.2014, 14:16 **[ТС]**

Сообщение от castaway

Покажи весь код с выводом

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
void CCOM_TestDlg::OnBnClickedButton17()
{
    // TODO: Add your control notification handler code here
int A[5];   
A[0] = 11;
A[1] = 7;
A[2] = 3;
A[3] = 10;
A[4] = 1;
int q;
for(int i = 0; i < 4; i++)
{
    if(A[i] > A[i+1])
    {
        q = A[i];
        A[i] = A[i+1];
        A[i+1] = q;
        i = 0;
    }
}
for(int w = 0; w !=5; w++)
{
    TCHAR str[5];
    _swprintf(str, _T("%d"), A[w]);
    MessageBox(str);}
}

Добавлено через 1 минуту
Щас тупо цикл сортировка еще скопипастил, и все заработало!

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
int A[5];   
A[0] = 11;
A[1] = 7;
A[2] = 3;
A[3] = 10;
A[4] = 1;
int q;
for(int i = 0; i < 4; i++)
{
    if(A[i] > A[i+1])
    {
        q = A[i];
        A[i] = A[i+1];
        A[i+1] = q;
        i = 0;
    }
}
for(int i = 0; i < 4; i++)
{
    if(A[i] > A[i+1])
    {
        q = A[i];
        A[i] = A[i+1];
        A[i+1] = q;
        i = 0;
    }
}
for(int w = 0; w !=5; w++)
{
    TCHAR str[5];
    _swprintf(str, _T("%d"), A[w]);
    MessageBox(str);}
}

Теперь интересно, почему ему первого цикла не хватало, и понадобился точно такой же второй?

@Voivoid · 11.07.2014, 14:20

Научись отладчиком пользоваться.

В данном случае происходит следующие:
После того, как у тебя выполнилось i = 0;
начинается новая итерация цикла и происходит инкремент i, т.е. i будет равен не 0, а 1

@takhvatulin · 11.07.2014, 14:23 **[ТС]**

Voivoid, Дада, только что доперло до меня

Добавлено через 2 минуты
В общем сделал дополнительную проверку условия в цикле после обнуления, все заработало:

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
int A[5];   
A[0] = 11;
A[1] = 7;
A[2] = 3;
A[3] = 10;
A[4] = 1;
int q;
for(int i = 0; i < 4; i++)
{
    if(A[i] > A[i+1])
    {
        q = A[i];
        A[i] = A[i+1];
        A[i+1] = q;
        i = 0;
        if(A[i] > A[i+1])
        {
            q = A[i];
            A[i] = A[i+1];
            A[i+1] = q;
            i = 0;
        }
    }
}

И не надо ее сравнивать с нормальными сортировками - это моя первая сортировка написанная мною)

@castaway · 11.07.2014, 15:17

А если в цикле убрать i++ и дописать в конец условия:

C++
1
2
...
} else i++;

Новые блоги и статьи Все статьи Все блоги /
PhpStorm 2025.3: WSL Terminal всегда стартует в ~ and_y87 14.12.2025 PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .	Access VikBal 11.12.2025 Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.	Новый ноутбук volvo 07.12.2025 Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro	Музыка, написанная Искусственным Интеллектом volvo 04.12.2025 Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .	От async/await к виртуальным потокам в Python IndentationError 23.11.2025 Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .
Поиск "дружественных имён" СОМ портов Argus19 22.11.2025 Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .	Сколько Государство потратило денег на меня, обеспечивая инсулином. Programma_Boinc 20.11.2025 Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .	Ломающие изменения в C#.NStar Alpha Etyuhibosecyu 20.11.2025 Уже можно не только тестировать, но и пользоваться C#. NStar - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .	Мысли в слух kumehtar 18.11.2025 Кстати, совсем недавно имел разговор на тему медитаций с людьми. И обнаружил, что они вообще не понимают что такое медитация и зачем она нужна. Самые базовые вещи. Для них это - когда просто люди. . .	Создание Single Page Application на фреймах krapotkin 16.11.2025 Статья исключительно для начинающих. Подходы оригинальностью не блещут. В век Веб все очень привыкли к дизайну Single-Page-Application . Быстренько разберем подход "на фреймах". Мы делаем одну. . .

@castaway 4986 / 3093 / 456 Регистрация: 10.11.2010 Сообщений: 11,170 Записей в блоге: 10
	11.07.2014, 13:47
	Если i = 4, тогда на что ссылается A[i + 1] ? 0

@pesenka1 1 / 1 / 3 Регистрация: 10.07.2014 Сообщений: 14
	11.07.2014, 13:49
	http://ru.wikibooks.org/wiki/%... 0%BE%D0%BC 0

@takhvatulin 11 / 11 / 0 Регистрация: 04.04.2014 Сообщений: 140
	11.07.2014, 13:50 [ТС]
	pesenka1, не надо мне статей, я сам хочу понять логику и написать. Я написал код, он должен работать, судя по моей логике, но почему то первый элемент не сортируется, видимо из за какой то невнимательной ошибке, которую я в упор не вижу( 0

@takhvatulin 11 / 11 / 0 Регистрация: 04.04.2014 Сообщений: 140
	11.07.2014, 13:54 [ТС]
	Supernatural, а в чем моя ошибка заключается? 0

@takhvatulin 11 / 11 / 0 Регистрация: 04.04.2014 Сообщений: 140
	11.07.2014, 13:59 [ТС]
	Supernatural, да елки палки, не интересует меня другой метод, конкретно в моем методе в чем ошибка? P.S. если б хотел тупо сортировать, а не научится сортировать, взял бы готовый код и не парился. 0

@castaway 4986 / 3093 / 456 Регистрация: 10.11.2010 Сообщений: 11,170 Записей в блоге: 10
	11.07.2014, 14:01
	А что ты там подправил ты покажешь? 0

@Supernatural 619 / 218 / 51 Регистрация: 22.11.2010 Сообщений: 1,781 Записей в блоге: 3
	11.07.2014, 14:06
	случай такой, при каждом проходе цикла количество участвующих элементов при обмене уменьшается, у тебя только один цикл, то бишь только один раз будет обмен. 0

@castaway 4986 / 3093 / 456 Регистрация: 10.11.2010 Сообщений: 11,170 Записей в блоге: 10
	11.07.2014, 14:13
	Не вижу проблемы. Покажи весь код с выводом и т.п. 0

@Voivoid 710 / 283 / 16 Регистрация: 31.03.2013 Сообщений: 1,340
	11.07.2014, 14:20
	Научись отладчиком пользоваться. В данном случае происходит следующие: После того, как у тебя выполнилось i = 0; начинается новая итерация цикла и происходит инкремент i, т.е. i будет равен не 0, а 1 1