diff options
author | Arno <arno@disconnect.de> | 2017-12-27 06:09:39 +0100 |
---|---|---|
committer | Arno <arno@disconnect.de> | 2017-12-27 06:09:39 +0100 |
commit | e2ac3a4b18573e8999138000e8af9e7b16192ea0 (patch) | |
tree | 26feb3e602fb14832cdb88ee44689b2ce33cb367 | |
parent | 931a1f45787719983cfac07c59a424655d0bedde (diff) | |
download | SheMov-e2ac3a4b18573e8999138000e8af9e7b16192ea0.tar.gz SheMov-e2ac3a4b18573e8999138000e8af9e7b16192ea0.tar.bz2 SheMov-e2ac3a4b18573e8999138000e8af9e7b16192ea0.zip |
Modernize mappingtablemodel.cpp
* use std::sort instead of deprecated qSort
* use c++ style for-loop instead of Q_FOREACH
-rw-r--r-- | mappingtablemodel.cpp | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/mappingtablemodel.cpp b/mappingtablemodel.cpp index 8d3752f..5ef0e0a 100644 --- a/mappingtablemodel.cpp +++ b/mappingtablemodel.cpp @@ -109,7 +109,7 @@ void MappingTableModel::setMappings(const QStringList &itemNames, int seriesPart if(seriesPartId == -1){ return; } - foreach(const QString name, itemNames){ + for(const QString &name : itemNames){ QModelIndex itemIdx = find(name); if(!itemIdx.isValid()){ addItem(name); @@ -125,13 +125,13 @@ QStringList MappingTableModel::findPartialName(const QString &value) const{ if(value.isEmpty()){ retval = mItemNames; }else{ - foreach(const QString cur, mItemNames){ + for(const QString &cur : mItemNames){ if(cur.contains(value)){ retval << cur; } } } - qSort(retval); + std::sort(retval.begin(), retval.end()); return retval; } @@ -217,7 +217,7 @@ QHash<QString, int> MappingTableModel::zeroUsers(){ QHash<QString, int> retval; if(!items.isEmpty()){ QList<int> ids = items.keys(); - foreach(int id, ids){ + for(int id : ids){ mItemCountQuery->bindValue(":id", id); if(mItemCountQuery->exec()){ mItemCountQuery->first(); @@ -236,7 +236,7 @@ void MappingTableModel::deleteItems(const QList<int> &ids){ if(ids.isEmpty()){ return; } - foreach(int id, ids){ + for(int id : ids){ mDeleteItemQuery->bindValue(":id", id); mDeleteItemQuery->exec(); } |