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

Homework day 2 - 11/28/17 #2

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

kjkeaston
Copy link

Comfort: 3/5
Completness: 2.5/3?

I had a tough time with the password login. I could get multiple prompts to display if the entered password was wrong, but then was having a hard time getting the subsequent entries to be validated correctly. Was also getting stuck in infinite loops of the alert displaying. Some instances of code I was trying would seem to throw alerts out of order. My current code works...as long as you enter your password right the first try.

The 99 bottles of beer song seemed fairly straight forward and felt easier than the other 2 parts. I seem to repeat myself more than the solution code though but I feel that my code is more readable and easier to see what is going on. Of course that is my opinion based on my limited knowledge

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