problem with my ea. ontick or timer?

hello there,

 I'm building my ea with a mql programmer. I'm a programmer also, but I know vanilla javascript, not mql. Things are not going the way i expected, the ea was made but it works in a certain way, that according with the mql programmer, is an issue with the mql4 language itself, how the API works. I will describe what i wanted the ea to do, the problem that is happening and  the full code i have from the EA, and if you folks could share some light i would really appreciate that

ea specs: 

opening(the first order) and closing orders are manual

-ea should initialise and listen/wait for/idle (not sure the best term) an order to get closed.

-once an order get closed, ea should open a new BUY or SELL with the same lot size. if a sell was closed, ea should place a buy and the other way around too.

The problem that is happening, is that EA is taking too long to open a new order once 1 gets closed, sometimes even 3 or 4 seconds! It should open a new order immediately after one gets closed. The developer says the EA is working on a tick basis and the code is the best he could do. What you say?

#property copyright ""
#property link      ""



extern bool EA_work=true;


//+------------------------------------------------------------------+
//| expert initialization function                                   |
//+------------------------------------------------------------------+



datetime starttime;
int init()
  {
    //set start time
    starttime=TimeCurrent();
   return(0);
  }
//+------------------------------------------------------------------+
//| expert deinitialization function                                 |
//+------------------------------------------------------------------+
int deinit()
  {

   return(0);
  }
//+------------------------------------------------------------------+
//| expert start function                                            |
//+------------------------------------------------------------------+


int start()
  {
    //checking status EA
    if(EA_work)  CheckOpenNewTrade();
    else return(0);
   return(0);
  }
//+------------------------------------------------------------------+
//function looking for open new trade 
void CheckOpenNewTrade()
{
  //get number ticket of last closed order
  int last=LastClosed();
  if(last!=0)
  {
    if(!Opened(last))//checking if no order open for last closed order
    {
      if(OrderSelect(last,SELECT_BY_TICKET))
      {
        int ticket=0;
        double lot=OrderLots();//get lot size
        switch(OrderType())//depending order type
        {
          case 0 :
          {
            ticket=OrderSend(Symbol(),OP_SELL,lot,Bid,20,0,0,DoubleToStr(last,0),0,0,Red);//send order open sell
            break;
          }
          case 1 :
          {
            ticket=OrderSend(Symbol(),OP_BUY,lot,Ask,20,0,0,DoubleToStr(last,0),0,0,Blue);//send open order buy
            break;          
          }
        }       
      }
    }
  }
  return;
}
//function checking for no open order by ticket, return true if order was opened false if not
bool Opened(int ticket)
{
  bool make=false;
  string ttt=DoubleToStr(ticket,0);
  for(int i=0;i<OrdersTotal();i++)
  if(OrderSelect(i,SELECT_BY_POS,MODE_TRADES))
  if(OrderSymbol()==Symbol()&&OrderType()<=OP_SELL)
  {
    if(OrderComment()==ttt)make=true;
  }
  for(i=0;i<OrdersHistoryTotal();i++)
  if(OrderSelect(i,SELECT_BY_POS,MODE_HISTORY))
  if(OrderSymbol()==Symbol()&&OrderType()<=OP_SELL)
  {
    if(OrderComment()==ttt)make=true;
  }
  return(make);
}
//function find last closed order, return number of ticket last closed order
int LastClosed()
{
  int tic=0;
  datetime ctt=0;
  for(int i=0;i<OrdersHistoryTotal();i++)
  if(OrderSelect(i,SELECT_BY_POS,MODE_HISTORY))
  if(OrderSymbol()==Symbol()&&OrderType()<=OP_SELL)
  {
    if(OrderCloseTime()>ctt&&OrderCloseTime()>starttime)
    {
      ctt=OrderCloseTime();
      tic=OrderTicket();
    }
  }
  return(tic);
}