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

Random password generator completed, Issue #36 #39

Closed
wants to merge 1 commit into from
Closed

Random password generator completed, Issue #36 #39

wants to merge 1 commit into from

Conversation

BrendanBurger
Copy link

Hello, I am new to contributing but I worked on Issue #36 Password Generator Function.
I included two methods for generating random strings,was unsure of password requirements.

One method returns a random string of length n with just (upper and lower)characters and at least one digit.
The other method returns a random string of length n with at least one upper character, one special character and one digit.

I also changed the passfield object to a textfield object to view the string.

@GeassCodeBall
Copy link

Thanks for your contribution! After reading your code, I think it may be better if we put one set of special character in together with another in the process of adding them into the arraylist. And it may be more convenient when we judge whether the special are added or not. : )

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.

2 participants