我尽量避免在QT计划中使用混合的QString
和char*
类型。我有一个会话函数,它返回QString
内部数据的指针,但我得到了非常奇怪的结果。这段代码怎么了?
编译器和环境gcc(Debian 4.9.2-10)4.9.2
通过qmake标记:QMAKE_CXXFLAGS += -std=c++11
代码段:
#include <QCoreApplication>
#include <iostream>
#define PCH(m) ( m.toAscii().data() )
// get the pointer short scripted
const char* CH(QString msg) {
const char* res = msg.toAscii().data();
return res;
}
int main(int argc, char *argv[])
{
QCoreApplication app(argc, argv);
const QString qformat = "%s does not work!";
const QString qvalue = "embedding";
// Works not
qDebug(CH(qformat), CH(qvalue));
// Works
qDebug(qformat.toAscii().data(), qvalue.toAscii().data());
// Works too but macro
qDebug(PCH(qformat), PCH(qvalue));
return app.exec();
}
结果
%s does not work! does not work!
embedding does not work!
embedding does not work!
答案 0 :(得分:2)
在这里,您将QString
复制到函数,并返回指向函数内已经破坏的数据的指针。这是已知的未定义行为,因为您返回指向垃圾的指针。返回QString。
const char* CH(QString msg) {
const char* res = msg.toAscii().data();
return res;
}
答案 1 :(得分:2)
您应该使用qDebug()
以及QString
内置的格式化机制。你也应该使用QStringLiteral
而不是依赖于默认的字符串转换 - 毕竟你已经表明你关心这些事情:
#include <QtCore>
int main()
{
const auto qformat = QStringLiteral("%1 does work!");
const auto qvalue = QStringLiteral("embedding");
qDebug() << qformat.arg(qvalue);
qDebug() << qformat.arg(0x7A01173C2, 0, 16);
}
在现代编译器上使用的QStringLiteral
不创建一个8位C字符串。它在编译时汇编a binary internal representation of a QString
,并将其存储在可执行文件的只读内存部分中。这是避免字符串编码的运行时成本的唯一方法。
如果您已经有UTF8编码的字符串文字或UTF8编码的C风格字符串,那么通过QString
进行往返是没有任何好处的。如果您需要使用C风格的字符串/字符串文字,请直接使用qDebug
:
#include <QtCore>
int main() {
const char format[] = "%s does not work!";
const char value[] = "embedding";
qDebug(format, value);
}
如果你需要传递临时C风格的字符串,请使用QByteArray
,但这真的是很多代码,很少显示它 - 它是如此冗长,以至于有一个非常糟糕的代码味道。所以不要这样做:
#include <QtCore>
QByteArray CH(const QString &msg) {
return msg.toLocal8Bit();
}
int main()
{
auto qformat = QStringLiteral("%s does work!");
auto qvalue = QStringLiteral("embedding");
qDebug(CH(qformat).constData(), CH(qvalue).constData());
}