Создание функции atoi

1. Вы объявляете массив char *, то есть массив строк, а не одну строку. Вы, вероятно, хотите:

   char number[MAXSIZE];
   

2. Ваша строка формата scanf неверна. Если вы хотите прочитать строку, вы должны использовать %s. %c читает только один символ.

3. Ваш параметр scanf неверен - передайте сам number (или &number[0], если хотите), а не &number.

4. Параметр, который вы передаете atoi_me, неверен. Назовите его с помощью number (или эквивалентно &number[0]), а не *number.

Собрав все это вместе, у вас должна получиться примерно такая процедура main:

int main(void)
{
    char number[MAXSIZE];
    int num;
    printf("Please enter a number: ");
    scanf("%s", number);
    num = atoi_me(number);
    printf("%d\n", num);
    return 0;
} 

Примечания редактора: У вас есть потенциальное переполнение буфера со строкой scanf. Лучше использовать такую ​​функцию, как fgets(3), которая упрощает защиту от такая проблема.

atoi(3) также традиционно поддерживает отрицательные числа (с ведущим -) и необязательным ведущим + для положительные числа, которые ваша реализация не обрабатывает.

Как я и думал, проблема в твоем звонке.

Измените свой основной на.

int main()
{
    char number[MAXSIZE];
    int num;

    printf("Please enter a number:\n");
    scanf("%s", number);
    num = atoi_me(number);
    printf("%d", num);
    return 0;
}

Помимо этого, не рекомендуется использовать scanf - http://c-faq.com/stdio/scanfprobs.html . В этом случае вы должны использовать fgets.

Это была не проблема с вашей функцией atoi_me(), а проблема с тем, как вы получили свой ввод. Ваша реализация показывает некоторую слабость в вашем понимании того, как работает scanf(). Это не проблема сама по себе, в конце концов, ошибки — это часть процесса обучения.

Как правило, безопаснее сначала собрать ввод в буфер, потому что scanf() из стандартного ввода слишком сильно зависит от пользователя программы, чтобы ввести ввод именно так, как вы ожидаете. В этом случае особого вреда не будет, так как вам нужна только одна строка ввода. Но, как правило, программа обрабатывает несколько строк ввода, и scanf() может застрять при возникновении ошибки. Итак, вы можете использовать что-то вроде этого, чтобы вместо этого получить строку ввода:

char line[MAXLINESIZE];

if (fgets(line, MAXLINESIZE, stdin) == 0) {
    fprintf(stderr, "no input was provided!\n");
    return 0;
}

Как упоминалось в другом месте, %c — это неправильный спецификатор формата, который следует использовать для ввода, которое вы собираете. Поскольку вам нужны десятичные цифры, семейство *scanf() имеет спецификатор формата, позволяющий собирать только эти символы.

char number[MAXSIZE];

if (sscanf(line, " %[0-9]", number) != 1) {
    fprintf(stderr, "no number found in input: %s", line);
    return 0;
}

Здесь я использую line, полученный с помощью fgets(), и анализирую часть ввода, содержащую цифры. Ведущий пробел заставляет sscanf() пропускать символы пробела, ведущие к цифрам.

Поскольку number должен быть массивом символов,

• вы должны объявить его как char number[MAXSIZE];,
• вы не должны вызывать свою функцию с &number, но с number напрямую: atoi_me(number);

Некоторые комментарии:

int atoi_me(const char *numstring)...

Лучше использовать указатель типа const, так как вы не собираетесь изменять содержимое строки.

int main()
{
    char number[MAXSIZE]; // array of chars
    int num;

    printf("Please enter a number:\n");
    scanf("%s", number);     // enter a string, not a char
    num = atoi_me(number);   // pointer to char, not pointer to pointer
    printf("%d", num);
    return 0;
}