Почему структуру выводит только последние введенные данные типа char*?

@DenisWeb · Регистрация: 15.08.2017

Не могу понять почему выводит только последние введенные данные. Как выводит и как должно быть, пример в коде есть.
Можете объяснить почему не работает, как задумано и как это исправить? Уже неделю думаю и пытаюсь понять...

C

#include <stdio.h>
#include <stdlib.h>
#define PN printf("\n");
 
/* 
Как выводит:
 
Какое у тебя имя? ->Денис
Какая у тебя фамилия? ->Денисов
Какая у тебя должность? ->пилот
Какой у тебя стаж? ->23
Какое у тебя имя? ->Михаил
Какая у тебя фамилия? ->Попов
Какая у тебя должность? ->Разраб
Какой у тебя стаж? ->32
 
Вы Михаил Попов
И ваша должность: Разраб со стажем в 32 лет
Вы Михаил Попов
И ваша должность: Разраб со стажем в 23 лет
 
_________________________________
 
Как должно выводить:
 
Какое у тебя имя? ->Денис
Какая у тебя фамилия? ->Денисов
Какая у тебя должность? ->пилот
Какой у тебя стаж? ->23
Какое у тебя имя? ->Михаил
Какая у тебя фамилия? ->Попов
Какая у тебя должность? ->Разраб
Какой у тебя стаж? ->32
 
 
 
Вы Михаил Попов
И ваша должность: Разраб со стажем в 32 лет
Вы Денис Денисов
И ваша должность: пилот со стажем в 23 лет
*/
 
 
 
struct works {
    char *name, *surname, *type;    // имя, фамилия, дложность
    int time;   // стаж
    struct works *next;
};
 
struct works *creater(struct works *, char *, char *, char *, int); // создание структуры
void list(struct works *);  // вывод структуры 
 
int main(){
    char *name, *surname, *type;
    name = (char*)malloc(30);
    surname = (char*)malloc(40);
    type = (char*)malloc(20);
    struct works *head;
    head = NULL;
 
    int time, n, i; // стаж, n - количество строчек
    scanf("%d", &n);
    for(i=0;i<n;i++){
        printf("Какое у тебя имя? ->"); scanf("%30s", name);
        printf("Какая у тебя фамилия? ->"); scanf("%40s", surname);
        printf("Какая у тебя должность? ->"); scanf("%20s", type);
        printf("Какой у тебя стаж? ->"); scanf("%d", &time);
        head = creater(head, name, surname, type, time);
        // printf("Вы %s %s\nИ ваша должность: %s со стажем в %d лет", (head)->name, (head)->surname, (head)->type, (head)->time);
    }
    list(head); // выводим
    free(head);
    return 0;
}
 
 
struct works *creater(struct works *head, char *name, char *surname, char *type, int time)
{
    struct works *element;
    (element) = (struct works *)malloc(sizeof(struct works));  // выделяем память новому элементу
    (element)->name = name;
    (element)->surname = surname;
    (element)->type = type;
    (element)->time = time;
    (element)->next = head;
    return element;
};
 
void list(struct works *p)
{
    PN PN PN
    while(p != NULL)    // пока не конец цикла
    {
            printf("Вы %s %s\nИ ваша должность: %s со стажем в %d лет", p->name, p->surname, p->type, p->time);
            PN
            p=p->next;
    }
    PN
}

@shadowmaan · 06.04.2019, 08:35

(element) = (struct works *)malloc(sizeof(struct works)); // выделяем память новому элементу
(element)->name = name;
(element)->surname = surname;
(element)->type = type;
(element)->time = time;
(element)->next = head;

Вот тут каждый элемент в списке ссылается на одни и те же строки name, surname, type. Надо каждому элементу выделять память для его собственных строк и копировать в них передаваемые со ввода значения. Можно использовать strdup(), например. Тогда
element->name = strdup(name);
и т.д.

И подключи заголовок string.h (для strdup).

Кстати, скобки вокруг element совершенно излишни.

@shadowmaan 73 / 54 / 24 Регистрация: 15.05.2013 Сообщений: 170
	06.04.2019, 08:35	2
	(element) = (struct works *)malloc(sizeof(struct works)); // выделяем память новому элементу (element)->name = name; (element)->surname = surname; (element)->type = type; (element)->time = time; (element)->next = head; Вот тут каждый элемент в списке ссылается на одни и те же строки name, surname, type. Надо каждому элементу выделять память для его собственных строк и копировать в них передаваемые со ввода значения. Можно использовать strdup(), например. Тогда element->name = strdup(name); и т.д. И подключи заголовок string.h (для strdup). Кстати, скобки вокруг element совершенно излишни. 1

Опции темы