Двойной указатель и символ

Question

У меня проблема с функцией set_ccs. Я не могу принимать элементы от пользователя. Как я могу это исправить?

int main(){ 

    char *ccs; 

    *ccs =(char*)malloc(sizeof(char) * 80);//i have to use dynamic memory allocation  
    printf("Enter CCS: "); 
    set_ccs(&ccs); 
    free(ccs); 
    return 0; 
} 

int set_ccs(char **ccs){ 

    int i = 0; 

    scanf("%s",*ccs);//Is it better to use fgets? Because scanf seems to count 'enter' 

    while(*ccs!='\0'){ 
     ccs++; 
     i++; 
    } 

    printf("Length of sequence : %d\n",i);//It always return 3 
    printf("%s",ccs); //with weird elements 
return i; 
} 

Спасибо.

Answer 1

В дополнение к раскручивается ответить, что вы должны просто использовать

char *ccs; 
ccs = malloc(80); 

Вы должны сделать функцию set_ccs() принять указатель:

int set_ccs(char *ccs) 

И называют это как это от вашей основной:

set_ccs(css); 

Тогда в вашей функции вы можете использовать scanf() как так:

scanf("%s", css); 

Теперь вы, если вы хотите, чтобы проверить '\0', это хорошая практика для инициализации «строка» в 0, прежде чем использовать его. Вы можете сделать это с помощью calloc(80) вместо malloc(80).

Если вам необходимо иметь указатель на указатель (char **ccs), вы должны сделать двойной указатель на вашем главном, проверить этот код:

int main(){ 

    char *ccs; 
    char **ccs2; //a pointer to a pointer 

    ccs = calloc(80); //i have to use dynamic memory allocation 
    ccs2 = &ccs; //pass the address of the pointer to the double pointer 

    printf("Enter CCS: "); 
    set_ccs(ccs2); //pass the double pointer 
    free(ccs); 
    return 0; 
} 

int set_ccs(char **ccs){ 

    int i = 0; 

    scanf("%s", *ccs); 
    char *c = *ccs; //copy to make increments 

    while(*c != '\0'){ 
     c++; 
     i++; 
    } 

    printf("Length of sequence : %d\n", i); 
    printf("%s", *ccs); 
    return i; 
} 

Answer 2

Это:

char *ccs; 

*ccs =(char*)malloc(sizeof(char) * 80); 

неправильно, и нет никакой нормальный компилятор, который будет принимать его. Вы стреляете указателем в char, и это, безусловно, не подойдет.

Это должно быть просто:

ccs = malloc(80); 

Там нет необходимости масштабировать по sizeof (char), что всегда 1.

Кроме того, please don't cast the return value of malloc() in C.