Code reviewMediumcr-p036

Subject CorrectnessLevel Mid–Senior~18 minCommon in Code quality & review interviewsIndustries Software development

Question

Review this code that removes inactive users.

What a strong answer looks like

Separate real bugs from style. Rank issues by severity, point at the root cause rather than the symptom, and suggest a concrete fix — specific and kind.

Learn the concepts

Talk through your review

Code to reviewpython

1for user in users:2    if not user.active:3        users.remove(user)

Run or narrate your approach, then ask the coach.