Seg Fault с isdigit () в C?

Поскольку tokenHolder представляет собой массив char *, когда вы индексируете tokenHolder[i], вы передаете char * в isdigit(), а isdigit() не принимает указатели.

Вероятно, вам не хватает второго цикла, или вам нужно:

if (isdigit(tokenHolder[i][0]))
    printf("working\n");

Не забывайте о новой строке.

Ваш тест в цикле тоже нечетный; обычно вы пишете «нулевой указатель» как 0 или NULL, а не как '\0'; это просто вводит людей в заблуждение.

Кроме того, вам нужно обратить внимание на предупреждения компилятора, которые вы получаете! Не публикуйте код, который компилируется с предупреждениями, или (по крайней мере) указывайте, что это за предупреждения, чтобы люди могли видеть, что говорит вам компилятор. Вы должны стремиться к нулевым предупреждениям с компилятором, настроенным на суетливость.

Если вы пытаетесь проверить, что все значения в массиве токенов являются числами, вам нужна функция test_integer(), которая пытается преобразовать строку в число и позволяет узнать, используются ли при преобразовании все данные в строке (или вы можете разрешить начальные и конечные пробелы). Ваша спецификация проблемы не совсем ясна, что вы пытаетесь сделать со строковыми токенами, которые вы нашли с помощью strtok() и т. д.

Что касается того, почему вы получаете дамп ядра:

Код макроса isdigit() часто примерно

#define isdigit(x) (_Ctype[(x)+1]&_DIGIT)

Когда вы предоставляете указатель, он обрабатывается как очень большое (положительное или, возможно, отрицательное) смещение массива из (обычно) 257 значений, и поскольку вы обращаетесь к памяти за пределами границ, вы получаете ошибку сегментации. +1 позволяет передать EOF в isdigit(), когда EOF равно -1, что является обычным значением, но не является обязательным. Макросы/функции, такие как isdigit(), принимают либо символ unsigned char, обычно в диапазоне 0..255, следовательно, либо EOF в качестве допустимых входных данных.

Вы объявляете массив указателя на char, а не простой массив только char. Вам также необходимо инициализировать массив или присвоить ему некоторое значение позже. Если вы читаете значение члена массива, который не был инициализирован или не назначен, вы вызываете неопределенное поведение.

char tokenHolder[2500] = {0};

for(int i = 0; tokenHolder[i] != '\0'; ++i){

     if(isdigit(tokenHolder[i])){ printf("worked"); }

С другой стороны, вы, вероятно, упускаете из виду предупреждения компилятора о том, что ваш код может быть неправильным. isdigit ожидает int, а char * не совместим с int, поэтому ваш компилятор должен был сгенерировать для этого предупреждение.

Вам нужно/хотите ввести свой ввод в unsigned char перед передачей его в isdigit.

if(isdigit((unsigned char)tokenHolder[i])){ printf("worked"); }

В большинстве типичных схем кодирования символы за пределами диапазона USASCII (например, любые буквы с умляутами, ударениями, гравюрами и т. д.) будут отображаться как отрицательные числа в типичном случае, когда char является знаком.

Что касается того, как это вызывает ошибку сегмента: isdigit (наряду с islower, isupper и т. д.) часто реализуется с использованием таблицы битовых полей, и когда вы вызываете функцию, значение, которое вы передаете, используется в качестве индекса в таблице. Отрицательное число заканчивается попыткой индексации (хорошо) за пределами таблицы.

Хотя сначала я этого не заметил, у вас также есть проблема, потому что tokenHolder (вероятно) не тот тип, который вы ожидали/планировали использовать. Судя по остальной части кода, вы действительно хотите определить его как:

char tokenHolder[2500];