Access Violation при попытке удалить "существующий объект"

@murphy · Регистрация: 14.05.2015

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

Есть такой вот код.

C++

    
for (int i = 0; i < Form1->ComponentCount; i++) {
    if (((TImage*)Form1->Components[i])->Hint == secondMaze[gameI1][gameJ1]) {
        delete Form1->Components[i];
    }
}

Примерно через раз(Это не точно, ибо после ошибки выключал, но ошибки ловил не каждый раз) вываливает ошибку Access Violation. Как такое может быть? Ведь код выполняется, если объект найден.

@nick42 · 14.05.2015, 23:10

Потому что неверно. У вас же в цикле каждый компонент формы "пробуется" на принадлежность классу TImage. По правилам сначала нужно отфильтровать (ClassNameIs ) из всех компонентов именно картинки, и с ними уже можно так обращаться. ( Hint -то по идее у подавляющего большинства компонентов присутствует, но тут - дело принципа

).

@murphy · 14.05.2015, 23:13 **[ТС]**

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

Потому что неверно. У вас же в цикле каждый компонент формы "пробуется" на принадлежность классу TImage. По правилам сначала нужно отфильтровать (ClassNameIs ) из всех компонентов именно картинки, и с ними уже можно так обращаться. ( Hint -то по идее у подавляющего большинства компонентов присутствует, но тут - дело принципа

).

Значит дописать что-нибудь типа

C++

1	&& Form1->Components[i]->ClassName == "TImage"

?

@nick42 · 14.05.2015, 23:13

И вдобавок такого рода цикл лучше начинать "сверху". А то удалите из середины..., индекс цикла уже инкрементирован, а под прежним индексом компонент окажется пропущенным.

@murphy · 14.05.2015, 23:14 **[ТС]**

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

И вдобавок такого рода цикл лучше начинать "сверху". А то удалите из середины..., индекс цикла уже инкрементирован, а под прежним индексом компонент окажется пропущенным.

Там только один элемент удаляется. Надо break поставить. Спасибо, что тыкнули.

@nick42 · 14.05.2015, 23:16

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

Значит дописать что-нибудь типа

Нет, первый анализ - на принадлежность классу, и только в положительном случае вот это: ((TImage*)Form1->Components[i]. А вообще, мэтры С++ (и С) избегают такого явного указания и применяют ..._cast .

@murphy · 14.05.2015, 23:18 **[ТС]**

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

Нет, первый анализ - на принадлежность классу, и только в положительном случае вот это: ((TImage*)Form1->Components[i]. А вообще, мэтры С++ (и С) избегают такого явного указания и применяют ..._cast .

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

@volvo · 14.05.2015, 23:21

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

и применяют ..._cast

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

@nick42 Практикантроп 4787 / 2686 / 518 Регистрация: 23.09.2011 Сообщений: 5,747
	14.05.2015, 23:10	2
	Сообщение было отмечено murphy как решение Решение Потому что неверно. У вас же в цикле каждый компонент формы "пробуется" на принадлежность классу TImage. По правилам сначала нужно отфильтровать (ClassNameIs ) из всех компонентов именно картинки, и с ними уже можно так обращаться. ( Hint -то по идее у подавляющего большинства компонентов присутствует, но тут - дело принципа ). 1

@nick42 Практикантроп 4787 / 2686 / 518 Регистрация: 23.09.2011 Сообщений: 5,747
	14.05.2015, 23:13	4
	И вдобавок такого рода цикл лучше начинать "сверху". А то удалите из середины..., индекс цикла уже инкрементирован, а под прежним индексом компонент окажется пропущенным. 0

@murphy 0 / 0 / 1 Регистрация: 14.05.2015 Сообщений: 34
	14.05.2015, 23:14 [ТС]	5
	Сообщение от nick42 И вдобавок такого рода цикл лучше начинать "сверху". А то удалите из середины..., индекс цикла уже инкрементирован, а под прежним индексом компонент окажется пропущенным. Там только один элемент удаляется. Надо break поставить. Спасибо, что тыкнули. 0

@nick42 Практикантроп 4787 / 2686 / 518 Регистрация: 23.09.2011 Сообщений: 5,747
	14.05.2015, 23:16	6
	Сообщение от murphy Значит дописать что-нибудь типа Нет, первый анализ - на принадлежность классу, и только в положительном случае вот это: *((TImage)Form1->Components[i]**. А вообще, мэтры С++ (и С) избегают такого явного указания и применяют ..._cast . 0

@murphy 0 / 0 / 1 Регистрация: 14.05.2015 Сообщений: 34
	14.05.2015, 23:18 [ТС]	7
	Сообщение от nick42 Нет, первый анализ - на принадлежность классу, и только в положительном случае вот это: *((TImage)Form1->Components[i]**. А вообще, мэтры С++ (и С) избегают такого явного указания и применяют ..._cast . Окей. Понял Вас. К сожалению, я не мэтр, да и сроки поджимают. Курсовую сдам, и обязательно посмотрю про ..._cast. 0

@volvo Супер-модератор 32591 / 21060 / 8134 Регистрация: 22.10.2011 Сообщений: 36,332 Записей в блоге: 8
	14.05.2015, 23:21	8
	Сообщение от nick42 и применяют ..._cast Не ..._cast, а именно dynamic_cast, чтобы, в случае невозможности каста (а значит, в случае когда тип компонента не тот, который нужен) вернулся NULL... 1

Опции темы

Access Violation при попытке удалить "существующий объект"

Решение