Nutze zurzeit BCB6 und BCB2007 und habe folgendes Problem:
Ich habe ein Shape (als Member in einer Klasse) folgendermaßen deklariert:

TShape  *Shapebewertung[4];

Weiters wird das Shapearray dynamisch erstellt.

Im Konstruktor ändere ich nun die Visibility auf false:

Shape->Visible = false; //Shapebewertung[i] wurde zum Initialisieren an eine Prozedur übergeben

Obiges funktioniert auch, doch jetzt ein Rätsel (zumindest für mich):
In einer Methode der Klasse, ein dynamisches OnClick-Event[u] eines Buttons, will ich nun die Shapes wieder sichtbar machen:

for(int i=0;i<4;i++)
{
	Shapebewertung[i]->Visible = true;
}

Jedoch zeigen mir genau bei dieser Zeile beide Compiler zur Laufzeit dieselbe Expection an:
http://www.imgdumper.com/file/img/2007/dec/30/img/jlnbkhsnbpxd3lisnt8yat5y0.jpg

Ich weiß nicht, was ich falsch mache, da es vorher (oben) schon funktioniert hat! Kann es vielleicht daran liegen, dass es sich bei der Methode um ein OnClick-Event handelt? Und wenn ja, wie kann ich es umgehen (das Problem). Viele meiner Kollegen wussten auch keinen Rat, somit hoffe ich auf Eure Mithilfe!

Ah, und noch eine allgemeine Frage, gibt es einen Grund, dass man z.B. einem dynamischen Label einen Namen (Caption) zuweist?

Bitte helft mir!

Falls es jemand braucht, habe ich hier noch das gesamte Projekt hochgeladen: (BCB6)[400KB]
http://rapidshare.com/files/80102086/bcb6_mastermind.rar
(Laut Kaspersky natürlich virenfrei)

TShape  *Shapebewertung[4];

Shape->Visible = false; //Shapebewertung[i] wurde zum Initialisieren an eine Prozedur übergeben

gibt es einen zusammenhang zwischen diesen beiden code-teilen?

for(int i=0;i<4;i++)
{
	Shapebewertung[i]->Visible = true;
}

Jedoch zeigen mir genau bei dieser Zeile beide Compiler zur Laufzeit dieselbe Expection an:
http://www.imgdumper.com/file/img/2007/dec/30/img/jlnbkhsnbpxd3lisnt8yat5y0.jpg

weißt du was ein debugger ist? damit könntest du sehen, dass mindestens eines der array-elemente 0 ist, wodurch die exception verursacht wird.
mehr ist mit diesen winzigen codefragmenten nicht rauszufinden.

Shape->Visible = false; //Shapebewertung[i] wurde zum Initialisieren an eine Prozedur übergeben

Zeig mal den Kopf dieser Funktion... (Parameter als Referenz übergeben???)

eventuele Konstanten cwidth werden vorher definiert!

In einer Header-Datei "File1.h" erzeuge ich meine Klasse namens Versuch:

class Versuch
{
private: //Membervariablen:
TShape  *Shapebewertung[4];
 //Auswertenbutton: Beim Klicken des Buttons, sollten die Shapes sichtbar gemacht werden <-- funzt nicht!
TButton *Button;

void initial(TShape *Shape,TForm *Form, const int x, const int y, const int width, const bool visibility, const bool typ); 
//die vielen Parameter geben u.a. die standardmäßige Sichtbarkeit (visibility), den Besitzer (Form) und die Position des Shapes an. Diese Prozedur wird im Konstruktor aufgerufen, da es somit feiner ist jedes Shape zu initialisieren

//und weitere Membervariablen, aber hierfür unwichtig...

public:
//Konstruktor:
Versuch(TForm *Form, const int x, const int y,const bool visibility, const int number); //übergeben wird die Startposition des Shapes, sowie die Visibility

//Auswertenbutton:
//Event, welches beim Klicken des Buttons "in Kraft tritt"
void __fastcall Auswerten(TObject *Sender);
}

Konstruktor:

Versuch::Versuch(TForm *Form, const int x, const int y,const bool visibility, const int number)
{
//Shapes initialisieren
	for(int i=0;i<4;i++)
	{
			initial(Shapekreis[i],Form,x+i*abstand,y,cwidth, visibility, true);
//das folgende hat alles seine Richtigkeit so:
//Übergeben der Shapes an die Prozedur "initial"			
if(i<2) initial(Shapebewertung[i],Form,(x+4*abstand+i*abstandklein),(y-cwidth/4),cwidthsmall,false, false);
			else initial(Shapebewertung[i],Form,x+4*abstand+(i-2)*abstandklein,y+cwidth/4,cwidthsmall,false, false);
	}

//weiters wird der Button initialisiert...
}

initial:

void Versuch::initial(TShape *Shape,TForm *Form, const int x, const int y, const int width,const bool visibility, const bool typ)
{
        Shape = new TShape(Form);
        Shape->Parent=Form;
        //Shape->Name = "Shape";
        Shape->Top = y;
        Shape->Left= x;
        Shape->Width=width;
        Shape->Brush->Style=bsSolid;
        //Auf Kreis umstellen:
        Shape->Shape=stCircle;
        //Farbe initialisieren
        change_color(Shape,clWhite);
        Shape->Visible = visibility;
        //da auch noch andere Shapes initialisiert werden, wird ein Parameter  typ übergeben
        if(typ) Shape->OnMouseDown=ShapeKlicken;
}//initial()

Auswerten:

void __fastcall Versuch::Auswerten(TObject *Sender)
{
	//Bewertungsshapes sichtbar machen:
	for(int i=0;i<4;i++)
	{
		Shapebewertung[i]->Visible = true;
		//Show();
	}
}

Aufruf in Form1 OnCreate <-- BesitzerForm:

for(int i=0;i<anzahl;i++)
        {
                //Nur dann sichtbar, wenn es der erste Versuch ist (true)
                if(!i) versuch[i]=new Versuch(Form1,xstart,ystart+(i*50),true,i);
                else versuch[i]=new Versuch(Form1,xstart,ystart+(i*50),false,i);
        }

Das war das ganze "Wunder", nur funktioniert das eben die Umstellung der Sichtbarkeit beim Klicken der Auswertebutton nicht und ja ich weiß auch was ein Debugger ist (ein kleines bisschen) aber dieser nützt mir hier (noch) nichts, weil ich einfach noch zu wenig Erfahrung damit habe (bin gerade erst neu im c-Builder und habe für meine vorherigen Programme kaum ein Debugger gebraucht. Und sorry, aber woher weißt du, dass mindestens ein Shape Null ist, etwa aus dem Bild (sorry für die blöde Frage, aber aus Fehlern wird man schlau)
Und wo kann nun in diesem Code ein Element 0 sein, ich habe doch alles dynamisch erstellt das einzige was ich nicht gemacht habe, ist den einzelnen Shapes Namen zugewiesen, könnte es vielleicht daran liegen?

Hoffe, dass das das Problem besser schildert! Vieleicht ist mir auch beim Kopieren der eine oder andere Fehler unterlaufen, bitte das zu entschuldigen
Und danke für Eure raschen Antworten!

void Versuch::initial(TShape *Shape,TForm *Form, const int x, const int y, const int width,const bool visibility, const bool typ)
{
        Shape = new TShape(Form);

das tut nicht das, was du vermutest. Shape ist eine lokale variable innerhalb der funktion initial(), das new hat außerhalb der funktion keine auswirkung, außer einem memleak.

du willst vielleicht das:

void Versuch::initial(TShape *&Shape,[...])

also eine referenz auf den shape-pointer übergeben.

o&oler schrieb:

void Versuch::initial(TShape *Shape,TForm *Form, const int x, const int y, const int width,const bool visibility, const bool typ)
{
        Shape = new TShape(Form);

das tut nicht das, was du vermutest. Shape ist eine lokale variable innerhalb der funktion initial(), das new hat außerhalb der funktion keine auswirkung, außer einem memleak.

du willst vielleicht das:

void Versuch::initial(TShape *&Shape,[...])

also eine referenz auf den shape-pointer übergeben.

Vielen Dank für deine Antwort, habs gerade ausprobiert und funktioniert spitze (endlich!), aber nur noch so, schauen ob ich alles kapiert hab:
Ich übergeben hier initial([...]) nur einen Zeiger auf ein Shape, dieser wird dann dynamisch erzeugt (in initial) und wenn ich die visibility auf true setze auch angezeigt. Nach Verlassen von initial([...]) entsteht nur ein Memoryleak, jedoch bleiben die Shapes immer noch angezeigt und deshalb waren auch im Debugger die Shapes 0
Jetzt nur noch die etwas blöde Frage:
Woran habt ihr im Debugger erkannt, dass das Shape nicht (richtig) erstellt wurde)

Aber jedenfalls danke für Eure rasche und sehr gute Mithilfe!!!
mfg. Helli

Im Debugger hättest du es gesehen, wenn du dir einen Breakpoint auf den Aufruf von 'initial' gesetzt hättest und nach Ausführen der Funktion (Step: F8) den Inhalt von 'Shapebewertung[i]' angesehen hättest, der hat sich nämlich bei deinem vorherigen Code nicht verändert.

Besser wäre aber folgende Funktion:

TShape* TMyForm::CreateShape(TForm *pForm, ...)
{
  TShape *pShape = new TShape(this);
  pShape->Parent = pForm;
  // ...

  return pShape;
}

// Aufruf

Shapebewertung[i] = CreateShape(this, ...);

Außerdem solltest du deinen Code dahingehend umschreiben, daß du die Funktionen nicht mehrfach mit verschiedenen Parametern aufrufst, sondern zuerst die Parameter bestimmst und dann die Funktion genau einmal aufrufst (ist übersichtlicher und eleganter):

for(int i=0; i<anzahl; i++)
{
  bool bVisible = !i; //Nur dann sichtbar, wenn es der erste Versuch ist (true)

  versuch[i] = new Versuch(Form1, xstart, ystart+(i*50), bVisible, i);
}

Außerdem solltest du deinen Code dahingehend umschreiben, daß du die Funktionen nicht mehrfach mit verschiedenen Parametern aufrufst, sondern zuerst die Parameter bestimmst und dann die Funktion genau einmal aufrufst (ist übersichtlicher und eleganter)

Schon erledigt und danke nochmal für die Erklärung mit dem Debuggen, werde fleißig weiterschreiben an meinem Spiel und wenn ich mal wieder ein Problem habe, melde ich mich wieder (-;

Dannke!