kz363/phase_0_unit_2

Code Review for 3_guessing_game_solo_challenge

Opened this issue · 0 comments

Pseudocode

Your pseudocode was pretty good. You broke down and described your process in a clear and concise way. My only criticism is that you would want to avoid using language specific syntax in pseudocode. So saying things like "@answer" or ":correct" in your pseudocode would be too language specific. You could say things like "initialize attribute answer" or "return a symbol correct."

Naming

Your naming is spot on. Nothing I would really change there. Just always keep in mind that naming is super super important. Developers will seriously take 30 minutes to fight over names.

Overall Solution

Your solution looks pretty good too. Not much I would change.

Driver test code

Looks pretty good.

Reflection

Good description of your process. Always be thinking not only about what you are doing but why you are doing it.