[C++] declareren array in 'protected' en maken in constructr

[Duim]

ik declareer in mij FlowObject.h file mijn arrays van floats (float* p_inputvector;):

C++:

   
class FlowObject
{
public:
    FlowObject();
    ~FlowObject();
    FlowObject(const FlowObject& orig);
    FlowObject& operator=(const FlowObject& orig);  
    

protected:
    float* p_inputvector;
    float* p_outputvector;
};

 

Deze wil ik in de class gebruiken en gebruik in de constructor dus:

C++:

  
FlowObject::FlowObject()
{
      p_inputvector  = new float[10];
      p_outputvector = new float[10];
           for (int i=0;i<10;i++)
           {
                   p_inputvector[i]=0;
                   p_outputvector[i]=0;
           }    

}
 

Alles compileert en bouwt prima, toch geefttie bij de uitvoer hier een "unhandled exception at..."

Is er wat jullie betreft iets fout aan het bovenstaande? Of moet ik de fout ergens anders zoeken?

[MSalters]

Ik gok dat je fout in de copy ctor zit. Als je zelf met pointer gaat stoeien, dan moet je die ook schrijven (en assignment, en destructor).

Gebruuik gewoon een std::vector<float>, dan heb je al dat gedoe niet.

[GX]

Ik krijg helemaal geen error wanneer ik het compile en uitvoer. Zou je kunnen zeggen welke regel er fout gaat zodat we wat meer inzicht kunnen krijgen.

Los daarvan kan je allicht beter de STL functies gebruiken inderdaad. Lekker simpel. Ik weet alleen niet wat de performance impact op je applicatie zal zijn.

_{@MSalters
Het gaat hier over de std::vector delete/insert ops, niet over array delete/insert}

[ Voor 15% gewijzigd door GX op 01-01-2006 14:01 ]

[jos707]

Zolang je een std::vector niet gebruikt voor heel veel delete/insert operaties maakt het de performantie
weinig uit. In het andere geval is een gewone array wel sneller.

[Soultaker]

Om concreet antwoord te geven op de vraag van de topicstarter (): de code die je post is zeker weten goed, dus de fout zit ergens anders.

Wat het dan wel is, is lastig te zeggen zo; het kan van alles zijn.

[MSalters]

jos707 schreef op zaterdag 31 december 2005 @ 14:15:
Zolang je een std::vector niet gebruikt voor heel veel delete/insert operaties maakt het de performantie weinig uit. In het andere geval is een gewone array wel sneller.

Hoe wil je uberhaupt 1 delete/insert doen op een gewone array? Een int[10] bevat altijd 10 elementen.

[jos707]

MSalters schreef op zaterdag 31 december 2005 @ 17:48:
[...]

Hoe wil je uberhaupt 1 delete/insert doen op een gewone array? Een int[10] bevat altijd 10 elementen.

ja idd maar ik doelde door de elementen te kopieren naar een nieuwe grotere/kleinere array bijvoorbeeld. Nu ik dacht dat die die insert en delete operaties die een vector voor jouw allemaal automatisch doet duurder waren.

[Duim]

Voor eerst bedankt, hier gaat dus niets mis, er gaat ergens anders iets mis. Ik hou jullie op de hoogte.

[MSalters]

jos707 schreef op zaterdag 31 december 2005 @ 18:27:
[...]
ja idd maar ik doelde door de elementen te kopieren naar een nieuwe grotere/kleinere array bijvoorbeeld. Nu ik dacht dat die die insert en delete operaties die een vector voor jouw allemaal automatisch doet duurder waren.

Waarom zou het kopieren van een float duurder zijn als ie in een vector zit? Ik kan me situaties voorstellen waarin het goedkoper is (betere alignment) maar duurder lijkt me erg onwaarschijnlijk.