Code reviewMediumcr-g383

Subject Iterator invalidationLevel Mid–Senior~18 minCommon in Databases & SQL interviewsIndustries Software development

Question

Review this Rust code removing finished tasks from a vector by index.

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 reviewrust

1let mut tasks: Vec<Task> = load();2let mut i = 0;3while i < tasks.len() {4    if tasks[i].done() {5        tasks.swap_remove(i);  // O(1) remove6    }7    i += 1;8}

Run or narrate your approach, then ask the coach.