diff --git a/src/blackcore/db/modeldatareader.cpp b/src/blackcore/db/modeldatareader.cpp index 463bd46ce..7fd194278 100644 --- a/src/blackcore/db/modeldatareader.cpp +++ b/src/blackcore/db/modeldatareader.cpp @@ -128,9 +128,9 @@ namespace BlackCore return this->getModels().size(); } - QList CModelDataReader::getModelDbKeys() const + QSet CModelDataReader::getModelDbKeys() const { - return this->getModels().toDbKeyList(); + return this->getModels().toDbKeySet(); } QStringList CModelDataReader::getModelStringList() const diff --git a/src/blackcore/db/modeldatareader.h b/src/blackcore/db/modeldatareader.h index 180b985fe..be45e1e74 100644 --- a/src/blackcore/db/modeldatareader.h +++ b/src/blackcore/db/modeldatareader.h @@ -105,7 +105,7 @@ namespace BlackCore //! Get model keys //! \threadsafe - QList getModelDbKeys() const; + QSet getModelDbKeys() const; //! Get model keys //! \threadsafe diff --git a/src/blackcore/webdataservices.cpp b/src/blackcore/webdataservices.cpp index b141838bb..a4ffab050 100644 --- a/src/blackcore/webdataservices.cpp +++ b/src/blackcore/webdataservices.cpp @@ -407,10 +407,10 @@ namespace BlackCore return 0; } - QList CWebDataServices::getModelDbKeys() const + QSet CWebDataServices::getModelDbKeys() const { if (m_modelDataReader) { return m_modelDataReader->getModelDbKeys(); } - return QList(); + return QSet(); } QStringList CWebDataServices::getModelStrings() const diff --git a/src/blackcore/webdataservices.h b/src/blackcore/webdataservices.h index d385bc911..98daf5830 100644 --- a/src/blackcore/webdataservices.h +++ b/src/blackcore/webdataservices.h @@ -41,6 +41,7 @@ #include #include +#include #include #include #include @@ -205,7 +206,7 @@ namespace BlackCore //! Model keys //! \threadsafe - QList getModelDbKeys() const; + QSet getModelDbKeys() const; //! Model strings //! \threadsafe diff --git a/src/blackgui/components/dbstashcomponent.cpp b/src/blackgui/components/dbstashcomponent.cpp index ff80c69f0..3b3a93552 100644 --- a/src/blackgui/components/dbstashcomponent.cpp +++ b/src/blackgui/components/dbstashcomponent.cpp @@ -131,7 +131,7 @@ namespace BlackGui ui->tvp_StashAircraftModels->updateContainerMaybeAsync(models); } - int CDbStashComponent::unstashModels(QList keys) + int CDbStashComponent::unstashModels(QSet keys) { if (keys.isEmpty()) { return 0; } return ui->tvp_StashAircraftModels->removeDbKeys(keys); diff --git a/src/blackgui/components/dbstashcomponent.h b/src/blackgui/components/dbstashcomponent.h index 617d13b9f..acbb3d451 100644 --- a/src/blackgui/components/dbstashcomponent.h +++ b/src/blackgui/components/dbstashcomponent.h @@ -74,7 +74,7 @@ namespace BlackGui BlackMisc::CStatusMessage validateStashModel(const BlackMisc::Simulation::CAircraftModel &model, bool allowReplace) const; //! Unstash given models with keys - int unstashModels(QList keys); + int unstashModels(QSet keys); //! Unstash given models by model string int unstashModels(QStringList modelStrings); diff --git a/src/blackmisc/db/datastoreobjectlist.cpp b/src/blackmisc/db/datastoreobjectlist.cpp index a80ef7a35..30a23250d 100644 --- a/src/blackmisc/db/datastoreobjectlist.cpp +++ b/src/blackmisc/db/datastoreobjectlist.cpp @@ -67,13 +67,13 @@ namespace BlackMisc } template - QList IDatastoreObjectList::toDbKeyList() const + QSet IDatastoreObjectList::toDbKeySet() const { - QList keys; + QSet keys; for (const OBJ &obj : ITimestampObjectList::container()) { if (!obj.hasValidDbKey()) { continue; } - keys.append(obj.getDbKey()); + keys.insert(obj.getDbKey()); } return keys; } @@ -81,7 +81,7 @@ namespace BlackMisc template KEYTYPE IDatastoreObjectList::getMaxKey(bool *ok) const { - QList keys(this->toDbKeyList()); + QSet keys(this->toDbKeySet()); if (keys.isEmpty()) { if (ok) { *ok = false; } @@ -93,7 +93,7 @@ namespace BlackMisc } template - int IDatastoreObjectList::removeObjectsWithKeys(const QList &keys) + int IDatastoreObjectList::removeObjectsWithKeys(const QSet &keys) { if (keys.isEmpty()) { return 0; } if (this->container().isEmpty()) { return 0; } @@ -133,7 +133,7 @@ namespace BlackMisc return this->container().size(); } CONTAINER newValues(this->container()); - const QList keys(container.toDbKeyList()); + const QSet keys(container.toDbKeySet()); newValues.removeObjectsWithKeys(keys); int removeSize = newValues.size(); // size after removing data newValues.push_back(container); diff --git a/src/blackmisc/db/datastoreobjectlist.h b/src/blackmisc/db/datastoreobjectlist.h index 21d617237..74c0c569c 100644 --- a/src/blackmisc/db/datastoreobjectlist.h +++ b/src/blackmisc/db/datastoreobjectlist.h @@ -15,7 +15,7 @@ #include "blackmisc/timestampobjectlist.h" #include -#include +#include #include namespace BlackMisc @@ -37,13 +37,13 @@ namespace BlackMisc void sortByKey(); //! All keys as list - QList toDbKeyList() const; + QSet toDbKeySet() const; //! Max.key value (making sense with integer key) KEYTYPE getMaxKey(bool *ok = nullptr) const; //! Remove objects with key - int removeObjectsWithKeys(const QList &keys); + int removeObjectsWithKeys(const QSet &keys); //! Remove objects without key int removeObjectsWithoutDbKey();