ListView и привязанный к нему объект

[Stilet]

код сомнительного качества.

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

[Пепел Феникса]

И вас совсем не смущает, что в базовом классе уже имеется метод (обратите внимание - метод, а не деструктор) с названием "Free"?

переопределяем метод, дальше то что?
если хочется то можно и так, но всетаки я советовал бы переопределять стандартный деструктор Destroy с override и потом вызывать через inherited базовый(в конце нашего нового)
всетаки от Free мы ожидаем сначало проверки

[dr.Chas]

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

А проверить на содержимое опять дядя Пушкин должен?
А вдруг .Data совсем не то содержит что нужно?

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

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

И еще как бы я не делал - класс и его объекты для меня вещь в себе, поэтому я бы не использовал никаких прямых вызовов чужих компонентов, таких как ListView1.AddItem(n, self); (который принадлежит форме).

Дело в том, что я привязываю к айтему каждый объект, мне так удобно. Можно было пойти другим способом, переписать компонент listview под себя. Но в чём разница того и другого? Лично мне приятнее ориентироваться в своём коде Хотя может я вас не так понял?

Сообщение от Пепел Феникса

но всетаки я советовал бы переопределять стандартный деструктор Destroy с override и потом вызывать через inherited базовый(в конце нашего нового)
всетаки от Free мы ожидаем сначало проверки

прям так и сделал

[Stilet]

Лишний if

IF никогда не бывает лишним

там точно путаницы не должно быть

Ой... Зарекаемся... Хозяин конечно барин, но и на проруху найдется старуха (и иногда с косой), по крайней мере я лично после некоторых граблей стал проверки тщательней ставить - поверь мне 30% проблем сразу исчезло.