demo_EventArbitrage例子中我发现一个问题，再次跟老师讨论下。是我搞错了，还是真有不妥的地方

//委托更新事件函数，参数ord表示更新的委托结构体

OnOrder(OrderRef ord)

{

LogFile(\"OnOrder:\"+ ord.symbol + \",\" + Text(ord.orderId));

//1.如果是左腿单，有成交，则锁定对手合约下单

if(ord.symbol==data0.Symbol())

{

If(!MapContain(leftOrderMap, ord.orderId))//如果这个orderid不在leftOrderMap中，说明没记录，那就直接返回

{

//防止收到重复的委托

Return;

}

leftOrderMap[ord.orderId]=ord;//如果order订单状态更新了，则重新更新leftOrderMap里面对应的订单信息

If(!isOrdersFinished(leftOrderMap))// 如果所有挂单还没都成交，则直接返回,等到所有单子都成交了，再往下执行，

{

Return;

}

fillVolume = sumFillVolume(leftOrderMap);//计算所有左腿单的总下单量

If(fillVolume <= 0)

{

//左腿没有成交量

finishArbitrage();

Return;

}

size=fillVolume-sumOrderVolume(rightOrderMap);//计算左腿总下单量-右腿总下单量，看看右腿还差多少手

if(size>0)

{

sendRightOrder(size);//再补多少手右腿的单子

}

Return;

}

//2.如果是右腿单，则判断否完成套利，如果完成清理缓存，重新挂单

if(ord.symbol==data1.Symbol())

{

If(!MapContain(rightOrderMap, ord.orderId))

{

//防止收到重复的委托

Return;

}

rightOrderMap[ord.orderId]=ord;

                       If(!isOrdersFinished(rightOrderMap))  //下面红色代码部分应该移动到这里

{

Return;

}

//2.1判断右腿委托是否满足，如果不满足，则继续委托

size=sumFillVolume(leftOrderMap)-sumOrderVolume(rightOrderMap);

if(size>0)

{

sendRightOrder(size);

Return;

}

If(!isOrdersFinished(rightOrderMap))  //这个红色字体的代码应该放到上面蓝色的地方

{

Return;

}

//2.2判断左右腿委托是否完成，如果都完成，则清空缓存，重新挂单

size=sumFillVolume(leftOrderMap)-sumFillVolume(rightOrderMap);

if(size==0)

{

finishArbitrage();

}

}

}

总得来说：就是把 2.如果是右腿单，则判断否完成套利 里面的 If(!isOrdersFinished(rightOrderMap))这个代码移动到上面去，要先确保rightOrderMap里面所有单子已经完结(要么成交，要么被删)。 再计算左腿总单量和右腿总单量的差，进行判断如果为0就套利完成，如果还有差额就补挂右腿单。

否则的话，右腿开单挂的单子都没有完结呢，你就开始判断差额了。那这个差额不用算了肯定等于左腿单了。也就是肯定等于0的。基本等于没用这个功能。