char* from QString

[bruccutler]

Hello,
Another really newbie question. What is the appropriate way to get a char * pointer from a QString object? I'm using object.toLocal8Bit().constData(), but doesn't always work (or maybe it doesn't work at all).
- BC

[wysota]

It works, just remember it is a const char*. BTW. there is a macro which is a shortcut to the above mentioned combination:

QString str = "xxxx";
printf("%s", qPrintable(str));

QString str = "xxxx";
printf("%s", qPrintable(str));

To copy to clipboard, switch view to plain text mode 

[jacek]

Please, search the forum before you ask a question. This topic was already discussed here: http://www.qtcentre.org/forum/f-qt-p...ting-3211.html

[bruccutler]

Please, search the forum before you ask a question. This topic was already discussed here: http://www.qtcentre.org/forum/f-qt-p...ting-3211.html

So, will the qPrintable work, or is the only (best) way to create a QByteArray that doesn't go out of scope? I looked at the previous post and can I assume yours is the only way to do this?

Incidentally, I find the Qt site difficult to search. This forum is excellent, however!
- BC

[jpn]

Incidentally, I find the Qt site difficult to search.

Have you tried Qt Assistant's indexing and searching facilities? In my opinion it's superior..

[jacek]

So, will the qPrintable work, or is the only (best) way to create a QByteArray that doesn't go out of scope? I looked at the previous post and can I assume yours is the only way to do this?

qPrintable() will only work as a function parameter, so either use:

char *cstr = qstrdup( qPrintable( str ) );
...
delete [] cstr;

char *cstr = qstrdup( qPrintable( str ) );
...
delete [] cstr;

To copy to clipboard, switch view to plain text mode 

or

QByteArray ba( str.toLocal8bit() );
const char *cstr = ba.constData(); // cstr is valid as long as ba is alive

QByteArray ba( str.toLocal8bit() );
const char *cstr = ba.constData(); // cstr is valid as long as ba is alive

To copy to clipboard, switch view to plain text mode 

[bruccutler]

Thanks. I've gone with the second method. Thanks for the help and I apologize that I didn't do a search first.

[sunil.thaha]

From the Faq

You can use toLatin1() to get a byte array and then call data() on that one which will give you a char*, i.e
toLatin1().data() // Is this wrong ??

http://doc.trolltech.com/4.2/qstring.html#toLatin1

To convert a QString to a char* you can do the following:
QString(QLatin1String(charstringhere))

See the documentation:

http://doc.trolltech.com/4.2/qlatin1string.html

So is the Faq wrong ??

I have tried this program

#include <iostream> 
#include <QString> 
 
int main(){
    QString str = "Sunil Thaha";
    std::cout << str.toLatin1().data() << std::endl;
 
    const char *charPointer = str.toLatin1().data();
    std::cout << charPointer << std::endl;
 
    if( charPointer == str.toLatin1().data() ){
        std::cout<< "Still pointing to same location ";
    }
    return 0;
}

#include <iostream> 
#include <QString> 

int main(){
    QString str = "Sunil Thaha";
    std::cout << str.toLatin1().data() << std::endl;
    
    const char *charPointer = str.toLatin1().data();
    std::cout << charPointer << std::endl;
    
    if( charPointer == str.toLatin1().data() ){
        std::cout<< "Still pointing to same location ";
    }
    return 0;
}

To copy to clipboard, switch view to plain text mode 

My results confirms with the Faq... I am able to get a const pointer to the data() and further calls returns the same memory address ?? Both Wiki and jaceks post discourages us from using this. So who is right here ?

[wysota]

str.toLatin1().data()

str.toLatin1().data()

To copy to clipboard, switch view to plain text mode 

This creates a temporary object (QByteArray) and returns a pointer to its data, but the temporary object is deleted immediately. The memory is not cleared, the compiler does some optimisations, etc. - the bottom line is the data IS there, but only because you're lucky. If you try to use the pointer after some time the byte array is deleted, you'll eventually receive trash which may cause your app to malfunction. This is strictly related to memory management on your system and to the compiler used. It is not safe to use such constructions in terms of the C++ standard.

You can think of it as the same as if you'd send a signal accross threads in Qt3 - basically it works (unfortunately only "most of the time"), but it doesn't mean that the code is correct.

[sunil.thaha]

Please refer to this Faq http://www.trolltech.com/developer/k...30.9032238253/

How can we ask for correction ?

[wysota]

There is nothing wrong in that entry, besides it seems to be an internal Trolltech tool and I'm sure they know about the quirk of storing the volatile pointer.

[sunil.thaha]

There is nothing wrong in that entry,

But all the posts above are against it isn't it ? And that being an official Faq; should not be pointing in a wrong direction

[wysota]

But all the posts above are against it isn't it ?

No, it doesn't say anything about storing the data pointer and only referring to it after the temporary object is destroyed is in violation with the specification. This is a perfectly legitimate statement:

QString str = "xxxx";
printf("%s\n", str.toLocal8Bit().data());

QString str = "xxxx";
printf("%s\n", str.toLocal8Bit().data());

To copy to clipboard, switch view to plain text mode 

whereas this one is not:

QString str = "xxxx";
char *ptr = str.toLocal8Bit().data();
printf("%s\n", ptr);

QString str = "xxxx";
char *ptr = str.toLocal8Bit().data();
printf("%s\n", ptr);

To copy to clipboard, switch view to plain text mode 

If you want to store the pointer, keep the byte array intact:

QString str = "xxxx";
QByteArray ba = str.toLocal8Bit();
char *ptr = ba.data(); // this is valid as long as "ba" is
//...
printf("%s\n", ptr);

QString str = "xxxx";
QByteArray ba = str.toLocal8Bit();
char *ptr = ba.data(); // this is valid as long as "ba" is
//...
printf("%s\n", ptr);

To copy to clipboard, switch view to plain text mode 

[sunil.thaha]

Waw !!

Beautiful answer... Thanks a lot for the time and patience.
Long live QtCentre

[sunil.thaha]

After this discussions and the subsequent discussion with Qt-Interest mailing list. I had requested the change told above. Trolltech has accepted the change request and has modified the faq

Here is the link to that

http://www.trolltech.com/developer/k...-30.9032238253


Thanks once again.

[VireX]

Btw, jacek wrote delete [] cstr, but you dont need to delete a pointer unless you initialized it with new keyword.

[sunil.thaha]

From the docs:

char * qstrdup ( const char * src ) Returns a duplicate string.
Allocates space for a copy of src, copies it, and returns a pointer to the copy. If src is 0, it immediately returns 0.
Ownership is passed to the caller, so the returned string must be deleted using delete[]