In der TForm lade ich Dateien und lege Daten von diesen in Variablen ab und möchte diese Variablen auch in den TFrame Quelltexten verwenden.

Normal lege ich in der Unit1.cpp eine Variable int XYZ an und in der Frame.cpp sage ich dann include die Unit1.h und extern int XYZ.
So klappts auf jedenfall.

Jetzt dachte ich, ach haust doch die Variablen direkt in die Unit1 Header, dann benötigst kein extern mehr, aber nun erhalte ich schon über 10.000 Warnungen vom Compiler und er rattert immernoch, das "Public symbol 'XYZ' defined in both module ....."

Muss man die extern-Schiene fahren oder habe ich einen Fehler gemacht?
Globale Konstanten muss ich jedenfalls im Header deklarieren, warum gehen Variablen dann nicht?

junix sein Beitrag versteh ich leider nur ansatzweise, bissle verständliche Quelltextbeispiele wären da sicherlich nicht deplaziert.

WebFritzi sein Beitrag ist für mein Problem auch nicht sinnvoll, ich arbeite ja mit den Variablen in den einzelnen Frames, verändert dort diese.

Also bleibts wohl bei globalen Variablen mit extern. (mist alles zurückmodeln)

So mach ichs momentan

TForm1 *Form1;

int X = 5;

//---------------------------------------------------------------------------
__fastcall TForm1::TForm1(TComponent* Owner)
    : TForm(Owner)
{
}
//---------------------------------------------------------------------------
void __fastcall TForm1::Button1Click(TObject *Sender)
{
 Caption = IntToStr(X);
}
//---------------------------------------------------------------------------

TForm2 *Form2;

extern int X;

//---------------------------------------------------------------------------
__fastcall TForm2::TForm2(TComponent* Owner)
    : TForm(Owner)
{
}
//---------------------------------------------------------------------------
void __fastcall TForm2::Button1Click(TObject *Sender)
{
 Caption = IntToStr(X);
 X = 6; // Form1->Caption würde nun 6 nach Form1->Button1Click() zeigen
}
//---------------------------------------------------------------------------

Ich würde nun gern auf X in TForm2 zugreifen wollen ohne dies extern erst bekannt machen zu müssen.

Wie er es gesagt hat :

Lege einfach 2 Dateien an, "globals.cpp" und "globals.h" ...
In der globals.h musst du auch das Schlüsselwort "extern" verwenden.

Builder-spezifisch heißt das eine neue Unit anlegen (enthält .cpp und .h) und im Header alle globalen Variablen deklarieren (mit extern) und in der cpp alle Variablen implementieren (ohne extern)

Aber auch ich rate dir von globalen Variablen ab, Membervariablen sind sicher besser geeignet. Überdenk dein Konzept.

bis bald
akari

Aufgrund des Beispiels (Berdi) hatte ich angenommen, dass du "hier und da" mal eine Variable für die globale Verwendung deklarierst. Man geht bei solch einem Stiel kaputt: Wurde denn eine Variable X schon in Unit xyz angelegt? Welche von beiden verwendet die Unit abc ....

Ich setze auch auf Membervariablen (und hasse diese Bezeichnung;)), aber in der Realität und in übernommenen Projekten kommt es öfter mal vor, dass es globale Variablen gibt. Sie sind aus der Programmierung nicht wirklich wegzudenken – andere Diskussion.
TForm1 * Form1; // == globale Variable

Wenn man also wenigstens eine eigene Unit für die globalen Variablen verwendet, sinkt die Fehleranfälligkeit erheblich.

thomas69 schrieb:

Ich setze auch auf Membervariablen (und hasse diese Bezeichnung;)), aber in der Realität und in übernommenen Projekten kommt es öfter mal vor, dass es globale Variablen gibt. Sie sind aus der Programmierung nicht wirklich wegzudenken – andere Diskussion.

Meistens fasse ich solche globalen Werte/Objekte aber in einem Singleton zusammen.

TForm1 * Form1; // == globale Variable

Das ist aber grad ein schlechtes Beispiel. Ich schalte in meinen Builder-Projekten immer alle automatischen Formgenerierungen aus (ausser das Main-Form) und benutze auch diese Pointer nicht. Alle anderen Dialoge werden dynamisch angelegt.

bis bald
akari

Mit Membervariablen meinst Klassenvariablen?

Berdi schrieb:

Mit Membervariablen meinst Klassenvariablen?

Genau

bis bald
akari