C струна проблема

Question

Моя проблема очень проста, но я действительно не знаю, как работать со строками в C

Проблема заключается в том: У меня есть структура называется человек

struct person 
{ 
    char name[25] // use char *name better? 
}p; 

У меня также есть функция называется p *addNewPerson(char *name)

p *addNewPerson(char *name) 
{ 
    p *newPerson = (p *)malloc(sizeof(person)); 
    //here how can I assign the name to this new person? 
    ... 
    return newPerson; 
} 

Таким образом, в основной функции

void main() 
{ 
    for(; ;) 
    { 
    char input[25]; 
    scanf("%s", input); // is this way possible? 
    //shoud I do something with this "input", like input[strlen(input)-1] = '\0' 
    //call addNewPerson() 
    p *newPerson = addNewPerson(&input); 
    //store this newPerson in some data structure 
    ... 
    } 
} 

Уточнение: Вопрос в том, как я могу присвоить имя новому человеку внутри p *addNewPerson(char *name)?

Answer 1

p *newPerson = (p *)malloc(sizeof(person)); 
//here how can I assign the name to this new person? 

Вы бы сделать это:

strcpy(p->name,name); 

Вы должны также изменить использование p к struct person поскольку p не тип, это глобальный veriable типа struct person. Изменение кода на:

struct person *addNewPerson(char *name) 
{ 
    struct person *newPerson = malloc(sizeof *newPerson); 

// Шоуда я сделать что-то с этим "вход", как вход [STRLEN (вход) -1] =

Нет, зсапЕ будет NUL прекратить строка для вас.

Помните, что обработка строк в C должна выполняться очень, очень осторожно. , например. когда вы делаете scanf("%s", input);, что произойдет, если вы введете имя длиной более 24 символов?

Все может произойти. scanf может переполнить ваш буфер, и вы получите неопределенное поведение. Вы должны сделать это по крайней мере:

int ret = scanf("%24s",buffer); //read max 24 chars, to make space for a final '\0' 
    if(ret == EOF) { //end of input reached. 
    break; 
    if(ret != 1) { 
    // for whatever reason, the conversion failed. exit, or alert the user, or whatever 
    } 

Аналогичных внутри вашего addNewPerson на strcpy(p->name,name);, зЬгср может счастливо писать мимо буфера, если name больше, чем то, что p->name может держать. В этом конкретном случае с указанной выше модификацией scanf длина всегда будет равна 24 или меньше, чтобы она была безопасной. Но будьте очень осведомлены об этом в целом.

Звонок addNewPerson должен просто передать имя buffer непосредственно, при использовании в качестве значения имя массива распадается на указатель на первый элемент этого массива;

struct person *newPerson = addNewPerson(input); 

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

Answer 2

Используйте strcpy, чтобы скопировать строку и сохранить его в структуры:

strcpy(p->name, name); 

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

Answer 3

1. Вы можете назначить name элемент по sprintf(p->name, "%s", name);, но вы должны быть осторожны переполнения буфера 25 символов. Например, вы могли бы сделать snprintf(p->name, 25, "%s", name);

2. Самое главное, что вы должны знать при написании этой программы является то, что все, что вы таНос() должен быть свободным() 'd. В противном случае произойдет утечка памяти. В конце main() убедитесь, что free(newPerson);. Если вы измените свой член структуры на указатель, вам нужно будет malloc его, а затем освободите его, когда это необходимо.

Answer 4

Вы можете сделать это двумя способами.

Во-первых, как вы предлагаете, вы можете изменить член name человека из массива в указатель. Тогда в addNewPerson, вы бы просто сделать:

p *newPerson = (p *)malloc(sizeof(person)); 
p->name = name; 

EDIT: Как было отмечено в комментариях, это по своей сути плохо, и вы должны скопировать байты. Я слишком долго работал в ссылке, подсчитывая землю.

В противном случае вам нужно скопировать байты из переданного по имени в свой массив структур. Я хотел бы использовать strlcpy как таковой:

strlcpy(p->name, name, sizeof(p->name)); 

если таковые имеются, в противном случае используйте:

strncpy(p->name, name, sizeof(p->name)); 

Единственное различие в том, что вы должны нуль-прекратить p->name если name был слишком длинным, чтобы вписаться в p->name.

Answer 5

Я скорее называю метод constructPerson. Это на самом деле конструктор в объектно-ориентированного жаргоне

struct person 
{ 
    char *name; //can hold string of different sizes 
}p; 

p *addNewPerson(char *name) 
{ 
    p *newPerson = (p *)malloc(sizeof(person)); 
p->name= (char *) malloc(sizeof(char)*(strlen(name)+1));//don't forget to claim space for the '/0' character at the end of the string 
strcpy(p->name,name); 
    return newPerson; 
} 

Answer 6

Существует фундаментальный изъян в вашем коде, который никто не не подобран на еще что стоит отметить. p не является типом, это объявление переменной типа struct person. Ваша функция addNewPerson() просто не работает. Либо измените функцию addNewPersion() соответствующим образом:

struct person *addNewPerson(char *name) 
{ 
    ... 
} 

или определить тип p:

typedef struct person { ... } p; 

Затем используйте strcpy() (вернее, я бы предложил strncpy() вместо этого), как и другие предложили.

struct person *addNewPerson(char *name) 
{ 
    struct person *newPerson = (struct person *)malloc(sizeof(struct person)); 
    //strcpy(newPerson->name, name); 
    strncpy(newPerson->name, name, sizeof(newPerson->name)); //more safe 
    return newPerson; 
} 

Answer 7

Вы не нуждаетесь в STRCPY, вам не нужен таНос, вам не нужен никакие SizeOf и т.д. Если вы используете обугленный-массив в вашей структуре, например:

typedef struct { 
char name[25]; 
} Person; 

main() { 
    Person person; 
    char name[25]; 
    if(1==scanf("%24s",name)) 
    person=*(Person*)name; 
    return 0; 
} 

или, если вам нужно больше людей:

main() { 
    Person person[2]={{"firstperson"}}; 
    char name[25]; 

    if(1==scanf("%24s",name)) 
    person[1]=*(Person*)name; 

    puts(person[0].name); 
    puts(person[1].name); 

return 0; 
}