Zitat:

(Objekts+0)->Pointer->Objekt0
(Objekts+1)->Pointer->Objekt1
(Objekts+2)->Pointer->Objekt2
usw...

Zitat:

void killObjekt(char pName[20])
{
for (int i=0; i<Objektcount; i++)
{
if ((*(Objekts+i))->compareNames(pName))
{
Objekt *Kill=*(Objekts+i);
Objekt **Temp=new Objekt* [Objektcount--];
int j=0;
for (int k=0; k<(Objektcount+1); k++)
{
if (*(Objekts+k)==Kill)
{
j=1;
}
else
{
*(Temp+(k-j))=*(Objekts+k);
}
}
Objekts=new Objekt* [Objektcount];
for (j=0; j<Objektcount; j++)
{
*(Objekts+j)=*(Temp+j);
}
delete []Temp;
delete Kill;
GUI->Objektscroller->Items->Delete(GUI->Objektscroller->Items->IndexOf(GUI->Objektscroller->Text));
GUI->Objektscroller->Text="Neues Objekt anlegen";
GUI->edName->Text="";
GUI->edX->Text="";
GUI->edY->Text="";
GUI->edVx->Text="";
GUI->edVy->Text="";
GUI->edMass->Text="";
break;
}
}
}

Zitat von xMATADORx:

...bist du nicht aufm TG in Ehingen?

Zitat von Polaris:

Bevor ich ausführlich Antwort geben kann, wären ein paar Erläuterungen zu dem Code nicht schlecht, denn der sieht sehr konfus und unlogisch teilweise aus ...

kleine Zusatzfragen:
Was bedeutet löschen bei dir?
Das eine Objekt vom Speicher entfernen und einfach an der Stelle ein neues Objekt anlegen??
(btw. Wenn du C++ verwendest, warum nimmst du nicht std::string, sondern ein char-Array?)

Zitat von Polaris:

Also ausführliche Antwort.
Mir sind ein paar Dinge an deinem Code aufgefallen:

1. Du mischst bei der direkten Verwalten den Zugriff mittels Pointern auf die Objekte (iteratoren) und mittels Indizes - das führt zu einem ziemlich undurchsichtigen Code.
2. Anstatt (*(objekts + i)) würde ich an deiner stelle immer objekts[i] schreiben, das erhöht die lesbarkeit doch deutlich und führt zu einer Vermeidung vieler unnötiger Klammern.


Direkt nachvollziehen kann ich den Code jetzt leider nicht, dazu fehlt mir gerade schlicht die Zeit bei dem Wirrwarr ...
for (int k=0; k< (Objektcount+1); k++) {
//...
}
Objekts=new Objekt* [Objektcount];
/* !!! !hier erzeugst du ein Speicherleck, weil der alte Speicher von Objekts nie freigegeben wird !!!!*/
for (j=0; j<Objektcount; j++) {
//...
}


ich habe aber mal ein Snippet kurz gemacht, dass den von dir gewünschten Zweck erfüllen sollte - das kannst du dir in Ruhe anschauen und dann versuchen von selber deine Fehler zu finden.

void killObjekt(std::string pName) {
bool ObjektKilled = false;
for(unsigned i = 0; i < Objektcount && !ObjektKilled; ++i) {
if(Objekts[i]->getName() == pName) {
// kannst natürlich auch mit char-array und deiner Comparefunktion arbeiten
Objekt ** Temp = new Objekt * [Objektcount - 1];
for(unsigned j=0; j < Objektcount-1; ++j) {
if(j < i)
Temp[j] = Objekts[j];
else
Temp[j] = Objekts[j+1];
}
for(unsigned j = 0; j < Objektcount; ++j)
{ delete Objekts[i]; }
delete [] Objekts;
Objekts = Temp;
GUI->Objektscroller->Items->Delete(GUI->Objektscroller->Items->IndexOf(GUI->Objektscroller->Text));
GUI->Objektscroller->Text="Neues Objekt anlegen";
GUI->edName->Text="";
GUI->edX->Text="";
GUI->edY->Text="";
GUI->edVx->Text="";
GUI->edVy->Text="";
GUI->edMass->Text="";
ObjektKilled = true;
}
}
}

Also ich hoffe ich habe keine Fehler reingemacht, bin auch gerade nicht mehr ganz da.
Und ich bete dafür, dass der Zuweisungsoperator von Objekt von dir korrekt überladen ist, denn sonst wirst du ziemlich schnell ziemlich viele Speicherlecks und undefiniertes Verhalten haben.

Zitat:

void Control::killObjekt(char pName[20])
{
Objekt **Temp=new Objekt*[Objektcount--];
int j=0;
for (int i=0; i<(Objektcount+1); i++)
{
if (Objekts[i]->compareNames(pName))
{
j=-1;
delete Objekts[i];
}
else
{
*(Temp+(i+j))=*(Objekts+i);
}
delete (Objekts+i);
}
Objekts=Temp;
GUI->Objektscroller->Items->Delete(GUI->Objektscroller->Items->IndexOf(GUI->Objektscroller->Text));
}

Zitat:

bool Objekt::compareNames(char pName[20])
{
for (unsigned int i=0; i<StrLen(pName); i++)
{
if (pName[i]!=Name[i])
{
return false;
}
}
return true;
}

Zitat von uwebaier:

mit strcmp gibt es bei meiner Funktion gar keine Treffer mehr...

Um dir zu sagen wie meine Strings entstehen:

Die Info liegt in
a) einem Editfeld beim erstellen (Ansistring)
b) in einer Combobox ebenfalls als Ansistring
beide Ansistrings werden durch funktion c_str()
also String.c_str() umgewandelt...

Ich melde mich später wieder, jetz grad hab ich anderes zu tun =)