按项目数据从 QStandardItemModel 中查找/删除一行

Question

我有一个QStandardItemModel，只有一列（代表一个列表）。列表中的每个项目都有一个唯一的整数 ID，该 ID 存储为 QStandardItem 的数据（通过 QStandardItem::setData 我猜默认是 Qt::UserRole+1）。

给定其中一个 ID，我想从模型中查找并删除相应的行。现在我正在这样做：

void NetworkManager::removeSessionFromModel (QStandardItemModel *model, int sessionId) {

    foreach (const QStandardItem *item, model->findItems("*", Qt::MatchWildcard)) {
        if (item->data() == sessionId) {
            model->removeRow(item->index().row());
            break;
        }
    }

}

它运行良好，但该函数的每一行都让我畏缩。有没有更清洁的方法来做到这一点？

Answer 1

直接遍历QStandardItemModel怎么样？像这样的：

void NetworkManager::removeSessionFromModel (QStandardItemModel *model, int sessionId) 
{
    for (int i = 0; i < model->rowCount(); ++i)
    {
        if (model->item(i)->data() == sessionId)
        {
            model->removeRow(i);
            break;
        }
    } 
}

不确定QStandardItemModel 在随机访问中的表现如何，也许你的方法更有效。

编辑：

其实有一个函数可以做你想做的：QAbstractItemModel::match

它返回一个QModelIndexList，其中包含在给定角色中具有匹配数据的所有条目。

void NetworkManager::removeSessionFromModel (QStandardItemModel *model, int sessionId)
{
    QModelIndexList list = model->match(model->index(0, 0), Qt::UserRole + 1, sessionId);

    if (!list.empty())
        model->removeRow(list .first().row());
}

将数据设置为特定角色可以如下完成：

model->setData(model->index(row, col), QVariant(13), Qt::UserRole + 1);

Answer 2

您需要从您的项目 ID 中获取行索引。

更有效的方法是使用 QMap，其中行索引作为值，项目 ID 作为键。

在这种情况下，您还需要在每次添加/删除行时维护映射值。

如果您的列表中没有 300 万个项目，请保持简单并使用您的代码。 通过优化此代码，您可能还会增加复杂性并降低可维护性，您得到的是 0.05 毫秒而不是 0.06 毫秒。

在 GUI 代码中，我经常有这样的代码：它很简单，每个人都立即得到它并且它完成了工作。它也足够快。

Answer 3

您使用findItems 错误，它已经可以通过传递您正在搜索的值返回您想要的项目。如果您像现在一样调用它，那么您至少要遍历您的项目两次，因为findItems 必须遍历所有项目以找到与您的模式匹配的项目，在您的情况下，所有项目都匹配，那么您再次迭代返回的项目以找到sessionId。

void NetworkManager::removeSessionFromModel (QStandardItemModel *model, int sessionId) {

    auto items = model->findItems(QString::number(sessionId));
    if (!items.empty()) {
        auto row = items.first()->index().row();
        model->removeRow(row);
    }
}

您也可以使用match 方法，因为findItems 在内部使用该方法，因此您避免分配StandardItem 只是为了获取其索引。同样match 在匹配模式的项目数之后立即返回，在这种情况下，sessionId 的值被发现，因此它并不总是迭代所有项目；这样更有效率。显然，如果在迭代所有项目后未找到该值，它将返回一个空列表。

auto start = model->index(0, 0);
auto indexes = model->match(start, Qt::UserRole + 1, QString::number(sessionId), 1, Qt::MatchExactly);
if (!indexes.empty()) {
    auto row = indexes.first().row();
    model->removeRow(row);
}