EA simple question

Hi 

I m trying to create a very simple EA (thats my first one) that will use a simple indicator to enter and exit a trade.
This indicator has two lines (Buffers) a green and a red. So I want to enter the trade when green & red cross and exit when they cross again AND RE-ENTER the trade.

I ve created the code bellow based on an MT4 EA  code:

#define MAGICMA  20050610

extern double Lots               = 0.1;

string customIndName="XYZ-Indicator";

//+------------------------------------------------------------------+
//| Calculate open positions                                         |
//+------------------------------------------------------------------+
int CalculateCurrentOrders(string symbol)
  {
   int buys=0,sells=0;
//----
   for(int i=0;i<OrdersTotal();i++)
     {
      if(OrderSelect(i,SELECT_BY_POS,MODE_TRADES)==false) break;
      if(OrderSymbol()==Symbol() /*&& OrderMagicNumber()==MAGICMA*/)
        {
         if(OrderType()==OP_BUY)  buys++;
         if(OrderType()==OP_SELL) sells++;
        }
     }
//---- return orders volume
   if(buys>0) return(buys);
   else       return(-sells);
  }

//+------------------------------------------------------------------+
//| Check for open order conditions                                  |
//+------------------------------------------------------------------+
void CheckForOpen()
  {
   double ma, red,green, red_1,green_1;
   int    res;
//---- go trading only for first tiks of new bar
   if(Volume[0]>1) return;
//---- get Moving Average 

        int history=10; // int

        green = iCustom(Symbol(),Period(),customIndName,history , 0 , 0);  
        green_1 = iCustom(Symbol(),Period(),customIndName,history , 0 , 1); 

        red   = iCustom(Symbol(),Period(),customIndName,history ,  1 , 0);   
        red_1   = iCustom(Symbol(),Period(),customIndName,history ,  1 , 1);    
        
          Print ("O T: green:"+green+" red:"+red)         ;


   //ma=iMA(NULL,0,MovingPeriod,MovingShift,MODE_SMA,PRICE_CLOSE,0);
//---- sell conditions
   if(red>green)  
     {
      res=OrderSend(Symbol(),OP_SELL,Lots,Bid,3,0,0,"",MAGICMA,0,Red);
      return;
     }
//---- buy conditions
   if(green>red)  
     {
      res=OrderSend(Symbol(),OP_BUY,Lots,Ask,3,0,0,"",MAGICMA,0,Green);
      return;
     }
//----
  }
//+------------------------------------------------------------------+
//| Check for close order conditions                                 |
//+------------------------------------------------------------------+
void CheckForClose()
  {
   double ma, red,green, red_1,green_1;
//---- go trading only for first tiks of new bar
   if(Volume[0]>1) return;
//---- get Moving Average 

        int history=10; // int
        green = iCustom(Symbol(),Period(),customIndName, history , 0 , 0);  
        green_1 = iCustom(Symbol(),Period(),customIndName,history , 0 , 1); 

        red   = iCustom(Symbol(),Period(),customIndName,history ,  1 , 0);   
        red_1   = iCustom(Symbol(),Period(),customIndName,history ,  1 , 1);   
         
          Print ("C T: green:"+green+" red:"+red)         ;

//----
   for(int i=0;i<OrdersTotal();i++)
     {
      if(OrderSelect(i,SELECT_BY_POS,MODE_TRADES)==false)        break;
      if(OrderMagicNumber()!=MAGICMA || OrderSymbol()!=Symbol()) continue;
      
      //---- check order type 
      if(OrderType()==OP_BUY) // if we are in a BUY order then SELL
        {
         if(red>green)   {OrderClose(OrderTicket(),OrderLots(),Bid,3,White);
                          OrderSend(Symbol(),OP_SELL,Lots,Bid,3,0,0,"",MAGICMA,0,Red);}
         break;
        }
      if(OrderType()==OP_SELL) // if we are in a SELL order then BUY
        {
         if(green>red )  {OrderClose(OrderTicket(),OrderLots(),Ask,3,White);
                          OrderSend(Symbol(),OP_BUY,Lots,Ask,3,0,0,"",MAGICMA,0,Green);}
         break;
        }
     }
//----
  }
//+------------------------------------------------------------------+
//| Start function                                                   |
//+------------------------------------------------------------------+
void start()
  {
//---- check for history and trading
   if(Bars<100 || IsTradeAllowed()==false) return;
//---- calculate open orders by current symbol
   if(CalculateCurrentOrders(Symbol())==0) CheckForOpen();
   else                                    CheckForClose();
//----

  }
//+------------------------------------------------------------------+

 Is this ok ? I m asking cause when i m also placing the indicator with the EA as well it doesn't show correct but the code looks ok to me.

Or should i say something like :

if(red>green && green_1>red_1)  
....
if(green>red && red_1>green_1)  
...

 Also during a backtest, is there a way to output anything to a file/log/Console/Print whatever?

 Thanks