|
279 / 156 / 52
Регистрация: 30.06.2011
Сообщений: 1,712
|
|
Изменить std::map08.12.2019, 15:06. Показов 5320. Ответов 42
Метки нет (Все метки)
Приветствую всех. Мне на работе приходится работать не совсем со свежим компилятором С++ и такой же старенькой библиотекой STL (там даже
auto_ptr не deprecated). Обновить все это хозяйство не возможно, поэтому приходится работать с тем, что есть.Я храню в std::map объекты своего класса. Так вот столкнулся с такой неприятной вещью, что при использовании оператора [] даже для существующего элемента вызывается конструктор по умолчанию. Сделав элементарный пример и проверив его на нескольких онлайн компиляторах я убедился, что такое поведение характерно только для моей STL.Открыв код std::map было обнаружено, что оператор [] это, по сути, обертка над методом insert. То есть, там не происходит проверки существует ли элемент, как, например, в библиотеке STL C++Builder.Поэтому такой вопрос. Могу ли я скопировать код файла map в файл, например my_map, добавить проверку в оператор [] и использовать этот свой файл для работы с std::map? Не возникнет ли в будущем каких-то неприятностей?
0
|
|
| 08.12.2019, 15:06 | |
|
Ответы с готовыми решениями:
42
переписать std::map Вопрос по std::map |
|
279 / 156 / 52
Регистрация: 30.06.2011
Сообщений: 1,712
|
|||||||||
| 10.12.2019, 10:12 [ТС] | |||||||||
Добавлено через 2 минуты
0
|
|||||||||
|
Комп_Оратор)
|
||||||||
| 10.12.2019, 11:30 | ||||||||
|
Я покажу как можно привести пример. Смотрите:
Вот об этом я хотел услышать.
0
|
||||||||
|
279 / 156 / 52
Регистрация: 30.06.2011
Сообщений: 1,712
|
||||||||
| 10.12.2019, 12:01 [ТС] | ||||||||
![]() Ну да ладно. Беру Ваш пример (чуть изменил, добавив cout-ов) и запускаю у себя (на своем компиляторе и "той самой" STL). Привожу результат работы кода онлайн компилятора и моего.Запускаемый код: Кликните здесь для просмотра всего текста
Онлайн компилятор: Кликните здесь для просмотра всего текста
mama
papa Bar() left value Bar() right value mama Мой компилятор: Кликните здесь для просмотра всего текста
mama
papa Bar() left value Bar() Bar(const Bar &) Bar(const Bar &) right value Bar() Bar(const Bar &) mama
0
|
||||||||
| 10.12.2019, 13:15 | |
|
Не по теме: d7d1cd, это у вас там случайно не VS 6?
0
|
|
| 10.12.2019, 13:18 [ТС] | ||||||
|
Не по теме: DrOffset, нет. Операционная система IBM i. Если интересно, ниже содержимое файла map. Копирайт 1998 года :D. Кликните здесь для просмотра всего текста
0
|
||||||
|
фрилансер
6461 / 5663 / 1130
Регистрация: 11.10.2019
Сообщений: 15,079
|
|
| 10.12.2019, 13:26 | |
|
ого, триграфы о_О
0
|
|
|
19500 / 10105 / 2461
Регистрация: 30.01.2014
Сообщений: 17,816
|
|
| 10.12.2019, 14:04 | |
|
Не по теме: d7d1cd, это поведение, конечно, баг. Я такое встречал в каких-то других реализациях, дело было давно. Добавлено через 2 минуты
1
|
|
|
279 / 156 / 52
Регистрация: 30.06.2011
Сообщений: 1,712
|
||
| 10.12.2019, 14:19 [ТС] | ||
|
Надеюсь, реализация заплатки годная...
1
|
||
|
19500 / 10105 / 2461
Регистрация: 30.01.2014
Сообщений: 17,816
|
|||||||
| 10.12.2019, 14:31 | |||||||
0
|
|||||||
|
Комп_Оратор)
|
|
| 10.12.2019, 15:17 | |
|
d7d1cd, эта библиотека написана корифеями. Вот как плохо жиреть на военных заказах... Я бы предложил отказаться от оператора индексации. И обязательно искать, сначала (см. выше).
Можно ещё поработать с указателями в качестве значений (придётся написать предикат на сравнение указателей) и переписать всё, что работало со значениями. Поэтому проще предварять insert поиском и использовать итератор подсказки если пара не найдена (upper_bound) и если найдена то удаляем её и вставляем новую или переписываем значение в паре, в зависимости от поддержки в классе значения. Тут также, есть выбор, - оставить прежнее без изменения, чего оператор[] не предоставляет. Это должно работать быстрее чем оператор [].
0
|
|
|
279 / 156 / 52
Регистрация: 30.06.2011
Сообщений: 1,712
|
|
| 10.12.2019, 15:25 [ТС] | |
|
IGPIGP, скажите, а чем плох метод переопределения оператора []?
0
|
|
|
19500 / 10105 / 2461
Регистрация: 30.01.2014
Сообщений: 17,816
|
|
| 10.12.2019, 15:59 | |
|
d7d1cd, я вам даже больше скажу, во времена, когда ваша версия компилятора и библиотеки были актуальными - в С++ сообществе вовсю практиковалась замена STD на стороннюю, потому что стандартная часто не выдерживала критики.
Так родился, например, такой продукт как STLPort, ныне почти полностью позабытый.
1
|
|
|
495 / 209 / 70
Регистрация: 27.05.2016
Сообщений: 557
|
||
| 10.12.2019, 16:27 | ||
![]() Искал инфу с доводами против наследования от стандартных STL контейнеров - есть те кто считает это неправильным в силу того что стандартные контейнеры не проектировались для наследования (у них нет, как минимум, виртуальных деструкторов), а есть и те кто считает это нормальной практикой. Нужно смотреть под какие нужды это делаеться и как будет использоваться и не возникнет ли в будущем проблем при работе с таким кодом. Кстати: boost program_options variables_map
1
|
||
|
19500 / 10105 / 2461
Регистрация: 30.01.2014
Сообщений: 17,816
|
||||||
| 10.12.2019, 16:50 | ||||||
|
Я не знаю кто вбросил эту мысль про отсутствие виртуальных деструкторов как критерий неиспользования в качестве базовых классов, но эта дурь ходит по сети лет 20 точно. И каждое новое поколение новичков постоянно на это ссылается.
А дурь это потому, что для того, чтобы виртуальный деструктор имел значение - класс должен предполагать полиморфное использование. И мало того, полиморфное удаление. У стандартных контейнеров нет ни одного виртуального метода, значит никакого полиморфного использования с ними не получится априори. И довод, что кто-то может сделать так:
Более того, если бы комитет поддерживал эту идею, то с приходом С++11 давно бы уже по-объявляли все стандартные контейнеры как final и закрыли вопрос. Но никто что-то этого делать не стал.
5
|
||||||
|
Комп_Оратор)
|
||
| 10.12.2019, 16:50 | ||
|
Кроме того, придётся перекомпилировать весь проект. Это иногда критический затык. Ну а проблемы с совместимостью с библиотеками, которые могут юзать этот класс и перспективой расширения кодовой базы тоже не радостны. Я бы не стал так делать. Можно переписать все критические места где вызывается оператор индексации и оно будет работать на всех компиляторах. Но это частное мнение, возможно есть варианты получше.
0
|
||
|
279 / 156 / 52
Регистрация: 30.06.2011
Сообщений: 1,712
|
||
| 11.12.2019, 08:37 [ТС] | ||
|
IGPIGP, по Вашему последнему посту. Я меняю поведение оператора [] в мапе только для использования в своем проекте. Этот проект будет и может работать только на конкретной системе. Код в стандартной библиотеке я не меняю (да и нет у меня авторизации на изменение этих файлов). Планирую все подобные баги STL в моей системе исправлять (видимо, как и тут, создавать наследников с переопределением кривых методов) и хранить в отдельных файлах. При необходимости использовать в заголовках их.
0
|
||
|
19500 / 10105 / 2461
Регистрация: 30.01.2014
Сообщений: 17,816
|
|
| 11.12.2019, 08:48 | |
|
d7d1cd, а если рассмотреть вариант полной замены STL на ту же STLPort? Она не только содержит меньше багов, но еще и быстрее работает. Если весь проект пересадить на нее, то он, возможно, станет работать лучше.
0
|
|
|
Комп_Оратор)
|
|||||||
| 11.12.2019, 12:34 | |||||||
Вы думаете без компаратора, карта не будет ущербна? Лучше уже затайпдефить карту каким-то промежуточным типом. Включить тайпдеф в условных директивах так чтобы можно было компилировать и так и эдак. И в варианте "'эдак" использовал бы включение объекта map. Да, - придётся переписать кучу тупого обёрточного кода. Но это не так уж и долго. Добавлено через 37 минут Но в принципе, я вас немного запутал. При прямом наследовании можно конечно похулиганить. Но стрёмно как-то Кликните здесь для просмотра всего текста
0
|
|||||||
|
19500 / 10105 / 2461
Регистрация: 30.01.2014
Сообщений: 17,816
|
|||||||
| 11.12.2019, 12:40 | |||||||
3
|
|||||||
| 11.12.2019, 12:40 | |
|
Помогаю со студенческими работами здесь
40
Обход элементов std::map в порядке их создания std::string, std::fstream, ошибка кучи Как проинициализировать std::stack<const int> obj ( std::stack<int>{} );
Где в настройках RAD Studio 10 Seattle изменить аргумент на -std=c99? Искать еще темы с ответами Или воспользуйтесь поиском по форуму: |
|
Новые блоги и статьи
|
|||
|
SDL3 для Web (WebAssembly): Синхронизация спрайтов SDL3 и тел Box2D
8Observer8 04.03.2026
Содержание блога
Финальная демка в браузере. Итоговый код: finish-sync-physics-sprites-sdl3-c. zip
На первой гифке отладочные линии отключены, а на второй включены:. . .
|
SDL3 для Web (WebAssembly): Идентификация объектов на Box2D v3 - использование userData и событий коллизий
8Observer8 02.03.2026
Содержание блога
Финальная демка в браузере. Итоговый код: finish-collision-events-sdl3-c. zip
https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11680&d=1772460536
Одним из. . .
|
Реалии
Hrethgir 01.03.2026
Нет, я не закончил до сих пор симулятор. Эта задача сложнее. Не получилось уйти в плавсостав, но оно и к лучшему, возможно. Точнее получалось - но сварщиком в палубную команду, а это значит, в моём. . .
|
Ритм жизни
kumehtar 27.02.2026
Иногда приходится жить в ритме, где дел становится всё больше, а вовлечения в происходящее — всё меньше. Плотный график не даёт вниманию закрепиться ни на одном событии. Утро начинается с быстрых,. . .
|
|
SDL3 для Web (WebAssembly): Сборка библиотек: SDL3, Box2D, FreeType, SDL3_ttf, SDL3_mixer и SDL3_image из исходников с помощью CMake и Emscripten
8Observer8 27.02.2026
Недавно вышла версия 3. 4. 2 библиотеки SDL3. На странице официальной релиза доступны исходники, готовые DLL (для x86, x64, arm64), а также библиотеки для разработки под Android, MinGW и Visual Studio. . . .
|
SDL3 для Web (WebAssembly): Реализация движения на Box2D v3 - трение и коллизии с повёрнутыми стенами
8Observer8 20.02.2026
Содержание блога
Box2D позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .
|
Конвертировать закладки radiotray-ng в m3u-плейлист
damix 19.02.2026
Это можно сделать скриптом для PowerShell. Использование
. \СonvertRadiotrayToM3U. ps1 <path_to_bookmarks. json>
Рядом с файлом bookmarks. json появится файл bookmarks. m3u с результатом.
# Check if. . .
|
Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI
Eddy_Em 18.02.2026
Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так:
https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347
Основана на STM32F303RBT6.
На борту пять. . .
|