0

Предположим, я хочу просто вывести содержимое словаря map используя константный метод класса, в котором этот словарь находится. Очевидно. что компилятор будет ругаться на то, что я хочу изменить элемент словаря, даже если этот элемент будет в словаре. Как оптимально обратиться к элементу, не создавая копию этого словаря и используя константный метод? Ведь если словарь большой, а надо вывести конкретные элементы, то копирование будет отнимать много времени.

5
  • 3
    Почему при выводе содержимого компилятор должен ругаться на то, что вы хотите что-то изменить? 8-/ Вы бы не могли привести кусок кода, где это происходит? map<int,string> m; for(auto& i: m) cout << i.first << " " << i.second << endl; - вот вариант вывода - где тут неприятности?
    – Harry
    6 сен 2018 в 6:32
  • "Очевидно. что компилятор будет ругаться" Ну так добавьте в вопрос код и текст ошибки. Мне вот не очевидно, как можно сделать итерацию по элементам map, чтобы словить такую ошибку. 6 сен 2018 в 6:33
  • class Test{ public: void Prnt()const{ cout << out[1]; } Test(){ out[1] = "out"; } private: map <int, string> out; }; Очевидно, потому что при обращении к несуществующему элементу в словаре map, добавляется элемент этого словаря. Компилятор не знает, есть или нет такой элемент заранее, поэтому вызывает ошибку, что я хочу изменить словарь в константном методе. Поэтому это очевидно. Ваш пример пробежится по существующим элементам 100%, поэтому и ошибку не выдаст.
    – Bahoo08
    6 сен 2018 в 6:36
  • На будущее, когда пишешь ответ на комментарий, желательно начинать его с @username , иначе человеку не приходит уведомление и он может не ответить. (Вам уведомления приходят всегда, потому что комменты под вашим постом.) 7 сен 2018 в 6:35
  • @HolyBlackCat спасибо за совет. Не знал этого, учту в будущем
    – Bahoo08
    7 сен 2018 в 13:57

2 ответа 2

3
const std::map<int, int> m = {
    { 1 , 1},
    {2 , 4},
    {3, 9},
    {4, 16}
};

for(std::map<int, int>::const_iterator it = m.begin(); it != m.end(); ++it)
{
    std::cout << "Key:" << it->first << " | Value:" << it->second << std::endl;
}

Для константного доступа по ключу, можно использовать метод at, но тут нужно быть аккуратный, потому что если ключа нет в коллекции - можно схватить исключение.

 std::cout << m.at(1); //1
 std::cout << m.at(2); //4
 std::cout << m.at(3); //9
5
  • Так я хотел использовать константный метод. Т.е есть класс, в нём есть словарь, есть константный метод, который обращается к конкретному элементу словаря.
    – Bahoo08
    6 сен 2018 в 6:43
  • @Bahoo08, нужно определится, какую логику Вы хотите получить, на случай, если ключа нет в std::map. Если точно знаете, чтоб ключ там будет, то используйте at вместо квадратных скобок []. Так же можно указать Ваш map как mutable, и тогда его можно будет менять в константном методе, но мне кажется это не лучший выход. Если ключа может и не быть, то лучше сначала поискать его с помощью find, потом самому решать, что делать в случае его отсутствия - бросать исключение или возвращать значение по умолчанию. 6 сен 2018 в 6:47
  • Спасибо, изучу этот вопрос.
    – Bahoo08
    6 сен 2018 в 6:49
  • 1
    Все-таки 2018 год на дворе. Если не ranged for, то хоть auto it = m.begin() написали бы. :) 6 сен 2018 в 6:53
  • @HolyBlackCat, решил записать именно так для наглядность. Когда то давно, я сам боролся невозможностью пройтись итератором по константному контейнеру, и проблема моя была как раз в том, что использовал std::map<int, int>::iterator it. 6 сен 2018 в 7:01
3

Извините, в комментарий не вписывается.

В чем все же проблема? Вот код, который в константном методе выводит содержимое члена map - что здесь не так?

#include <iostream>
#include <string>
#include <map>

using namespace std;

struct Test
{

    void out() const
    {
        for(auto& i: m)
            cout << i.first << " " << i.second << endl;
    }
    map<int,string> m = {{1,"one"},{2,"two"}};

};

int main(int argc, const char * argv[])
{
    Test().out();
}
4
  • я писал уже выше. Вы просто перебираете известные элементы, это компилятор знает и конфликта нет. Ответ выше уже был дан. Если вы замените range based for на прямое указывание, то компилятор выведет ошибку. Не верите, замените ваше тело метода на cout << m[1] << endl;
    – Bahoo08
    6 сен 2018 в 10:30
  • Об ЭТОМ вы ничего не писали в вопросе. Перечитайте - где в нем хоть слово об обращении через оператор индекса? Если вам "надо вывести конкретные элементы" - то всегда есть find(), который и нужно применять в вашем случае как самый надежный способ. Вы в очередной раз, как многие, спросили не о том, как забить гвоздь, а в какой руке держать микроскоп для забивания гвоздя. Вернее, даже не так - пытаясь выяснить, в какой руке держать микроскоп для забивания гвоздя, вы спрашиваете, как завернуть шуруп :( Без точного ТЗ результат всегда один - ХЗ :) Формулируйте вопросы верно...
    – Harry
    6 сен 2018 в 10:53
  • Ну написал,, что нужно обратиться к элементу, а не к элементам. В вашем примере выводятся элементы. Так что и вы немного не корректно прочли, не смотря на мою не до конца точную формулировку.
    – Bahoo08
    6 сен 2018 в 11:58
  • Ну, если для вас обращение к элементу тождественно применению оператора индекса... Мне даже нечего вам возразить :) - настолько на разных языках мы говорим...
    – Harry
    6 сен 2018 в 12:28

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.