[警告は閉鎖されました！】フォーラムを乱雑にしないために、どんな初心者の質問でも。プロフェッショナルは、通り過ぎないでください。あなたなしでは、どこにも行けない。 - ページ 113

新しいコメント
 
mukata >> :

では、その質問はどうでしょうか？

いちいち関数を呼び出すより、変数に格納したほうが早い。

ちなみに、テスターでの確認は簡単です。

 
StatBars >> :
どこにコードを隠したんだ？ちょっといじりたいだけなのに、もう......。


//+------------------------------------------------------------------+
//| Copyright © 2009, ::: -- SAMER -- ::: |
//| E-MAIL: camep@inbox.ru icq: 422372555 |
//+------------------------------------------------------------------+

extern double TakeProfit = 500;
extern double Lots = 0.1;
extern double Stoploss = 30;
extern double TrailingStop = 30;
extern int iWPRperiod1 = 50;
extern int iWPRperiod2 = 60;
extern int iWPRperiod3 = 60;
extern int iWPRperiod4 = 50;
extern int slowing = 3;
extern int period = 3;
extern int period2 = 5;

int start()
{
double a;
int b;
int total;
int ticket;
int cnt;
int iWPRperiod1;
int iWPRperiod2;
int iWPRperiod3;
int iWPRperiod4;
int slowing;
int period;
int period2;

if(Bars<100)
{
Print("bars less than 100");
return(0);
}

if(TakeProfit<10)
{
Print("TakeProfit less than 10");
return(0);
}

total=OrdersTotal();
if(total<1)
{

if(AccountFreeMargin()<(1000*Lots))
{
Print("We have no money. Free Margin = ", AccountFreeMargin());
return(0);
}

if((iWPR(NULL,0,iWPRperiod1,0)>iWPR(NULL,0, iWPRperiod2,0)) < iStochastic(NULL,0,period2,period,slowing,0,0,0,0) )

{
ticket = OrderSend(Symbol(),OP_BUY,Lots,Ask,3,Ask-Stoploss*Point, Ask+TakeProfit*Point,"comment",16384,0,Green);

if(ticket>0)
{
if(OrderSelect(ticket,SELECT_BY_TICKET,MODE_TRADES)) Print("BUY order opened : ",OrderOpenPrice());
{
else Print("Error opening BUY order : ",GetLastError());
return(0);
}

if((iWPR(NULL,0,iWPRperiod1,0)>iWPR(NULL,0, iWPRperiod2,0)) > iStochastic(NULL,0,period2,period,slowing,0,0,0,0) )
{
ticket = OrderSend(Symbol(),OP_SELL,Lots,Bid,3,Bid+Stoploss*Point,Bid-TakeProfit*Point,"macd sample",16384,0,Red);
if(ticket>0)
{
if(OrderSelect(ticket,SELECT_BY_TICKET,MODE_TRADES)) Print("SELL order opened : ",OrderOpenPrice());
}
else Print("Error opening SELL order : ",GetLastError());
return(0);
}
return(0);
}

for(cnt=0; cnt < total;cnt++)
{
OrderSelect(cnt, SELECT_BY_POS, MODE_TRADES);
if(OrderType()<=OP_SELL &&
OrderSymbol()==Symbol())

{
if(OrderType()==OP_BUY)
{
if(iWPR(NULL,0,iWPRperiod1,0)>iWPR(NULL,0, iWPRperiod2,0))
{
OrderClose(OrderTicket(),OrderLots(),Bid,3,Violet)
return(0);
}

if(TrailingStop>0)
{
if(Bid-OrderOpenPrice()>Point*TrailingStop)
{
if(OrderStopLoss()<Bid-Point*TrailingStop)
{
OrderModify(OrderTicket(),OrderOpenPrice(),Bid-Point*TrailingStop,OrderTakeProfit(),0,Green);
return(0);
}
}
}
}
else
{
if(iWPR(NULL,0,iWPRperiod3,0)<iWPR(NULL,0, iWPRperiod4,0))
{
OrderClose(OrderTicket(),OrderLots(),Ask,3,Violet);
return(0);
}
if(TrailingStop>0)
{
if((OrderOpenPrice()-Ask)>(Point*TrailingStop))
{
if((OrderStopLoss()>(Ask+Point*TrailingStop)) || (OrderStopLoss()==0))
{
OrderModify(OrderTicket(),OrderOpenPrice(),Ask+Point*TrailingStop,OrderTakeProfit(),0,Red);
return(0);
}
}
}
}
}
}
return(0);
}

 
TheXpert >> :

いちいち関数を呼び出すより、変数に格納したほうが早い。

ちなみに、テスターで簡単に確認できる場合もあります。

ありがとうございます、私もそう思っていました。

しかし、私がこのフォーラムで読んだ多くのコードでは、誰もそんなことはせず、誰もが毎回関数を呼び出しています。私も同じ場所にいます。

関数ならわかるが、配列はどうするんだ？

追伸：最近ここ...

  
//+------------------------------------------------------------------+
//| Copyright © 2009, ::: -- SAMER -- ::: |
//| E-MAIL: camep@inbox.ru icq: 422372555 |
//+------------------------------------------------------------------+

extern double TakeProfit = 500;
extern double Lots = 0.1;
extern double Stoploss = 30;
extern double TrailingStop = 30;
extern int iWPRperiod1 = 50;
extern int iWPRperiod2 = 60;
extern int iWPRperiod3 = 60;
extern int iWPRperiod4 = 50;
extern int slowing = 3;
extern int period = 3;
extern int period2 = 5;

int start()
{
   double a;
   int b;
   int total;
   int ticket;
   int cnt;
   int iWPRperiod1;
   int iWPRperiod2;
   int iWPRperiod3;
   int iWPRperiod4;
   int slowing;
   int period;
   int period2;

   if(Bars<100)
   {
      Print("bars less than 100");
      return(0);
   }

   if( TakeProfit<10)
   {
      Print("TakeProfit less than 10");
      return(0);
   }

   total=OrdersTotal();
   if( total<1)
   {

      if(AccountFreeMargin()<(1000* Lots))
      {
         Print("We have no money. Free Margin = ", AccountFreeMargin());
         return(0);
      }

      if((iWPR(NULL,0, iWPRperiod1,0)>iWPR(NULL,0, iWPRperiod2,0)) < iStochastic(NULL,0, period2, period, slowing,0,0,0,0) )
      {
         ticket = OrderSend(Symbol(),OP_BUY, Lots,Ask,3,Ask- Stoploss*Point, Ask+ TakeProfit*Point,"comment",16384,0,Green);

         if( ticket>0)
         {
            if(OrderSelect( ticket, SELECT_BY_TICKET, MODE_TRADES)) Print("BUY order opened : ",OrderOpenPrice());
            else Print("Error opening BUY order : ",GetLastError());
            return(0);
         }
      }
      if((iWPR(NULL,0, iWPRperiod1,0)>iWPR(NULL,0, iWPRperiod2,0)) > iStochastic(NULL,0, period2, period, slowing,0,0,0,0) )
      {
         ticket = OrderSend(Symbol(),OP_SELL, Lots,Bid,3,Bid+ Stoploss*Point,Bid- TakeProfit*Point,"macd sample",16384,0,Red);
         if( ticket>0 && OrderSelect( ticket, SELECT_BY_TICKET, MODE_TRADES))Print("SELL order opened : ",OrderOpenPrice());
         else 
         {
            Print("Error opening SELL order : ",GetLastError());
            return(0);
         }
      }
      return(0);
   }  

   for( cnt=0; cnt < total; cnt++)
   {
      OrderSelect( cnt, SELECT_BY_POS, MODE_TRADES);
      if(OrderType()<=OP_SELL && OrderSymbol()==Symbol()) 
      {
         if(OrderType()==OP_BUY)
         {
            if(iWPR(NULL,0, iWPRperiod1,0)>iWPR(NULL,0, iWPRperiod2,0))
            {
               OrderClose(OrderTicket(),OrderLots(),Bid,3,Violet);
               return(0); 
            }
            if( TrailingStop>0)
            {
               if(Bid-OrderOpenPrice()>Point* TrailingStop)
               {
                  if(OrderStopLoss()<Bid-Point* TrailingStop)
                  {
                     OrderModify(OrderTicket(),OrderOpenPrice(),Bid-Point* TrailingStop,OrderTakeProfit(),0,Green);
                     return(0);
                  }
               }
            }
         }
         else
         {
            if(iWPR(NULL,0, iWPRperiod3,0)<iWPR(NULL,0, iWPRperiod4,0))
            {
               OrderClose(OrderTicket(),OrderLots(),Ask,3,Violet);
               return(0);
            }
            if( TrailingStop>0)
            {
               if((OrderOpenPrice()-Ask)>(Point* TrailingStop))
               {
                  if((OrderStopLoss()>(Ask+Point* TrailingStop)) || (OrderStopLoss()==0))
                  {
                     OrderModify(OrderTicket(),OrderOpenPrice(),Ask+Point* TrailingStop,OrderTakeProfit(),0,Red);
                     return(0);
                  }
               }
            }
         }
      }
   }
return(0);
}

私が直した中で最も簡単な構文エラーは、「簡単」なもの、つまり論理的なものが残されているのです。:)

  
if((iWPR(NULL,0, iWPRperiod1,0)>iWPR(NULL,0, iWPRperiod2,0)) < iStochastic(NULL,0, period2, period, slowing,0,0,0,0) )
if((iWPR(NULL,0, iWPRperiod1,0)>iWPR(NULL,0, iWPRperiod2,0)) > iStochastic(NULL,0, period2, period, slowing,0,0,0,0) )
奇抜なデザインですね。
 
StatBars >> :

どのようにコピーすればいいのでしょうか？

をプログラムにコピーすると、すべて1行になります。

 
FOREXMASTER писал(а)>>

どのようにコピーすればいいのでしょうか？

をプログラムにコピーすると、すべて1行になります。

ファイル:
forxmaster.mq4  4 kb
 
StatBars >> :

>>ありがとうございました！)

 
TheXpert >> :

いちいち関数を呼び出すより、変数に格納したほうが早い。

ちなみに、テスターでの確認は簡単です。

関数はわかったけど、配列はどうするんだ？

テスターで動かすにも、かなり書き直さないといけないんです。

いちいち関数を呼び出すより変数を格納した方が早いし、配列の要素を読み込むのも単純な変数と同じくらい早いのか？

あるいはもっと長く、インデックスを調べる必要がある配列なのか、それとも変数と同じなのか。

>> こういう微妙なところがよくわからない。

 
mukata >> :

関数は明確ですが、配列はどうでしょうか？

こういうのって、よくわからないんですよね。

コード例で示すのがよいでしょう。


配列には曖昧な点があります。


私は、EAにはデータ解釈に関わる配列を含めるべきではないと考えています。指標に属するものです。

ここでは、多くの人が違う考えを持っています。そして、Expert Advisorにすべてを詰め込もうとする。

1...106107108109110111112113114115116117118119120...1145
新しいコメント