C: Очень важно, что параметр является параметром int

Question

В моей программе мне нужно убедиться, что параметр, переданный в программу, является целым числом, поэтому я создал эту маленькую функцию для обработки случая, когда пользователь вводил «1-й» вместо «1».

Проблема в том, что он вообще не работает.Я пробую отладку, и все, что я могу вам сказать, это то, что параметры равны 12, а long равен 2. (12 - это значение, которое я хочу проверить, а 2 - это число чисел, переданных в функцию)*

Надеюсь, что все понятно, спасибо за помощь;)

Answer 1

Если вы хотите проверить, является ли символ одной из десяти цифр, вам необходимо использовать символьные константы '0', '1' и т. Д. (Не целочисленные константы, 0, 1 и т. Д.).

Answer 2

После того, как вы исправили проблемы, которые уже были указаны, вам, вероятно, следует поискать isdigit и isspace и использовать их, чтобы написать функцию намного более четко.Другая возможность - использовать что-то вроде strcspn или strpbrk, чтобы выполнить поиск за один шаг.

Answer 3

Просто используйте strtol, это преобразует целое число, и вы можете определить по третьему параметру, удалось ли это.

Описание см. strtol (3)

Answer 4

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

Answer 5

Вы можете упростить свою жизнь и сократить код, если будете использовать ctype.h , в частности isdigit или

if(toCheck[i] >= '0' && toCheck[i] <= '9' || toCheck[i] == ' '){

Answer 6

Вы можете использовать функции isdigit и isspace как:

int intOnly(char *toCheck, int len) { // long is reserved word..use len.
    int i = 0; 
    while (i < len) {
        if(!isdigit(toCheck[i]) && !isspace(toCheck[i]))
            return 0;
        i++;
    }
    return 1;
}

Также long - это ключевое слово, которое нельзя использовать в качестве идентификатора.

Answer 7

В дополнение к другим ответам рассмотрите возможность использования isdigit.Также рассмотрите возможность переименования вашей функции, чтобы отразить ее действительность (например, intsAndSpacesOnly)

Answer 8

Возможно, вы захотите добавить оператор break; после оператора i++;. В противном случае вы всегда будете «проваливаться» через ветку default и возвращать 0.

Answer 9

Во-первых, я удивлен, что ваш код проходит компиляцию с переменной long. Однако ваша реальная проблема заключается в том, что вы используете цифры 0-9, а не символы '0' - '9' в переключателе.