Adding function to foreground service lifecycle methods can create an issue? Is this best practise?

Question

I created a timer app. So it\'s counting time basically and I\'m saving the time to room when service close in OnDestroy scope. Is it best practise? Or is there any case tha