Qt中隐式共享带来的坑

先看代码：
这里向list列表中放入了10条数据，每条数据存放不同的内容。为了防止循环多次构建QByteArray，这里将QByteArray的初始化放在外面。这段代码咋一看好像没什么问题，我们打印一下看看结果。

#include <QDebug>

int main()
{
    constexpr int SIZE = 10;
    QByteArray byte;
    byte.resize(SIZE);
    quint8 *data = reinterpret_cast<quint8*>(byte.data());
    QList<QByteArray> list;
    for (int i = 0;i < 10;++i) {
        memset(data,'0'+i,SIZE);
        list << byte;
    }
    for (const auto &x : qAsConst(list)) {
        qDebug().noquote().nospace() << x << " addr:0x" << QString::number(reinterpret_cast<qintptr>(x.constData()),16);
    }

    return 0;
}

输出结果：

9999999999 addr:0x18723ad62e0
9999999999 addr:0x18723ad62e0
9999999999 addr:0x18723ad62e0
9999999999 addr:0x18723ad62e0
9999999999 addr:0x18723ad62e0
9999999999 addr:0x18723ad62e0
9999999999 addr:0x18723ad62e0
9999999999 addr:0x18723ad62e0
9999999999 addr:0x18723ad62e0
9999999999 addr:0x18723ad62e0

好家伙全是一摸一样的数据，连地址都是一样的。
我们稍微改下代码试试：

#include <QDebug>

int main()
{
    constexpr int SIZE = 10;
    QByteArray byte;
    byte.resize(SIZE);
    QList<QByteArray> list;
    for (int i = 0;i < 10;++i) {
        quint8 *data = reinterpret_cast<quint8*>(byte.data());
        memset(data,'0'+i,SIZE);
        list << byte;
    }
    for (const auto &x : qAsConst(list)) {
        qDebug().noquote().nospace() << x << " addr:0x" << QString::number(reinterpret_cast<qintptr>(x.constData()),16);
    }

    return 0;
}

输出结果：

0000000000 addr:0x1d690db62e0
1111111111 addr:0x1d690db6360
2222222222 addr:0x1d690db6390
3333333333 addr:0x1d690db6310
4444444444 addr:0x1d690db6440
5555555555 addr:0x1d690db63c0
6666666666 addr:0x1d690db63f0
7777777777 addr:0x1d690db6580
8888888888 addr:0x1d690db65b0
9999999999 addr:0x1d690db65e0

这里把quint8 *data = reinterpret_cast<quint8*>(byte.data());由循环外面挪到了循环里面结果就正确了。
因为Qt的隐士共享具备写时复制的特性，所以在进行写数据的操作时采用深拷贝，在只读操作时采用浅拷贝。
因为习惯这里把QByteArray的内部数据指针取出来直接操作指针，进行数据的读写操作。在第一段代码中，我们在循环外面从data接口获取内部指针地址，此时QByteArray检测到当前还没有任何引用所以未进行任何操作直接返回内部指针。
在后续的赋值时我们都是操作的同一个指针地址，虽然在list中有10个QByteArray数据。但他们内部都指向同一个内存地址，所以是相同的数据。

第二段代码中我们将指针的获取移动到了循环里面。第一次QByteArray返回原始指针地址，在写入数据后存入list列表。第二次QByteArray检测到当前的数据地址包含有多个引用，所以新创建了一个内存地址，将原来的数据拷贝过去后再返回新的地址。循环每次到来都返回的是新的内存地址所以不冲突。

所以这里建议在操作QByteArray的原始指针时最好直接用byte.data()的形式，不要再用一个指针变量来操作。

建议写法：

#include <QDebug>

int main()
{
    constexpr int SIZE = 10;
    QList<QByteArray> list;
    for (int i = 0;i < 10;++i) {
        QByteArray byte;
        byte.resize(SIZE);
        memset(byte.data(),'0'+i,SIZE);
        list << byte;
    }
    for (const auto &x : qAsConst(list)) {
        qDebug().noquote().nospace() << x << " addr:0x" << QString::number(reinterpret_cast<qintptr>(x.constData()),16);
    }

    return 0;
}

虽然每次QByteArray的构造都会花上一些时间，但却可以节省一次深拷贝的时间。在QByteArray包含大量数据时把QByteArray的构造写在循环里面要比写在循环外面速度要快一些。当然小数据量下差别就不大了。

经过测试，在QByteArray包含200MB数据的情况下把QByteArray的构造写在循环里面要比写在循环外面要快个大约500ms的样子。