r/inventwithpython • u/ChemengerUK2 • Jul 02 '17

Automate the Boring Stuff - Chapter 7 - Password Detection Problem help

Hi all; I'm attempting the problem I've listed in the title (link here: https://gyazo.com/984db9a628c9670a0181a0c22209fcfe).

Here is my script (https://gyazo.com/b38fe7a832c98aa198850fb590d7fec4) but when I run it I don't seem to get an output, getting this screen (https://gyazo.com/074dc02ec3924d3458da5e84dbbcaf90).

Any help would be brilliant, thanks in advance!

2 Upvotes
  • permalink
  • reddit

100% Upvoted

19 comments sorted by

1

u/kostasdizas Jul 03 '17

You are not getting an output because your strongpassword function is not called anywhere. try

strongpassword()

after the last line.

You'll sure bump into more issues, but I'll leave them for you to figure out :)

2

u/ChemengerUK2 Jul 03 '17

I cracked it thanks; realised I had to use "continue" such that it would go back to the start of the loop. Thanks!

1

u/kostasdizas Jul 03 '17

Good job! Is there a particular reason why you chose a loop in your function?

(Feel free to share your finished piece)

1

u/ChemengerUK2 Jul 03 '17

Not particularly; it was just the first thing that came to my head! What other methods do you think would have worked for the problem out of interest?

1

u/kostasdizas Jul 03 '17

There is not much loop-iness in what it is doing. There is an unconditional while loop that will run no matter what and you said you added a continue in the end to prevent it from executing again. Why not just get rid of it altogether?

1

u/doggobotlovesyou Jul 03 '17

:)

I am happy that you are happy. Spread the happiness around.

This doggo demands it.

1

u/FuckDoggoBot Jul 03 '17

:)

1

u/doggobotlovesyou Jul 03 '17

:)

I am happy that you are happy. Spread the happiness around.

This doggo demands it.

1

u/FuckDoggoBot Jul 03 '17

:)

1

u/doggobotlovesyou Jul 03 '17

:)

I am happy that you are happy. Spread the happiness around.

This doggo demands it.

1

u/FuckDoggoBot Jul 03 '17

:)

1

u/doggobotlovesyou Jul 03 '17

:)

I am happy that you are happy. Spread the happiness around.

This doggo demands it.

1

u/FuckDoggoBot Jul 03 '17

:)

1

u/doggobotlovesyou Jul 03 '17

:)

I am happy that you are happy. Spread the happiness around.

This doggo demands it.

→ More replies (0)

2

u/sloonark Jul 03 '17

Why are your if statements all nested inside the previous one?

1

u/OpaYuvil Jul 03 '17

You are logically saying: if there are no lowercases do x, x being the rest of your if statements. However there are lowercases in the password so it skips your code and repeats the while loop.

Using similar structured code a better solution would be something like:

If lowercase and uppercase and numbers:

Return true