Извлечение чисел из строки не работает! c

Question

Я бы удалил «-» из строки ISBN. Но мой код не выдаст мне значение. Где ошибка?

char *ISBN[20]; //example: 3-423-62167-2 
*p = ISBN; 

strcpy(ISBN, ptr); //Copy from a Buffer 
printf("\nISBN Array: %s", ISBN); //This works! 

while(*p) 
{ 
    if (isdigit(*p)) 
    { 
     long val = strtol(p, &p, 10); 
     printf("%ld\n", val);   //Do not show anything! 
    } 
    else 
    { 
     p++; 
    } 
} 

Answer 1

Неверно использовать strtol не нужно; второй аргумент не является входом, а выходом, то есть он устанавливает его на последний интерпретируемый символ. Прежде всего, почему вы хотите преобразовать символ в длинный, а затем преобразовать его обратно в символ, когда все, что вам нужно, это символ для печати?

char ISBN[] = "3-423-62167-2"; 
char *p = ISBN; 
while (*p) 
{ 
    if (isdigit(*p)) 
     printf("%c", *p); 
    ++p; 
} 

EDIT:

Чтобы сделать всю строку в длинный:

unsigned long long num = 0; 
while (*p) 
{ 
    if (isdigit(*p)) 
    { 
     const char digit = *p - '0'; 
     num = (num * 10) + digit; 
    } 
    ++p; 
} 

Answer 2

насчет:

for (char* p = ISBN; *p != '\0'; p++) 
{ 
    if (isdigit(*p)) 
    { 
     printf("%c", *p); 
    } 
} 

Если вы хотите long: Сохранить символы в char[] (вместо printf()), а затем, когда все сделано, что преобразования в long. Можно даже использовать ISBN массив сделать обращенный на месте:

int i = 0; 
for (char* p = ISBN; *p != '\0'; p++) 
{ 
    if (isdigit(*p)) 
    { 
     ISBN[i++] = *p; 
    } 
} 

ISBN[i] = '\0'; 

long isbn = strtol(ISBN, NULL, 10); 

BTW, вы забыли p++ когда is digit() верно.

Answer 3

Предполагая p является char * указателем вы должны обновить свой код

//-----v no * 
    char ISBN[20]; //example: 3-423-62167-2 
    p = ISBN; 
//^-- no * 

сохранить остальную часть кода, как есть.

Answer 4

Следующий код работает для меня:

#include <stdio.h> 
#include <string.h> 

int main() 
{ 

char *ptr = "3-423-62167-2"; 
char ISBN[20]; // should be either ISBN[] or char *ISBN for a string 
char *p = ISBN; // declared this one here.. else should be p = ISBN 

strcpy(ISBN, ptr); 
printf("\nISBN Array: %s\n", ISBN); 

while(*p) 
{ 
    if (isdigit(*p)) 
    { 
     long val = strtol(p, &p, 10); 
     printf("%ld\n", val); 
    } 
    else 
    { 
     p++; 
    } 
} 

} 

ознаменовали корректировки в комментариях!

Answer 5

Я думаю, что ОП хочет, чтобы преобразовать строку с переносами в длинное целое. Эта функция преобразует десятичные цифры строки в длинную. Дефис (в любом месте) игнорируется, другие символы, в том числе места, привести к читающей ошибке:

/* 
*  Return ISBN as long or -1L on format error 
*/ 
long isbn(const char *str) 
{ 
    long n = 0L; 

    if (*str == '\0') return -1L; 

    while (*str) { 
     if (isdigit(*str)) { 
      n = n * 10 + *str - '0'; 
     } else { 
      if (*str != '-') return -1L; 
     } 
     str++; 
    } 

    return n; 
} 

Обратите внимание, что long имеет такой же размер, как int на некоторых машинах и не может быть достаточно широкими, чтобы сохранить числовой ISBN.