phecht / five_letters Goto Github PK

Clippy says the code has 20 issues.

You are only randomly picking a word. While unlikely, it is possible to pick the same word before the list is exhausted. The list should have a way to pick only words that have not been chosen yet until all words are chosen.

Produce a keyboard that shows what letters are still available, ones in correct position and ones that should no longer be used.

Use a Vec that contains a char and three i32. Such as: char, row, pos, value.

enum
green =1, yellow =2, red =3, blue = 4
Update value based on return of checkanswer. If 0 the word isn't used, put a red in value.
If 1 word is in right pos, set value to green
If 2 letter is in wrong pos, set value to yellow
default is blue
Use row and pos to print out the keyboard. Use colored::Colorize; like:
char.black() for unused, yellow() for in word, wrong case. Use green() for proper position.

If the word was lying and you enter going, it incorrectly says the first 'g' is in the wrong position and doesn't correctly handle the last 'g'.

The display should show all previous picks as opposed to just the last one.