scanf зависает после присвоения массива 0.o?

Эта программа снова и снова печатает «Scanf Test # 0 и 1».

Это следствие отсутствия проверки результата scanf. Если scanf не может прочитать целое число, то он оставляет поток нетронутым и сообщает об ошибке. Вы не проверяете это, так что это сработало один раз и не удалось 9 раз.

Если вы введете 10 целых чисел, все будет работать нормально; однако очевидно, что вы вводите какие-то другие вещи, такие как буквы, что приводит к сбою извлечения целого числа.

Когда в строке формата есть одна спецификация преобразования ("%d"), scanf может возвращать:

• 1 - успех
• 0 - введены ненужные символы
• EOF - ошибка потока

Вы должны проверить это значение и принять соответствующие меры. Вы можете просто прервать программу, если это не 1, хотя было бы полезно очистить мусор и снова спросить в случае 0.

Переходим к фактической сортировке. Ваш интерфейс странный, поскольку вы передаете N-1 функции, а затем функция получает доступ к 1 элементу за концом массива. Было бы понятнее сделать selection_sort(N, a), а потом внутри функции high = n-1 и for (i = 0; i < n-1; i++) и т.д.

Теперь есть тонкая проблема со строкой:

void selection_sort(int n, int a[n])

Эта строка не соответствует прототипу; вы можете решить свою проблему, заменив строку на:

void selection_sort(int n, int a[])

Первоначальная версия заставляла программу вести себя неожиданно для меня, но только если я вызываю gcc в режиме по умолчанию (т.е. несовместимом со стандартом C). Должно быть какое-то расширение GNU, вызывающее проблемы с int a[n].

В стандартном C исходный код должен работать (и работает, если gcc вызывается в стандартном режиме через -std=c99, что я настоятельно рекомендую делать в любом случае).

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

Спасибо, что помогли мне с этим.

    // Chapter 9 Programming Project #1

    #include <stdio.h>
    #include <ctype.h>

    #define N count

    void selection_sort(int n, int a[]);

    int main(void)
    {
        int i, temp, temp2, a[20], count_dgts = 0, count = 0;
        char ch;

        printf("Enter a series of integers\n");
        printf("Enter only one or two digit numbers: ");
        ch = getchar();
        for (i = 0; ch != '\n';) {
            switch (ch) {
                case '0': temp = 0; count_dgts += 1; break;
                case '1': temp = 1; count_dgts += 1; break;
                case '2': temp = 2; count_dgts += 1; break;
                case '3': temp = 3; count_dgts += 1; break;
                case '4': temp = 4; count_dgts += 1; break;
                case '5': temp = 5; count_dgts += 1; break;
                case '6': temp = 6; count_dgts += 1; break;
                case '7': temp = 7; count_dgts += 1; break;
                case '8': temp = 8; count_dgts += 1; break;
                case '9': temp = 9; count_dgts += 1; break;
                default: count_dgts = 0; break;
            }
            ch = getchar();
            if (count_dgts == 2) {
                a[i++] = (temp2 * 10) + temp;
                count += 1;
            }
            if (ch != ' ' && count_dgts == 1) {
                temp2 = temp;
            }
            if ((ch == ' ' && count_dgts == 1) ||
                (ch == '\n' && count_dgts == 1)) {
                a[i++] = temp;
                count += 1;
            }
        }
        selection_sort(N, a);
        // iterate through array
        printf("\nYour numbers in ascending order are: \n");
        for (i = 0; i < N; i++) {
            printf("%d ", a[i]);
        }
        printf("\n");
        return 0;
    }

    void selection_sort(int n, int a[])
    {
        // Search for the highest value
        // Place that value at the end
        // Call the function with the end of the array removed
        // Create a way for it to break the cycle
        int i, temp, high = n - 1;
        for (i = 0; i < n; i++) {
            if (a[i] > a[high]) {
                temp = a[high];
                a[high] = a[i];
                a[i] = temp;
                if (n - 1 > 1)
                    selection_sort(n - 1, a);
            }
        }
    }