Copying struct's dynamic array to another struct

Question

I have a struct defined like this:

struct Queries {

    uint64_t Id;
    uint64_t from;  
    uint32_t counter; // total queries
    char queries[];
};

What I am trying to do is create a new struct "object" and copy the values from an existing one to this new object.

What I tried

void function(Queries* oldq){

    Queries* q = new Queries();

    // values are copied correctly
    q->Id = oldq->Id;
    q->from = oldq->from;
    q->counter = oldq->counter;

    // copy is not correct
    for (unsinged i = 0; i < oldq->counter; i++)
          q->queries[i] = oldq->queries[i];

}

1) I also tried:

q = oldq;

but this does not work.

2) I think I have to allocate counter * sizeof(char) space for the queries array but since the struct's member is not a pointer I don't know how to do this.

If you define proper copy ctor you can use Queries *q = new Queries( *oldq ); — Slava
– Slava, Commented Jan 8, 2016 at 15:55
Your thinking is correct. You need to pre-allocate counter * sizeof(char) bytes before copying the array. But you don't need an explicit loop to copy it - you can use memcpy. Alternatively, you may implement this as a copy constructor, as cited by Slava. — jweyrich
– jweyrich, Commented Jan 8, 2016 at 15:55
But how should I do the assignment in the array case inside the copy constructor? — pirox22
– pirox22, Commented Jan 8, 2016 at 16:18
Is this your struct or are you using some library code? If it's yours, I suggest using either char* or std::string instead of a zero-sized array (you should've gotten a warning in a C++ compiler). If it's from library API, take a look at this and this. — prazuber
– prazuber, Commented Jan 8, 2016 at 16:21
Unfortunately, I am not allowed to change the structure of the struct, so changing char to stl's string is impossible. I '' ll check your links. Thanks — pirox22
– pirox22, Commented Jan 8, 2016 at 16:25

Community · Accepted Answer · 2017-05-23 12:03:59Z

2

Here you are dealing with a C-style flexible array member. It's not valid C++ code, but it is valid C since C99 (see link for details). To use such structure, you need to allocate sizeof(Queries) + counter bytes, where the array field will use that counter bytes part. (Note: if you had array field other than char you would have to multiply accordingly.)

Now, you cannot use C++ features here like copy constructor since compiler doesn't know the size of your structure. Instead, you have to use the pure C approach:

Queries *cloneQueries(Queries *oldQ)
{
    size_t sizeQ = sizeof(Queries) + oldQ->counter;
    Queries *newQ = (Queries*)malloc(sizeQ);
    memcpy(newQ, oldQ, sizeQ);
    return newQ;
}

edited May 23, 2017 at 12:03

CommunityBot

11 silver badge

answered Jan 8, 2016 at 16:57

prazuber

1,36210 silver badges29 bronze badges

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

Comments

Bathsheba · Accepted Answer · 2016-01-08 16:04:27Z

2

The simplest thing to do is to use a std::string for queries.

Then you can simply write Queries* q = new Queries(*oldq); and rely on the compiler-generated constructor: you can remove all your copying code.

edited Jan 8, 2016 at 16:04

answered Jan 8, 2016 at 15:58

Bathsheba

235k35 gold badges376 silver badges504 bronze badges

Comments

Ziezi · Accepted Answer · 2016-01-08 17:10:34Z

0

You could do it by using copy constructor that performs a deep copy of your object.

This could be done when instead of function() you define a copy constructor like so:

Queries(const Queries& q)
    : Id(q.Id), from(q.from), counter(q.counter)
{
    // allocate the new memory
    queries = new char[counter];

    // copy each element
    for (size_t i = 0; i < counter; ++i) {
        queries[i] = q.queries[i];
    }
}

and then in your code, you could use the line:

Queries *q = new Queries(*oldq);

where the object on the right hand side is created by copy construction, i.e. by copying the object oldq.

See, how operator new[] works.

answered Jan 8, 2016 at 17:10

Ziezi

6,5134 gold badges43 silver badges54 bronze badges

1 Comment

jwd Over a year ago

I think this is wrong — you cannot assign a pointer (new char[...]) to a flexible-length array member (char queries[]). You could if it were char *queries, but that's not the question.

Collectives™ on Stack Overflow

Copying struct's dynamic array to another struct

3 Answers 3

Comments

Comments

1 Comment

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Linked

Related