Проблема с указателем на элемент вектора

@Fobes · Регистрация: 19.09.2012

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

Всем доброго времени суток. Не могу никак выкрутиться)
Допустим, у нас есть класс "А" с параметрами:

C++
1
    std::vector <TNode*> nodes;

И есть зависимый от него класс "В" с параметрами:

C++
1
    TNode* firstNode;

В классе "В" сохранен указатель на элемент класса "А". Но элементы можно удалять независимо друг от друга. При удалении объекта класса "В" нет никаких проблем. Но при удалении объекта класса "А" (delete по вектору) указатель в классе "В" невозможно разыменовать и прога падает.
TNode:

C++
1
    int getBaseY(){return *this->baseY;}

Как быть, чет не приходят идеи в голову

@DrOffset · 16.07.2015, 16:41

Не по теме:

Avazart, это не присваивание.

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

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

if(boost::shared_ptr<int> spt = ref.lock())/* "=" ?? */

Аналог:

C++
1
if(boost::shared_ptr<int> spt(ref.lock()))

@Avazart · 16.07.2015, 16:50

Не по теме:

Мм и не смущает конструирование в if ? Как по мне, как минимум непривычно.
Я бы скорее использовал expired().

@DrOffset · 16.07.2015, 16:57

Не по теме:

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

Мм и не смущает конструирование в if ? Как по мне, как минимум непривычно.

"Это так и задумано" (с). Цель этого намеренно сузить область видимости переменной, дабы исключить ее случайное применение. Очевидно, что нулевой пойтер нельзя разыменовывать, поэтому нет смысла оставлять видимость переменной после условия. Когда такое происходит в for, то вопросов же не возникает:

C++
1
2
3
4
for(int i = 0; i < 10; ++i)
//vs
int i;
for(i = 0; i < 10; ++i)

Как раз таки второе - это сишный подход. А первое - С++. В if тоже самое.

@Avazart · 16.07.2015, 16:59

Не по теме:

Я ему про Фому ...

@DrOffset · 16.07.2015, 17:07

Не по теме:

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

Я ему про Фому ...

Я просто объяснил зачем это сделано.
А Фома - это "непривычно"? Ну так это нестрогое понятие. Кому-то и std:: непривычно и т.п. Что ж теперь не пользоваться std::?
В языке эта возможность существует именно для таких случаев, не вижу причин почему ее нужно игнорировать.

А случайное присваивание в if уже давно диагностируется всеми современными компиляторами, если предупреждения не отключать совсем.

@Avazart · 16.07.2015, 17:13

Не по теме:

Фома это читаемость/понятность кода.

@DrOffset · 16.07.2015, 17:19

Не по теме:

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

Фома это читаемость/понятность кода.

Это субъективная шкала.

@Avazart · 16.07.2015, 17:33

Не по теме:

И все же конструировать объект раньше времени что бы только "проверить"...

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

Это субъективная шкала.

Да и никто не говорил что так строго запрещено делать...

@DrOffset · 16.07.2015, 17:43

Не по теме:

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

И все же конструировать объект раньше времени что бы только "проверить"...

В смысле раньше времени?
Этот код полностью эквивалентен такому, за исключением области ивдимости spt

C++
1
2
3
4
5
6
    boost::shared_ptr<int> spt = ref.lock();
    if(spt)
    {
        //........
    }
    // тут spt все еще доступна

В остальном код полностью эквивалентный. Никого "раньше времени" нет и в помине.

@Avazart · 16.07.2015, 17:51

Не по теме:

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

за исключением области ивдимости spt

И ясности, как по мне тут область не так уж важна.

А вообще я про это:

C++
1
2
3
4
if(!ref.expired())
{
    // use  ref.lock();
}

@DrOffset · 16.07.2015, 17:55

Не по теме:

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

И ясности, как по мне тут область не так уж важна.

Мы оба понимаем, что это холиварная тема. Предлагаю на этом на эту тему закончить.

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

А вообще я про это:

Не заметил как ты перевел фокус на другой аспект. С этим соглашусь.

@Avazart · 16.07.2015, 17:56

Не по теме:

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

Не заметил как ты перевел фокус на другой аспект. С этим соглашусь.

Как другой? Все тот же конструктор в if это странно...

@DrOffset · 16.07.2015, 18:02

Не по теме:

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

С этим соглашусь.

Но с оговоркой, код c expired потребует дополнительной синхронизации в многопоточном окружении. Объект может протухнуть сразу после проверки, но до первого обращения к нему.

Не по теме:

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

Как другой? Все тот же конструктор в if это странно...

Мой код более универсален и применим в многопточной среде. Если элементы вектора чистятся из другого потока.
Но для учебного примера можно было бы и обойтись expired, с этим я согласен.
Все остальное - твои субъективные ощущения насчет читаемости.

@Avazart · 16.07.2015, 18:08

Не по теме:

В многопоточной среде стоит будет пересмотреть полностью всю программу.
Т.е обычно это означает что почти весь код в уйдет в мусорку.

@DrOffset · 16.07.2015, 18:12

Не по теме:

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

В многопоточной среде стоит будет пересмотреть полностью всю программу.

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

Новые блоги и статьи Все статьи Все блоги /
Учебный мини "цифровой двойник" = (Сервер = Java + Flask_питон )+(3D = Гугловскаая 4T nvideo + google colab) +( WMS = Odoo на Docker) anaschu 07.07.2026 Увидел в требованиях к вакансии создателя имитационных моделей связку (REst + Api + WMS + Anylogic + питон + 3D Nvideo), и решил посмотреть, что это такое. Из этого всего я работал более менее. . .	Море в объективе kumehtar 06.07.2026 Хотел написать много проникновенных слов, но передумал. Оно само - говорит. Кто слышит тот слышит.	Doom для терминала без стрельбы и монстров. 3D Raycasting на ascii. dcc0 05.07.2026 Попросил нейронную сеть deepai. org написать рейкастинг 3D с библиотекой ncurses для Linux. Чтобы можно было ходить на стрелочки. Чтобы стены были отрисованы символами. Справилась. Первый вариант. . .	Установка статуса документа по условию Maks 05.07.2026 Алгоритм из решения ниже реализован на нетиповом документе "НарядПутевка" разработанного в КА2. Задача: в табличной части "Материалы" документа при записи автоматически устанавливать статус. . .
Сезонность и суточность закисления почв anaschu 04.07.2026 200 часов это все равно моловато. Есть ситуации, но нестандартные, когда смена происходит за 5 лет. Но обычно это 50 лет и более. Наверное, закисление почвы происходит сезонно в средней. . .	В чем ценность человеческого опыта в глобальном смысле? kumehtar 03.07.2026 Возможно, ценность человека не в том, что он однажды достигает мудрости, а в том, что он становится носителем карты пути. Он знает не только истину, но и последовательность внутренних изменений,. . .	интеграция AnyLogic с самописным REST API и переход на Odoo anaschu 03.07.2026 Успешная интеграция AnyLogic с самописным REST API и переход на промышленную Odoo WMS Сегодня проделал огромный путь от простой симуляции физических процессов до построения полноценной. . .	Поиск всех путей на ориентированном графе. Linux dcc0 02.07.2026 Переработка старого кода из моей статьи. Через несколько переработок от PHP кода к C89 (надеюсь, 89). Но довольно запутанно получилось. Код для Linux. Но если убрать time и то, что с ним. . .

@Avazart
	16.07.2015, 16:50
	Не по теме: Мм и не смущает конструирование в if ? Как по мне, как минимум непривычно. Я бы скорее использовал expired(). 0

@Avazart
	16.07.2015, 16:59
	Не по теме: Я ему про Фому ... 0

@Avazart
	16.07.2015, 17:13
	Не по теме: Фома это читаемость/понятность кода. 0

@Avazart
	16.07.2015, 18:08
	Не по теме: В многопоточной среде стоит будет пересмотреть полностью всю программу. Т.е обычно это означает что почти весь код в уйдет в мусорку. 0