Копирование альфанумерных символов в C

Question

У меня есть массив символов из массива, вы хотите сохранить буквенно-цифровое строчное значение в массиве указателей. т.е. mystr должен указывать на char [] из «50sometexthere»

char[] myline = " 50 Some Text Here "; 
char *mystr = (char *)malloc(128 * sizeof(char)); 

char *tmp = myline; 

while (*tmp != '\0'){ 
if (isalnum(*tmp)) 
    strcat(mystr,(char*) tolower(*tmp)); 
tmp++; 
} 

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

Answer 1

char *myline = " 50 Some Text Here "; 
char *mystr = (char *)malloc(128); //sizeof char is always 1 

char *tmp = myline; 
char *tmpdest = mystr; 

while (*tmp != '\0'){ 
if (isalnum(*tmp)) 
    *tmpdest++ = tolower(*tmp); //this line is changed! 
tmp++; 
} 

*tmpdest = '\0'; 

НТН

Answer 2

Ваша ошибка отлитый в вызове strcat.

почти всегда литая, в C, неправильно

strcat занимает 2 указателя на символ, вы подачи указатель на символ и INT (ошибочно приводится к указателю на символ).

Answer 3

Функция tolower возвращает целое число, и вы ошибочно набрасываете его на char *.

Лучший способ сделать это, чтобы скопировать буквенно-цифровые символы из исходного массива в массив назначения

char myline[] = " 50 Some Text Here "; // put the [] after the variable. 
char *mystr = malloc(128); 

char *tmp = myline; 
char *destPtr = mystr; 

while (*tmp != '\0'){ 
if (isalnum(*tmp)) { 
    *destPtr++ = *tmp; 
} 
tmp++; 
} 
*destPtr = 0; // terminating nul character. 

Если вы действительно хотите использовать strcpy, вам необходимо инициализировать строку назначения в пустую строку и сделайте символ скопированным частью массива символов и добавьте этот массив в строку назначения:

char myline[] = " 50 Some Text Here "; // put the [] after the variable. 
char *mystr = malloc(128); 

char *tmp = myline; 
mystr[0] = 0; // initialize the destination string. 
while (*tmp != '\0'){ 
     char str[2] = {0}; // temp string of size 2. 
     if (isalnum(*tmp)) 
       str[0] = tolower(*tmp); // put the char to be copied into str. 
       strcat(mystr,str);  // append. 
     tmp++; 
}