Это использование QMap потенциально вредно?

Question

#include <QMultiMap>

template <typename TKey, typename TValue>
TKey lastKeyOf(const QMap<TKey, TValue>& map)
{
    if (map.isEmpty())
        throw something;
    return (map.end() - 1).key();
}

Причина, по которой я спрашиваю:

template <typename TKey, typename TValue>
QMultiMap<TKey, TValue>;

наследуется QMap<TKey, TValue> публично. Так что, если я позвоню:

QMultiMap<int, std::string> multimap;
...
lastKeyOf(multimap);

Все звонки внутри lastKeyOf статически связаны с их QMap версии вместо QMultiMap версии, так как QMap не был предназначен для полиморфного использования (без виртуального деструктора).

Я даже не уверен, как называется это использование. Это нарезка объектов?

Answer 1

Я считаю, что это должно быть безопасно. QMultiMap просто добавляет новый набор перегруженных методов в QMap, вы получите только неожиданные результаты, если будете использовать функцию внутри lastKeyOf(), которая перегружена в QMultiMap, чтобы иметь другое поведение. Единственный перегруженный метод с совместимой сигнатурой и другим поведением, которое я вижу, это insert().

Сказав это, можно сделать более простой код, чтобы избежать QMultiMap и использовать QMap только с QMap::insertMulti() вместо QMap::insert().