Copy Datatype



  • THE POST BELOW IS MORE THAN 5 YEARS OLD. RELATED SUPPORT INFORMATION MIGHT BE OUTDATED OR DEPRECATED

    On 20/05/2009 at 01:51, xxxxxxxx wrote:

    So it is.
    Here the operator.

    > AixGeometry& AixGeometry::operator=(const AixGeometry& src){ \> \>      if(this == &src;){ \>            \>           return \*this; \>      } \>       \>         //Allocate memory Points \>      GeFree(this->pointArray); \>      this->pointArray = NULL; \>      this->pointArray = (AixPoint\* ) GeAlloc(sizeof(AixPoint) \* src.pointArraySize); \>       \>      this->pointArraySize = src.pointArraySize; \>       \>         //Allocate memory lines \>      GeFree(this->lineArray); \>      this->lineArray = NULL; \>      this->lineArray = (AixNline\* ) GeAlloc(sizeof(AixNline) \* src.lineArraySize); \>       \>      this->lineArraySize = src.lineArraySize; \>       \>      //Allocate memory Ngons \>      GeFree(this->ngonArray); \>      this->ngonArray = NULL; \>      this->ngonArray = (AixNgon\* ) GeAlloc(sizeof(AixNgon) \* src.ngonArraySize); \>       \>      this->ngonArraySize = src.ngonArraySize; \>       \>      //Points \>      for(LONG i = 0; i < src.pointArraySize - 1; i++){ \>            \>           this->pointArray[i] = src.pointArray[i]; \>      } \>       \>      //Lines \>      for(LONG l = 0; l < src.lineArraySize - 1; l++){ \>            \>           this->lineArray[l] = src.lineArray[l]; \>      } \>       \>      //Ngons \>      for(LONG j = 0; j < src.ngonArraySize - 1; j++){ \>            \>           this->ngonArray[j] = src.ngonArray[j]; \>      } \>            \>      //dirtyCount \>      this->dirtyCount = src.dirtyCount; \>       \>      this->additionalAttributes = src.additionalAttributes; \> \>      return \*this; \> }

    But if i comment it out, Cinema builds its own =operator internally, which does the same garbage :-).

    I have tried to write my DataType into GeData, copy the GeData into an other with its copy-function and read out the copied GeData. This doesn't work too.



  • THE POST BELOW IS MORE THAN 5 YEARS OLD. RELATED SUPPORT INFORMATION MIGHT BE OUTDATED OR DEPRECATED

    On 20/05/2009 at 02:53, xxxxxxxx wrote:

    Did you try to pass the reference as a = *b; ?



  • THE POST BELOW IS MORE THAN 5 YEARS OLD. RELATED SUPPORT INFORMATION MIGHT BE OUTDATED OR DEPRECATED

    On 20/05/2009 at 03:46, xxxxxxxx wrote:

    Ok this is funny.
    I have found a line in my code where i wrote something like this.

    AixGeometry cubeCache = *defaultCube; //defaultCube is a pointer on an    
                                                                              //AixGeometry

    In this case it seems to work correct. But why it doesn't work if i have two "concrete" AixGeometries.

    AixGeometry a;
    AixGeometry b;

    b = a;

    Is it not the same as b = *a if a is a pointer (AixGeometry* a)?

    Thanks,

    Oli



  • THE POST BELOW IS MORE THAN 5 YEARS OLD. RELATED SUPPORT INFORMATION MIGHT BE OUTDATED OR DEPRECATED

    On 20/05/2009 at 07:48, xxxxxxxx wrote:

    Ok, i found another line where a = *b doesn't work :-(.
    I have written a copy-constructor with the same success. It can't be mere chance, that the function one time does it's work and one time not.

    I' m glad about every idea i get.

    cheers,

    Oli



  • THE POST BELOW IS MORE THAN 5 YEARS OLD. RELATED SUPPORT INFORMATION MIGHT BE OUTDATED OR DEPRECATED

    On 20/05/2009 at 11:56, xxxxxxxx wrote:

    Do you have a copy constructer defined in your AixGeometry class?



  • THE POST BELOW IS MORE THAN 5 YEARS OLD. RELATED SUPPORT INFORMATION MIGHT BE OUTDATED OR DEPRECATED

    On 25/05/2009 at 01:56, xxxxxxxx wrote:

    Hi,

    yes, i have tried this too. But no chance; same garbage.



  • THE POST BELOW IS MORE THAN 5 YEARS OLD. RELATED SUPPORT INFORMATION MIGHT BE OUTDATED OR DEPRECATED

    On 25/05/2009 at 02:46, xxxxxxxx wrote:

    Actually the assignment operator must be called as a.operator=(b) is called in the case of a = b.

    But if you have both defined (you should have both defined), an explicit copy constructor and an assignment operator, the only chance (cause I don´t know the structure of the full class you have there) you should in the end try a copy assignment operator, to avoid shallow copying.



  • THE POST BELOW IS MORE THAN 5 YEARS OLD. RELATED SUPPORT INFORMATION MIGHT BE OUTDATED OR DEPRECATED

    On 26/05/2009 at 02:26, xxxxxxxx wrote:

    So, i have tried this too; without success. I have also compared the memory-adresses - all correct. So i have to search anywhere else in my code.

    Thanks for your help,

    Oli



  • THE POST BELOW IS MORE THAN 5 YEARS OLD. RELATED SUPPORT INFORMATION MIGHT BE OUTDATED OR DEPRECATED

    On 26/05/2009 at 02:30, xxxxxxxx wrote:

    hmm, then I really have no other idea either. :-/ I am curious in knowing the solution to your problem if you find out.

    Good luck



  • THE POST BELOW IS MORE THAN 5 YEARS OLD. RELATED SUPPORT INFORMATION MIGHT BE OUTDATED OR DEPRECATED

    On 26/05/2009 at 06:16, xxxxxxxx wrote:

    I got it :-).
    AixGeometry works like a container for the other datatypes (points, lines etc). In one of them i have commented out the operator= not so long ago to debug something.

    So I summarise.
    To make deep copies of your datatype(s), you need the CopyData-function of each datatype, the operator= and the copyconstructor.
    If one of the last two functions doesn't exist, Cinema only makes a shallow copy.

    So thanks for your help,

    cheers Oli



  • THE POST BELOW IS MORE THAN 5 YEARS OLD. RELATED SUPPORT INFORMATION MIGHT BE OUTDATED OR DEPRECATED

    On 26/05/2009 at 06:19, xxxxxxxx wrote:

    ah there you go. Glad you finally worked it out. The good ol power of three. :)


Log in to reply