При обращении к QMap и записи в файл, меняется последовательность выводимой информации

QMap<QString, QString> *map = new QMap<Qstring, QString>();
map->insert("key1", "value1");
map->insert("key2", "value2");
map->insert("key3", "value3");

QMap<Qstring, QString>::iterator it = map->begin();
for(;it != map->end(); ++it)
{
    if(it.key() == "key1")
    {
        добавили значение в файл;
    }
    if(it.key() == "key2")
    {
        добавили значение в файл;
    }
    if(it.key() == "key3")
    {
        добавили значение в файл;
    }
}

Рекомендуємо хостинг TIMEWEB

Стабільний хостинг, на якому розміщується соціальна мережа EVILEG. Для проектів на Django радимо VDS хостинг.

добрий день, так а що вам заважає зробити по документації? ніколи таких пробелм не виникало

  QMap<QString, int> map;
  QMapIterator<QString, int> i(map);
  while (i.hasNext()) {
      i.next();
      cout << i.key() << ": " << i.value() << endl;
  }

Here's the same code, but using an STL-style iterator this time:

  QMap<QString, int>::const_iterator i = map.constBegin();
  while (i != map.constEnd()) {
      cout << i.key() << ": " << i.value() << endl;
      ++i;
  }

і можна без new

QMap<QString, QString> map;

Используешь не тот контейнер. Если хочешь получить последовательный вывод как и добавлял - используй QList. а вообще смотря для каких целей ты все это юзайешь.

Мне нужно чтобы было однозначное соответсвие ключ-значение. Разве QList это позволяет? Если да, то как это реализовать?

вы можете испльзовать QList, просто помещайте туда QPair, будет примерно тоже самое. Просто QMap автоматически сортируется по ключу.

QList<QPair<QString, QString>> list_with_pair;
list_with_pair.push_back(QPair("key1", "value1"));

Добрый день.
Я бы хотел добавить, что в вашем коде можно было бы использовать слово auto и цикл for с перебором. Так код получился бы компактнее:

auto map = new QMap<Qstring, QString>();
map->insert("key1", "value1");
map->insert("key2", "value2");
map->insert("key3", "value3");

for (const auto& it : map)
{
    if(it.key() == "key1")
    {
        добавили значение в файл;
    }
    if(it.key() == "key2")
    {
        добавили значение в файл;
    }
    if(it.key() == "key3")
    {
        добавили значение в файл;
    }
}

Но не очень ясен вообще смысл подобного перебора, т.к. вы и так можете проверять наличие ключа:

if (map->contains("key1")) {
    Do something;
  }

и т.д.

И, как уже сказали ранее, QMap не гарантирует сохранения порядка элементов. Воспользуйтесь другим контейнером, например, как предложил Евгений. Правда, в таком случае вы потеряете возможность обращения по ключу, а также будет возможно возникновение ошибки из-за дублирования ключей.

Александр, добрый день.

Проверка нужна мне для случая, когда в словаре может не оказаться среди ключей того, что мне нужно. А так да, я с Вами полностью согласен. То что предложил Евгений, мне очень понравилось, но это усложняет код. Мне со словарем нравится больше.

Спасибо Вам за помощь.

Вообще, это можно переписать так

auto map = new QMap<Qstring, QString>();
map->insert("key1", "value1");
map->insert("key2", "value2");
map->insert("key3", "value3");

for (const auto& [key, value] : map)
{
    if(key == "key1")
    {
        // добавили значение в файл;
    }
    if(key == "key2")
    {
        // добавили значение в файл;
    }
    if(key == "key3")
    {
        // добавили значение в файл;
    }
}

Не помню только, какой это стандарт, C++14 или C++17

Евгений, данная конструкция не работает

for (const auto& [key, value] : map)
{
    if(key == "key1")
    {
        // добавили значение в файл;
    }
    if(key == "key2")
    {
        // добавили значение в файл;
    }
    if(key == "key3")
    {
        // добавили значение в файл;
    }
}

Я переписал цикл следующим образом, потому что проверка в этом случае не нужна. Мне просто нужно было просто создать атрибуты xml тега. И поэтому в словаре будет лежать только то, что я туда положу.
Но все равно порядок атрибутов внутри тега меняется каждый раз, хотя последовательность одна и та же и в словарь кладется каждый раз в одинаковом порядке.
А сам цикл я переписал вот так:

auto *map = object.getMap();
QMap<QString, QString>::iterator it = map->begin();
for(;it != map->end(); ++it)
{
    domElement.setAttribute(it.key(), it.value());
}

Но вот почему каждый раз последовательность берется по-разному не понимаю. Единственное предположение - так работает итератор. Можно конечно пойти по пути QList > map... Попробую.

Евгений, данная конструкция не работает

Данная конструкция работает, но работает при использовании стандарта C++17. Чтобы его включить, необходимо прописать в pro файле следующее

CONFIG += c++17
QMAKE_CXXFLAGS += /std:c++17

Но вот почему каждый раз последовательность берется по-разному не понимаю. Единственное предположение - так работает итератор.

Дело не в работе итератора, а в том, что QMap автоматически сортируется. Я об этом уже говорил выше.
Если вы постоянно что-то добавляете или удаляете, то естественно, что последовательность элементов может изменяться.

Если вы постоянно что-то добавляете или удаляете, то естественно, что последовательность элементов может изменяться.

Даже если добавляется одно и тоже количество пар "ключ-значение"? Ну если так, то тогда чуть позже перепишу на конструкцию QList.

    CONFIG += c++17
    QMAKE_CXXFLAGS += /std:c++17

Если я это напишу, то мне придется весь код писать на новом стандарте или использовать можно будет и старые конструкции?

Евгений,

Если вы постоянно что-то добавляете или удаляете, то естественно, что последовательность элементов может изменяться.

Даже если добавляется одно и тоже количество пар "ключ-значение"? Ну если так, то тогда чуть позже перепишу на конструкцию QList.

    CONFIG += c++17
    QMAKE_CXXFLAGS += /std:c++17

Если я это напишу, то мне придется весь код писать на новом стандарте или использовать можно будет и старые конструкции?

Если я это напишу, то мне придется весь код писать на новом стандарте или использовать можно будет и старые конструкции?

Вам ничего не придётся переписывать, просто получите возможность использовать новый стандарт. Единственные проблемы могут возникать, если вы пишите библиотеку, которую используют другие проекты, которые работают только со старыми стандартами, но это видимо не ваш случай на данный момент.

Спасибо за помощь, Евгений.

и всё равно не понимаю, зачем перебирать всю мапу? можно просто проверять наличие ключа и при его наличии (или отсутствии) делать какие-то действия.
у вас получается избыточный код.

Александр, мне не нужно перебирать. Вы говорите правильно, сначала я написал избыточный код просто не подумав. Задача такая, мне нужно просто переложить из QMap в атрибуты xml тега все, что там лежит. И еще раз повторюсь, изначально я решил задачу в лоб не более того. Вот при перекладывании последовательность атрибутов всегда меняется при неизменности словаря. Поэтому я и спросил почему так, может быть я что-то делаю неправильно.