Quelle est la bonne façon de quitter Windows Service OnStart si la configuration est incorrecte et que rien à faire?

Question

C'est ce que je suis:

protected override void OnStart(string[] args) 
{ 
    if (SomeApp.Initialize()) 
    { 
     SomeApp.StartMonitorAndWork(); 
     base.OnStart(args); 
    } 
} 

protected override void OnStop() 
{ 
    SomeApp.TearDown(); 
    base.OnStop(); 
} 

Ici Initialiser lit un fichier de configuration et si elle est mal il n'y a rien à faire le service devrait cesser! Si la configuration est correcte, StartMonitorAndWork démarre:

Timer(new TimerCallback(DoWork), null, startTime, loopTime); 

et DoWork interroge périodiquement la base de données.

Si Initialiser échoue (je vérifie le fichier journal) et je tente d'arrêter le service de Outils- administratif> Services que je reçois:

 
Could not stop the SomeService on Local Computer. The service did not return an error. 
This could be internal Windows error or an internal service error. 
If the problem persists, contact system administrator. 

 
The question is: 
"Is exiting OnStart without doing nothing enough if Initialize returns false? 

ou devrait-il y avoir quelque chose comme ceci:

private void ExitService() 
{ 
    this.OnStop(); 
    System.Environment.Exit(1); 
} 

protected override void OnStart(string[] args) 
{ 
    if (ObjectFolderApp.Initialize()) 
    { 
     SomeApp.StartMonitorAndWork(); 
     base.OnStart(args); 
    } 
    else 
    { 
     ExitService(); 
    } 
} 

Merci & BR - Matti

EDIT: Je suis venu avec quelque chose comme ceci:

protected override void OnStart(string[] args) 
{ 
    try 
    { 
     if (SomeApp.Initialize()) 
     { 
      SomeApp.StartMonitorAndWork(); 
      base.OnStart(args); 
     } 
     else 
     { 
      Stop(); 
     } 
    } 
    catch 
    { 
     Stop(); 
    } 
} 

protected override void OnStop() 
{ 
    try 
    { 
     SomeApp.TearDown(); 
     base.OnStop(); 
    } 
    catch 
    { 
     base.OnStop(); 
    } 
} 

Answer 1

je serais enregistrer une erreur au journal des événements si Initialize() faux rapports avec un certain message sensé dire alors qu'il échoue, et vous devriez que vous suggérez appeler OnStop() si elle échoue. C'est une bonne pratique d'assurer une fermeture correcte du service.

Voir aussi this related SO question et dev newsgroup thread.

Answer 2

après avoir testé toutes les approches, je préfère appeler

Environment.FailFast("Configuration is wrong."); 

objectif principal est que l'explication est l'échec écrit dans le journal des événements et FailFast affecte les paramètres de récupération de votre service. Donc, si vous configurez la récupération et que la configuration est correcte, votre service démarrera automatiquement.

Answer 3

Je sais que ce n'est pas joli mais lancer une exception dans OnStart fonctionne également. Si votre service est configuré sur "Autolog", votre message d'exception sera automatiquement écrit dans EventLog.

protected override void OnStart(string[] args) 
{ 
    if (ObjectFolderApp.Initialize()) 
    { 
     SomeApp.StartMonitorAndWork(); 
     base.OnStart(args); 
    } 
    else 
    { 
     throw new Exception("What went wrong"); 
    } 
}