ich habe da ein Problem mit o.g. Komponente. Diese erzeuge ich dynamisch in einer eigenen Klasse. Wenn ich nun die Anwendung starte und in der CheckListBox Eintrage aktiviere bzw. deaktiviere bekomme ich beim Beenden des Programms eine EAccessViolation. Dies passiert nur enn ich Einträge 'anklicke'. Auch passiert es ab und an, das das programm abbricht, wenn ich 'viele' Einträge aktiviere.

Im Debugger ist alles i.O., also die Komponente kann ich sehen und sie hat auch entsprechende Daten.

Normalerweise passiert sowas ja wenn man einen 'verbogenen Zeiger' zu beschreiben versucht, ich möchte jedoch behaupten, da ist kein Zeiger verbogen.

Hat jemand eine Idee?

Danke im voraus

Manfred

Ich möchte behauten das TCheckListBox selber fehlerfrei ist. Der eigentliche Fehler wird also durch deinen Code verursacht, damit ist es aber auch für uns ohne relevate Codeausschnitte schwer dazu etwas zu sagen.
Das Abstürzen beim Beenden deutet darauf hin das du vermutlich bei den Aufräumarbeiten im Speicher einen Zeiger doppelt mit delete behandelst.

bis bald
akari

iss schon klar das ohne code schnipsel wenig zu sagen ist.

Ich poste hier mal den OnClick event des Button rein, der die CheckListBox erzeugt und den FormDestroy-Event der den Speicher wieder freigibt.

void __fastcall TForm3::Button1Click(TObject *Sender)
{
TAttribut *neu;

neu = new TAttribut();
neu->checklist = new TCheckListBox(this);
neu->checklist->Parent = this;
neu->checklist->Top = 100;
for( int i=0; i<400; i++ ) {
neu->checklist->Items->Add(IntToStr(i));
}
neu->checklist->Checked[2] = true;
neu->checklist->Checked[3] = true;
//ShowMessage(neu->checklist->Owner->ClassName());
usrHead = neu;
}
//---------------------------------------------------------------------------
void __fastcall TForm3::FormDestroy(TObject *Sender)
{
if( usrHead != NULL ) {
delete usrHead; usrHead = NULL;
}

}

Dioe AccessViolation passiert tatsächlich beim Speicher freigeben in der eigenen Klasse (TAttributListe). Eigenartigerweise passiert dies aber nur wenn ich einen Eintrag in der CheckListBox aktiviere bzw. deaktiviere. Doppeltes Löschen müßte ja auch passieren wenn ich nichts aktiviere und das Programm beende.

Habe ich hier grundsätzlich was falsch verstanden?

Erzeuge ich in zusätzlich noch TEdit, TLabel oder sonstig VCL Komponenten dynamisch funktionieren dies einwandfrei; auch das delete dieser Komponenten bringt keinen Fehler.

Immer nocht ratlos bin.

Manfred

Nein vor dem Löschen auf NULL setzen ist Unsinn, denn dann wird ja nichts gelöscht.
Auch die Abfrage auf NULL und das nachträgliche setzen auf NULL ist überflüssig, denn delete NULL ist gültig und macht nichts. Wichtiger wäre da im Konstruktor ds Forms den Zeiger auf NULL zu setzen weil Zeiger nicht automatisch initialisert werden.

Allerdings solltest du niemals OnDestroy/OnCreate benutzen sondern stattdessen den Destruktor (wenn nötig manuell hinzufügen) bzw. Konstruktor der Klasse. Begründung kannst du über SuFu finden.
Da du aber beim Erstellen der CheckListBox this als Owner angibst brauchst du dich aber auch gar nicht um das manuelle Löschen des Controls zu kümmern. Also entferne OnDestroy samt Implementation komplett. Wahrscheinlich behebt das schon das Problem.

Und bitte beachten : sfds

bis bald
akari

es erklärt auch nicht warum bei mehrfachem aktivieren einzelner Checkboxen in der CheckListBox eine AccessViolation kommt. ich weiß auch nicht wie ich das tracken soll.

Wenn das Form gelöscht wird (also spätestens beim Schließen des Programms) werden auch alle Controls gelöscht die das Form besitzt. Da du der CheckListBox this als Owner übergibst kümmert sich also dann das Form darum.
Manuell löschen must du nur wenn du NULL als Owner angibst (bzw. keine Owner-Eigenschaft existiert)

Aus deinem Code geht jedenfalls kein weitere Grund für einen Absturz hervor. Werden denn irgendwelche Aktionen im OnClickCheck-Event der ListBox ausgeführt?

bis bald
akari

Auffällig ist das beim Beenden des programms dann und nur dann der Fehler auftritt wenn ich mindestens eine Checkbox verändert habe, sonst nicht.

Im OnClick-Event passiert nichts. es wird kein Event beim Erzeugen der Komponente gesetzt.

Habe die Erzeugung jetzt in den Konstruktor und das delete von usrHead in den destruktor des TForm verpflanzt; behebt den fehler leider auch nicht!

Ich bin mir recht sicher, dass akaris erster Post bereits die richtige Anwort war.

Die Frage ist, warum Du die Controls überhaupt löschst und neu erzeugst? Notwendig ist dies sicherlich nicht.

Also, die Anwendung soll eine Benutzerverwaltung darstellen, aber nicht jeder Benutzer hat die gleichen Attribute (Daten) und nicht zur gleichen Zeit; das ganz ist dynamisch. Jedesmal wenn ein Benutzer ausgewählt wird erzeuge ich je Attribut ein Instanz von TAttribut (eigene Klasse) mit einem TLabel und entweder einem TEdit, einer TComboBox, ..... oder aber einer TCheckListBox. Deshalb ist das notwendig.

Die Frage ist auch warum das Ganz bei den TEdit, TLabel, TComboBox, usw. keine Probleme macht? ich lösche ja nicht doppelt und die Anwendung steigt beim delete in der eigenen Klasse aus - aber NUR DANN wenn ich eine Checkbox aktiviert habe. Im Debugger ist alles genauso wie ohne Aktivierung einer Checkbox, mit dem Unterschied das ich eine AccessViolation bekomme.

Lösche ich die Komponenten nicht manuell, bekomme ich bei der nächsten benutzerauswahl ein "Eine Komponente mit diesem Namen existiert bereits" - Fehlermeldung.

Bin ja für andere Ansätze offen ........

Manfred

Als Workaround würde ich vielleicht versuchen, das ganze Form jeweils neu zu erstellen, die Controls erzeugen und dann das komplette Form löschen.
Das Form würde ich dann komplett aus der Liste der automatisch erzeugten Formulare herausnehmen und grundsätzlich 'von Hand' erstellen und löschen.

Zumindest Testweise solltest Du das mal machen. Wenn es danach keine Probleme mehr gibt, weißt Du zumindest, dass es an dem manuellen Löschen der Controls liegt.

Möglicherweise wäre auch die Verwendung eines TFrames sinnvoll.