Metatrader Forum | Forex Expert-Advisor | Broker & Forex Tools

Metatrader Forum | Forex Expert-Advisor | Broker & Forex Tools (http://www.expert-advisor.com/forum/index.php)

- Programmierung MQL4 (http://www.expert-advisor.com/forum/forumdisplay.php?f=220)

- - GlobalVariables nicht alle gelöscht (http://www.expert-advisor.com/forum/showthread.php?t=6682)

GlobalVariables nicht alle gelöscht

Ich verstehe die Welt nicht mehr. Ich versuche ein Set von GlobalVariablen zu löschen, der MT macht aber was er grad mal so als lustig empfindet.

Code:

// string PFX="OSCVolDir";

// string GPFX=PFX+"_"+_Symbol+"_";



void DeleteGlobals()

{

   ResetLastError();

   Print(" total Glob vars=",IntegerToString( GlobalVariablesTotal() ),

         " StringLen(",GPFX,")=",IntegerToString( StringLen(GPFX) )    );

   for(int i=0; i<GlobalVariablesTotal(); i++ )

   {

      string glob_name=GlobalVariableName(i);

      if(StringSubstr(glob_name,0,StringLen(GPFX))==GPFX)

      {

         Print(" i=",IntegerToString(i),

               " GlobalVariableName=",GlobalVariableName(i) );

         if( GlobalVariableDel(GlobalVariableName(i)) )

              Print(" i=",IntegerToString(i)," deleted: ",GlobalVariableName(i));

         else Print(" i=",IntegerToString(i)," delete error, code: ",IntegerToString(_LastError));

      }

   }

}

In dem Logfile MQL4/Logs/JJJJMMDD.log wird auch nichts mehr eingetragen :mad::mad: - darum die Meldungen im Anhang.
In der GlobVarList sieht man, daß alle Globals gesetzt wurden, 18 Stück.
In der GlobVarDels wurde die Anzahl richtig erkannt, aber der Durchlauf in der Schleife ist nicht korrekt - dabei sind die Klammern doch eindeutig.

Meiner Meinung nach ist der Code korrekt - oder bin ich etwa schon wieder an allem Schuld ??
Hat jemand einen Kommentar dazu, dann bitte her damit ;) AVT

Probiere es mal mit

if(StringSubstr(glob_name,0,StringLen(GPFX)-1)==GPFX)

traderdoc

Es liegt es daran, dass du die For-Schleife mit 0 anfängst, anstatt bei der Max-Position der externen Variablen.

Ist dann das gleiche Problem, welches du hättest, wenn du z.B. alle Orders Closen willst, aber bei der Ersten anfängst.

Du löschst ja eine globale Variable. Dann nimmt die nächste in der Reihe direkt den Platz der gerade gelöschen ein. Diese fragst du im Schleifendurchlauf aber nicht mehr ab, sondern springst zur nächsten!

Var0 ist gelöscht -> Var1 geht an i =0. for-schleife springt auf i = 1, var2 wird jetzt gelöscht, da var1 ja auf i =0 sitzt, var3 nimmt die Position von Var2 ein usw.

Daher löscht du nur 9 Variablen, obwohl 18 vorhanden waren. Außerdem fragt deine For-Schleife ja auch jedes mal die Anzahl der vorhandenen externen Variablen ab, daher halt auch nur die 9 Durchläufe!

Lösung zum Problem:

Code:

for(int i=GlobalVariablesTotal()-1; i>=0; i--)

Außerdem könntest du deine If-Abfrage sicherer gestalten, wenn du das folgendermaßen machst, zumindest solange der jeweilige SubString nicht nochmal irgendwo auftaucht!

Code:

if(StringFind(glob_name,GPFX,0)>=0)

Zitat:

Zitat von traderdoc (Beitrag 43824)

Probiere es mal mit

if(StringSubstr(glob_name,0,StringLen(GPFX)-1)==GPFX)

traderdoc

Danke für den Vorschlag. Das klappt nicht.
Der Substring ist ja genau 17 Zeichen lang (hab ich auch noch mal manuell nachgezählt) - und wenn ich von 0 beginnend 17 Zeichen haben will (die den GPFX ergeben müssen), dann ist das letzte Zeichen zwar an Position 16 (wegen der Null), aber ich will damit ja nicht weiter arbeiten, sondern nur wissen ob die Buchstabenfolge stimmt, damit ich löschen kann.
Wenn ich die Länge, wie Du vorgeschlagen hast auf 16 kürze, dann entsprechen diese 16 Zeichen nicht mehr dem GPFX (denn der ist 17 lang und somit == nicht erfüllt).
Der Test zeigt, daß alle Variablen noch vorhanden sind.
Aber nochmal danke, daß Du Dir die Zeit genommen hast.:) AVT

Zitat:

Zitat von AVT (Beitrag 43826)

das das nicht klappt wird daran liegen, dass du die Lösung aus einer Antwort nutzen solltest ?!?

Zitat:

Zitat von Indikator-Trading (Beitrag 43825)

Es liegt es daran, dass du die For-Schleife mit 0 anfängst, anstatt bei der Max-Position der externen Variablen.

Danke Dir, Du hast Recht :) mach ich ja bei Orders und anderen Sachen, die eine Gesamtmenge untersuchen ja sonst auch !!

Zitat:

Außerdem könntest du deine If-Abfrage sicherer gestalten, wenn du das folgendermaßen machst, zumindest solange der jeweilige SubString nicht nochmal irgendwo auftaucht!

Code:

if(StringFind(glob_name,GPFX,0)>=0)

Dann sind sie wenigstens weg :D AVT

Zitat:

Zitat von Indikator-Trading (Beitrag 43827)

das das nicht klappt wird daran liegen, dass du die Lösung aus einer Antwort nutzen solltest ?!?

So schnell bin ich nicht :eek:

sorry, dachte du hättest es übersehen, sonst hätte ich auch nicht so forsch geantwortet :o :D