Сортировки выдают неверный результат

@nasty7chukova · Регистрация: 08.11.2012

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

Помогите, пожалуйста.
Написала три вида сортировки: подсчетом, методом вставок, обменом (пузырьком). Они работают, но результат выдают неправильный. Может я где-то ошиблась? Помогите, пожалуйста.

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
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
#include <stdio.h>
#include <stdlib.h>
 
#define SIZE 5
 
void count (int X[])
{
    int Y[SIZE], i, j;
 
    for (i = 0; i < SIZE; i++) 
    { 
        j = i; 
            while (j > 0 && Y[j-1] > X[i])
            {
                Y[j] = Y[j-1];
                    j = j - 1;
            }
 
        Y[j] = X[i];
    }
 
printf("\n Sorted massive: \n");
 
    for(i=0; i<SIZE; i++)
    printf("%d\t", X[i]);
}
 
 
void insert (int X[])
{
    int i, j, k; 
 
    for (i=1; i<SIZE; i++)
    {
        k=X[i] ; // запоминание элемента для вставки
            for (j=i-1; j>=0 && k<X[j] ; j--) // ищем место для вставки
                X[j+1] =X[j] ; // сдвиг на одну позицию
                    X[j+1] =k;
    }
 
    printf("\n Sorted massive: \n");
 
        for(i=0; i<SIZE; i++)
        printf("%d\t", X[i]);
}
 
void exchange (int X[])
{
    int i, j, n;
 
    for(i=0; i<SIZE-1; i++)
        for (j=SIZE-2; j>=i; j--)
 
    if (X[j]>X[j+1])
    {
        n=X[j];
            X[j]=X[j+1];
                X[j+1] = n;
    }
printf("\n Sorted massive: \n");
 
    for(i=0; i<SIZE; i++)
    printf("%d\t", X[i]);
}
 
int main()
{
  int S[SIZE], a, i; 
  
  for (i=0; i<SIZE; i++)
  {
      printf ("S[%d]=", i+1); 
      scanf ("%d", &S[i]); 
  }
  
   while(a!=0)
  {  
      printf ("Input number of menu: \n 1. Sorting by counting; \n 2. Simple method inserts; \n 3. Exchange sorting; \n 4. Exit\n"); 
      scanf ("%d", &a); 
      
      if(a==1)    count(S[SIZE]);
      if(a==2)    insert(S[SIZE]);
      if(a==3)    exchange(S[SIZE]);
      if(a==4)    break; 
      else printf ("Wrong number of menu\n"); 
   } 
  
   
   system("PAUSE"); 
   return 0;
}

@easybudda · 09.06.2013, 13:17

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

Может я где-то ошиблась?

Первое, что увидел:

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

C
1
2
3
void count (int X[])
//...
while (j > 0 && Y[j-1] > X[i])

Массив Y у Вас в функции создаётся и никак не инициализируется. По сути в нём случайные значения хранятся, и эти случайные значения Вы со значениями массива X, переданного в функцию параметром, сравниваете.

Добавлено через 6 минут
А не, понял, там всё хитрее...

Добавлено через 1 минуту

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

C
1
printf("%d\t", X[i]);

Только тогда уж этот хитроотсортированный Y и выводите...

@nasty7chukova · 09.06.2013, 13:28 **[ТС]**

А если я проинициализирую этот Y нулями, мне это поможет?

Добавлено через 6 минут
Нет, не поможет. Он все равно эти же самые случайные значения и выдает.

@easybudda · 09.06.2013, 13:41

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

А если я проинициализирую этот Y нулями, мне это поможет?

Не...
Вот так работает

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
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
#include <stdio.h>
#include <stdlib.h>
 
#define SIZE 5
 
void count (int X[])
{
    int Y[SIZE], i, j;
 
    for (i = 0; i < SIZE; i++) 
    { 
        j = i; 
            while (j > 0 && Y[j-1] > X[i])
            {
                Y[j] = Y[j-1];
                    j = j - 1;
            }
 
        Y[j] = X[i];
    }
 
printf("\n Sorted massive: \n");
 
    for(i=0; i<SIZE; i++)
    printf("%d\t", Y[i]);
}
 
 
void insert (int X[])
{
    int i, j, k; 
 
    for (i=1; i<SIZE; i++)
    {
        k=X[i] ; // запоминание элемента для вставки
            for (j=i-1; j>=0 && k<X[j] ; j--) // ищем место для вставки
                X[j+1] =X[j] ; // сдвиг на одну позицию
                    X[j+1] =k;
    }
 
    printf("\n Sorted massive: \n");
 
        for(i=0; i<SIZE; i++)
        printf("%d\t", X[i]);
}
 
void exchange (int X[])
{
    int i, j, n;
 
    for(i=0; i<SIZE-1; i++)
        for (j=SIZE-2; j>=i; j--)
 
    if (X[j]>X[j+1])
    {
        n=X[j];
            X[j]=X[j+1];
                X[j+1] = n;
    }
printf("\n Sorted massive: \n");
 
    for(i=0; i<SIZE; i++)
    printf("%d\t", X[i]);
}
 
int main()
{
  int S[SIZE], a, i; 
  
  for (i=0; i<SIZE; i++)
  {
      printf ("S[%d]=", i); 
      scanf ("%d", &S[i]); 
  }
  
   a = -1;
   while(a!=0)
  {  
      printf ("\nInput number of menu: \n 1. Sorting by counting; \n 2. Simple method inserts; \n 3. Exchange sorting; \n 4. Exit\n"); 
      scanf ("%d", &a); 
      
      if(a==1)    count(S);
      else if(a==2)    insert(S);
      else if(a==3)    exchange(S);
      else if(a==4)    return 0; 
      else printf ("Wrong number of menu\n"); 
   } 
  
   
   //system("PAUSE"); 
   return 0;
}

Но вообще это ужас ужасный! Будь я преподом - больше трёх баллов не поставил бы...

@nasty7chukova · 09.06.2013, 13:44 **[ТС]**

Блин, спасибо большое. Я сама даже кое-где не понимаю, как это вообще работает. Нам преподаватель дает метод. указания, и там на псевдокоде написаны эти сортировки. Попыталась реализовать.

@easybudda · 09.06.2013, 13:50

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

Нет, не поможет. Он все равно эти же самые случайные значения и выдает.

А оно у Вас ещё и работало?

Code
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
~/cpp/numbers $ gcc -o nasty_sorts nasty_sorts.c 
nasty_sorts.c: In function 'main':
nasty_sorts.c:82: warning: passing argument 1 of 'count' makes pointer from integer without a cast
nasty_sorts.c:83: warning: passing argument 1 of 'insert' makes pointer from integer without a cast
nasty_sorts.c:84: warning: passing argument 1 of 'exchange' makes pointer from integer without a cast
~/cpp/numbers $ ./nasty_sorts 
S[0]=3
S[1]=1
S[2]=5
S[3]=2
S[4]=4
Input number of menu: 
 1. Sorting by counting; 
 2. Simple method inserts; 
 3. Exchange sorting; 
 4. Exit
1
Bus error

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

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

if(a==1) count(S[SIZE]);

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

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

Я сама даже кое-где не понимаю, как это вообще работает.

Почитайте Алгоритмы сортировок - может что и прояснится...

@nasty7chukova · 09.06.2013, 13:54 **[ТС]**

Да работала, как ни странно.
Doc1.docx

@nasty7chukova · 09.06.2013, 13:56 **[ТС]**

Вы мне очень помогли. Спасибо большое

@easybudda · 09.06.2013, 14:19

nasty7chukova,

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Desktop (MinGW): Создаём пустое окно с нуля для 2D-графики на SDL3, Си и C++ 8Observer8 10.03.2026 Содержание блога Финальные проекты на Си и на C++: hello-sdl3-c. zip hello-sdl3-cpp. zip Результат:	Установка CMake и MinGW 13.1 для сборки С и C++ приложений из консоли и из Qt Creator в EXE 8Observer8 10.03.2026 Содержание блога MinGW - это коллекция инструментов для сборки приложений в EXE. CMake - это система сборки приложений. Здесь описаны базовые шаги для старта программирования с помощью CMake и. . .	Как дизайн сайта влияет на конверсию: 7 решений, которые реально повышают заявки Neotwalker 08.03.2026 Многие до сих пор воспринимают дизайн сайта как “красивую оболочку”. На практике всё иначе: дизайн напрямую влияет на то, оставит человек заявку или уйдёт через несколько секунд. Даже если у вас. . .	Модульная разработка через nuget packages DevAlt 07.03.2026 Сложившийся в . Net-среде способ разработки чаще всего предполагает монорепозиторий в котором находятся все исходники. При создании нового решения, мы просто добавляем нужные проекты и имеем. . .
Модульный подход на примере F# DevAlt 06.03.2026 В блоге дяди Боба наткнулся на такое определение: В этой книге («Подход, основанный на вариантах использования») Ивар утверждает, что архитектура программного обеспечения — это структуры,. . .	Управление камерой с помощью скрипта OrbitControls.js на Three.js: Вращение, зум и панорамирование 8Observer8 05.03.2026 Содержание блога Финальная демка в браузере работает на Desktop и мобильных браузерах. Итоговый код: orbit-controls-threejs-js. zip. Сканируйте QR-код на мобильном. Вращайте камеру одним пальцем,. . .	SDL3 для Web (WebAssembly): Синхронизация спрайтов SDL3 и тел Box2D 8Observer8 04.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-sync-physics-sprites-sdl3-c. zip На первой гифке отладочные линии отключены, а на второй включены:. . .	SDL3 для Web (WebAssembly): Идентификация объектов на Box2D v3 - использование userData и событий коллизий 8Observer8 02.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-collision-events-sdl3-c. zip Сканируйте QR-код на мобильном и вы увидите, что появится джойстик для управления главным героем. . . .

@nasty7chukova 0 / 0 / 0 Регистрация: 08.11.2012 Сообщений: 8
	09.06.2013, 13:28 [ТС]
	А если я проинициализирую этот Y нулями, мне это поможет? Добавлено через 6 минут Нет, не поможет. Он все равно эти же самые случайные значения и выдает. 0

@nasty7chukova 0 / 0 / 0 Регистрация: 08.11.2012 Сообщений: 8
	09.06.2013, 13:44 [ТС]
	Блин, спасибо большое. Я сама даже кое-где не понимаю, как это вообще работает. Нам преподаватель дает метод. указания, и там на псевдокоде написаны эти сортировки. Попыталась реализовать. 0

@nasty7chukova 0 / 0 / 0 Регистрация: 08.11.2012 Сообщений: 8
	09.06.2013, 13:54 [ТС]
	Да работала, как ни странно. Doc1.docx 0

@nasty7chukova 0 / 0 / 0 Регистрация: 08.11.2012 Сообщений: 8
	09.06.2013, 13:56 [ТС]
	Вы мне очень помогли. Спасибо большое 0

@easybudda Модератор 12843 / 7592 / 1766 Регистрация: 25.07.2009 Сообщений: 13,973
	09.06.2013, 14:19
	nasty7chukova, 1