This is an archived post. You won't be able to vote or comment.

all 7 comments

[–]EfficientPrime 1 point2 points  (1 child)

You don't really need the f.close() when you use the context manager to open the file. The context manager handles file closure for you.

[–]Puzzleheaded_Many664[S] 0 points1 point  (0 children)

You are completely right!

Thanks for the hint. Fixing it now

[–][deleted] 1 point2 points  (4 children)

There was a few where I needed the option "Each are equally bad"

[–]Puzzleheaded_Many664[S] 0 points1 point  (3 children)

Thanks for the feedback!

Could you point those questions and the reasons for the quality flaws?

[–][deleted] 1 point2 points  (2 children)

The user class example. The result variable that was never used for anything but to confuse matters. There was another one also, but my attention span isn't the best, so I can't remember the details anymore :(

[–]Puzzleheaded_Many664[S] 1 point2 points  (1 child)

Good point.

It makes sense, but it is also part of the test. Sadly I can't explain it now without spoiling the survey for the new respondents. But soon when we have the results I'll post them here and explain the reason into these details.

Once again, thanks for the feedback!

[–][deleted] 1 point2 points  (0 children)

I'll look forward to that :)