void Suche_Positionen(AnsiString String,AnsiString Zeichen,int *Positionen)
{
//......
Positionen[j] = i;
//......
}

main()
{
//...
int i;
int *z;
i = z[0]
//....
}

Zugriffsverweigerung

Schöne Bezeichnung.
Dein z ist ja auch nicht gültig, da du keinen Speicher allokiert hast. Du hast doch new schon erwähnt, solltest also wissen wie das geht.

oder besser gleich std::vector benutzen.

bis bald
akari

,und da das ein Zeiger ist (Positionen) bezieht er sich doch darauf.

Zum einen war dieser Ausschnitt nicht schon im ersten Post zu sehen. Zum Andern sagt er auch nicht viel aus, zum Beispiel ob j zur Laufzeit wirklich größer 1 ist und ob der Inhalt von Positionen über z wirklich an Suche_Positionen gegeben wird. Daalso noch immer nicht der relevante und komplette Ablauf für uns zu sehen ist, können wir dazu nicht viel sagen, außer das du den Debugger benutzen sollst und/oder std::vector.

bis bald
akari

/Edit : Okay langsam wird klarer.

int *z; // z wird nie initialisert.
Suche_Positionen("12i456i890i","i",z);
i=z[0];

Du müßtest schon Suche_Positionen mit der Adresse von z, und bei der Funktion selber mit einer Pointer auf Pointer arbeiten. Oder mit einer Referenz auf Pointer.
Das ganze Gefuddle kannst du dir aber sparen wenn du std::vector verwendest (Ja das ist schon der dritte Hinweis...)

bis bald
akari

grundsätzlich

void __fastcall TForm1::Button1Click(TObject *Sender)
{
        //Zum Testen
        int i;
        int *z;
        Suche_Positionen("12i456i890i","i",z);
        i=z[0];
}

void Suche_Positionen(AnsiString String,AnsiString Zeichen,int *&Positionen) // Referenz auf Pointer
{
  ...
        Positionen = new int[j-1];
  ...
}

Allerdings erstellst du durch das new ein Element zu wenig, und außerdem must du ja das Suchen zweimal machen.

Deshalb solltest du das ganze lieber so machen

#include <vector> // Am Anfang der Datei
...
void __fastcall TForm1::Button1Click(TObject *Sender)
{
        //Zum Testen
        int i;
        std::vector<int> z;
        Suche_Positionen("12i456i890i","i",z);
        if (!z.empty())
        {
          i=z[0];
        }
}

void Suche_Positionen(AnsiString String,AnsiString Zeichen,std::vector<int>& Positionen)
{
        int i=0
        while (i != 0)
        {
          i = Suchen(String,Zeichen,i+1);
          if (i!=0) Positionen.push_back(i);
        }
}

Desweiteren weiß ich ja nicht was die Funktion Suche() machen, aber hoffentlich etwas anders als AnsiString::Pos

bis bald
akari