Не работает сортировка

@SRq · Регистрация: 18.11.2010

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

Добрый вечер! У меня проблемка с сортировкой, вместо сортировки происходит вообше не знаю что =\
Остальное все вроде правильно.

Вот собственно код:

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
#include <stdio.h>
#include <windows.h>
#define N 100
 
char size_arr(char Array[N])
{
    char i=0,k=0;
    while (Array[i]!='\0') i++,k++;
    return k;
}
 
char RandomFill(char basic_arr[N])
{
    char i,j;
    srand(time(NULL));
    for (i=0;i<N;basic_arr[i]=21+rand()%(254+1-21),i++);
    basic_arr[N]='\0';
}
 
char Select_print(char sel_arr[N])
{
    char str[50];
 
    if (size_arr(sel_arr)==N) CharToOem("Исходный массив:\n", str), printf("%s", str);
      else
        if ((sel_arr[0]>='0') && (sel_arr[0]<='9')) CharToOem("ЦИФРЫ:\n", str), printf("%s", str);
          else if ((sel_arr[0]=='!') || (sel_arr[0]==';') || (sel_arr[0]==':') || (sel_arr[0]==',') || (sel_arr[0]=='.') || (sel_arr[0]=='?'))
                 CharToOem("Знаки:\n", str), printf("%s", str);
}
 
char Print(char print_arr[N])
{
    char i=0;
    Select_print(print_arr);
    while (print_arr[i]!='\0') printf("%c",print_arr[i]),i++;
    printf("\n\n\n");
}
 
char sorting(char sort_arr[N])
{
    char tmp, str[50];
    int i, j;
 
    for(i=0; i<N; i++)
    {
        tmp=sort_arr[i];
        for(j=i-1; j>=0&&sort_arr[j]>tmp; --j)
        {
            sort_arr[j+1]=sort_arr[j];
        }
        sort_arr[j+1]=tmp;
    }
    CharToOem("Сортировка:\n", str), printf("%s", str);
    Print(sort_arr);
}
 
char Select(char new_arr[N])
{
    char numb_arr[N],chr_arr[N];
    char i=0,j=0,k=0;
 
    numb_arr[0]='\0'; chr_arr[0]='\0';
    while (new_arr[i]!='\0')
    {
        if ((new_arr[i]>='0') && (new_arr[i]<='9')) numb_arr[j]=new_arr[i],numb_arr[j+1]='\0',j++;
        if ((new_arr[i]=='!') || (new_arr[i]==';') || (new_arr[i]==':') || (new_arr[i]==',') || (new_arr[i]=='.') || (new_arr[i]=='?'))
            chr_arr[k]=new_arr[i],chr_arr[k+1]='\0',k++;
        i++;
    }
    Print(numb_arr); Print(chr_arr);
    sorting(numb_arr); sorting(chr_arr);
}
 
int main()
{
    char mas[N];
    RandomFill(mas);
    Print(mas);
    Select(mas);
    return 0;
}

Добавлено через 4 минуты
Забыл задание.
Заполнить рандомно массив, выделить цифры и знаки препинания в отдельные массивы, отсортировать их.

@Thinker · 10.12.2011, 09:29

А ведь позиции с номером N не существует:

C
1
basic_arr[N]='\0';

@SRq · 10.12.2011, 10:57 **[ТС]**

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

А ведь позиции с номером N не существует:

C
1
basic_arr[N]='\0';

И как это влияет на программу?

@Thinker · 10.12.2011, 11:14

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

И как это влияет на программу?

В некоторых случаях ваша программа может упасть

@SRq · 10.12.2011, 11:59 **[ТС]**

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

В некоторых случаях ваша программа может упасть

Ясно, а насчет сортировки можете что-нибудь сказать? Почему после сортировки массивы портятся?

go · 10.12.2011, 19:37

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

В некоторых случаях ваша программа может упасть

Вообще то это грубая ошибка. Выход за пределы выделенной памяти, нельзя так делать и надеяться на "авось"

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

Ясно, а насчет сортировки можете что-нибудь сказать? Почему после сортировки массивы портятся?

C
1
2
int arr_cmp (const void *a, const void *b) { return strcmp ( ((char *) a), ((char *) b) ) ; }
qsort (arr,strlen(arr),1,arr_cmp);

@AncinetHero · 10.12.2011, 19:50

Вот рабочая

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
void sorting(char sort_arr[N])
{
    int i, j;
    for(i=0; i<N-1; i++)
    {
        int index=i;
        for(j=i+1; j<N; j++)
         if(sort_arr[j]<sort_arr[index])
            index=j;
        swap(sort_arr[i],sort_arr[index]);
    }
    Print(sort_arr);
}

Я вообще не понял что за метод сортировки вы использовали, и что у вас за вывод, и зачем у вас функция типа char, и зачем неизвестный массив char'ов str..

@SRq · 10.12.2011, 21:06 **[ТС]**

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

Вот рабочая

C
1
2
3
4
5
6
7
8
9
10
11
12
13
void sorting(char sort_arr[N])
{
    int i, j;
    for(i=0; i<N-1; i++)
    {
        int index=i;
        for(j=i+1; j<N; j++)
         if(sort_arr[j]<sort_arr[index])
            index=j;
        swap(sort_arr[i],sort_arr[index]);
    }
    Print(sort_arr);
}

Я вообще не понял что за метод сортировки вы использовали, и что у вас за вывод, и зачем у вас функция типа char, и зачем неизвестный массив char'ов str..

С этим кодом такая же байда.

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

C
1
2
int arr_cmp (const void *a, const void *b) { return strcmp ( ((char *) a), ((char *) b) ) ; }
qsort (arr,strlen(arr),1,arr_cmp);

Спасибо. Все заработало. Если вам не сложно можете объяснить как этот код работает? Ну или ссылку киньте, где нормально описывается все, а то я не очень понимаю его. Буду благодарен. Погуглил ниче путевого не нашел.

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

Вообще то это грубая ошибка. Выход за пределы выделенной памяти, нельзя так делать и надеяться на "авось"

Эм.. а почему за пределы выделеной памяти? N - это же константа. В Си на сколько я знаю конец массива записывается символом \0 или как его там... и поэтому не помня делается ли это автоматически я решил удостовериться, чтобы нормально работала функция size_arr

Добавлено через 7 минут
P.S. Я эти задачи делал в прошлом году и все было норм. В этом году друг попросил помочь, я решил не париться и брать функции с разных задач, потому что когда я это себе делал, более-менее понимал все юзал интернет и т.п. и у меня нормально все равботало с этим кодом сортировки, а после вставки в эту задачу, такая байда пошла...
Кстате с этим кодом:

C
1
2
3
4
5
6
7
8
9
for(i=0; i<N; i++)
    {
        tmp=sort_arr[i];
        for(j=i-1; j>=0&&sort_arr[j]>tmp; --j)
        {
            sort_arr[j+1]=sort_arr[j];
        }
        sort_arr[j+1]=tmp;
    }

мне помогли на этом форуме.
Вот.))

go · 10.12.2011, 21:34

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

Если вам не сложно можете объяснить как этот код работает?

http://www.cplusplus.com/refer... lib/qsort/
Я объясню на примере

C
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
#include <stdio.h>
#include <conio.h>
#include <stdlib.h>
#include <string.h>
 
int arr_cmp (const void *a, const void *b) { return strcmp ( ((char *) a), ((char *) b) ) ; }
 
int main(int argc, char* argv[])
{
    char s[BUFSIZ] = "qwerty"; // На самом деле размер памяти выдеоенной под указатель равен BUFSIZ
       // Но длина строки = кол-во символ (не путайте эти понятия) и она должна быть <= BUFSIZ - 1
 
    printf ("Str : %s", s);
    qsort (s,strlen(s),1,arr_cmp);  //
    printf ("\nNewSrt : %s", s);
 
    getch ();
    return 0;
}

C
1
2
3
4
char s[3] = "qw";
s[0] = 'q';
s[1] = 'w';
s[2] = '\0';

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

C++
1
void qsort ( void * base, size_t num, size_t size, int ( * comparator ) ( const void *, const void * ) );

Вот прототип функции сортировки

Новые блоги и статьи Все статьи Все блоги /
Новый ноутбук 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 . Быстренько разберем подход "на фреймах". Мы делаем одну. . .	Фото: Daniel Greenwood kumehtar 13.11.2025	Расскажи мне о Мире, бродяга kumehtar 12.11.2025 — Расскажи мне о Мире, бродяга, Ты же видел моря и метели. Как сменялись короны и стяги, Как эпохи стрелою летели. - Этот мир — это крылья и горы, Снег и пламя, любовь и тревоги, И бескрайние. . .

Не работает сортировка

Решение