Skip to content

ViennaNeo_27-2#562

Open
vien0001 wants to merge 1 commit intorocketacademy:mainfrom
vien0001:main
Open

ViennaNeo_27-2#562
vien0001 wants to merge 1 commit intorocketacademy:mainfrom
vien0001:main

Conversation

@vien0001
Copy link

@vien0001 vien0001 commented Feb 5, 2024

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

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

How many hours did you spend on this assignment?
2.5

Please fill in one error and/or error message you received while working on this assignment.

my p1WinCounts and p2WinCounts were undefined at first. this is because i didnt initialize it to 0. instead of var p1WinCounts =0, i wrote let p1WinCounts;

so when i tried to +1 to the count, it returned undefined and i couldnt figure out why for a while until i dumped everything into chatgpt and asked.

What part of the assignment did you spend the most time on?
getting used to arrays.

Comfort Level (1-5):
3

Completeness Level (1-5):
3

What did you think of this deliverable?
fun to work on!

Is there anything in this code that you feel pleased about?
i tried using more helper functions this time around.

What's one aspect of your code you would like specific, elaborate feedback on?
how to shorten the code more? maybe how to make use of objects and arrays more instead of using variables.

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