QDataStream with vector of pointers

[jsulm]

@VincentLiu said in QDataStream with vector of pointers:

LightObj _standardLightObj

I don't see how it could be incorrect except that you seem to use a global variable which is bad.
Did you try to debug as I suggested before?

[VRonin]

Your problem, almost surely comes from commandObj->setCommandName(commandName); with commandObj being null or dangling.

A few things:

• Could you show us the implementation of QDataStream &operator<<(QDataStream &out, const LightObj &lightObj); and QDataStream &operator>>(QDataStream &in, LightObj &lightObj); ? (i'm 99.99% sure the problem is here)
• if AoiCommandObj and CurrentCommandObj have different members to serialise you'll need a more general approach.
• make sure you have clear the concept of virtual

The usual approach in this case is declare two virtual protected methods in the base class to do the serialisation. something like

class CommandObj{
// other stuff
protected:
virtual void serialise(QDataStream& stream) const {stream << _commandName;}
virtual void deserialise(QDataStream& stream) {stream >> _commandName;}
firend QDataStream &operator<<(QDataStream &out, const CommandObj& commandObj);
firend QDataStream &operator>>(QDataStream &in, CommandObj& commandObj);
};
QDataStream &operator<<(QDataStream &out, const CommandObj& commandObj);
QDataStream &operator>>(QDataStream &in, CommandObj& commandObj);

and in the definition:

QDataStream &operator<<(QDataStream &out, const CommandObj& commandObj){commandObj.serialise(out); return out;}
QDataStream &operator>>(QDataStream &in, CommandObj& commandObj){commandObj.deserialise(in); return in;}

now in the classes derived from CommandObj you just need to override serialise and deserialise to make everything work

[VincentLiu]

@VRonin Thanks. You are right. The commandObj is not accessible in commandObj->setCommandName(commandName); when I am under debug mode.

Here is my implementation:

QDataStream &operator<<(QDataStream &out, const LightObj &lightObj)
{
    out<< lightObj.getCommandObjs();
    return out;
}

QDataStream &operator>>(QDataStream &in, LightObj &lightObj)
{
    QVector<CommandObj*> commandObjs;

    in >>  commandObjs;

    lightObj.setCommandObjs(commandObjs);

    return in;
}

I am trying your suggestion now~~~

[VRonin]

Bingo!

The language/library is not as smart as you think it is. even if out<< lightObj.getCommandObjs(); might be smart enough to serialise pointers correctly when you use in >> commandObjs; how can the system know how much memory to allocate and whether a CommandObj* in the vector is a AoiCommandObj or a CurrentCommandObj?

You just need to do more manual work

[jsulm]

@VincentLiu

QVector<CommandObj*> commandObjs;

    in >>  commandObjs;

Your vector contains pointers and nobody creates instances as far as I can see. You need to read in a loop something like:

while(...) {
    CommandObj* commandObj = new CommandObj();
    in>>commandObj;
    commandObjs.append(&commandObj);
}

[VincentLiu]

@jsulm Hi, how can I know the condition should be in while loop ??? I don't have other informations

[VincentLiu]

@VRonin Hi, I got your points. What is the 'manual work' your mentioned?

[jsulm]

@VincentLiu While serializing you will need to write type information for each object, so later, when you're de-serializing you know what you're going to read next.
Serializing:

1. Write type information (could be just a byte)
2. Write object

De-serializing:

1. Read type information
2. Depending on the type create a CommandObj, AoiCommandObj or a CurrentCommandObj object
3. Read the object

while( !in.atEnd())...

[VRonin]

@VincentLiu

class CommandObj{
public:
enum CommandIDs : qint8 {
cmdInvalid
,cmdAoiCommand
,cmdCurrentCommand
};
CommandObj()
:classID(cmdInvalid)
{}
virtual  ~CommandObj()=default; //you probably want a virtual destructor too
void setCommandName(const QString& commandName);
QString getCommandName() const;
qint8 getClassID() const {return classID;}
protected:
QString _commandName;
const qint8 classID; // use this to identify AoiCommandObj from CurrentCommandObj, set it in the constructor
virtual void serialise(QDataStream& stream) const {stream << _commandName;}
virtual void deserialise(QDataStream& stream) {stream >> _commandName;}
firend QDataStream &operator<<(QDataStream &out, const CommandObj& commandObj);
firend QDataStream &operator>>(QDataStream &in, CommandObj& commandObj);
};
QDataStream &operator<<(QDataStream &out, const CommandObj& commandObj);
QDataStream &operator>>(QDataStream &in, CommandObj& commandObj);

QDataStream &operator<<(QDataStream &out, const LightObj &lightObj)
{
const QVector<CommandObj*> allCmds=lightObj.getCommandObjs();
    out<< static_cast<qint32>(allCmds.size());
for(CommandObj* cmd : allCmds)
out<< cmd->getClassID() << *cmd;
    return out;
}

QDataStream &operator>>(QDataStream &in, LightObj &lightObj)
{
qint32 vecSize=0;
  in >>vecSize;
qint8 cmdType=CommandObj::cmdInvalid;
    QVector<CommandObj*> commandObjs;
for(int i=0;i<vecSize;++i){
  in >>cmdType;
CommandObj* tempObj = nullptr;
switch(cmdType){
case CommandObj::cmdAoiCommand:
tempObj =new AoiCommandObj;
break;
case CommandObj::cmdCurrentCommand:
tempObj = new CurrentCommandObj;
break;
default:
Q_ASSERT_X(false,"QDataStream &operator>>(QDataStream &in, LightObj &lightObj)","Trying to load unsupported command");
}
if(!tempObj) continue;
in >> *tempObj;
commandObjs << tempObj;
}
    lightObj.setCommandObjs(commandObjs);

    return in;
}

[VincentLiu]

@VRonin and @jsulm

Finally it works!!!!!
Really thanks for your patience and kindness.