Review concurrent code quality

You are given a short code snippet during a pull-request review and asked to identify engineering issues and suggest improvements. The review focuses on the following topics: - incorrect or unnecessary use of `volatile` - thread-safety problems with shared mutable state - violations of the Single Responsibility Principle - hard-coded constants or configuration values in the code Explain what is wrong, why it is risky in production, and how you would refactor the code to make it safer, cleaner, and easier to maintain.