ich führe beim Klick auf eine ListBox folgenden Code aus

void __fastcall TForm1::mediaListClick(TObject *Sender)
{
        char * mediaListVal ;
        //char mediaListVal ;
        String mList = mediaList->Items->Text;
        mediaListVal = new char[mList.Length()];
        strcpy(mediaListVal,mList.c_str());

        String id = IntToStr(GetIdFromCompleteName(mediaListVal));

        MyQuery1->SQL->Clear();
        MyQuery1->SQL->Add("Select vl.verliehenam as 'verliehen am', vl.zurueckam as 'zurück am' , bn.benutzername as 'verliehen an' from verleih as vl left outer join benutzer as bn on bn.benutzer_id = vl.benutzer_id where vl.medium_id='" + id + "' order by vl.verliehenam");
        MyQuery1->Execute();
        grd_history->DataSource = DataSource1;
        mediaListVal = NULL;
}

int GetIdFromCompleteName(char completeName[])
{
     char * posOfSearchString = strrchr(completeName,' ');
     int id = atoi(posOfSearchString);
     posOfSearchString = NULL;
     return id;
}

Das funktioniert unmittelbar noch Programmstart auch...
So, nun klicke ich ein zweites Mal auf die ListBox und dann bekomme ich eine "Zugriffsverletztung bei Adresse ....","Prozess wird angehalten"...

Ich versteh nicht ganz warum???
Bin über jede Hilfe dankbar.

1. Bei chars mußt Du ein Zeichen mehr reservieren, als der String lang ist (für das abschließende \0).

2. Gibst Du nirgendwo den char wieder frei (delete[] mediaListVAL).

3. Einfacher wäre es mit:
char* blablabla = AnsiString::c_str();
Dabei mußt Du dann nicht mal den Speicher für den char* mit new reservieren. Und auch nicht wieder freigeben.

Ich versteh nicht ganz warum???

Dann solltest du schnellstenst den Debugger anwerfen und erstmal herausfinden, in welcher Zeile der Fehler auftritt.

Davon abgesehen ist die Umwandlung in char* wie auch die ganze GetIdFromCompleteName-Funktion relativ fragwürdig, weil völlig unnötig. Das wollte ich dir eigentlich schon in dem diesbezüglichen Thread sagen.

String mList = mediaList->Items->Text; 
int pos = mList.LastDelimiter(" ");
String id = mList.SubString(pos + 1, mList->Length() - pos);

Et voila!

String mList = mediaList->Items->Text; 
int pos = mList.LastDelimiter(" ");
String id = mList.SubString(pos + 1, mList->Length() - pos);

Et voila!

Naja, leider bekomme ich hier jetzt

[C++ Fehler] Unit1.cpp(52): E2288 Zeiger auf Struktur auf linker Seite von -> oder von ->* erforderlich*

3. Einfacher wäre es mit:
char* blablabla = AnsiString::c_str();
Dabei mußt Du dann nicht mal den Speicher für den char* mit new reservieren. Und auch nicht wieder freigeben.

Eher ein Irrglaube... lies doch schnell den Hinweis in der FAQ zu dem Thema...
Der Ansatz war absolut richtig.

-junix

danke, tut mir leid.

Bin eigentlich C#-Coder, so dass ich solche Feinheiten sehr gerne übersehe...

Du glaubst gar nicht, was du mir durch diese Funktion abgenommen hast. DANKE!!!!

Mein Code ist dadurch um einiges kürzer geworden. Super...

Joe_M. schrieb:

3. Einfacher wäre es mit:
char* blablabla = AnsiString::c_str();
Dabei mußt Du dann nicht mal den Speicher für den char* mit new reservieren. Und auch nicht wieder freigeben.

Eher ein Irrglaube... lies doch schnell den Hinweis in der FAQ zu dem Thema...
Der Ansatz war absolut richtig.

Das sehe ich anders. Laut Hilfe ist der Zeiger bis zum Ende der Anwendung gültig. Natürlich kann der Zeiger ungültig werden, aber in der gegebenen Routine bezweifle ich das.

Wenn man sowas im Windows Quellcode finden würde wäre vollkommen klar, warum das Teil sich manchmal von der blauen Seite zeigt :p

läßt sich nicht compilieren, wenn dann müßte man schon so schreiben:

char blablabla = AnsiString().c_str();*

Dadurch wird aber kein Speicherplatz allokiert, woher soll er auch wissen wieviel benötigt wird. Kopiert man in so einen String/Pointer was rein geht das oft auch gut da dann warscheinlich nichts kritisches im Speicher überschrieben wird. Macht man aber z.B. sowas:

memset(blablabla, 'x', 1000);

läuft das Programm aller Voraussicht nach nicht mehr ganz so geschmeidig.

Für Dich dann im Klartext:

AnsiSting asTemp;
char* pcTemp;

pcTemp = asTemp.c_str();

Dieser Zeiger darf natürlich nur und ausschließlich für Leseoperationen verwendet werden. Aber mehr war in dem Thread ja auch nicht verlangt. Er wollte den char* auswerten nicht ihn modifizieren.
Um die Länge eines chars zu ermitteln gibt es dann auch noch strlen(), aber wie gesagt, das ist ein anderes Thema.

Gruß
skydiver1983

Hakt doch bitte nach, wenn ihr was nicht versteht. Es nützt euch doch gar nichts, wenn ihr nur abtippselt was andere posten. Schließlich wollt ihr doch was lernen, oder nicht?