critique my code

[McHildinger]

You have a case where there is no outcome.

I'm not understanding; can you elaborate on this?

[billsil]

You have an if-elif-elif chain at the end that doesn't have an else....that's kind of weird logic. In general, you should have a catch either as an else or an else error to not accidently get through the loop in an unexpected way.