Проверьте мой класс на наличие ошибок и недочётов

[Человек_Борща]

Всем доброго времени суток!

Переписываю свою программу на d2010 полностью.

Переписал класс хранения списка отлавливаемых окон-ошибок, нужно проверить на наличие ошибок сам класс. Ссылка на Pastebin.


Как говорится "В чужом глазу сучок видим, а в своем бревна не замечаем."

помогите найти все бревна и сучки в коде=)

[veniside]

Без главной формы список ну никак не может существовать? Даёшь каждому списку по форме!

uses MainFormUnit;
 
  { TErrorWindowsList }
 
procedure TErrorWindowsList.AddWindow(aError: TErrorWindow);
var
  LI: TlistItem;
begin
  With MainForm.ErrorsList do
  begin

это чё, новый вид рекурсии?

procedure TErrorWindowsList.DeleteWindow(aIndx: integer);
begin
  eWindows[aIndx].Free;
  DeleteWindow(aIndx);
end;

[Человек_Борща]

Нужно же куда-то данные показывать.
MainForm.ErrorsList - это ListView.

Если я правильно думаю, то можно в ручную создавать такую процедуру? Не обязательно делать её частью класса.

А за второе спасибо(ввиде +сега)! Не заметил, мой косяк=)

Ещё замечания по коду будут?

Люди добрые, прошу уделить 10 минут времени=)

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

Если я правильно думаю, то можно в ручную создавать такую процедуру? Не обязательно делать её частью класса.

правильнее сделать процедуру в которую уже передавать ListView и процедура заполнит уже что надо.

[Человек_Борща]

Сделал. Ещё предложения,замечания,критика по коду будут?