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)
-   -   EA: Pseudo Stoploss und Takeprofit (http://www.expert-advisor.com/forum/showthread.php?t=6893)

noSkill06s 21.01.21 22:52

EA: Pseudo Stoploss und Takeprofit
 
Liste der Anhänge anzeigen (Anzahl: 3)
Sehr häufig kämpft man damit das der gewünschte Stoploss (meistens) oder Takeprofit vom Broker verneint wird, weil er zu eng ist, natürlich kann man per Funktion "MarketInfo(Symbol(),MODE_STOPLEVEL);" die vom Broker verlangte bzw. gewünschte Stop levels abfragen, jedoch habe ich eine andere Idee verfolgt, was wenn ich überhaupt keinen Stoploss oder Takeprofit an den Broker weiterleite und im EA selber den Kurs ständig mit meinem Pseudo Stoploss und Takeprofit abgleiche? ich habe mit dieser Funktion schon bereits erfolg, der Grund für den Post ist das er manchmal funktioniert und manchmal nicht. Mein Frage ist jetzt könntet ihr bitte mal einen Blick auf den Code werfen warum er manchmal nicht funktioniert? kann es vielleicht sein das es daran liegt das ich keine "automatic generated" magic numbers verwende? funktion "GetLastError();" wirft "0" aus was so ziemlich überhaupt nicht weiterhilft.
HTML-Code:

//+------------------------------------------------------------------+
//|                                          breakOut_TRF_v4.0.0.mq4 |
//|                        Copyright 2021, MetaQuotes Software Corp. |
//|                                            https://www.mql5.com |
//+------------------------------------------------------------------+
#property copyright "Copyright 2021, MetaQuotes Software Corp."
#property link      "https://www.mql5.com"
#property version  "1.00"
#property strict
#include <stderror.mqh>
#include <stdlib.mqh>

int StopLoss=3;
int TakeProfit=20;
int Slippage=2;
int Magic=280456;
double myPoint;
int countBar=0;
double TP=0,SL=0,TR;
//+------------------------------------------------------------------+
//| Expert initialization function                                  |
//+------------------------------------------------------------------+
int OnInit()
  {
//--- 

//---
  return(INIT_SUCCEEDED);
  }
//+------------------------------------------------------------------+
//| Expert deinitialization function                                |
//+------------------------------------------------------------------+
void OnDeinit(const int reason)
  {
//---
 
  }
//+------------------------------------------------------------------+
//| Expert tick function                                            |
//+------------------------------------------------------------------+
void OnTick()
  {
//---
  CheckForSignal();
  }
//+------------------------------------------------------------------+
//| Function "CheckForSignal()"                                      |
//+------------------------------------------------------------------+
void CheckForSignal(){
  static datetime candletime=0;
  if(candletime!=Time[0]){
      double upArrow=iCustom(Symbol(),PERIOD_CURRENT,"breakOut_TRF_v4.0.0",0,1);
      if(upArrow != EMPTY_VALUE){
        buyEnterTrade(OP_BUY,0.1,Ask,2,0,0);
      }
      double downArrow=iCustom(Symbol(),PERIOD_CURRENT,"breakOut_TRF_v4.0.0",1,1);
      if(downArrow != EMPTY_VALUE){
        sellEnterTrade(OP_SELL,0.1,Bid,2,0,0);
      }
      candletime=Time[0];
  }
}
//+------------------------------------------------------------------+
//| buyEnterTrade function                                          |
//+------------------------------------------------------------------+
void buyEnterTrade(int type,double volumen,double price,int slippage,double stoploss,double takeprofit){
  int longTicket=OrderSend(Symbol(),type,volumen,price,slippage,stoploss,takeprofit,"Funktion: buyEnterTrade",Magic,0,clrGreen);
  stopLoss();
}
//+------------------------------------------------------------------+
//| sellEnterTrade function                                          |
//+------------------------------------------------------------------+
void sellEnterTrade(int type,double volumen,double price, int slippage,double stoploss, double takeprofit){
  int shortTicket=OrderSend(Symbol(),type,volumen,price,slippage,stoploss,takeprofit,"Funktion: sellEnterTrade",Magic,clrRed);
  stopLoss();
}
//+------------------------------------------------------------------+
//| stopLoss function                                          |
//+------------------------------------------------------------------+
void stopLoss()
  {
  double MyPoint=Point;
  if(Digits==3 || Digits==5) MyPoint=Point*10;
  TP=TakeProfit*MyPoint;
  SL=StopLoss*MyPoint;

  double OrdP=0,OrdTP=0,OrdSL=0;
  for(int i=0; i<OrdersTotal(); i++)
    {
      if(OrderSelect(i,SELECT_BY_POS,MODE_TRADES))
        if(OrderMagicNumber()==Magic && Symbol()==OrderSymbol())
          {
            OrdP=OrderProfit()-MathAbs(OrderSwap())-MathAbs(OrderCommission());
            OrdSL=(-1)*SL*OrderLots()*MarketInfo(OrderSymbol(),MODE_TICKVALUE)/Point;
            OrdTP=TP*OrderLots()*MarketInfo(OrderSymbol(),MODE_TICKVALUE)/Point;
            int error=0;
            Print("Checking 'OrdP>OrdTP OR OrdP<OrdSL'",Symbol());
            if(OrdP>
OrdTP || OrdP<OrdSL)
              {
              if(OrderType()==OP_BUY){
                  bool OrdClP=OrderClose(OrderTicket(),OrderLots(),Bid,Slippage,clrYellow);
              }else{
                  error=GetLastError();
                  Print("Long OrderClose() error: "+(string)error+" "+ErrorDescription(error));
              }
                 
              if(OrderType()==OP_SELL){
                  bool OrdClL=OrderClose(OrderTicket(),OrderLots(),Ask,Slippage,clrYellow);                 
              }else{
                  error=GetLastError();
                  Print("Short OrderClose() error: "+(string)error+" "+ErrorDescription(error));
              }
              }
          }
    }
  }
//+------------------------------------------------------------------+


Indikator-Trading 21.01.21 23:09

Ich konnte jetzt den Code nur schnell per Handy überfliegen, aber was mir auffällt, ist das du Point in der Berechnung nutzt obwohl du myPoint vorher anpasst

Indikator-Trading 21.01.21 23:15

Das deine Anpassungen der SL Werte nicht angenommen wurden, wird aber an der gesetzen slippage liegen, welche mit 2 denkbar klein ist. Was mich etwas wundert, dass du den Swapwert mit betrachtest, wenn du so enge SSL Werte setzen willst. Swap hättest du ja nur, wenn du deine Position über Nacht halten wollen würdest. Gruß Timo

traderdoc 22.01.21 20:34

Ohne den Indikator zu kennen und das Programm laufen gesehen, kann ich folgendendes kurz dazu sagen:
1. Eine Aktion findet immer nur nach Kerzenwechsel statt und dass auch nur für die Dauer genau eines Ticks und auch nur dann, wenn die Indikatorwerte nicht gleich EMPTY_VALUE entsprechen.
2. D.h. wiederum, wenn die Indikatorwerte gleich EMPTY_VALUE entsprechen, werden weder Orders geöffnet (das wäre ja noch klar), aber auch keine offenen Orders gemanaged., weil in die Funktion stopLoss() nur dann eingesprungen wird, wenn in die Funktion buyEnterTrade() bzw. sellEnterTrade() eingesprungen wird.
Und in die beiden Funktionen wird wiederum nur eingesprungen, wenn die Indikatorwerte nicht gleich EMPTY_VALUE sind.
3. Somit word es sehr häufig passieren, dass der Profit während des Kerzenaufbaus über dem errechneten TakeProfit in Währung bzw. unter dem StopLoss in Währung liegt.
4. Mit der gravierendste Fehler liegt aber in diesen beiden Zeilen:

Code:

OrdSL=(-1)*SL*OrderLots()*MarketInfo(OrderSymbol(),MODE_TICKVALUE)/Point;
OrdTP=TP*OrderLots()*MarketInfo(OrderSymbol(),MODE_TICKVALUE)/Point;

Das Point hat da nichts zu suchen.

Einfaches Beispiel für EURUSD auf $-Kontowährung:

OrdTP = 100*1*1/0.00001
OrdTP = 10.000.000$

Ohne!! Point:

OrdTP = 100*1*1
OrdTP = 100$

D.h. bei einem TP von 100 (das sind dann 10 Pips) und einer Lotgröße von 1 ergibt das 100$, weil ja 1 Pip bei 1 Lot auch 10$ Profit entspricht.

So, und nun mach was draus.

traderdoc

noSkill06s 24.01.21 14:26

Danke für deine Antwort, könntest du nochmal einen Blick drüber werfen habe es jetzt mal aufgerufen.

HTML-Code:

//+------------------------------------------------------------------+
//|                                          breakOut_TRF_v4.0.0.mq4 |
//|                        Copyright 2021, MetaQuotes Software Corp. |
//|                                            https://www.mql5.com |
//+------------------------------------------------------------------+
#property copyright "Copyright 2021, MetaQuotes Software Corp."
#property link      "https://www.mql5.com"
#property version  "1.00"
#property strict

extern int      hidden_sl    = 10;
extern  int    hidden_tp    = 20;
int magicNumber = 123456;
//+------------------------------------------------------------------+
//| Expert initialization function                                  |
//+------------------------------------------------------------------+
int OnInit()
  {
//--- 

//---
  return(INIT_SUCCEEDED);
  }
//+------------------------------------------------------------------+
//| Expert deinitialization function                                |
//+------------------------------------------------------------------+
void OnDeinit(const int reason)
  {
//---
 
  }
//+------------------------------------------------------------------+
//| Expert tick function                                            |
//+------------------------------------------------------------------+
void OnTick()
  {
//---
  //
  CheckForSignal();
  //
  stealthControl();
  }
//+------------------------------------------------------------------+
//| Function "CheckForSignal()"                                      |
//+------------------------------------------------------------------+
void CheckForSignal(){
  static datetime candletime=0;
  if(candletime!=Time[0]){
      double upArrow=iCustom(Symbol(),PERIOD_CURRENT,"breakOut_TRF_v4.0.0",0,1);
      if(upArrow != EMPTY_VALUE){
        buyEnterTrade(OP_BUY,1.0,Ask,100,0,0);
      }
      double downArrow=iCustom(Symbol(),PERIOD_CURRENT,"breakOut_TRF_v4.0.0",1,1);
      if(downArrow != EMPTY_VALUE){
        sellEnterTrade(OP_SELL,1.0,Bid,100,0,0);
      }
      candletime=Time[0];
  }
}
//+------------------------------------------------------------------+
//| buyEnterTrade function                                          |
//+------------------------------------------------------------------+
void buyEnterTrade(int type,double volumen,double price,int slippage,double stoploss,double takeprofit){
  int longTicket=OrderSend(Symbol(),type,volumen,price,slippage,stoploss,takeprofit,"Funktion: buyEnterTrade",magicNumber,0,clrGreen);
}
//+------------------------------------------------------------------+
//| sellEnterTrade function                                          |
//+------------------------------------------------------------------+
void sellEnterTrade(int type,double volumen,double price, int slippage,double stoploss, double takeprofit){
  int shortTicket=OrderSend(Symbol(),type,volumen,price,slippage,stoploss,takeprofit,"Funktion: sellEnterTrade",magicNumber,0,clrRed);
}
//+------------------------------------------------------------------+
//| stopLoss function                                          |
//+------------------------------------------------------------------+
void stealthControl(){

    double myPoint=Point();
    if(Digits()==5 || Digits()==3){
      myPoint *=10;
    }
    int totalorders = OrdersTotal();
    for(int i=totalorders-1;i>=0;i--){
      OrderSelect(i, SELECT_BY_POS);
      bool result = false;
      if ( OrderSymbol()==Symbol()){
        //profitTargeting
        if (OrderType() == OP_BUY &&  OrderOpenPrice()+hidden_tp*myPoint<=Bid )  result = OrderClose( OrderTicket(), OrderLots(), MarketInfo(OrderSymbol(), MODE_BID),10, Red );
        if (OrderType() == OP_SELL &&  OrderOpenPrice()-hidden_tp*myPoint>
=Ask  )  result = OrderClose( OrderTicket(), OrderLots(), MarketInfo(OrderSymbol(), MODE_ASK), 10, Red );
        //looseTargeting
        if (OrderType() == OP_BUY &&  OrderOpenPrice()-hidden_sl*myPoint>=Bid )  result = OrderClose( OrderTicket(), OrderLots(), MarketInfo(OrderSymbol(), MODE_BID), 10, Red );
        if (OrderType() == OP_SELL &&  OrderOpenPrice()+hidden_sl*myPoint<=Ask  )  result = OrderClose( OrderTicket(), OrderLots(), MarketInfo(OrderSymbol(), MODE_ASK), 10, Red );
      }
  }
  return;
}
//+------------------------------------------------------------------+


noSkill06s 24.01.21 14:27

Zitat:

Zitat von Indikator-Trading (Beitrag 45006)
Das deine Anpassungen der SL Werte nicht angenommen wurden, wird aber an der gesetzen slippage liegen, welche mit 2 denkbar klein ist. Was mich etwas wundert, dass du den Swapwert mit betrachtest, wenn du so enge SSL Werte setzen willst. Swap hättest du ja nur, wenn du deine Position über Nacht halten wollen würdest. Gruß Timo

danke für deine Antwort habe, den Code mal jetzt aufgeräumt könntest du mal einen Blick darauf werfen :)

MA-EA 25.01.21 11:00

Zitat:

Zitat von noSkill06s (Beitrag 45015)
extern int hidden_sl = 10;
extern int hidden_tp = 20;

Dürfte zu klein sein. Bei so manchen WP sind Ask/Bid wesentlich größer als der TP, bei vielen viel größer als der SL.

Indikator-Trading 25.01.21 11:54

Falls der EA noch nicht laufen sollte, hier ist ein Punkt, welcher evtl. ein Problem auslösen könnte:

Code:

if(OrderType() == OP_BUY &&  OrderOpenPrice()+hidden_tp*myPoint<=Bid)
Klammersetzung wird hier leider von vielen nicht so wichtig gesehen, kann aber bei falscher/keiner Anwendung zu Fehlern führen, welche nicht so schnell gefunden werden können.

Schreib das mal so:
Code:

if((OrderType() == OP_BUY) &&  ((OrderOpenPrice()+hidden_tp*myPoint)<=Bid))
Gruß Timo

noSkill06s 25.01.21 18:24

vielen Dank für die Antwort, aber der EA läuft bzw. der hidden Stoploss und Takeprofit funktionieren perfekt :) nehme deine Anregung mit in meine Muster Dokumentation mit der Klammer :)

Für die nachfolgenden die genau das gleiche Problem haben einfach meinen letzten "code" als Muster nehmen mehr oder weniger einfach durch eigenen "iCustom" austauschen.

danke an alle die mir geholfen haben.

noSkill06s 25.01.21 18:25

Zitat:

Zitat von noSkill06s (Beitrag 45015)
Danke für deine Antwort, könntest du nochmal einen Blick drüber werfen habe es jetzt mal aufgerufen.

HTML-Code:

//+------------------------------------------------------------------+
//|                                          breakOut_TRF_v4.0.0.mq4 |
//|                        Copyright 2021, MetaQuotes Software Corp. |
//|                                            https://www.mql5.com |
//+------------------------------------------------------------------+
#property copyright "Copyright 2021, MetaQuotes Software Corp."
#property link      "https://www.mql5.com"
#property version  "1.00"
#property strict

extern int      hidden_sl    = 10;
extern  int    hidden_tp    = 20;
int magicNumber = 123456;
//+------------------------------------------------------------------+
//| Expert initialization function                                  |
//+------------------------------------------------------------------+
int OnInit()
  {
//--- 

//---
  return(INIT_SUCCEEDED);
  }
//+------------------------------------------------------------------+
//| Expert deinitialization function                                |
//+------------------------------------------------------------------+
void OnDeinit(const int reason)
  {
//---
 
  }
//+------------------------------------------------------------------+
//| Expert tick function                                            |
//+------------------------------------------------------------------+
void OnTick()
  {
//---
  //
  CheckForSignal();
  //
  stealthControl();
  }
//+------------------------------------------------------------------+
//| Function "CheckForSignal()"                                      |
//+------------------------------------------------------------------+
void CheckForSignal(){
  static datetime candletime=0;
  if(candletime!=Time[0]){
      double upArrow=iCustom(Symbol(),PERIOD_CURRENT,"breakOut_TRF_v4.0.0",0,1);
      if(upArrow != EMPTY_VALUE){
        buyEnterTrade(OP_BUY,1.0,Ask,100,0,0);
      }
      double downArrow=iCustom(Symbol(),PERIOD_CURRENT,"breakOut_TRF_v4.0.0",1,1);
      if(downArrow != EMPTY_VALUE){
        sellEnterTrade(OP_SELL,1.0,Bid,100,0,0);
      }
      candletime=Time[0];
  }
}
//+------------------------------------------------------------------+
//| buyEnterTrade function                                          |
//+------------------------------------------------------------------+
void buyEnterTrade(int type,double volumen,double price,int slippage,double stoploss,double takeprofit){
  int longTicket=OrderSend(Symbol(),type,volumen,price,slippage,stoploss,takeprofit,"Funktion: buyEnterTrade",magicNumber,0,clrGreen);
}
//+------------------------------------------------------------------+
//| sellEnterTrade function                                          |
//+------------------------------------------------------------------+
void sellEnterTrade(int type,double volumen,double price, int slippage,double stoploss, double takeprofit){
  int shortTicket=OrderSend(Symbol(),type,volumen,price,slippage,stoploss,takeprofit,"Funktion: sellEnterTrade",magicNumber,0,clrRed);
}
//+------------------------------------------------------------------+
//| stopLoss function                                          |
//+------------------------------------------------------------------+
void stealthControl(){

    double myPoint=Point();
    if(Digits()==5 || Digits()==3){
      myPoint *=10;
    }
    int totalorders = OrdersTotal();
    for(int i=totalorders-1;i>=0;i--){
      OrderSelect(i, SELECT_BY_POS);
      bool result = false;
      if ( OrderSymbol()==Symbol()){
        //profitTargeting
        if (OrderType() == OP_BUY &&  OrderOpenPrice()+hidden_tp*myPoint<=Bid )  result = OrderClose( OrderTicket(), OrderLots(), MarketInfo(OrderSymbol(), MODE_BID),10, Red );
        if (OrderType() == OP_SELL &&  OrderOpenPrice()-hidden_tp*myPoint>
=Ask  )  result = OrderClose( OrderTicket(), OrderLots(), MarketInfo(OrderSymbol(), MODE_ASK), 10, Red );
        //looseTargeting
        if (OrderType() == OP_BUY &&  OrderOpenPrice()-hidden_sl*myPoint>=Bid )  result = OrderClose( OrderTicket(), OrderLots(), MarketInfo(OrderSymbol(), MODE_BID), 10, Red );
        if (OrderType() == OP_SELL &&  OrderOpenPrice()+hidden_sl*myPoint<=Ask  )  result = OrderClose( OrderTicket(), OrderLots(), MarketInfo(OrderSymbol(), MODE_ASK), 10, Red );
      }
  }
  return;
}
//+------------------------------------------------------------------+


für nachfolgende User mit dem gleichen Problem dieser Code funktioniert


Alle Zeitangaben in WEZ +1. Es ist jetzt 01:36 Uhr.

Powered by vBulletin® Version 3.8.5 (Deutsch)
Copyright ©2000 - 2024, Jelsoft Enterprises Ltd.
SEO by vBSEO 3.6.1
Powered by vBCMS® 2.7.0 ©2002 - 2024 vbdesigns.de
Copyright ©2009 - 2023 by Expert-Advisor.com - Das Metatrader Forum