2

我尽量避免在 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!
4

2 回答 2

2

在这里，您正在复制QString到函数，并返回指向函数内已销毁数据的指针。这是已知的未定义行为，因为您正在返回指向垃圾的指针。返回 QString。

const char* CH(QString msg) {
    const char* res = msg.toAscii().data();
    return res;
}
于 2016-12-05T20:53:06.247 回答
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 的二进制内部表示QString，并将其存储在可执行文件的只读内存部分中。这是避免字符串编码的运行时成本的唯一方法。

QString如果您已经拥有 UTF8 编码的字符串文字或 UTF8 编码的 C 样式字符串，那么进行往返没有任何好处。如果您需要使用 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());
}
于 2016-12-05T21:09:42.187 回答