多线程程序中QJSEngine随机崩溃及线程安全改造咨询

核心问题分析 #

QJSEngine及所有由它创建的QJSValue对象都具有严格的线程亲和性——所有操作（包括显式的方法调用、隐式的内存管理、值的创建/销毁）必须在引擎的创建线程执行。你遇到的崩溃，大概率是因为除了QJSValue::call()之外，还有其他涉及引擎或JS值的操作（比如QJSValue的构造、析构、属性读取/设置，甚至引擎内部的垃圾回收触发）在非创建线程执行了，哪怕没有并发访问，跨线程的内存操作也会触发QV4底层存储的段错误（对应QTBUG-83410的场景）。

你的封装可能存在的疏漏 #

你用QMetaObject::invokeMethod()处理了call()，但可能忽略了这些场景：

• 在非创建线程中创建/销毁QJSValue对象（比如从引擎获取全局对象、构造JS值）
• 跨线程传递QJSValue对象（哪怕只是持有，其内部的引用计数操作也会在当前线程执行）
• 引擎的垃圾回收触发时，当前线程不是创建线程（比如其他线程调用了引擎的collectGarbage()）

正确的线程安全封装方案 #

要实现真正的线程安全，必须确保所有与QJSEngine及QJSValue相关的操作都严格限定在引擎创建线程，以下是具体实现思路：

1. 让ThreadSafeQJSEngine自身运行在独立线程 #

将引擎的生命周期完全绑定到一个专用线程，所有操作通过信号槽或invokeMethod()异步/同步调用：

class ThreadSafeQJSEngine : public QObject
{
    Q_OBJECT
public:
    ThreadSafeQJSEngine(QObject *parent = nullptr) : QObject(parent) {
        // 创建专用线程并将引擎移到该线程
        engineThread = new QThread(this);
        engine = new QJSEngine();
        engine->moveToThread(engineThread);
        engineThread->start();
    }

    ~ThreadSafeQJSEngine() {
        // 必须在引擎线程销毁引擎
        QMetaObject::invokeMethod(engine, [this](){
            delete engine;
        }, Qt::BlockingQueuedConnection);
        engineThread->quit();
        engineThread->wait();
    }

    // 封装同步调用JS函数的方法
    QVariant callFunction(const QString &funcName, const QVariantList &args) {
        QVariant result;
        QMetaObject::invokeMethod(this, "_callFunction",
                                  Qt::BlockingQueuedConnection,
                                  Q_RETURN_ARG(QVariant, result),
                                  Q_ARG(QString, funcName),
                                  Q_ARG(QVariantList, args));
        return result;
    }

private slots:
    QVariant _callFunction(const QString &funcName, const QVariantList &args) {
        // 此槽函数运行在引擎线程，所有操作都是安全的
        QJSValue global = engine->globalObject();
        QJSValue func = global.property(funcName);
        if (!func.isCallable()) {
            return QVariant();
        }
        QJSValueList jsArgs;
        for (const auto &arg : args) {
            jsArgs.append(engine->toScriptValue(arg));
        }
        QJSValue jsResult = func.call(jsArgs);
        if (jsResult.isError()) {
            // 处理错误
            return QVariant();
        }
        return jsResult.toVariant();
    }

private:
    QThread *engineThread;
    QJSEngine *engine;
};

2. 禁止跨线程传递QJSValue #

绝对不要将QJSValue对象从引擎线程传递到其他线程——哪怕是只读访问，其内部的引用计数操作会在当前线程执行，破坏QV4的存储结构。所有需要从JS获取的数据，必须先转换为QVariant或其他线程安全的类型后再传递。

3. 确保垃圾回收在引擎线程执行 #

不要在非引擎线程调用QJSEngine::collectGarbage()，如果需要触发垃圾回收，同样通过invokeMethod()在引擎线程执行：

void triggerGC() {
    QMetaObject::invokeMethod(engine, "collectGarbage", Qt::QueuedConnection);
}

4. 避免线程销毁时的竞态 #

销毁引擎和线程时，必须用BlockingQueuedConnection确保引擎的销毁操作在其线程完成后，再退出线程，避免残留的内存操作导致崩溃。

验证崩溃是否解决 #

检查所有代码中是否存在以下违规操作：

• 在非引擎线程中直接创建QJSValue（比如engine->newObject()）
• 跨线程存储或传递QJSValue对象
• 在非引擎线程中调用QJSValue的任何方法（包括析构，比如局部变量在其他线程销毁）

只要严格遵守“所有引擎相关操作都在创建线程执行”的规则，就能解决QTBUG-83410相关的崩溃问题。

内容的提问来源于stack exchange，提问作者Nikita