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)
-   -   Heiken Ashi (http://www.expert-advisor.com/forum/showthread.php?t=6144)

traderdoc 26.06.18 15:00

Evtl. solltestst Du doch mal andere Buffer benutzen, z.B. 0 und 1.

FX4Life 26.06.18 22:09

Leider geht es auch nicht mit der 0&1



Gesendet von iPhone mit Tapatalk

MA-EA 26.06.18 22:21

Kann er damit eigentlich was anfangen?

"Heiken_Ashi_Smoothed"

Soviel ich weiß, darf nur der korrekte Name des HA eingetragen werden. :confused:


Edit:Gibts eigentlich Fehlermeldungen oder Warnungen?

traderdoc 26.06.18 22:21

Das geht mit Sicherheit:

Code:

double val0, val1;
int HA1_Kerze = 0;

val0 = NormalizeDouble(iCustom(Symbol(), 0, "Heiken_Ashi_Smoothed", HA_MaMethod, HA_MaPeriod, HA_MaMethod2, HA_MaPeriod2, 0, 1), Digits);
val1 = NormalizeDouble(iCustom(Symbol(), 0, "Heiken_Ashi_Smoothed", HA_MaMethod, HA_MaPeriod, HA_MaMethod2, HA_MaPeriod2, 1, 1), Digits);
 
if (val0 < val1) {
  HA1_Kerze = 1;      //blauer HA
} else {
  HA1_Kerze = -1;      //roter HA
}

Ansonsten liegt es wo anders in Deinem Code.

traderdoc

PS: "Kann er damit eigentlich was anfangen?

"Heiken_Ashi_Smoothed"

Soviel ich weiß, darf nur der korrekte Name des HA eingetragen werden."

Das ist der korrekte Name!!

FX4Life 29.06.18 13:37

Hi,

ich habe es mit allem möglichen probiert und es will einfach nicht laufen.

1. Es passt nicht zu meinem Indikator-Bild (Heiken_Ashi_Smoothed). Ich habe den Indikator in den Tester (als template) eingebaut, aber das was ich sehe und was der EA macht, das sind zwei verschiedene Welten.

2. Es werden nur buy orders ausgelöst.

Hier einmal der gesamt Code

Code:

//+------------------------------------------------------------------+
//|                                                      ProjectName |
//|                                      Copyright 2012, CompanyName |
//|                                      http://www.companyname.net |
//+------------------------------------------------------------------+
#property version  "1.00"
#property strict

double total,ticket;
double Lots=1.0;
int sl =80;
int tp = 200;
double TrailingStop=150;

double val0,val1;
int HA1_Kerze=0;
int  Digits();
//+------------------------------------------------------------------+
//| Expert initialization function                                  |
//+------------------------------------------------------------------+
int OnInit()
  {

  val0 = iCustom(Symbol(), 0, "Heiken_Ashi_Smoothed", 2,4,2,1,0,1);
  val1 = iCustom(Symbol(), 0, "Heiken_Ashi_Smoothed", 2,4,2,1,1,1);

  if(val0<val1)
    {
      HA1_Kerze=1;      //blauer HA
        } else {
      HA1_Kerze=-1;      //roter HA
    }
  return(INIT_SUCCEEDED);
  }
//+------------------------------------------------------------------+
//| Expert deinitialization function                                |
//+------------------------------------------------------------------+
void OnDeinit(const int reason)
  {

  }
//+------------------------------------------------------------------+
//| Expert tick function                                            |
//+------------------------------------------------------------------+
void OnTick()
  {
//---
//| Prüfen der Anzahl der Orders
//+------------------------------------------------------------------+
  total=OrdersTotal();
  if(total==0)
      //| Buy
      //+------------------------------------------------------------------+
    {
      if(HA1_Kerze==1)
        {
        ticket=OrderSend(Symbol(),OP_BUY,Lots,Ask,3,Ask-sl*Point,Ask+tp*Point,NULL,0,0,clrGreen);
        if(ticket>0)
          {
            if(OrderSelect(ticket,SELECT_BY_TICKET,MODE_TRADES));
            Print("BUY order opend : ",OrderOpenPrice());
          }
        else Print("Error opening BUY Order : ",GetLastError());
        }

      //| Sell
      //+------------------------------------------------------------------+
      if(total==0)
        if(HA1_Kerze==-1)
          {
            ticket=OrderSend(Symbol(),OP_SELL,Lots,Bid,3,Bid+sl*Point,Bid-tp*Point,NULL,0,0,clrRed);
            if(ticket>0)
              {
              if(OrderSelect(ticket,SELECT_BY_TICKET,MODE_TRADES));
              Print("Sell order opend : ",OrderOpenPrice());
              }
            else Print("Error opening Sell Order : ",GetLastError());

          }
    }
//+------------------------------------------------------------------+
//---Zähler & Close Bedingungen
//+------------------------------------------------------------------+
for(int cnt=0;cnt<total;cnt++)
    {
      OrderSelect(cnt,SELECT_BY_POS,MODE_TRADES);
      if(OrderType()<=OP_SELL && OrderSymbol()==Symbol())
        {
        if(OrderType()==OP_BUY)
          {
            //Check for Trailling Stop
            //+------------------------------------------------------------------+
            if(TrailingStop>=0)
              {
              if(Bid-OrderOpenPrice()>Point*TrailingStop)
                {
                  if(OrderStopLoss()<Bid-Point*TrailingStop)
                    {
                    OrderModify(OrderTicket(),OrderOpenPrice(),Bid-TrailingStop*Point,OrderTakeProfit(),0,Green);
                    return;
                    }
                }
              }
          }
        else // got to short Position
          {
            //Check for TrailingStop
            //+------------------------------------------------------------------+
            if(TrailingStop>0)
              {
              if((OrderOpenPrice()-Ask)>(Point*TrailingStop))
                {
                  if(OrderStopLoss()>(Ask+Point*TrailingStop))
                    {
                    OrderModify(OrderTicket(),OrderOpenPrice(),Ask+Point*TrailingStop,OrderTakeProfit(),0,Red);
                    return;
                    }
                }
              }
          }
        //+------------------------------------------------------------------+
        } // "else" Close
      //+------------------------------------------------------------------+
    } //"for" Close
//+------------------------------------------------------------------+
  } //"void" Close
//+------------------------------------------------------------------+

Ich bin für jede weitere Unterstützung dankbar!

traderdoc 29.06.18 14:33

Die Indikatorabfrage gehört auf jeden Fall in die Funktion OnTick(), egal ob direkt per Code oder aufgerufen über eine externe Funktion, z.B. CheckIndicator(). Steht das alles nur! In OnInit(), dann wird die Indikatorabfrage auch nur einmal! durchlaufen.
Diese Funktion braucht auch nur dann aufgerufen zu werden, wenn eine neue Kerze gebildet wird, z.B. per

if (OldTime != Time[0]) {
CheckIndicator();
OldTime = Time[0];
}

Das zweite if (total == 0) ist überflüssig und irgendwas ist auch an der {} Setzung nicht korrekt.

traderdoc

MA-EA 30.06.18 00:04

double total,ticket;

Muss das nicht int sein?

Lad vielleicht mal hoch.

FX4Life 30.06.18 12:56

Zitat:

Zitat von MA-EA (Beitrag 41167)
double total,ticket;

Muss das nicht int sein?

Lad vielleicht mal hoch.



Danke MA-EA,

hast recht. Die Wert sind ganze Zahlen.

Und ebenfalls vielen Dank an Traderdoc, denn jetzt läuft die EA [emoji16]


Alle Zeitangaben in WEZ +2. Es ist jetzt 00:26 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