Upload Assignment for xylo

[devyarustagi]

No description provided.

[TanmayArya-1p]

• multiple smaller atomic commits > one large commit for everything - ref (read it)
• prefer to make your commit messages meaningful and in the present tense - guide: https://www.conventionalcommits.org/en/v1.0.0/

[TanmayArya-1p]

If i don't type anything then it errors out, take care of edge cases like this

[TanmayArya-1p]

Also, try to make an interactive TUI something like this that highlights the currently typed text red if its wrongly typed in.
Other stuff you can do if you get time after making the TUI: #9 (comment)

[TanmayArya-1p]

Good work on the TUI.

More things to do:

• Center align the entire TUI
• Take the word count in via a command line flag to the bash script. If the user doesn't specify a word count using this flag, then prompt them normally.
• Also, don't do the base64 hack you are doing right now for the banner cuz it looks very sus at first glance(using b64 is a common trick to load malware in bash scripts 💀). Instead copy paste the ascii into the file and use a multi line cursor to echo out each line individually.
• Maybe include multiple word lists for different difficulties. Higher difficulties will have words with punctuation. Take input for difficulty through a CLI flag or TUI prompt just like word count.

Once you do the above things, you can take a 2nd review from another 2y before submission.

I also want to give you something interesting to do: Add a timed mode which should display a timer that updates every second. Think about how you would do this with common OS primitives.

Hints

fork and signal handlers(you can also do it other ways if you find them to be simpler)

[peopleig]

Good job
I tried to break it, couldn't find any bugs
I noticed that you haven't implemented the timer display that tcan suggested. But no issues
LGTM

[peopleig]

Minor suggestion
You should add a simple help flag: -h, explaining which flags are used, what values are accepted
Since getopts won't accept longer flags, so only -h, not --help

[peopleig]

Could just use exit 1
Nothing wrong with this though