EA setzt keine Trades ab
 

Hallo zuammen,

ich habe (versucht) einen Indikator in einen EA zu wandeln, aber er setzt einfach keine Trades ab. Habe ich irgendwo einen grundelgeneden Fehler den ich übersehe?

Wäre um jede Hilfe dankebar.

Gruß Gondiego

Ohne den Code vollständig seziert zu haben, fallen diese beiden Zeilen auf, da stimmt so einiges nicht:

OrderSend(OP_BUY,Lot,MarketInfo(Symbol(), MODE_ASK),2,StopLoss,Ask-TakeProfit*Point,Lime);

OrderSend(OP_SELL,Lot,MarketInfo(Symbol(), MODE_BID),2,StopLoss,Bid-TakeProfit*Point,Red);

1. keine Angabe des Symbols
2. bei StopLoss müsste der Kurs des StopLoss angegeben werden, was ich bezweifle, da diese Variable so in den Eigenschaften geschrieben steht
3. beim TakeProfit der BuyOrder würde der TP unter!! dem Open der Order liegen - das geht natürlich gar nicht.

Besser wäre es, sich an die Vorgabe der Dokumentation zu halten:

int OrderSend(
string symbol, // symbol
int cmd, // operation
double volume, // volume
double price, // price
int slippage, // slippage
double stoploss, // stop loss
double takeprofit, // take profit
string comment=NULL, // comment
int magic=0, // magic number
datetime expiration=0, // pending order expiration
color arrow_color=clrNONE // color
);

traderdoc

Hallo traderdoc,

danke für deine Antwort. Ja, das sind dann so Sachen, die sieht man nicht mehr wenn man zu lange davor sitzt. ;)

Aber....auch durch ändern der Zeilen tut sich nichts...selbst wenn ich sie ohne Verweise schreibe.

OrderSend(Symbol(),OP_BUY,0.1,MarketInfo(Symbol(), MODE_ASK),2,0,0,NULL,0,0,Green);

OrderSend(Symbol(),OP_SELL,0.1,MarketInfo(Symbol() , MODE_BID),2,0,0,NULL,0,0,Red);


Ich habe einfach mal den ursprünglichen Indikator mitangehangen, vielleicht hilft das.

Gondiego

OK, dann muss jetzt jemand ran, der viel Zeit hat, um den Fehler zu finden.

traderdoc

*Hust...

Hab auch mal kurz einen Blick auf deinen Code im 1 Post geworfen:

Da fehlt der Wert.

X ist nirgendwo definiert.

Leere Funktion, die du aber weiter unten mit dem Wert "0" aufrufst.

Im ersten Durchlauf der Schleife ist die Variable "PREV" 0. Der Code geht also an der blau hervorgehobenen Stelle weiter.
Dort hast du aber einen Fehler (rot hervorgehoben), denn "nPips" initialisierst du oben mit der nicht vorhandenen Variable "x", somit
wird die Variable "TrStopLevel" auch nicht "optimal" belegt.

• Fehler 1:
  
  Code:

  ---

  #property indicator_buffers 4
#property indicator_chart_window

  ---

  ===>>> Gehört nicht in einen EA.

• Fehler 2:
  
  Code:

  ---

  ...
double  ExtHistoBuffer[];
double  ExtHistoBuffer2[];
double  BlueBar[],RedBar[];
...

  ---

  ===>>> In einem EA sind das einfach nur dynamische Arrays, OHNE Bindung zu einem "Indicator-Buffer" und somit auch OHNE "automatischer Größenverwaltung".
  Zu diesem Zeitpunkt haben diese Arrays keine wirkliche größe.

• Fehler 3:
  
  Code:

  ---

  void SetIndexValue(int shift, double value)
{
  ExtHistoBuffer[shift] = value;
}

void SetIndexValue2(int shift, double value)
{
  ExtHistoBuffer2[shift] = value;
}

  ---

  ===>>> 1. Diese beiden "Buffer" sind, wie gerade schon erwähnt normale, dynamische Arrays, momentan ohne Größe.
  Den Index "shift" gibt es somit nicht.
  ===>>> 2. Hier würdest du einfach nur einen Wert in das Array setzen. Dieses Array ist jedoch nicht an einen Indikator-Buffer gebunden.
  Diese Arrays werden also nicht "automatisch" für irgendetwas genutzt.

• Fehler 4:
  
  Code:

  ---

  BlueBar[shift]=Low[shift];
RedBar[shift]=High[shift];
SetIndexValue2(shift, TrStopLevel);

  ---

  ===>>> Auch hier setzt du lediglich Werte in ein dynamisches Array, welches zu diesem Zeitpunkt noch nichtmal eine Größe hat.
  Der Index "shift" ist also auch hier nicht existent.

Ich frag mich auch grad, ob du beim Compilieren überhaupt keine Errors und Warnungen erhalten hast... ?

Jetzt aber schlafen...

Na siehst Du, @Gondiego, hat sich doch schon jemand gefunden, der einfach zu viel Zeit hat.

traderdoc

Stimmt. Wenn ihr meinen Namen unter den "Onlineusern" mal nicht seht, das ist ein Bug den Master noch nicht behoben hat.
Bin 24/7 präsent...

Spaß bei Seite, mein Post waren die letzten Minuten vorm Schlafen gehen. Da hab ich die mal schnell noch sinnvoll genutzt.

Hallo,

erstmal vielen Danke @next user für deine Mühe.

Das mit den x ist mir schon bewusst, das habe ich einfache weggelassen. Sollte ja darum gehen, ob es grundlegende Fehler im Aufbau gibt - was du ja letztendlich festgestellt hast. ;)

Aber dafür stecke ich einfach zu wenig in der Materie :(

Kann mir denn jetzt jemand sagen, was ich tun muss, damit er als EA einfach nur eine Order, bei Auftreten der Pfeile, absetzt?
(Vielleicht hilft ja der Ursprungscode des Indis, welchen ich mit angehangen habe)

Wenns garnicht anders geht, bin ich auch bereit, das auf Honorarbasis machen zu lassen. (wenns nicht zu teuer wird ;-P )

Gondiego

Hey Doc, biste da :D

Der @doc ist da, steckt aber zu tief in anderen Aufträgen.
Übernimm Du mal.

traderdoc