std::transform over QMap

Christian Ehrlicher

@Venkateswaran said in std::transform over QMap:

std::back_inserter(map2)

How should this work? QMap has no push_back() and can't have it since it needs a key and a value.

Venkateswaran

@Christian-Ehrlicher is there any other way to achieve this without general for loop?

Christian Ehrlicher

No, but why?

for (auto it = map.begin(); it != map.end(); ++i)
  map2.insert(it.key + "testing", it.value);

Venkateswaran

@Christian-Ehrlicher Because of cppcheck in my project :) it's not allowing me to use for loop for this.

jsulm

@Venkateswaran said in std::transform over QMap:

it's not allowing me to use for loop for this

?
Can you post what it tells you?

Christian Ehrlicher

@Venkateswaran said in std::transform over QMap:

it's not allowing me to use for loop for this.

It's just suggesting to use std functions when possible. But it is not. Blame cppcheck.

qwertzui11

I guess the correct code could would look like

void fun() {
  QMap<QString, int> map;
  QMap<QString, int> map2;
  std::transform(map.begin(), map.end(), std::inserter(map2, map2.end()),
                 [](auto &todo) { return todo; });
}

however there seems to be no QMap<QString, int>::value_type defined. Qts container seem do not stick to the standard, so the std::inserter paradigma does not work.

using the standard map, it compiles.

void works() {
  std::map<QString, int> map;
  std::map<QString, int> map2;
  std::transform(map.begin(), map.end(), std::inserter(map2, map2.end()),
                 [](auto &todo) { return todo; });
}

Have fun
Markus

Christian Ehrlicher

@qwertzui11 said in std::transform over QMap:

Qts container seem do not stick to the standard,

correct because QMap was there earlier :)

A range-based for-loop for QMap similar to std::map can be done with https://doc.qt.io/qt-5/qmap-key-iterator.html afair

Venkateswaran

Ok. My actual conversion is from QMap<quint16, QPair<QString, quint16>> to QMap<quint16, QPair<QUuid, quint16>> and QHash<QString, int> to QHash<QUuid, int>

QMap<quint16, QPair<QUuid, quint16>> map2;
for(auto it = map.begin(); it != map.end(); ++it)
    map2.insert(it.key(), qMakePair(QUuid(it.value().first), it.value().second));

QHash<QUuid, int>map2;
for(auto it = map.begin(); it != map.end(); ++it)
    map2.insert(QUuid(it.key()), it.value());

and the cppCheck not complaining. Looks like cppcheck complain only if didn't do much inside the loop (I'm not sure).

I get the below warning when I convert QList<enum> to QList<QString>.

Controller.cpp:120:21: warning: Consider using std::transform algorithm instead of a raw loop. [useStlAlgorithm]
      versionsInStr.push_back(convertApiVersionToString(ver));

Thank you @Christian-Ehrlicher

qwertzui11

@Christian-Ehrlicher agreed. I hope Qt6 fixes these smalls gaps :-)

Christian Ehrlicher

@qwertzui11 It can't since it will break a lot of existing stuff - you won't explain all customers that the range-based for loop for containers now returns an iterator instead the value on a QMap and that they have to adjust all of it's code. :(

SGaist

Hi,

You can use QMap:: keyValueBegin and QMap::keyValueEnd for your use case.