Немножко обобщу и дополню ответ @i-one. Для того чтобы корректно завершать сервис, нужно две вещи:
- уметь сообщать нужному коду о том, что пора закругляться
- дожидаться пока этот код завершит свое исполнение
Сигнализировать о сообщении можно разными способами, самый разумный -- CancellationToken
, который при старте передается всему необходимому коду.
Дожидаться завершения исполнения тоже можно разными способами. В случае с потоками/синхронным исполнением -- @i-one привел рецепт.
В случае если приложение полностью асинхронное (с использованием async/await
), в винсервисе на запуске запоминается таск, а на стопе этот таск await'ится.
Еще один момент, который стоит упомянуть: по умолчанию, Service Console Manager выделяет 30 секунд на то, чтобы сервис завершился (если это значение не было переопределено в реестре). Если сервис не уложился в отведенное время, то в консоли он помечается как остановленный, но процесс при этом продолжает работать, что не очень корректно. Если корректное завершение требует много времени, то можно запросить дополнительное время -- вплоть до 125 секунд в сумме:
protected override void OnStop()
{
int timeout = 10000;
while (!mainTask.Wait(timeout))
{
RequestAdditionalTime(timeout);
}
}
Однако в любом случае полезно иметь некий общий таймаут (например, 120 секунд) и завершать работу принудительно (если вдруг какой-то код завис).