اشكالية في .. اكسبيرت تنفيذ بحسب خط على الشارت

[أبو نـاصر]

كنت ساشغله على الحقيقي

فرأيت هذا لا ادري كيف حصل

[أبو نـاصر]

عزيزي
لازالت الاشكالية قائمة

2016.02.23 13:10:25.378 AboNaser_v3 EURAUDpro,M5: Error modifing Breakeven at order #102789719 Error: invalid stops

[awran5]

عزيزي
لازالت الاشكالية قائمة

2016.02.23 13:10:25.378 AboNaser_v3 EURAUDpro,M5: Error modifing Breakeven at order #102789719 Error: invalid stops

لا يا اخي هذا خطأ مختلف يعنى عدم القدرة على تعديل الستوب الى نقطة البريك ايفن .. اظن ان هذا الخطأ بسبب الخطأ الاول وهو فتح عدد كبير من الصفقات في نفس الوقت فلا يستطيع تعديل القيم ..

تحتاج استخدام وظيفة OrdersTotal حتى لا يحدث تكرار لفتح للصفقات (مع انه لم تظهر معى هذه المشكلة على الباك تست)

ايضاً .. لاحظ يا اخي ان القيمة الافتراضية لعدد النقاط التى تحدد بدء تفعيل البريك ايفن هي 25 نقطة وهي نفس قيمة طرح الهدف - سعر فتح الصفقة .. لان القيمة الافتراضية لرسم الخط كانت 50 نقطة لذلك يحدث هذا الخطأ بسبب تقارب الاسعار ..

قمت ببعض التعديلات لتجنب هذه الاخطاء واتمنى ان تعمل بشكل سليم .. ولكن يبقى عليك مرعاة فارق النقاط مستقبلاً ..

ملاحظة: قمت بالغاء قيمة السبريد التي كانت مضافة الى قيم الخطوط ..

[أبو نـاصر]

لا يا اخي هذا خطأ مختلف يعنى عدم القدرة على تعديل الستوب الى نقطة البريك ايفن .. اظن ان هذا الخطأ بسبب الخطأ الاول وهو فتح عدد كبير من الصفقات في نفس الوقت فلا يستطيع تعديل القيم ..

تحتاج استخدام وظيفة OrdersTotal حتى لا يحدث تكرار لفتح للصفقات (مع انه لم تظهر معى هذه المشكلة على الباك تست)

ايضاً .. لاحظ يا اخي ان القيمة الافتراضية لعدد النقاط التى تحدد بدء تفعيل البريك ايفن هي 25 نقطة وهي نفس قيمة طرح الهدف - سعر فتح الصفقة .. لان القيمة الافتراضية لرسم الخط كانت 50 نقطة لذلك يحدث هذا الخطأ بسبب تقارب الاسعار ..

قمت ببعض التعديلات لتجنب هذه الاخطاء واتمنى ان تعمل بشكل سليم .. ولكن يبقى عليك مرعاة فارق النقاط مستقبلاً ..

ملاحظة: قمت بالغاء قيمة السبريد التي كانت مضافة الى قيم الخطوط ..

الله ينور عليك

---

مع اننا استخدمنا دالةchek_For_BUY_Order
والتي يفترض ألا تسمح بفتح صفقة أخرى بمنطقة السبريد
فأين الثغرة في هذه الدالة

double chek_For_BUY_Order(double Price_)
  {
   double Aa=Price_+Spread;
   double Vv=Price_-Spread;

   for(int i=OrdersTotal()-1; i>=0; i--)
     {
      if(!OrderSelect(i,SELECT_BY_POS)) break;
      if(OrderSymbol()==Symbol() && OrderMagicNumber()==Magicnumber)
        {
         int type=OrderType();
         double OpEn_Price=OrderOpenPrice();
         if(type==OP_BUYSTOP || type==OP_BUYLIMIT || type==OP_BUY)
           {
            if(OpEn_Price<Aa && OpEn_Price>Vv)
              {
               return(1);
              }
           }
        }
     }

   return(0);
  }

بطريقة الأستاذ وضاح عدلتها الى هذه

  if(MathAbs(OpEn_Price-Price_)>Spread)

عزيزي
OrdersTotal سوف تقيدنا بعدد صفقات محددة على الزوج

ونحن نريد ألا ينفذ على الخط مرة أخرى مهما ذهب ورجع اليه .. فقط
(علما انه يوجد عدة خطوط بيع على الشارت كل منها له رقم مختلف )

يوجد شرطين تجاوزها النيوزلندي

            if(T_Old_entry!=T_Line)
              {
               T_Old_entry=T_Line;
               T_order_No=chek_For_SELL_Order(T_Line);

[أبو نـاصر]

ايضاً .. لاحظ يا اخي ان القيمة الافتراضية لعدد النقاط التى تحدد بدء تفعيل البريك ايفن هي 25 نقطة وهي نفس قيمة طرح الهدف - سعر فتح الصفقة .. لان القيمة الافتراضية لرسم الخط كانت 50 نقطة لذلك يحدث هذا الخطأ بسبب تقارب الاسعار ..

قمت ببعض التعديلات لتجنب هذه الاخطاء واتمنى ان تعمل بشكل سليم .. ولكن يبقى عليك مرعاة فارق النقاط مستقبلاً ..

ملاحظة: قمت بالغاء قيمة السبريد التي كانت مضافة الى قيم الخطوط ..

اعتذر منك جدا

حيث كنت وضعتها في السابق بحيث تكون نصف المسافة للهدف .. وبقيت في ذهني هذه الفكرة << وهي ما نحتاج

[أبو نـاصر]

لاحظت شيئا آخر

كان الله في عونك


في امر مدافي
ان وجد لديك أكثر من صفقة من مستويات متعددة

سيعدل كامل أهدافها الى آخر خط (hadf) رسم ضمن قائمة الاوبجيكت

مع أننا من البداية قيدناه برقم يربط كل خط بهدفه

            if(T_Zone_No_==StringSubstr(ObjectName(i),18,10))

[أبو نـاصر]

استاذي الفاضل

اضفت

                  T_Line=NormalizeDouble(ObjectGet(ObjectName(i),OBJPROP_PRICE1),Digits);

مع 
                if(T_Line-Bid<0.00002 &&T_Line-Bid>=0)

فاصبح ينشئ جميع الصفقات ولا يتجاهل اي خط حتى الآن


وأدخلت كود انشاء الصفقات داخل شرط

         if(Name22=="hadf")
           {  
            if(T_Zone_No_==StringSubstr(ObjectName(i),18,10))
              {

فأصبح يتقيد بكل صفقة وهدفها
واضفت احترازا التقييد بالتيكيت

                 if(ticket>0) CheckForModify(ticket,sl,T_Line_hadaf);

ارجوا التكرم بمراجعتها

ملاحظة : جميع التعديلات معتمدا على النسخة رقم 3

[awran5]

مع اننا استخدمنا دالةchek_For_BUY_Order
والتي يفترض ألا تسمح بفتح صفقة أخرى بمنطقة السبريد
فأين الثغرة في هذه الدالة

الحقيقة يا اخي، لم اراجع هذه الوظيفة قبل ذلك على فرض ان الكود لم يكتمل وان المشكلة كانت في فتح وتعديل الصفقات ..

- لست بحاجة الى هذه الوظيفة اساساً فالحل ابسط من ذلك .. فقط قم بتعريف متغير من نوع دبل في الجلوبال وخصص له قيمة الخط + السبريد

 double SpreadZone;
 
SpreadZone = T_Line - (Spread*pips2double); // in Sell section 
SpreadZone = B_Line + (Spread*pips2double); // in Buy section 
  

لاحظ ان متغير السبريد معرف مسبقاً وهو يجلب لك قيمة السبريد الحالية وقت استدعائه فلا داعي لعمل وظيفة وحلقة وخلافه ..

 Spread=MarketInfo(Symbol(),MODE_SPREAD) 
  

هكذا .. وتصرف في هذا المتغير كيفما تشاء مثلاً

  
bool SellSpreadZone = Bid < SpreadZone;
 
if(SellSpreadZone)
{ ..... }
//----
if(!SellSpreadZone)
{ ..... } 
  

- أكرر .. لا داعي لاستخدام NormalizeDouble .. وظيفة ObjectGet تعيد القيم معدلة بالفعل .. قمت باختبار القيم ووجدتها متساوية في الحالتين

- حول شرط الاختراق .. لا اعلم يا اخي .. مازلت اري انه من الافضل ان تضيف قيمة السبريد الى قيمة الخطوط وتحسب حسباتك على هذا الاساس .. لاحظ ان السبريد شيئ لا مفر منه ومهما كانت قيمته متغيرة (او كبيرة) ، فيمكن اضافتها الى السعر واعتبارها جزء من السعر .. ويكون امر الاختراق تجاوز قيمة السبريد وبذلك فهو اختراق فعلي ..

----------

OrdersTotal سوف تقيدنا بعدد صفقات محددة على الزوج

ونحن نريد ألا ينفذ على الخط مرة أخرى مهما ذهب ورجع اليه .. فقط
(علما انه يوجد عدة خطوط بيع على الشارت كل منها له رقم مختلف )

يوجد شرطين تجاوزها النيوزلندي

            if(T_Old_entry!=T_Line)
              {
               T_Old_entry=T_Line;
               T_order_No=chek_For_SELL_Order(T_Line);

هذا الشرط موجود ولكن لم يستخدم اساساً ..
- انت قمت بتعريف متغير B_Old_entry قم وضعت له شرط ان لم يساوي B_Line تخصص له قيمة ال B_Line ولكنك لم تستعمل B_Old_entry مرة اخرى.
والحل هو نفس الفكرة الاولى يا اخي .. قم بتعريف متغير جديد من نوع bool مع قيمة افتراضية false ثم حول القيمة الى true في موضع اختراقه وبذلك تضمن عدم اختراقه مجدداً ..

  
bool SellLineHit = false;
 
if(Bid < B_Line && Ask > B_Line)
{
   SellLineHit = true;
   .....
} 
  

[awran5]

لاحظت شيئا آخر

في امر مدافي
ان وجد لديك أكثر من صفقة من مستويات متعددة

سيعدل كامل أهدافها الى آخر خط (hadf) رسم ضمن قائمة الاوبجيكت

مع أننا من البداية قيدناه برقم يربط كل خط بهدفه

if(T_Zone_No_==StringSubstr(ObjectName(i),18,10))

هذه هي المشكلة الحقيقة والتى ليس لها حل للاسف .. هذه المشكلة تحدث بسبب FIFO الذي تحدثت عنه سابقاً .. والفيفو ان لم تفهمه جيدا يا اخي هو اختصار لجملة First In, First Out بمعنى انك مقيد باول صفقة دخلت بها للسوق ولابد من إغلاقها اولا قبل اغلاق اي صفقة اخرى واذا كانت هناك اكثر من صفقة مفتوحة، فيجب عليك اغلاق الصفقات تنازلياً .. اي اول صفقة قمت بفتحها تغلق اولا ثم التى تليها وهكذا ..

معنى ذلك ايضاً، انه في حالة الاكسبيرت، عندما يقوم بفتح اكثر من صفقة في نفس الاتجاه سيقوم البروكر بتخصيص اول سعر هدف يجده امامه اجبارياً لجميع الصفقات مهما كانت اهداف باقي الصفقات وعندما يصل السعر الى الهدف تغلق جميع الصفقات .. والامر ينطبق على الستوب.

ملاحظة: هذا الامر يتم على الصفقات التى من نفس النوع وعلى نفس الزوج

[awran5]

واضفت احترازا التقييد بالتيكيت

if(ticket>0) CheckForModify(ticket,sl,T_Line_hadaf);

عفواً نسيت هذه النقطة .. الحقيقة لا اري داعي لها يا اخي .. وان كان ولابد فلاحظ ان نوع ticket هو int وليس double ووضع المتغير يكون باستبدال وظيفة OrderTicket()

فتكون الوظيفة بالشكل التالي

 void CheckForModify(int ticket,double stoploss,double takeprofit)
  {
//---
   for(int i=OrdersTotal()-1; i>=0; i--)
     {
      if(!OrderSelect(i,SELECT_BY_POS,MODE_TRADES)) break;
      if(OrderSymbol()==Symbol() && OrderMagicNumber()==Magicnumber)
        {
         if(OrderType()==OP_BUY)
           {
            if(!OrderModify(ticket,OrderOpenPrice(),stoploss,takeprofit,0,clrYellow))
              {
               Print("Error modifing BUY order #",OrderTicket()," Error: ",ErrorDescription(GetLastError()));
              }
            else Print("BUY Order #",ticket," modified successfully.");
           }
 
         if(OrderType()==OP_SELL)
           {
            if(!OrderModify(ticket,OrderOpenPrice(),stoploss,takeprofit,0,clrYellow))
              {
               Print("Error modifing SELL order #",OrderTicket()," Error: ",ErrorDescription(GetLastError()));
              }
            else Print("SELL Order #",ticket," modified successfully.");
           }
        }
     }
//---
  } 
  

[أبو نـاصر]

يسعد صباحك أخي الكريم .. ويوم موفق باذن الله


سأوضح دالةchek_For_BUY_Order لو تكرمت


هي تأخذ قيمة الخط وتبحث في الأوامر السابقة عن اي صفقة أو أمر معلق في نفس المكان +- السبريد

         if(type==OP_BUYSTOP || type==OP_BUYLIMIT || type==OP_BUY)
           {
             if(MathAbs(OpEn_Price-Price_)>Spread)
              {
               return(1);
              }
           }

فإن وجد أي نتيجة يعيد رقم 1


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

               B_order_No=chek_For_BUY_Order(B_Line);
              
               if(B_order_No==0)
                 {

-------------

  if(T_Old_entry!=T_Line)
              {
               T_Old_entry=T_Line;

هذا الشرط موجود ولكن لم يستخدم اساساً ..
- انت قمت بتعريف متغير B_Old_entry قم وضعت له شرط ان لم يساوي B_Line تخصص له قيمة ال B_Line ولكنك لم تستعمل B_Old_entry مرة اخرى.
والحل هو نفس الفكرة الاولى يا اخي .. قم بتعريف متغير جديد من نوع bool مع قيمة افتراضية false ثم حول القيمة الى true في موضع اختراقه وبذلك تضمن عدم اختراقه مجدداً ..

الفكرة أن T_Old_entry سيسجل اخر قيمة جديدة لـ T_Line (الخط)

فلا يدخل اليها الا اذا اصبحت قيمة الخط مختلفة
ومتى ما رجع هنا بنفس قيمة الخط السابق الذي يساوي T_Old_entry يفترض ألا يسمح له

[أبو نـاصر]

عفواً نسيت هذه النقطة .. الحقيقة لا اري داعي لها يا اخي .. وان كان ولابد فلاحظ ان نوع ticket هو int وليس double ووضع المتغير يكون باستبدال وظيفة OrderTicket()

فتكون الوظيفة بالشكل التالي

 void CheckForModify(int ticket,double stoploss,double takeprofit)
  {
//---
   for(int i=OrdersTotal()-1; i>=0; i--)
     {
      if(!OrderSelect(i,SELECT_BY_POS,MODE_TRADES)) break;
      if(OrderSymbol()==Symbol() && OrderMagicNumber()==Magicnumber)
        {
         if(OrderType()==OP_BUY)
           {
            if(!OrderModify(ticket,OrderOpenPrice(),stoploss,takeprofit,0,clrYellow))
              {
               Print("Error modifing BUY order #",OrderTicket()," Error: ",ErrorDescription(GetLastError()));
              }
            else Print("BUY Order #",ticket," modified successfully.");
           }
 
         if(OrderType()==OP_SELL)
           {
            if(!OrderModify(ticket,OrderOpenPrice(),stoploss,takeprofit,0,clrYellow))
              {
               Print("Error modifing SELL order #",OrderTicket()," Error: ",ErrorDescription(GetLastError()));
              }
            else Print("SELL Order #",ticket," modified successfully.");
           }
        }
     }
//---
  } 
  

استاذي الفاضل

حاليا على الديمو طبعا
لا اتوقع انها من FIFO


جربت ذلك
فبدون تحديد التيكيت

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

فهو يدمج الأهداف

ربما بسبب أننا داخل حلقة فيتعرف على اخر هدف فقط

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

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

  if(Name22=="hadf")
           {  
            if(T_Zone_No_==StringSubstr(ObjectName(i),18,10))
              {

[أبو نـاصر]

وضعته على الحقيقي

وأول صفقة حققت 23 سنت

فأوقفت العمل على هذا الزوج واشباهه

لأن السبريد كان 6

وأضفت شرطا

if (Spread < 3){

[النظير]

ممتاز هل هناك نسخه جاهزه للاستخدام

[أبو نـاصر]

ممتاز هل هناك نسخه جاهزه للاستخدام

لم تكتمل كامل الأفكار التي في أول الموضوع حتى الآن

حاليا انا استخدم النسخة في المشاركة رقم 37

ولا زال العمل جاري مع الشكر الجزيل للاستاذ awran5


ضع اي عدد خطوط للشراء

باسم مثلا
eBUY_Line123_hadf_80

eBUY_ <<< ثابته للشراء

Line123 <<<< اكتب اي وصف من 7 احرف

hadf <<< لخط الهدف open <<< لخط فتح الصفقة

80 <<< رقم تسلسلي لابد أن يتساوي بين خط الدخول وخط الهدف