Dynamic memory allocation in c arrays of structs [closed]

Question

Closed. This question needs debugging details. It is not currently accepting answers.

Edit the question to include desired behavior, a specific problem or error, and the shortest code necessary to reproduce the problem. This will help others answer the question.

Closed last year.

Improve this question

void addWorkerToProject(Worker *worker, Project *project) {
    worker->projects =  malloc(sizeof (strlen(project)+1));
    worker->projects[worker->projectCount]->name =  project->name;
    worker->projectCount++;
}

    struct Worker

    char *name;
    Project **projects;
    int projectCount;

    struct Project

    char *name;
    Worker **workers;
    int workerCount;
    FeatureNode *features;

How do I make this function run correctly? It supposed to assign workers to project, but it fails to assign 2 projects to the same worker.

Let's say I have a worker named Sean and two projects name first and second. I need to attach them to Sean so for example when I print workers[i]->projects[j]->name they both needs to appear, when each iteration adds him 1 project. When I run it the first iteration is correct but the second one doesn't work.

Please post a Minimal, Reproducible Example and describe "run correctly" (for example, how the data should be set). — MikeCAT
– MikeCAT, Commented Mar 10, 2024 at 12:13
(strlen(project->name) + 1) * sizeof(Project*) makes no sense. Why do you want to multiply the string length with the size of a pointer (which will likely always be 8)? And if you want to copy a number of Project structures, you need a loop. — Some programmer dude
– Some programmer dude, Commented Mar 10, 2024 at 12:22

Amit · Accepted Answer · 2024-03-10 14:32:58Z

1

malloc(sizeof (strlen(project)+1)) will always return size of an integer. i.e.4 bytes/8 bytes based on your machine/os. What I felt you are trying to do is as below:

void addWorkerToProject(Worker *worker, Project *project) {
    worker->projects[worker->projectCount] =  malloc(sizeof(Project));
    worker->projects[worker->projectCount]->name = project->name;
    worker->projectCount++; 
}

edited Mar 10, 2024 at 14:32

answered Mar 10, 2024 at 14:29

Amit

112 bronze badges

Sign up to request clarification or add additional context in comments.

Collectives™ on Stack Overflow

Dynamic memory allocation in c arrays of structs [closed]

1 Answer 1

Comments

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Related