Question

这是我所说的:

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

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

这里,如果STOP! 如果会议开始:

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

和定期调查数据库。

如果开端失败(核对记录),我试图停止从行政工具及服务中提供服务:

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?

还是应该这样:

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();
    }
}

附录

EDIT: I came up with something like this:

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

在测试所有做法之后,我个人希望呼吁

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

主要目标是,未能做出解释,最终会影响到你工作的恢复环境。因此,如果你把恢复和配置混为一谈,你的工作就会自动开始。

Answer 2

我知道,这并不奇怪,而是在《OnStart》中提出例外。

如果你的服务是“Autolog”服务,这还将自动向事件记录仪发出例外信息。

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

Answer 3

如果(<>Initialize(>>)以一些令人敏感的信息告别,但如不正确的话,我将对事件表示错误。这是确保适当关闭服务的良好惯例。

另见, 与此相关的SO问题, 和。

Answer 4

我建议你这样做:

protected override void OnStop()
{
    try
    {
        EventLog.WriteEntry("MyService", "Service is going to stop because of ...", EventLogEntryType.Information);        
        // Dispose all your objects here        
    }
    catch (Exception ex)
    {
        EventLog.WriteEntry("MyService", "Exception : " + ex.ToString(), EventLogEntryType.Error);                
    }
    finally
    {
        GC.Collect();                
        base.OnStop();
    }
}

友情链接