Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

#27-1: Beat That Submission - C J Thia #571

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

ThiaCH
Copy link

@ThiaCH ThiaCH commented Feb 6, 2024

Please fill out the survey before submitting the pull request. Thanks!

🚀🚀🚀🚀🚀🚀🚀🚀🚀🚀🚀🚀🚀🚀🚀🚀

How many hours did you spend on this assignment?
3.5 - 4 hours

Please fill in one error and/or error message you received while working on this assignment.
Undefined of the dice value. Early part of the coding due to missing out the return value in one of the helper function.

What part of the assignment did you spend the most time on?
The coding part at the win, lose or tie segment.

Comfort Level (1-5):
3.5 - 4 (Due to having some help from the tutorial video.)

Completeness Level (1-5):
4

What did you think of this deliverable?
Certain coding portion is repetitive, e.g. like Math.random, if / else due to pass assignment / exercises.
Therefore I was able to quickly understand what is roughly needed to be coded.
Only issue for myself is that I wish I can fit in more time to explore more.

Is there anything in this code that you feel pleased about?
I can read the codes easier and the joy of able to redesign the Html page (Designer's craving).

What's one aspect of your code you would like specific, elaborate feedback on?
Simplification of the codes and methods using Pseudo Code. It was explained in lesson 9 (05 Feb 2024) during class.

@ThiaCH ThiaCH changed the title Beat That Submission - C J Thia #27-1: Beat That Submission - C J Thia Feb 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant