Avatar billede cdrasmussen Nybegynder
24. februar 2011 - 10:38 Der er 4 kommentarer og
1 løsning

Dynamisk array af struct

Det er efterhånden nogle år siden jeg har kodet C++ og nu er jeg stødt på et problem. Jeg kan angivligt ikke ændre størrelse på mit array af struct.

I koden kopierer jeg det oprindelig array over i et nyt og tilføjer en mere.

Jeg har følgende kode. hvor GasMatlabOut er en struct.

void CGasMatlab::AddOutput(std::string Line)
{
    GasMatlabOut out;
    GasMatlabOut *ptr_out;
    ptr_out = &out;
    ptr_out->stationName = this->GetStationName(Line);
    ptr_out->tagName = this->GetTagName(Line);

    this->_gasMatlabOut = this->_gasMatlabOut+1;
    GasMatlabOut *resize_arr = new GasMatlabOut[this->_gasMatlabOut];
    GasMatlabOut *ptr1, *ptr2;
        for(int i = 0; i < this->_gasMatlabOut; i++)
        {
        ptr1 = &_out[i];
        ptr2 = &resize_arr[i];
        ptr2 = ptr1;
        }
        *this->_out = *resize_arr;
    ptr1 = &this->_out[this->_gasMatlabOut-1];
    ptr1 = ptr_out;

    std::cout << ptr1->stationName + " ";
    std::cout << ptr1->tagName << std::endl;
    delete[] resize_arr;
}
Avatar billede arne_v Ekspert
24. februar 2011 - 15:14 #1
Jeg kan ikke lig ehelt gennemskue koden, men

1) det ser altsaa lidt ud som om du gemmer adressen paa en lokal variabel / det duer ikke

2) hvus du bruge vectir<GasMatlabOut> fremfor GasMatlabOut[] tror jeg at din kode ville blive meget paenere
Avatar billede segmose Nybegynder
25. februar 2011 - 11:32 #2
En af fejlene må være

*this->_out = *resize_arr;

some hvis der ikke er defineret andet laver en bit-vis kopiering af resize_arr over i _out, men da _out ikke er resized så fejler det da der kopieres det antal der er i resize_arr men det antal der er i _out som er en for lidt.

Hvad er formålet med::
        ptr1 = &_out[i];
        ptr2 = &resize_arr[i];
        ptr2 = ptr1;
Den gør ikke noget???

Hvis den skulle have kopieret skulle det have været
*ptr2=*ptr1;
men så kunne man lige så godt have skrevet
        resize_arr[i]=_out[i];
istedet???

JEg tror jeg ville have skrevet det om til::

void CGasMatlab::AddOutput(std::string Line)
{
    // Make new element
    GasMatlabOut out;
    out->stationName = this->GetStationName(Line);
    out->tagName = this->GetTagName(Line);

    this->_gasMatlabOut = this->_gasMatlabOut+1;
    GasMatlabOut *resize_arr = new GasMatlabOut[this->_gasMatlabOut];

        for(int i = 0; i < this->_gasMatlabOut; i++)
        {
                resize_arr[i] = _out[i];
        }

    // delete old data
    delete[] _old;  // if its allocated with new else that is a bug too.
    // replace _out with resize_arr
    this->_out = resize_arr;
   
   
    _out[this->_gasMatlabOut-1] = out; // copy of content of out

    // Test
    std::cout << _out[this->_gasMatlabOut-1].stationName + " ";
    std::cout << _out[this->_gasMatlabOut-1].tagName << std::endl;
}
Avatar billede segmose Nybegynder
25. februar 2011 - 11:32 #3
Bemærk utestet.
Avatar billede cdrasmussen Nybegynder
26. marts 2011 - 06:53 #4
Hej

Fandt en anden løsning. I fortjener begge et svar. Kan man det?
Avatar billede arne_v Ekspert
26. marts 2011 - 14:47 #5
ok
Avatar billede Ny bruger Nybegynder

Din løsning...

Tilladte BB-code-tags: [b]fed[/b] [i]kursiv[/i] [u]understreget[/u] Web- og emailadresser omdannes automatisk til links. Der sættes "nofollow" på alle links.

Loading billede Opret Preview
Kategori
Kurser inden for grundlæggende programmering

Log ind eller opret profil

Hov!

For at kunne deltage på Computerworld Eksperten skal du være logget ind.

Det er heldigvis nemt at oprette en bruger: Det tager to minutter og du kan vælge at bruge enten e-mail, Facebook eller Google som login.

Du kan også logge ind via nedenstående tjenester