kdeconnect-kde/core/networkpacket.cpp
Alexander Lohnau 0b4d0757dc NetworkPacket: Optimize serialization/deserialization
We don't need to convert the data to an intermediate QVariantMap just to
make JSON out of it. This siginificantly improves performance.
The measurements were done with 3000000 iterations each:

Before serialization: 0.0045009 msecs
Before deserialization: 0.002293 msecs

After serialization: 0.0034216 msecs (32% faster)
After deserialization: 0.001776 msecs (29% faster)
2024-09-17 08:45:39 +00:00

116 lines
3.4 KiB
C++

/**
* SPDX-FileCopyrightText: 2013 Albert Vaca <albertvaka@gmail.com>
*
* SPDX-License-Identifier: GPL-2.0-only OR GPL-3.0-only OR LicenseRef-KDE-Accepted-GPL
*/
#include "networkpacket.h"
#include "core_debug.h"
#include <QByteArray>
#include <QDataStream>
#include <QDateTime>
#include <QDebug>
#include <QJsonDocument>
#include <QMetaObject>
#include <QMetaProperty>
#include "dbushelper.h"
#include "filetransferjob.h"
#include "kdeconnectconfig.h"
#include "pluginloader.h"
QDebug operator<<(QDebug s, const NetworkPacket &pkg)
{
s.nospace() << "NetworkPacket(" << pkg.type() << ':' << pkg.body();
if (pkg.hasPayload()) {
s.nospace() << ":withpayload";
}
s.nospace() << ')';
return s.space();
}
const int NetworkPacket::s_protocolVersion = 7;
NetworkPacket::NetworkPacket(const QString &type, const QVariantMap &body)
: m_id(QString::number(QDateTime::currentMSecsSinceEpoch()))
, m_type(type)
, m_body(body)
, m_payload()
, m_payloadSize(0)
{
}
QByteArray NetworkPacket::serialize() const
{
QJsonObject obj;
obj.insert(QLatin1String("id"), m_id);
obj.insert(QLatin1String("type"), m_type);
obj.insert(QLatin1String("body"), QJsonObject::fromVariantMap(m_body));
if (hasPayload()) {
obj.insert(QLatin1String("payloadSize"), m_payloadSize);
obj.insert(QLatin1String("payloadTransferInfo"), QJsonObject::fromVariantMap(m_payloadTransferInfo));
}
QByteArray json = QJsonDocument(obj).toJson(QJsonDocument::Compact);
if (json.isEmpty()) {
qCDebug(KDECONNECT_CORE) << "Serialization error:";
} else {
json.append('\n');
}
return json;
}
template<class T>
void qjsonobject2qobject(const QJsonObject &obj, T *object)
{
for (auto iter = obj.begin(); iter != obj.end(); ++iter) {
const int propertyIndex = T::staticMetaObject.indexOfProperty(iter.key().toLatin1().data());
if (propertyIndex < 0) {
qCWarning(KDECONNECT_CORE) << "missing property" << object << iter.key();
continue;
}
QMetaProperty property = T::staticMetaObject.property(propertyIndex);
bool ret = property.writeOnGadget(object, iter.value().toVariant());
if (!ret) {
qCWarning(KDECONNECT_CORE) << "couldn't set" << object << "->" << property.name() << '=' << *iter;
}
}
}
bool NetworkPacket::unserialize(const QByteArray &a, NetworkPacket *np)
{
// Json -> QVariant
QJsonParseError parseError;
auto parser = QJsonDocument::fromJson(a, &parseError);
if (parser.isNull()) {
qCDebug(KDECONNECT_CORE) << "Unserialization error:" << parseError.errorString();
return false;
}
const QJsonObject obj = parser.object();
qjsonobject2qobject(obj, np);
// Will return an empty qvariantmap if was not present, which is ok
np->m_payloadTransferInfo = obj.value(QLatin1String("payloadTransferInfo")).toVariant().toMap();
// Ids containing characters that are not allowed as dbus paths would make app crash
if (np->m_body.contains(QStringLiteral("deviceId"))) {
QString deviceId = np->get<QString>(QStringLiteral("deviceId"));
DBusHelper::filterNonExportableCharacters(deviceId);
np->set(QStringLiteral("deviceId"), deviceId);
}
return true;
}
FileTransferJob *NetworkPacket::createPayloadTransferJob(const QUrl &destination) const
{
return new FileTransferJob(this, destination);
}
#include "moc_networkpacket.cpp"