Simple MQL Help

This is taken from the Coder's guru course. Instead of plotting the indicator in a seperate window, I want to plot the indicator on the chart. What am I doing wrong?

//+------------------------------------------------------------------+
//| TCALERTv1.mq4 |
//| Copyright 2007, MetaQuotes Software Corp. |
//| http://www.metaquotes.net |
//+------------------------------------------------------------------+
#property copyright "Copyright 2007, MetaQuotes Software Corp."
#property link "http://www.metaquotes.net"

#property indicator_chart_window
#property indicator_buffers 1
#property indicator_color1 Red

double ExtMapBuffer1[];

//---- input parameters
//--extern int Len=21;
//--extern int Len=21;
//+------------------------------------------------------------------+
//| Custom indicator initialization function |
//+------------------------------------------------------------------+
int init()
{
//---- indicators
SetIndexStyle(0,DRAW_LINE,STYLE_SOLID,3,Red);
IndicatorDigits(MarketInfo(Symbol(),MODE_DIGITS));
IndicatorShortName("TCALERT");
SetIndexBuffer(0,ExtMapBuffer1);

//----
return(0);
}
//+------------------------------------------------------------------+
//| Custom indicator deinitialization function |
//+------------------------------------------------------------------+
int deinit()
{
//----

//----
return(0);
}
//+------------------------------------------------------------------+
//| Custom indicator iteration function |
//+------------------------------------------------------------------+
int start()
{
int counted_bars=IndicatorCounted();

//----Error Check
if (counted_bars < 0) return(-1);

//----Last Counted Bar Will Be Counted
if (counted_bars >0) counted_bars--;

int pos = Bars - counted_bars;

double dHigh, dLow, dResult;

//----Main Calc Loop

while(pos>=0)
{
dHigh = High[pos];
dLow = Low[pos];
dResult = dHigh - dLow;
ExtMapBuffer1[pos] = dResult;
pos--;
}
return(0);
}
//+------------------------------------------------------------------+

OK, forget about that one. I made this a script instead. All this is supposed to do is sound an alarm if the current bid on the EURO$ comes with +/- 5 pips of the 21EMA. I can't get the @#%@! parenthesis to 'balance'. Any thoughts???

//+------------------------------------------------------------------+
//| TCALERTv1.mq4|
//| Palmer |
//| http://www. .com |
//+------------------------------------------------------------------+
#property copyright "Palmer"
#property link "http://www. .com"

//+------------------------------------------------------------------+
//| script program start function |
//+------------------------------------------------------------------+
int start() {
double EMA21Now = iMA(NULL,0,21,0,MODE_EMA,PRICE_CLOSE,0);
double PriceNow = Bid;
if (EMA21Now == (PriceNow + .0005)) || (EMA21Now == (PriceNow - .0005))


{
PlaySound("sirenhilo.wav");
}
return (0);
}

//+------------------------------------------------------------------+

1. You need to put a zero before floating point numbers or else it thinks "." is an operator. (0.0005)

2. You need another set of parenthesis around your entire if statement

if ( (EMA21Now == (PriceNow + 0.0005)) || (EMA21Now == (PriceNow - 0.0005)) )

{
PlaySound("sirenhilo.wav");
}

I didn't test for accuracy but it compiles now.

Hi Palmer,

what the heck is that supposed to be ?
Please use:
It always is if(...)!

Haha, that's funny, you edited it!
Well...anyways. I doubt your first statement about '.0005'. You're right, it's not a good style but MQL doesn not care, I think.

What do you mean?

Try it without the leading 0, It won't work on my platform.