Code reviewMediumcr-g462

Subject CorrectnessLevel Mid–Senior~16 minCommon in Concurrency · Code quality & review interviewsIndustries Software development

Question

Review this Python code that removes processed jobs from a list while iterating over it.

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 i, job in enumerate(jobs):2    if job.done:3        del jobs[i]4    else:5        job.retry()

Run or narrate your approach, then ask the coach.