Заполнение словаря из функции

@Alex_Solo_Mars · Регистрация: 29.05.2024

Студворк — интернет-сервис помощи студентам

Добрый день,

заклинило меня на простой проблеме, не знаю, где туплю. Подскажите, пожалуйста, где прочитать про это.

Создаю пустой словарь:

C++
1
map<string, string> test_dic;

Пишу функцию:

C++
1
2
3
4
void test_foo(map<string, string>* t_mapping)
{
    t_mapping->at("ключ") = "значение";
}

Компилируется прекрасно. При исполнении выдаёт ошибку "out of range". Кто знает почему так происходит и как это лечить?

@SmallEvil · 14.08.2024, 12:08

Читаем описание метода std::map::at

Returns a reference to the mapped value of the element with specified key. If no such element exists, an exception of type std::out_of_range is thrown.

Добавлено через 5 минут

Сообщение от Alex_Solo_Mars

и как это лечить?

Зависит от задачи.
Если заполнять из функции, без проверки наличия ключа + с заменой значения, с вашими параметрами функции :

C++
1
    (*t_mapping)["ключ"] = "значение";

@KSergey9 · 14.08.2024, 12:13

Не по вопросу

Сообщение от Alex_Solo_Mars

void test_foo(map<string, string>* t_mapping)

А вот указатель тут точно нужен?
Может ссылка будет кошернее?

@Alex_Solo_Mars · 14.08.2024, 12:55 **[ТС]**

Сообщение от KSergey9

Не по вопросу

А вот указатель тут точно нужен?
Может ссылка будет кошернее?

Ссылка, она всегда кошернее будет, т.к. указатель уже воспринимается как рудимент. Однако, как советует Гугл, если в функции вы хотите менять передаваемый объект, то лучше использовать указатель, а не ссылку.

@KSergey9 · 14.08.2024, 12:58

Сообщение от Alex_Solo_Mars

если в функции вы хотите менять передаваемый объект, то лучше использовать указатель, а не ссылку

Думаю, гугл вам нехорошие советы даёт в данном случе.
Можно ссылочку, почитать чего там автор пишет и в каком контексте?

@SmallEvil · 14.08.2024, 13:09

Сообщение от Alex_Solo_Mars

Ссылка, она всегда кошернее будет, т.к. указатель уже воспринимается как рудимент. Однако, как советует Гугл, если в функции вы хотите менять передаваемый объект, то лучше использовать указатель, а не ссылку.

Указатель никуда не делся и используется.
Но ваш случай, подразумевает что вам должен передаваться объект, всегда валидный. Значит использование ссылки напрашивается само собой.

@Alex_Solo_Mars · 14.08.2024, 13:28 **[ТС]**

Сообщение от KSergey9

Думаю, гугл вам нехорошие советы даёт в данном случе.
Можно ссылочку, почитать чего там автор пишет и в каком контексте?

Вот тут я это прочитал (типа я умею ссылки вставлять):

Code
1
ru ТОЧКА stackoverflow ТОЧКА com/questions/994785/%D0%9A%D0%BE%D0%B3%D0%B4%D0%B0-%D1%81%D1%82%D0%BE%D0%B8%D1%82-%D0%B8%D1%81%D0%BF%D0%BE%D0%BB%D1%8C%D0%B7%D0%BE%D0%B2%D0%B0%D1%82%D1%8C-%D0%B2-%D0%BA%D0%B0%D1%87%D0%B5%D1%81%D1%82%D0%B2%D0%B5-%D0%BF%D0%B0%D1%80%D0%B0%D0%BC%D0%B5%D1%82%D1%80%D0%BE%D0%B2-%D1%84%D1%83%D0%BD%D0%BA%D1%86%D0%B8%D0%B8-%D1%83%D0%BA%D0%B0%D0%B7%D0%B0%D1%82%D0%B5%D0%BB%D0%B8-%D0%B0-%D0%BA%D0%BE%D0%B3%D0%B4%D0%B0-%D0%BF%D0%B5%D1%80%D0%B5%D0%BC%D0%B5

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

Добавлено через 5 минут

Сообщение от SmallEvil

Читаем описание метода std::map::at

Добавлено через 5 минут

Зависит от задачи.
Если заполнять из функции, без проверки наличия ключа + с заменой значения, с вашими параметрами функции :

C++
1
    (*t_mapping)["ключ"] = "значение";

Спасибо, это действительно работает. Проверять не надо, т.к. только заполняю словарь. Пойду писать гневные письма умникам, которые уверяли, что после С++11 запись

C++
1
t_mapping->at("ключ") = "значение";

более правильная, чем

C++
1
(*t_mapping)["ключ"] = "значение";

[/QUOTE]

Добавлено через 9 минут

Сообщение от SmallEvil

Указатель никуда не делся и используется.
Но ваш случай, подразумевает что вам должен передаваться объект, всегда валидный. Значит использование ссылки напрашивается само собой.

Спасибо, сделал и так и так. Работает в обоих случаях. Теперь пытаюсь, понять что из этого best practices.

C++
1
2
3
4
5
6
7
8
9
void dic_sample_pointer(std::map<std::string, std::string>* t_mapping)
{
    (*t_mapping)["ключ"] = "значение";
}
 
void dic_sample_ref(std::map<std::string, std::string>& t_mapping)
{
    (t_mapping)["ключ"] = "значение";
}

@SmallEvil · 14.08.2024, 13:36

Сообщение от Alex_Solo_Mars

Вот тут я это прочитал (типа я умею ссылки вставлять)

По ссылке на Google Guid.

Avoid defining functions that require a reference parameter to outlive the call. In some cases reference parameters can bind to temporaries, leading to lifetime bugs. Instead, find a way to eliminate the lifetime requirement (for example, by copying the parameter), or pass retained parameters by pointer and document the lifetime and non-null requirements.

Мне уже хочется увидеть этот случай. Что же они такого начудили что бы добиться подобного эффекта.

map::at и map::operator[] имеют различное поведение изначально.

@Alex_Solo_Mars · 14.08.2024, 13:39 **[ТС]**

Сообщение от SmallEvil

По ссылке на Google Guid.

Мне уже хочется увидеть этот случай. Что же они такого начудили что бы добиться подобного эффекта.

map::at и map::operator[] имеют различное поведение изначально.

К сожалению, это пока не мой уровень дискуссии. Ваше решение работает, спасибо.

@KSergey9 · 14.08.2024, 14:15

Сообщение от Alex_Solo_Mars

Пойду писать гневные письма умникам, которые уверяли, что после С++11 запись

Скорее всего там контекст был, где это было разумно.

@leproza · 14.08.2024, 14:18

Сообщение от SmallEvil

Мне уже хочется увидеть этот случай. Что же они такого начудили что бы добиться подобного эффекта.

Как всегда, сами же выстрелили себе в ногу =)

C++
1
2
3
4
5
6
7
8
9
class StringHolder {
public:
  // The input `val` must live as long as this object, not just the call to this c'tor
  StringHolder(const string& val) : val_(val) {}
 
  const string& get() { return val_; }
private:
  const string& val_;
};

https://github.com/google/styleguide/issues/561

@KSergey9 · 14.08.2024, 14:22

По поводу приведённой ссылки на Google Style Guide.

Я могу понять откуда у них взялось такое в Style Guide. Если нас никто не слышит, то по секрету признаюсь, что я тоже больше предпочитаю стиль, когда возвращаемые значения передаются через указатель, т.к. в тексте программы в месте вызова функции по & перед аргументом явно видишь, что это возвращаемый параметр. Во всяком случае это надежно.
Впрочем, этого же эффекта можно достигнуть другими соглашениями в Style Guide, однако столь же эффективно добиться соблюдения таких соглашений от команды, как в случае с указателями, будет сложнее.

Но с точки зрения языка ссылки, конечно, уместнее, т.к. уже на этапе компиляции гарантируют, что будет передан валидный объект (понятно, это можно обойти трюками, но троюки сразу явно видно будет).

@Alex_Solo_Mars · 14.08.2024, 14:24 **[ТС]**

Сообщение от KSergey9

Скорее всего там контекст был, где это было разумно.

Или я неправильно понял и, не разобравшись, сделал слишком обобщённый вывод.

@Ballantrae · 14.08.2024, 14:33

Сообщение от Alex_Solo_Mars

К сожалению, это пока не мой уровень дискуссии. Ваше решение работает, спасибо.

На сколько я помню, at(key) возвращает ссылку на переменную соответсвющую ключу, а оператор [] проверяет, есть ли такой ключ и если нет добавляет его и возвращает ссылку на переменную, хранящуюся по заданному ключу, если есть, то просто возвращает ссылку.

Алексей1153 · 14.08.2024, 14:51

Ballantrae, Alex_Solo_Mars, возможно, это будет шоком, но есть ещё insert и emplace

@Ballantrae · 14.08.2024, 14:56

Сообщение от Алексей1153

возможно, это будет шоком, но есть ещё insert и emplace

С чего бы это должно быть шоком? Речь шла про at() и operator[]. Про них и написал.

Алексей1153 · 14.08.2024, 15:01

Ballantrae, это я про метания ТС между at и []

@SmallEvil · 14.08.2024, 15:34

Сообщение от leproza

сами же выстрелили себе в ногу =)

А где же ссылки (я вижу лишь константные ссылки) и багостное продление жизни в примере?

@Alex_Solo_Mars · 14.08.2024, 15:38 **[ТС]**

Сообщение от Алексей1153

Ballantrae, это я про метания ТС между at и []

У ТС метания между тем, как передавать словарь в функцию: ссылкой и указателем.

Алексей1153 · 14.08.2024, 16:55

Alex_Solo_Mars, как удобнее. Компилятору, по большому счёту, без разницы

Добавлено через 3 минуты
Alex_Solo_Mars, а если объект там инициализируется, а не просто дополняется, то лучше вообще вернуть целиком

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
#include <map>
 
auto test_foo()
{
    std::map<std::string, std::string> m
    {
        {"1","1111"},
        {"2","2222"},
    };
 
    m["3"]="3333";
 
    return m;
}
 
int main()
{
    auto m=test_foo();
 
    return 0;
}

Новые блоги и статьи Все статьи Все блоги /
Новый ноутбук volvo 07.12.2025 Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro	Музыка, написанная Искусственным Интеллектом volvo 04.12.2025 Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .	От async/await к виртуальным потокам в Python IndentationError 23.11.2025 Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .	Поиск "дружественных имён" СОМ портов Argus19 22.11.2025 Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .	Сколько Государство потратило денег на меня, обеспечивая инсулином. Programma_Boinc 20.11.2025 Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .
Ломающие изменения в C#.NStar Alpha Etyuhibosecyu 20.11.2025 Уже можно не только тестировать, но и пользоваться C#. NStar - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .	Мысли в слух kumehtar 18.11.2025 Кстати, совсем недавно имел разговор на тему медитаций с людьми. И обнаружил, что они вообще не понимают что такое медитация и зачем она нужна. Самые базовые вещи. Для них это - когда просто люди. . .	Создание Single Page Application на фреймах krapotkin 16.11.2025 Статья исключительно для начинающих. Подходы оригинальностью не блещут. В век Веб все очень привыкли к дизайну Single-Page-Application . Быстренько разберем подход "на фреймах". Мы делаем одну. . .	Фото: Daniel Greenwood kumehtar 13.11.2025	Расскажи мне о Мире, бродяга kumehtar 12.11.2025 — Расскажи мне о Мире, бродяга, Ты же видел моря и метели. Как сменялись короны и стяги, Как эпохи стрелою летели. - Этот мир — это крылья и горы, Снег и пламя, любовь и тревоги, И бескрайние. . .

@KSergey9 631 / 526 / 104 Регистрация: 05.08.2022 Сообщений: 2,810
	14.08.2024, 14:22
	По поводу приведённой ссылки на Google Style Guide. Я могу понять откуда у них взялось такое в Style Guide. Если нас никто не слышит, то по секрету признаюсь, что я тоже больше предпочитаю стиль, когда возвращаемые значения передаются через указатель, т.к. в тексте программы в месте вызова функции по & перед аргументом явно видишь, что это возвращаемый параметр. Во всяком случае это надежно. Впрочем, этого же эффекта можно достигнуть другими соглашениями в Style Guide, однако столь же эффективно добиться соблюдения таких соглашений от команды, как в случае с указателями, будет сложнее. Но с точки зрения языка ссылки, конечно, уместнее, т.к. уже на этапе компиляции гарантируют, что будет передан валидный объект (понятно, это можно обойти трюками, но троюки сразу явно видно будет). 0

Алексей1153 фрилансер 6440 / 5634 / 1127 Регистрация: 11.10.2019 Сообщений: 14,978
	14.08.2024, 14:51
	Ballantrae, Alex_Solo_Mars, возможно, это будет шоком, но есть ещё `insert` и `emplace` 0

Алексей1153 фрилансер 6440 / 5634 / 1127 Регистрация: 11.10.2019 Сообщений: 14,978
	14.08.2024, 15:01
	Ballantrae, это я про метания ТС между `at` и `[]` 0

Заполнение словаря из функции

Решение