What methods and criteria do you use when checking the quality of other developers' code?

When reviewing the quality of other developers' code, I pay attention to the following aspects:

• Readability and clarity of the code: the code should be easily readable, with understandable variable and method names.
• Compliance with code style: adherence to the standards of code formatting accepted in the project or company.
• Logic and architecture: ensuring that the code correctly implements the task and does not contain redundancies.
• Error handling: proper and predictable management of exceptions.
• Testability: presence of unit tests and the ability to easily cover the code with tests.
• Optimality and performance: avoiding unjustified resource costs.

Methods of review:

• Code Review: systematic review of code with comments and recommendations.
• Static analysis: using tools (e.g., Lint, SonarQube) for automatic checking.
• Running tests: ensuring all tests pass successfully.

Example of a comment in code review:

// It is better to use 'when' instead of 'if-else' to improve readability
when (status) {
    Status.SUCCESS -> handleSuccess()
    Status.FAILURE -> handleFailure()
    else -> handleUnknown()
}