//---------------------------------------------------------------------------

#include <vcl.h>
#pragma hdrstop
#include "IdBaseComponent.hpp"
#include "IdComponent.hpp"
#include "IdHTTP.hpp"
#include "IdTCPClient.hpp"
#include "IdTCPConnection.hpp"
#include "sms_unit.h"
#include "fstream.h"
#include <iostream>
//---------------------------------------------------------------------------
#pragma package(smart_init)
#pragma resource "*.dfm"
TForm1 *Form1;

int AddSize;
int len;
int x;
int Size = 1;
String* adds;
String* Searchword;
String Meldung;

//---------------------------------------------------------------------------
__fastcall TForm1::TForm1(TComponent* Owner)
        : TForm(Owner)
{

}
//---------------------------------------------------------------------------

void __fastcall TForm1::Button1Click(TObject *Sender)
{

         Meldung = Searchword[5];
}

void __fastcall TForm1::FormCreate(TObject *Sender)
{
//TIdHTTP *IdHTTP1 = new TIdHTTP(NULL); // Instanz erstellen
IdHTTP1->HandleRedirects=true;

fstream File2("List.dat", ios::in|ios::binary);
if (File2.is_open())
{
Box->Items->Add("List.dat wurde geöffnet");
   // cout << "List.dat has been opened" << endl << endl;
    File2.read((char*)&Size, sizeof(Size));
   // cout << "The Searchwords are:" << endl << endl;
    }
    String* Searchword = new  String[Size+1];
    if (File2.is_open())                       //ändern!!!!!
{

    for(int i=0;i < Size; i++)
    {
        File2.read((char*)&len, sizeof(len));
        char* buffer = new char[len+1];
        File2.read(buffer,len);
        buffer[len] = '\0';
        Searchword[i] = buffer;
        delete[] buffer;
      //  cout << Searchword[i].c_str() << endl;
    }

    }
 File2.close();

 fstream File3("adds.dat", ios::in|ios::binary);
if (File3.is_open())
{
Box->Items->Add("adds.dat wurde geöffnet");
    File3.read((char*)&AddSize, sizeof(AddSize));
    }
    String* adds = new  String[AddSize+1];
    if (File3.is_open())                       //ändern!!!!!
{

    for(int i=0;i < AddSize; i++)
    {
        File3.read((char*)&len, sizeof(len));
        char* buffer = new char[len+1];
        File3.read(buffer,len);
        buffer[len] = '\0';
        adds[i] = buffer;
        delete[] buffer;
    //    cout << adds[i].c_str() << endl;
    }

    }
 File3.close();
}
//---------------------------------------------------------------------------

Das Problem ist das wenn ich auf den Button drücke
die Meldung kommt "Zugriffsverletzungen bei Adresse ..."
Sind meine Zeiger nicht mehr gültig??
Wenn ja wie kann ich das beheben?
Ich will nicht dass die Dateien bei jedem druck auf den Knopf geladen werden, sondern nur einmal am Anfang...

Mit Zeigern muss ich arbeiten da nicht von vornerein feststeht wie groß die Datei ist...

Könnt ihr mir helfen!

Wäre echt sehr dankbar!!!!!!!!!

Schonmal vielen Dank!

MfG Straightflush

is sone Windows MessageBox...

Aber hier: Meldung = Searchword[5];

muss der Fehler liegen, bzw tritt er auf, da die Meldung erst kommt wenn man auf den Knopf drückt

es scheint irgendwas mit der gültigkeit der zeiger tun zu haben, da wenn ich die Dateiöffnung auch in die Methode von Button1 setze es keine Probleme gibt...

auch wenn ich versuche den Wert von Searchword[0] zu übergeben geht es nicht

der Debugger meldet ja garkeine Probleme

Diese Aussage disqualifizert sie. Denn sie zeigt das du noch nicht mal begriffen hast, was ein Debugger eigentlich macht. Schade. Ich dachte immer man liest links, die deinem als HIlfe angeboten werden.

Straightflush schrieb:

insofern gibt es auch keine Zeile es reicht auch schon aus wenn man nur die kompilierte *.exe ausführt...

Aha... und das hat ja bestimmt nichts mit dem Quelltext zu tun? Tip: Startest du das Programm in der IDE, bleibt die Anwendung nach dem Klick auf OK stehen und das was du dann siehst mit vielen Farben, etc. ist der Debugger. Damit lässt sich auch rausfinden wo "sone Windows MessageBox" her kommt. Denn entgegen vielleicht deinem Verdacht ist die Meldung nicht von Zauberhand auf den Bildschrim gelangt sondern die Generierung muss ebenfalls im Programm stattfinden. Windows hat damit aber gar nix zu tun.

Ansonsten hat Linnea wohl recht. Willkommen in der Hölle der globalen Variablen. Das Problem ist einfach zu lösen: Erklär mir bitte wieso du "String* Searchword" global benötigst. In einem zweiten Schritt würde ich gerne wissen, wieso du eine erneute Variablendeklaration machst, wenn du doch nur Speicher für Searchword reservieren willst.

Ahja und noch eine empfehlung: Verwende doch eine Stringlist statt ein String-Array für Searchword. Das vereinfacht das Handling erheblich.

[edit="junix"]ahja, ich vergass: Bei Verwendung eines Debuggers lässt sich übrigens ein NULL-Zeiger wunderbar rausfinden.
Und noch ein Tipp am Rande: Wer mit zeigern hantiert sollte sie zum Einen bei der Deklaration immer mit NULL initialisieren (selbiges gilt für deletes) und zum Anderen vor der Verwendung auf NULL prüfen. das hilft dann solche üblen Spiele wie das was du grad mitmachst zu verhindern.[/edit]

Damit lässt sich auch rausfinden wo "sone Windows MessageBox" her kommt.

Aber wie soll bitte der Debugger mir zeigen woher die Fehlermeldung kommmt??
Tut mir ja leid wenn ich voll aufm Schlauch stehe aber in finde einfach nix derartiges...