EA Start() Loop

Is there any reason why a loop (like the one below) wouldn't be an effective method?

My EA actively monitors a couple of indicators and then monitors active orders for trailing stop's and take profit's as well as reads/writes to multiple files.

start()                    // Special function start()
   {
   while(!IsStopped())     // Until user.. 
      {                    // ..stops execution of the program
      RefreshRates();      // Data renewal
      //......................The main code of the program is specified here
      Sleep(5);            // Short pause
      }
   return;                 // Control is returned to the terminal
   }