is it a C++ error or Qt/Qwt related?

[jacek]

Qwt docs say:

setRawData is provided for efficiency. It is important to keep the pointers during the lifetime of the underlying QwtCPointerData class.

but you pass pointers to local variables to every call to setRawData().

[jiveaxe]

Sorry if I ask, but how should I modify the code?

Thanks

[mcostalba]

Allocate data on the heap.

Put somewhere in your private section:

QVector<double*> samplesPtr;
	int samplesSize;

QVector<double*> samplesPtr;
	int samplesSize;

To copy to clipboard, switch view to plain text mode 

Then modify as follows

void Dialog::getData()
{
 
    samplesSize = 1+rand()%10 + 10;
 
    double* redCurveDataPtr = new double[samplesSize];
    double* greenCurveData = new double[samplesSize];
    double* xval = new double[samplesSize];
 
    samplesPtr << redCurveDataPtr << greenCurveData << xval;
 
   .......
 
}

void Dialog::getData()
{

    samplesSize = 1+rand()%10 + 10;

    double* redCurveDataPtr = new double[samplesSize];
    double* greenCurveData = new double[samplesSize];
    double* xval = new double[samplesSize];

    samplesPtr << redCurveDataPtr << greenCurveData << xval;

   .......

}

To copy to clipboard, switch view to plain text mode 

Of course you need to add a Dialog::freeData() to release the memory when no more needed. That's the reason you may want to set samplesPtr and samplesSize as class memeber data.

[mcostalba]

Interestin enough delete[] works also from copied pointers, it means that you don't need

int samplesSize;

as member data but a local variable will suffice and freeData() became:

void Dialog::freeData()
{
     for(int i = 0; i < samplesPtr.size(); i++)
            delete[] samplesPtr.at(i);
}

void Dialog::freeData()
{
     for(int i = 0; i < samplesPtr.size(); i++)
            delete[] samplesPtr.at(i);
}

To copy to clipboard, switch view to plain text mode 

[jiveaxe]

Thank you, mcostalba, your code works good.

Regards

[jiveaxe]

One more question: the freeData() function must be called in the destructor only or before each getData()?

Thanks

[mcostalba]

After

redCurve.setRawData(xval,redCurveData,size);
greenCurve.setRawData(xval,greenCurveData,size);

redCurve.setRawData(xval,redCurveData,size);
greenCurve.setRawData(xval,greenCurveData,size);

To copy to clipboard, switch view to plain text mode 

you don't need the previous samples anymore (it seems) so you can call freeData() after that, of course delaying assign of new pointers until after the free of old data:

void Dialog::getData()
{
    int size = 1+rand()%10 + 10;
 
    double* redCurveDataPtr = new double[size];
    double* greenCurveData = new double[size];
    double* xval = new double[size];
 
    for(int i = 0; i < size; i++)
    {
        redCurveData[i] = 0+rand()%10;
        greenCurveData[i] = 0+rand()%10;
        xval[i] = i + 1;
    }
 
    redCurve.setRawData(xval,redCurveData,size);
    greenCurve.setRawData(xval,greenCurveData,size);
 
    // old data is no more needed now
    freeData();
 
    // now we can add newly created samples pointers
    samplesPtr << redCurveDataPtr << greenCurveData << xval;
 
    plot->setAxisScale(QwtPlot::xBottom, 1, size);
    plot->setAxisScale(QwtPlot::yLeft, 0, 10);
 
    drawCurves();
}

void Dialog::getData()
{
    int size = 1+rand()%10 + 10;

    double* redCurveDataPtr = new double[size];
    double* greenCurveData = new double[size];
    double* xval = new double[size];

    for(int i = 0; i < size; i++)
    {
        redCurveData[i] = 0+rand()%10;
        greenCurveData[i] = 0+rand()%10;
        xval[i] = i + 1;
    }

    redCurve.setRawData(xval,redCurveData,size);
    greenCurve.setRawData(xval,greenCurveData,size);

    // old data is no more needed now
    freeData();

    // now we can add newly created samples pointers
    samplesPtr << redCurveDataPtr << greenCurveData << xval;

    plot->setAxisScale(QwtPlot::xBottom, 1, size);
    plot->setAxisScale(QwtPlot::yLeft, 0, 10);

    drawCurves();
}

To copy to clipboard, switch view to plain text mode 

[mcostalba]

A couple of bugs

Because pointer vector is reused freeData() should be:

void Dialog::freeData()
{
   for(int i = 0; i < samplesPtr.size(); i++)
        delete[] samplesPtr.at(i);
 
   samplesPtr.clear(); 
}

void Dialog::freeData()
{
   for(int i = 0; i < samplesPtr.size(); i++)
        delete[] samplesPtr.at(i);

   samplesPtr.clear(); 
}

To copy to clipboard, switch view to plain text mode 

And also remember to add a freeData() call in the d'tor to be sure the last samples data set is properly released when closing the dialog.

[jiveaxe]

Inserting freeData() where you say gives me strange graphs. Removing it works again right.

Regards