Danke für die Antwort Traderdoc. Es hat mir geholfen und ich hab es so umgesetzt. Er setzt jetzt schon zuerst beim Kreuzen die richtige Order - aber nach dem Teke Profit setzt er gleich die nächste Order ohne, dass tatsächlich eine Kreuzung stattgefunden hat (siehe Bild). Da kann ja nicht sein.
Beim kompilieren kommen aber nur noch Wrnhinweise 'implicit conversion from string to number'. Aber das kann doch nicht der Grund sein oder doch?

Wenn nach dem Schließen durch TP/SL die Öffnungsbedingungen
immer noch gegeben sind... Poste mal Bilder zum besseren Verständnis.

Hallo MA-EA - danke für die Antwort. Ja ich glaube es könnte daran liegen. Ich probiere schon etwas jetzt dort zu modifizieren. Also es ist auffällig, dass immer direkt nach dem TP sofort wieder die gleiche Ordereröffnet wird. Die Bedingungen dazu bestehen zu der Zeit noch (siehe Bild). Nur wie erwähnt ich bin erst Anfänger und deswegen ist das bei mir noch nicht so schnell umgesetzt.

hier sind noch mal meine Bedingungen im Code:

if (LongSignal == false && ShortSignal == false)
{
// Handelssignale ermitteln
if(NeuePeriodeBegonnen == true)
{
// Buy Signal
if(MA_lang < MA_kurz && MA_lang_Vorperiode >= MA_kurz_Vorperiode)
{
LongSignal = true;
}
else LongSignal = false;

// Sell Signal
if(MA_lang > MA_kurz && MA_lang_Vorperiode <= MA_kurz_Vorperiode)
{
ShortSignal = true;
}
else ShortSignal =false;
}
else
{
LongSignal = false;
ShortSignal = false;
}

Mit dem TP ist das so ne Sache. Er wird nach diesem immer
wieder ne neue Order öffnen, solange entsprechende Bedingungen
gegeben sind. Versuch vielleicht z.B. nen Buy ohne TP zu schließen
wenn

Close Bar 1 < MA_kurz.


Buy öffnen wenn

Close Bar 1 > MA_kurz && MA_kurz > MA_lang

Umgedreht mit Sells. Dann kann er mal verschnaufen bzw.
Seitwärtsphasen auslassen.

Wobei ich immer noch nicht ganz kapiere, was das mit der
Vorperiode soll. :confused: Kannst Du das mal mit Chart-Bildern erklären?

Zitat:

Zitat von MA-EA (Beitrag 35908) Mit dem TP ist das so ne Sache. Er wird nach diesem immer wieder ne neue Order öffnen, solange entsprechende Bedingungen gegeben sind. Versuch vielleicht z.B. nen Buy ohne TP zu schließen wenn Close Bar 1 < MA_kurz. Buy öffnen wenn Close Bar 1 > MA_kurz && MA_kurz > MA_lang Umgedreht mit Sells. Dann kann er mal verschnaufen bzw. Seitwärtsphasen auslassen. Wobei ich immer noch nicht ganz kapiere, was das mit der Vorperiode soll. :confused: Kannst Du das mal mit Chart-Bildern erklären?

Mit der Vorperiode meine ich die Periode, die genau vor der anderen Vergleichsperiode kommt - deswegen ist bei Vorperiode der Shift auf 2 gesetzt und bei den anderen auf 1 (siehe unten in rot)


double MA_lang = iMA(NULL,0,MA_Periode_lang,0,0,0,1);
double MA_lang_Vorperiode = iMA(NULL,0,MA_Periode_lang,0,0,0,2);
double MA_kurz = iMA(NULL,0,MA_Periode_kurz,0,0,0,1);
double MA_kurz_Vorperiode = iMA(NULL,0,MA_Periode_kurz,0,0,0,2);


Im Bild sieht man dass die sich da genau kreuzen - nach dem TP wird sofort die neue Order eröffnet (siehe 2. Bild). Das komische ist aber, dass doch dort garnicht die gesamte Bedingung erfüllt ist also Der rote Teil ist nicht erfüllt aus meiner Sicht:

// Buy Signal
if(MA_lang < MA_kurz && MA_lang_Vorperiode >= MA_kurz_Vorperiode)

schon komisch :)

Ich habe jetzt nur kurz drübergelesen über deinen Code, aber auf den ersten Blick würde ich sagen, dass du dein LongSignal / ShortSignal zurücksetzen musst.

Du checkst die Bedingung mit
if (LongSignal == false && ShortSignal == false)
{....}
und setzt nur dort drinnen die Signale zurück.
Wenn du aber bereits LongSignal=true hast, ein Trade eröffnet wird, dieser per TP geschlossen wird, kommt sofort der nächste Trade da LongSignal=true ist. (noch immer)

hab auch nur über den thread geschaut, aber ich bin mir sicher dass Susanne recht hat.

Wenn immer wieder neue Order eröffnet werden, dann nur weil die EntrySignale nicht zurück gesetzt werden oder die Variable einfach falsch ist. Glaub damit hat jeder Neuling zu kämpfen, kenn das auch ganz gut. Versuchs mal so:

so fragst du die Einstiegssignale nur einmal pro neuer Kerze ab, was bei Kreuzungen ja auch eher sinn macht. So sparst du dir auch das rücksetzten der Signale. Must oben eben nur noch BarCount initialisieren oberhalb von start(). Bedeutet du setzt einfach folgende Zeile hin : int BarCount =0 ;

Es gibt zich Tutorials zu C++. Würd dir empfehlen erst mal Basics zu lernen bevor du weiter machst, denn dann tust du dich nicht so schwer, das hält doppelt und dreifach auf.Wenn man nicht weiß wofür int, double oder string steht,dann macht das echt keinen Sinn. Bin auch kein Profi also quereinsteiger und tuh mich oft trotz einigem Wissen noch schwer. Gelernte Programmierer haben es eben leichter :o

LG