MQL4多条件含OR运算符时意外快速开平仓问题求助

问题根源 #

语法错误类 #

• 多单开仓时错误使用==比较运算符而非=赋值运算符，同时混用了Ticket和TicketBuy两个变量，导致开单成功后全局变量没有正确记录票号，平仓逻辑判断完全失效
• 全局变量定义漏写分号：datetime SwitchBuy、datetime SwitchSell行末尾缺少分号，会触发编译异常，变量赋值逻辑混乱
• 平仓函数遍历越界：for (int i =OrdersTotal(); i>=0;i--) 应该从OrdersTotal()-1开始遍历，否则首次循环就会访问不存在的订单索引，导致逻辑异常

逻辑错误类 #

• 平仓时间参数写错：需求是持仓1020秒平仓，但代码里写的是SwitchBuy+60，持仓60秒就会触发平仓，平仓后马上满足开仓条件，形成高频开平仓循环
• 时间变量混用：卖单平仓判断错误使用了买单的时间变量SwitchBuy，只要买单开仓满60秒，所有卖单都会被强制平仓
• 全局变量未初始化：SwitchBuy、SwitchSell默认初始值为0，程序启动后TimeCurrent()>(SwitchBuy+1020)会直接判定为真，刚开的仓会被立即平仓
• 开平仓顺序颠倒：代码先执行开仓逻辑，再执行平仓逻辑，刚开的订单在同一个tick内就会被平仓条件命中，直接触发平仓，形成循环
• OrderType()调用无效：时间平仓判断中直接调用OrderType()，没有提前执行OrderSelect选中订单，返回值为无效值，判断条件完全失效
• 平仓函数参数错误：平仓时传入全局的Ticket/TicketSell作为平仓票号，遍历到其他订单时会调用失败，甚至触发全局变量被意外重置

修复方案 #

1. 修正变量定义与赋值错误，删除无用的Ticket变量，补全语法分号，给所有全局变量加初始值：

//Global Variables
//int Value = ReadCSVFile and get cell value(Either -1 or 1)
int TicketBuy = 0;
int TicketSell = 0;
datetime SwitchBuy = 0;
datetime SwitchSell = 0;
int magic = 12345; // 自行替换为你的魔术手数值

2. 修正多单开仓的赋值逻辑：

if(Value == 1){
    if(getBuyOrderCount(Symbol(),magic) < 1){
        TicketBuy = OrderSend(NULL,OP_BUY,0.01,Ask,2,0,0,NULL,magic,0,clrAliceBlue);
        SwitchBuy = TimeCurrent();
    }
}

3. 调整开平仓执行顺序：先执行所有平仓判断，再执行开仓逻辑，避免同tick内开仓即平仓
4. 修正平仓条件的时间参数与变量引用：

//Close Trades based on value in CSV file changing
if(TicketBuy > 0){
    if(Value == (-1)||(TimeCurrent()>(SwitchBuy+1020))){
        CloseAllTradesBuy();
        TicketBuy = 0;
        Alert("Buy Trade Closed!");
    } 
}                

if(TicketSell > 0){
    if(Value == 1||(TimeCurrent()>(SwitchSell+1020))){
        CloseAllTradesSell();
        TicketSell = 0;
        Alert("Sell Trade Closed!");
    }
}
// 原有单独的1020秒平仓逻辑可以删除，已经整合到上面的判断中

5. 修正平仓函数的遍历与平仓逻辑：

void CloseAllTradesBuy(){
    for (int i = OrdersTotal()-1; i >= 0; i--)
    {
        if(OrderSelect(i,SELECT_BY_POS)==true){
            if(OrderType()==OP_BUY && OrderSymbol()==Symbol() && OrderMagicNumber() == magic){
                OrderClose(OrderTicket(),OrderLots(),MarketInfo(OrderSymbol(),MODE_BID),5,clrRed);
            }
        }
    }
}

void CloseAllTradesSell(){
    for (int i = OrdersTotal()-1; i >= 0; i--)
    {
        if(OrderSelect(i,SELECT_BY_POS)==true){
            if(OrderType()==OP_SELL && OrderSymbol()==Symbol() && OrderMagicNumber() == magic){
                OrderClose(OrderTicket(),OrderLots(),MarketInfo(OrderSymbol(),MODE_ASK),5,clrRed);
            }
        }
    }
}

内容的提问来源于stack exchange，提问作者Rahul2304