سؤال برمجى مهم جدا

[ahmedessam58]

الله اكبر
الاكسبيرت معتش بيفتح صفقات خالص

سورى الخطأ كان من المنصة
جارى التجربة
ولسه عاوز اعرف انته عدلت ايه ؟علشان اتعلم

[ساهر الفوركس]

      if((Close[1]>Tenkan)&&(Close[1]>Kijun)&&(Close[1]>Senkoub)&&(Close[1]>Senkoua)&&(Ichimoku_B==3)&&(Senkou>Senkoubplus&&Senkou>Senkouaplus)){        // ------------------------ 22
       if(orderscnt()<MaxTrades){
         if(stoploss==0){sl=0;}else{sl=Ask-stoploss*Point;}
           if(takeprofit==0){tp=0;}else{tp=Ask+takeprofit*Point;}
            OrderSend(Symbol(),OP_BUY,Lots,Ask,2,sl,tp,"Ambush",MagicNumber,0,Blue);
             PlaySound("Alert.wav");
          }
       }
   if(Close[1]<Tenkan)
   {
    CloseBuyOrders();
   } 
    if ((Close[1]<Tenkan)&&(Close[1]<Kijun)&&(Close[1]<Senkoua)&&(Close[1]<Senkoub)&&(Ichimoku_S==3)&&(Senkou<Senkoubplus&&Senkou<Senkouaplus)){       // ------------------------ 23
       if(orderscnt()<MaxTrades){
         if(stoploss==0){sl=0;}else{sl=Bid+stoploss*Point;}
           if(takeprofit==0){tp=0;}else{tp=Bid-takeprofit*Point;}
            OrderSend(Symbol(),OP_SELL,Lots,Bid,2,sl,tp,"Ambush",MagicNumber,0,Red); 
             PlaySound("Alert.wav");
          }
       }
if(Close[1]>Tenkan)
   {
    CloseBuyOrders();
   } 
  

الخطئ هنا

انت استخدمت دالة اغلاق الشراء مرتين CloseBuyOrders

والصحيح ان تستخدم مرة دالة اغلاق البيع ومرة دالة اغلاق الشراء حسب فكرة الاكسبريت

انا عدلت هنا كان الاكود في حالة الشراء والبيع يغلق الصفقات

[ahmedessam58]

كان المطلوب من الكود
افتتاح شمعه واغلاقها فوق التنكن
ولكن الاكسبيرت يقوم بغلق الصفقات اذا اغلقت الشمعه فقت فوق التنكن

[ساهر الفوركس]

الكود كان فيه الدالة CloseBuyOrders

لغلق صفقات الشراء وايضا لغلق صفقات البيع

انا نزعتها ووضعت الدالة CloseSellOrders

[ساهر الفوركس]

انا لم اعدل في الكود فقط صلحت خطئ غلق الصفقات

ساضيف الان اغلاق الافتتاح ايضا

[ساهر الفوركس]

انا عدلت الشرط الان يغلق في حالة تكون الشمعة كامل فوق او تحت التنكن

 //+------------------------------------------------------------------+
 //|                                                        Ambush    |
 //|                                   Copyright, created 20 july 2012|
 //|                                                                  |
 //|                                            Desined by Ahmed Essam|
 //|                                           [email protected] |
 //+------------------------------------------------------------------+
 
 #property copyright "Ambush"
 #property link      "[email protected]"
 
 extern int   MaxTrades = 1;    // ---------------- 1 
 extern int   stoploss = 50;       // ---------------- 2 
 extern int   takeprofit = 50;      // ---------------- 3
 extern double TrailingStop =0;       // ---------------- 4
 extern bool  UseHourTrade = false;        // ---------------- 5
 extern int   FromHourTrade = 7;            // ---------------- 6
 extern int   ToHourTrade = 17;          // ---------------- 7
 extern string  MM_Parameters = "---------- Money Management";
 extern double  Lots = 0.01;                                           
 extern bool  MM = false,                  // ---------------- 8
              AccountIsMicro = true;
 extern int   Risk = 10;                      // ---------------- 9
 
 int MagicNumber=148;                             // -------------------- 10
 extern int tenkansen=9; 
 extern int kijunsen=26; 
 extern int senkospan=52;
 extern int shift=0;
 extern int shift2=26;
 //+------------------------------------------------------------------+
 //| FUNCTION DEFINITIONS    deinitialization function                |
 //+------------------------------------------------------------------+
 
 void deinit() {
    Comment("");
  }
 
 int orderscnt(){
 int cnt=0;
   for(int i =0;i<OrdersTotal();i++){
     if(OrderSelect(i,SELECT_BY_POS,MODE_TRADES)){
       if(OrderSymbol()==Symbol() && MagicNumber==OrderMagicNumber()){
         cnt++;
       }
     }
   }
    return(cnt);
  }
 
 //+------------------------------------------------------------------+
 //| FUNCTION DEFINITIONS   Start function                            |
 //+------------------------------------------------------------------+
 
 int start()
   {
       if (UseHourTrade){
        if(!(Hour()>=FromHourTrade&&Hour()<=ToHourTrade)){
           Comment("Non-Trading Hours!");
           return(0);
         }
       }
     
double  Tenkan=iIchimoku(NULL,0,tenkansen,kijunsen,senkospan,MODE_TENKANSEN,shift); 
double  Kijun=iIchimoku(NULL,0,tenkansen,kijunsen,senkospan,MODE_KIJUNSEN,shift); 
double  Senkou=iIchimoku(NULL,0,tenkansen,kijunsen,senkospan,MODE_CHINKOUSPAN,1+kijunsen); 
double  Senkoua=iIchimoku(NULL,0,tenkansen,kijunsen,senkospan,MODE_SENKOUSPANA,shift); 
double  Senkouaplus=iIchimoku(NULL,0,tenkansen,kijunsen,senkospan,MODE_SENKOUSPANA,shift+27);
double  Senkoub=iIchimoku(NULL,0,tenkansen,kijunsen,senkospan,MODE_SENKOUSPANB,shift); 
double  Senkoubplus=iIchimoku(NULL,0,tenkansen,kijunsen,senkospan,MODE_SENKOUSPANB,shift+27);
double  Ichimoku_B=iCustom(Symbol(),0,"Ichimoku Trend Magic Histogram with Alert",1,shift);
double  Ichimoku_S=iCustom(Symbol(),0,"Ichimoku Trend Magic Histogram with Alert",0,shift);
  
   
      double sl,tp; 
 
    if(MM) Lots = subLotSize();
    
          //-----------------buy&sell   ----------------
     if((Close[1]>Tenkan)&&(Close[1]>Kijun)&&(Close[1]>Senkoub)&&(Close[1]>Senkoua)&&(Ichimoku_B==3)&&(Senkou>Senkoubplus&&Senkou>Senkouaplus)){        // ------------------------ 22
       if(orderscnt()<MaxTrades){
         if(stoploss==0){sl=0;}else{sl=Ask-stoploss*Point;}
           if(takeprofit==0){tp=0;}else{tp=Ask+takeprofit*Point;}
            OrderSend(Symbol(),OP_BUY,Lots,Ask,2,sl,tp,"Ambush",MagicNumber,0,Blue);
             PlaySound("Alert.wav");
          }
       }
   if(Close[1]<Tenkan&&Open[1]<Tenkan)
   {
    CloseBuyOrders();
   } 
    if ((Close[1]<Tenkan)&&(Close[1]<Kijun)&&(Close[1]<Senkoua)&&(Close[1]<Senkoub)&&(Ichimoku_S==3)&&(Senkou<Senkoubplus&&Senkou<Senkouaplus)){       // ------------------------ 23
       if(orderscnt()<MaxTrades){
         if(stoploss==0){sl=0;}else{sl=Bid+stoploss*Point;}
           if(takeprofit==0){tp=0;}else{tp=Bid-takeprofit*Point;}
            OrderSend(Symbol(),OP_SELL,Lots,Bid,2,sl,tp,"Ambush",MagicNumber,0,Red); 
             PlaySound("Alert.wav");
          }
       }
if(Close[1]>Tenkan&&Open[1]>Tenkan)
   {
    CloseSellOrders();
   }   
        
 //+------------------------------------------------------------------+
 //| FUNCTION DEFINITIONS   TrailingStop                              |
 //+------------------------------------------------------------------+
    
    if(TrailingStop>0)  
        {                 
          if(Bid-OrderOpenPrice()>Point*TrailingStop)
           {
            if(OrderStopLoss()<Bid-Point*TrailingStop)
              {
               OrderModify(OrderTicket(),OrderOpenPrice(),Bid-Point*TrailingStop,
               OrderTakeProfit(),0,Green);
               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);
   }
 void CloseBuyOrders()
{
  for (int cnt = 0 ; cnt < OrdersTotal() ; cnt++)
  {
    OrderSelect(cnt,SELECT_BY_POS,MODE_TRADES);
    if (OrderMagicNumber() == MagicNumber && OrderSymbol()==Symbol())
    {
      if (OrderType()==OP_BUY)
      {
        OrderClose(OrderTicket(),OrderLots(),NormalizeDouble(Bid,Digits),3);
      }
    }
  }
}  
 
void CloseSellOrders()
{
 for (int cnt = 0 ; cnt < OrdersTotal() ; cnt++)
  {
    OrderSelect(cnt,SELECT_BY_POS,MODE_TRADES);
    if (OrderMagicNumber() == MagicNumber && OrderSymbol()==Symbol())
    {
      if (OrderType()==OP_SELL)
      {
       OrderClose(OrderTicket(),OrderLots(),NormalizeDouble(Ask,Digits),3);
      }
     }
    }
}
 //+------------------------------------------------------------------+
 //| FUNCTION DEFINITIONS   Money Managment                           |
 //+------------------------------------------------------------------+ 
 
 double subLotSize()
  {
      double lotMM = MathCeil(AccountFreeMargin() *  Risk / 1000) / 100; 
      
       if(AccountIsMicro==false)
       {
          if(lotMM < 0.1)                  lotMM = Lots;
          if((lotMM > 0.5) && (lotMM < 1)) lotMM = 0.5;
          if(lotMM > 1.0)                  lotMM = MathCeil(lotMM);
          if(lotMM > 100)                  lotMM = 100;
       }
       else
       {
          if(lotMM < 0.01)                 lotMM = Lots;
          if(lotMM > 1.0)                  lotMM = MathCeil(lotMM);
          if(lotMM > 100)                  lotMM = 100;
       }
       
       return (lotMM);
  }
 
 //+---------------------------------------------------------------------------------+ 
  

[ahmedessam58]

انا لم اعدل في الكود فقط صلحت خطئ غلق الصفقات

ساضيف الان اغلاق الافتتاح ايضا

والله انى سعيد جدا .... لأن صديقى صار محترف

[ساهر الفوركس]

لسا والله لم افهم شروط البيع والشراء والتريلنج جيدا ساحاول الليلة فهمهم ان شاء الله

ممكن اجد الاخطاء لاكن البناء من 0 ليس بعد

[ahmedessam58]

جربت الكود الجديد
وبقه سلطه
يفتح صفقات شرا تحت الكيجن والتنكن
الله اكبر

[ساهر الفوركس]

ههههههههههه

كلما اتحققت الشروط يفتح صفقة

مفهمتش ايه المشكل بالتحديد

[ساهر الفوركس]

لو ممكن صورة لتوضيح المشكل

[ahmedessam58]

لو ممكن صورة لتوضيح المشكل

والله انى هيست من اللخبطة اللى حصلت
انا مضطر اققفل كل شئ الآن ونكمل غدا بمشيئة الله تعالى

[ساهر الفوركس]

لو ممكن اخي احمد شروط الشراء التي وضعتها وشروط البيع وشروط الخروج من الصفقة لكي افهم ماذا تريد من البرنامج بالتحديد

[ساهر الفوركس]

والله انى هيست من اللخبطة اللى حصلت
انا مضطر اققفل كل شئ الآن ونكمل غدا بمشيئة الله تعالى

ان شاء الله

[ahmedessam58]

تم تعديل الاكواد وجارى عمل الباك تيست