Von Modulen geteilte Variable?

08/31/2012 13:54 Mi4uric3#1
Hai,
Ich hab mir c++ so ohne Buch oder sonstwas ein wenig angeignet, deshalb ist mein Coding-Stil (wahrscheinlich) recht miserabel..

Ich versuche zur Zeit eine Variable aus mehreren Modulen auszulesen/zu verändern. Klappt jedoch nicht so wie ich will. Ich lasse ein Modul eine Patternsuche machen. Das Ergebnis davon soll in eine Variable geschrieben werden, die von anderen Modulen aus erreichtbar ist.

Informationen vorab:
Quote:
Compiler: Borland C++
Entwicklungsumgebung: Embarcadero C++ Builder XE2
Beispiel-Projekt:
Quote:
FindPattern.cpp
FindPattern.h
Hack.cpp
Hack.h
Main.cpp
Main.h
Values.h
Aufbau des Values-Moduls:
Code:
namespace Values {
    DWORD Addresse1 = 0;
}
Erklärung:
Das Modul "Main" ruft im Modul "FindPattern" eine PatternSuch-Funktion auf. Diese Funktion schreibt das Ergebnis dann in die Variable "Addresse1" im "Values"-Modul. (Ich habs mit ner MessageBox getestet, das Ergebnis ist != 0)
Dann ruft das Modul "Main" im "Hack"-Modul eine Funktion auf, die wiederum auf die Variable "Addresse1" aus dem "Values"-Modul zugreift, sagt aber dann, dass "0" drinsteht.

c++ Builder gibt mir auch eine Warnung aus:
Quote:
[ILINK32 Warnung] Warning: Public Symbol 'Values::Addresse1' ist in Modul PROJEKT\WIN32\DEBUG\HACK.OBJ und PROJEKT\WIN32\DEBUG\MAIN.OBJ definiert
(Das "Main"-Modul includiert die 2 anderen Module).


Wie kann ich das jetzt so machen, dass die Variable für beide Module erreichbar ist (dass die die selbe Value teilen) ..?

Danke im Voraus!
08/31/2012 14:20 nkkk#2
du musst die varaiable als extern makieren (also nur deklarieren) nur in einer datei nicht (in der definierst sie dann also).

also üblicherweise würdest du ein der Values.h stehen haben:
Code:
extern int value1;
extern char value2;
..
und in der Values.cpp:
Code:
int value1;
char value2;
..
08/31/2012 14:22 Mi4uric3#3
Danke, das probier ich mal aus :)

#Edit:

Das funktioniert sogar, vielen Dank!
Jetzt bin ich dieses blöde Problem endlich los und beim Compilen kommen keine Warnungen mehr :)

Danke ^-^

Closerequest ist gestellt :)
08/31/2012 15:01 Nightblizard#4
Das klingt nach einem Designfehler deinerseits. Globale Variablen sollte man wenn möglich vermeiden.
08/31/2012 15:06 Mi4uric3#5
Quote:
Originally Posted by Nightblizard View Post
Das klingt nach einem Designfehler deinerseits. Globale Variablen sollte man wenn möglich vermeiden.
Warum?
Ich hab doch erklärt, worfür sie da sind (FindPattern).
08/31/2012 16:35 Nightblizard#6
Wenn dein FindPattern auf globale Variablen getrimmt wurde, dann ist da was falsch. Kann auch sein, dass ich deine Erklärung missverstehe.

Für mich klingt das einfach so, als wolltest du folgendes machen:
Code:
namespace Values //Warum du namespaces "Module" nennst, ist mir auch nicht so ganz klar
{
     adresse1 = 0;
}

namespace Hack
{
     void foo()
     {
          doSomething(Values::adresse1);
     }
}

namespace FindPattern
{
     void findPattern1()
     {
          Values::adresse1 = implFindPattern("\xB\x0\x0\xB\x5", 0, UINT_PTR(-1));
     }
}

namespace Main
{
     void foo()
     {
          FindPattern::findPattern1();
          Hack::foo();
     }
}

So kommt das für mich rüber und das ist einfach nur unschön und ganz und gar nicht flexibel.
Code:
namespace Hack
{
     void foo(UINT_PTR address)
     {
          doSomething(address);
     }
}

namespace FindPattern
{
     UINT_PTR findPattern(const std::string& pattern, UINT_PTR begin, UINT_PTR end)
     {
          //...
          return address;
     }
}

namespace Main
{
     void foo()
     {
          auto address = FindPattern::findPattern("\xB\x0\x0\xB\x5", 0, UINT_PTR(-1));
          Hack::foo(address);
     }
}
Das ist kürzer und sehr viel flexibler als das da oben und kommt ganz ohne globale Variablen aus.
08/31/2012 17:18 MrSm!th#7
Mal so als Frage in den Raum geworfen, da ich da selbst etwas unsicher bin:

Wie siehts mit static aus? Würde das nicht ebenso den Zweck erfüllen? Ungeachtet der Tatsache, dass es schlechter Stil ist.
08/31/2012 17:48 Mi4uric3#8
Quote:
Originally Posted by Nightblizard View Post
Wenn dein FindPattern auf globale Variablen getrimmt wurde, dann ist da was falsch. Kann auch sein, dass ich deine Erklärung missverstehe.

Für mich klingt das einfach so, als wolltest du folgendes machen:
Code:
*Siehe Original-Post
So kommt das für mich rüber und das ist einfach nur unschön und ganz und gar nicht flexibel.
Code:
*Siehe Original-Post
Das ist kürzer und sehr viel flexibler als das da oben und kommt ganz ohne globale Variablen aus.
Ist nicht sonderlich praktisch, da ich dann jedes mal FindPattern aufrufen müsste, um wieder auf die Adresse zuzugreifen. Da such ich sie lieber nur 1x, speichere sie global und greif dann immer wieder darauf zu, ist wesentlich performanter.

Das mit den Namespaces mache ich, damit ich die Übersicht bewahre.
08/31/2012 17:57 .SkyneT.#9
Quote:
Originally Posted by MrSm!th View Post
Mal so als Frage in den Raum geworfen, da ich da selbst etwas unsicher bin:

Wie siehts mit static aus? Würde das nicht ebenso den Zweck erfüllen? Ungeachtet der Tatsache, dass es schlechter Stil ist.
Afaik sollte das funktionieren, durch die Speicherklasse "static" wird
der Speicher ja erst am Ende des Programms freigegeben, also sollte
es möglich sein darauf zuzugreifen.
(aber kann mir vorstellen das es da mal Probleme geben könnte,
besonders wenn man dann noch mehrere Threads verwendet)
08/31/2012 18:24 nkkk#10
also static bei globalen variablen bedeutet eigentlich das man nicht von anderen dateien darauf zugreifen kann (also eben nicht wie im beispiel oben mit extern), das globale variablen erst bei programmende freigegeben werden ist immer so, auch ohne static.

das es erst bei programmende freigegeben wird spielt nur bei static variablen innerhalb von funktion eine rolle.
08/31/2012 18:27 xNopex#11
Mit static begrenzt du die Sichtbarkeit einer globalen Variable auf ihr Modul. Enstpricht also nicht dem was der TS wollte.

EDIT: Ich werd zu alt und langsam..
08/31/2012 19:25 Nightblizard#12
Quote:
Originally Posted by Mi4uric3 View Post
Ist nicht sonderlich praktisch, da ich dann jedes mal FindPattern aufrufen müsste, um wieder auf die Adresse zuzugreifen. Da such ich sie lieber nur 1x, speichere sie global und greif dann immer wieder darauf zu, ist wesentlich performanter.

Das mit den Namespaces mache ich, damit ich die Übersicht bewahre.
Ich denke was du suchst sind Klassen.

Du solltest uns zumindest grob sagen was du vor hast, damit man dir spezifische Tipps geben kann. Z.B. solltest du dir Behavior Trees anschauen (zumindest den Gedanken dahinter), falls du einen Bot programmieren möchtest.
08/31/2012 19:29 Mi4uric3#13
Quote:
Originally Posted by Nightblizard View Post
Ich denke was du suchst sind Klassen.

Du solltest uns zumindest grob sagen was du vor hast, damit man dir spezifische Tipps geben kann. Z.B. solltest du dir Behavior Trees anschauen (zumindest den Gedanken dahinter), falls du einen Bot programmieren möchtest.
Nun ja, ich hab doch gesagt, was ich mache.
Ich such ne Adresse, welche dann von anderen Funktionen in unterschiedlichen Modulen (für die ganz pingeligen, ich meine die .cpp Dateien) genutzt wird. Und das mit vielen Adressen.
Außerdem bin ich doch vollkommen zufrieden. Die 1. Antwort hat mir zu 100% geholfen, das war genau das, was ich wissen wollte.