Почему не работает программа (работа со строками в Си)?

Question

[nkorobkov]

Задание: пользователь вводит строку из 20 символов. Нам необходимо определить какое кол-во раз встретилась в ней каждая цифра. Я решил сделать вот так (почему-то не работает):

#include <stdio.h>
#include <stdlib.h>
#define N 20
int main()
{
    int i, j;
    char string[N];
    int numbers[10];
    for(i = 0; i < 10; i++){
        numbers[i] = 0; //Заполняем массив результатов нулями
    }
    for(i = 0; i < N; i++){
        string[i] = getchar(); //Вводим строку из 20 символов
    }
    for(i = 0; i < N; i++){
        putchar(string[i]); //Выводим строку на экран
    }

    for(i = 0; i < 10; i++){
        for(j = 0; j < N; j++)
            if(string[j] == (char)i) numbers[i]++; //Определяем частоту, с которой встречается каждая цифра
    }

    for(i = 0; i < 10; i++){
        printf("\nNumbers[%d] = %d", i, numbers[i]); //Выводим массив результатов
    }
    return 0;
}

Что я сделал неправильно?

Answer 1

[diskonnect]

У вас небольшая ошибка в сравнении символов с цифрами:

if(string[j] == (char)i) numbers[i]++; //Определяем частоту, с которой встречается каждая цифра

Цифры, которые вы вводите с консоли, записываются в память не как десятичные цифры, а как ASCII символы, т.е. если вы ввели с клавиатуры символ '0' и вы считываете его с помощью getchar(), то он у вас сохранится как ASCII значение символа '0' (в десятичной системе представляет собой число 48). Чтобы программа работала, как вы и задумали, нужно из введеного символа вычитать 'смещение' нуля в таблице ASCII:

if((string[j] - '0') == i) numbers[i]++; //Определяем частоту, с которой встречается каждая цифра

Comments

[diskonnect]

Ну и стоит не забывать, что называть переменные как типы в c++ (как у вас string), это является не лучшей практикой.

[nkorobkov]

Я не понял фразу "нужно из введеного символа вычитать 'смещение' нуля в таблице ASCII", и, соответственно, строку "string[j] - '0') == i"... Можете объяснить поподробнее, как это работает?

[diskonnect]

Если посмотрите таблицу ascii символов (www.asciitable.com/), то Вы увидите, что символы цифр (начиная с '0' и до '9') идут подряд начиная с 0. Соответственно, введеному символу '0' с клавиатуры соответствует значение 48, символу '1' - 49, и так до символа '9' - 57. Поэтому для того, чтобы получить вхождения символа '0', вам нужно сравнивать введеный символ (string[j]) не с нулем, а с символом '0' (значение 48). Можно было бы написать
'string[j] == i + 48', но мне привычнее видеть код так, как я указал в ответе (дело привычки/вкусов).

[Денис Загаевский]

nkorobkov: ну и ещё внутренний цикл не нужен. Достаточно
numbers[string[j] - '0']++;

[abcd0x00]

diskonnect:

Ну и стоит не забывать, что называть переменные как типы в c++ (как у вас string), это является не лучшей практикой.

Вообще пофиг, во-первых, это разные языки, а во-вторых, в C++ принято пространство имён писать. То есть оно и не должно пересечься там.