snoopier wrote:Hi,

i modified the balance function for myself to get only one currency in one direction
like eurusd long and usdchf long is allowed
The original code doesnt work properly

Best Regards
Dennis


>>>>>>>>>>>>>>>>>>>>>
bool BalancedPair(int type)
{
if (IsTesting()) return (true);

string BuyCcy1, SellCcy1, BuyCcy2, SellCcy2, sType1, sType2;

if (type == OP_BUY || type == OP_BUYSTOP)
{
sType1 = "Buy";
BuyCcy1 = first.currency;
SellCcy1 = second.currency;
}//if (type == OP_BUY || type == OP_BUYSTOP)
else
{
sType1 = "Sell";
BuyCcy1 = second.currency;
SellCcy1 = first.currency;
}//else

for (int cc = OrdersTotal() - 1; cc >= 0; cc--)
{
if (!OrderSelect(cc, SELECT_BY_POS)) continue;
if (OrderSymbol() == symbol) continue;
if (OrderType() == OP_BUY || OrderType() == OP_BUYSTOP)
{
sType2 = "Buy";
BuyCcy2 = StringSubstr(OrderSymbol(), 0, 3);
SellCcy2 = StringSubstr(OrderSymbol(), 3, 3);
}//if (OrderType() == OP_BUY || OrderType() == OP_BUYSTOP)
else
{
sType2 = "Sell";
BuyCcy2 = StringSubstr(OrderSymbol(), 3, 3);
SellCcy2 = StringSubstr(OrderSymbol(), 0, 3);
}//else

if ( (BuyCcy1 == BuyCcy2) || (SellCcy1 == SellCcy2) )
{
//Print("BalancedPair checks ", symbol, "(", sType1, ") -> blocked by ", OrderSymbol(), "(", sType2, ")");
//last.balance.block.time = TimeCurrent();
//last.balance.block.curr = OrderSymbol()+"-"+sType2;
//last.balance.block.type = sType1;
return(false);
}
}//for (int cc = OrdersTotal() - 1; cc >= 0; cc--)

//Got this far, so it is ok to send the trade
return(true);

}//End bool BalancedPair(int type)
>>>>>>>>>>>>>>>>>>>>>

Pippopotamus wrote:George, wish I could help, but I'm not a good coder. I must fiddle around with this until I get it to work... a tatons, comme toujours...

jlcgarcia wrote:Hi,

I have replaced the old balanced pair function with the new posted by snoopier.


Please check if it functions correctly.

Simple Beast! with new balanced pair.mq4

Regards



José Luis