How to pass a QString to another class ?

[probine]

I must be doing something very wrong !!!

I am able to pass an integer from one class to another.

From class Client, I am passing this integer to class MessageIn:

messageIn->parseMessage(8);

messageIn->parseMessage(8);

To copy to clipboard, switch view to plain text mode 

The MessageIn class receives this call:

void MessageIn::parseMessage(int dataIn)
{
 //something here
}

void MessageIn::parseMessage(int dataIn)
{
 //something here
}

To copy to clipboard, switch view to plain text mode 

I want to do the exact same thing, but I want to pass a QString from Client to MessageIn.

How does the signature and expected variables for the functions should look like ?

[yogeshm02]

This seems very simple; may be I'm not understanding you question


If you want to change contents of string directly, then

void MessageIn::parseMessage(QString &str){
/////////
}

void MessageIn::parseMessage(QString &str){
/////////
}

To copy to clipboard, switch view to plain text mode 

If you want to return changed value, then

QString MessageIn::parseMessage(const QString &str){
/////////
}

QString MessageIn::parseMessage(const QString &str){
/////////
}

To copy to clipboard, switch view to plain text mode 

or

QString &MessageIn::parseMessage(const QString &str){
/////////
}

QString &MessageIn::parseMessage(const QString &str){
/////////
}

To copy to clipboard, switch view to plain text mode 

If you just want to receive value, then

void MessageIn::parseMessage(const QString &str){
/////////
}

void MessageIn::parseMessage(const QString &str){
/////////
}

To copy to clipboard, switch view to plain text mode 

[probine]

I guess the code you posted is passing by reference, so changes to the string in one class will affect the whole string.

How about passing a copy of the string, so I work in the copy ?

[probine]

It should be very simple....

in Class ClienThread I have this:

void ClientThread::readData(){
  dataIn.clear();
  dataIn=(_tcpSocket->readAll());
  textBrowser->append(dataIn);
  [B]messageIn->parseMessage(dataIn);[/B]
}

void ClientThread::readData(){
  dataIn.clear();
  dataIn=(_tcpSocket->readAll());
  textBrowser->append(dataIn);
  [B]messageIn->parseMessage(dataIn);[/B]
}

To copy to clipboard, switch view to plain text mode 

The "dataIn" is a QString that has been declared earlier in the class.

In class MessageIn I have:

void MessageIn::parseMessage(QString &dataIn){
}

void MessageIn::parseMessage(QString &dataIn){
}

To copy to clipboard, switch view to plain text mode 

Class "MessageIn" has to receive the string from class "ClientThread". The string should be a copy of the original string, because the original will be cleared and I just want to work in the copy !!!

How ?

[jacek]

I just want to work in the copy !!!

Then use this:

void MessageIn::parseMessage(QString dataIn){ ... }

void MessageIn::parseMessage(QString dataIn){ ... }

To copy to clipboard, switch view to plain text mode 

But beware that if you just invoke that method from client thread, it will be executed within that thread. If you want to pass data between threads better use signals and queued connections.

[probine]

ok, not working... here again in detail:

Class ClientThread.h

..
QString * dataIn;
..

..
QString * dataIn;
..

To copy to clipboard, switch view to plain text mode 

Class ClientThread.cpp

..
void ClientThread::readData(){
  dataIn->clear();
  dataIn->append(_tcpSocket->readAll());
  textBrowser->append(dataIn->toAscii());
  messageIn->parseMessage(*dataIn);
}
..

..
void ClientThread::readData(){
  dataIn->clear();
  dataIn->append(_tcpSocket->readAll());
  textBrowser->append(dataIn->toAscii());
  messageIn->parseMessage(*dataIn);
}
..

To copy to clipboard, switch view to plain text mode 

Class MessageIn.h

..
void parseMessage(Qstring dataIn);
..

..
void parseMessage(Qstring dataIn);
..

To copy to clipboard, switch view to plain text mode 

Class MessageIn.cpp

..
void MessageIn::parseMessage(Qstring dataIn){
}
..

..
void MessageIn::parseMessage(Qstring dataIn){
}
..

To copy to clipboard, switch view to plain text mode 

This gives me the following error:

messagein.h:10: error: variable or field `parseMessage' declared void
messagein.h:10: error: expected `;' before '(' token
clientthread.cpp: In member function `void ClientThread::readData()':
clientthread.cpp:34: error: 'class MessageIn' has no member named 'parseMessage'
mingw32-make[1]: *** [release\clientthread.o] Error 1
mingw32-make[1]: Leaving directory `E:/infospeed/program/server'
mingw32-make: *** [release] Error 2

Any idea ?

[hickscorp]

According to me...

messagein.h:10: error: variable or field `parseMessage' declared void

messagein.h:10: error: variable or field `parseMessage' declared void

To copy to clipboard, switch view to plain text mode 

This message most likely tells you that in your .h you declared the method "parseMessage" as void, and in your implementation you tried to return something or declared it with any other type.

Also:

messagein.h:10: error: expected `;' before '(' token

messagein.h:10: error: expected `;' before '(' token

To copy to clipboard, switch view to plain text mode 

This message probably points you that you didn't put a ";" at the end bracket of your class definition. E.G.:

class A { ... }

class A { ... }

To copy to clipboard, switch view to plain text mode 

instead of

class A { ... };

class A { ... };

To copy to clipboard, switch view to plain text mode 

Hope it will help,
Pierre.

[probine]

That is not the problem... I can compile and execute the program without any problem if I just pass an integer from one class to the other, but as soon as I try to pass a QString or a string, then I get estrange error messages.

[jacek]

messagein.h:10: error: variable or field `parseMessage' declared void
messagein.h:10: error: expected `;' before '(' token

Are you sure that these are the first errors you get? Maybe you are missing #include directive somewhere?

[probine]

Thanks JACEK,

I included <QString> in all the classes.

I don't know why I have to included it through !!! I thought QString was a part of the core of Qt.