Replace bool * array by std::vector<char> in MultiFilterInterface

This replaces a dynamically allocated array of bool by std::vector<char>.
1) This makes the code shorter and less error prone, because memory
   management has not to be done by hand.
2) It fixes a bug in the old code:
   memset(checkState, false, list.count()) is wrong, because bool is
   not guaranteed to be the same size as char!

Two notes:
1) QMap<>, QVector<>, etc. are used numerous times in the code, so
   this doesn't introduce a new C++ concept. Here, the std:: version
   is used, because there is no need for reference counting, COW
   semantics, etc.
2) std::vector<char> is used instead of std::vector<bool>, because
   the latter does a pessimization where a bitfield is used!

Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
This commit is contained in:
Berthold Stoeger 2017-11-25 15:04:38 +01:00 committed by Dirk Hohndel
parent 739b27427c
commit 4da6a0a732
2 changed files with 13 additions and 20 deletions

View file

@ -10,6 +10,7 @@
#endif
#include <QDebug>
#include <algorithm>
#define CREATE_INSTANCE_METHOD( CLASS ) \
CLASS *CLASS::instance() \
@ -39,8 +40,7 @@ bool CLASS::setData(const QModelIndex &index, const QVariant &value, int role) \
#define CREATE_CLEAR_FILTER_METHOD( CLASS ) \
void CLASS::clearFilter() \
{ \
memset(checkState, false, rowCount()); \
checkState[rowCount() - 1] = false; \
std::fill(checkState.begin(), checkState.end(), false); \
anyChecked = false; \
emit dataChanged(createIndex(0,0), createIndex(rowCount()-1, 0)); \
}
@ -128,10 +128,8 @@ void SuitsFilterModel::repopulate()
qSort(list);
list << tr("No suit set");
setStringList(list);
delete[] checkState;
checkState = new bool[list.count()];
memset(checkState, false, list.count());
checkState[list.count() - 1] = false;
checkState.resize(list.count());
std::fill(checkState.begin(), checkState.end(), false);
anyChecked = false;
}
@ -153,10 +151,8 @@ void TagFilterModel::repopulate()
qSort(list);
list << tr("Empty tags");
setStringList(list);
delete[] checkState;
checkState = new bool[list.count()];
memset(checkState, false, list.count());
checkState[list.count() - 1] = false;
checkState.resize(list.count());
std::fill(checkState.begin(), checkState.end(), false);
anyChecked = false;
}
@ -248,10 +244,8 @@ void BuddyFilterModel::repopulate()
qSort(list);
list << tr("No buddies");
setStringList(list);
delete[] checkState;
checkState = new bool[list.count()];
memset(checkState, false, list.count());
checkState[list.count() - 1] = false;
checkState.resize(list.count());
std::fill(checkState.begin(), checkState.end(), false);
anyChecked = false;
}
@ -301,10 +295,8 @@ void LocationFilterModel::repopulate()
qSort(list);
list << tr("No location set");
setStringList(list);
delete[] checkState;
checkState = new bool[list.count()];
memset(checkState, false, list.count());
checkState[list.count() - 1] = false;
checkState.resize(list.count());
std::fill(checkState.begin(), checkState.end(), false);
anyChecked = false;
}

View file

@ -5,13 +5,14 @@
#include <QStringListModel>
#include <QSortFilterProxyModel>
#include <stdint.h>
#include <vector>
class MultiFilterInterface {
public:
MultiFilterInterface() : checkState(NULL), anyChecked(false) {}
MultiFilterInterface() : anyChecked(false) {}
virtual bool doFilter(struct dive *d, QModelIndex &index0, QAbstractItemModel *sourceModel) const = 0;
virtual void clearFilter() = 0;
bool *checkState;
std::vector<char> checkState;
bool anyChecked;
};