The code is quite clear in what it does, good job.
# We define a function here.
def ...
...
... = {} # Empty dictionary
...
... = False # Set the flag to 'False'
Many of these comments are not so useful. When I see the code, I can see that the indicator is set to false or that a function has been defined. Generally, I would comment on the reason why there is a code snippet. If you deleted a comment next to some code, could you figure out what the code does? If so, the comment is probably not useful.
def display_comments():
"""Displays User Input (comments)"""
if user_comments: # If the dictionary 'user_comments' has at least one key value pair then execute this block of code.
print("nTop Comments:")
for username, comment in user_comments.items(): # Define a for loop for each key value pair in the dictionary 'user_comments'.
print(f"nt{username}: {comment}.") # Print each key value pair.
This looks pretty good. It does what it says it does. Delete comments online and it's golden.
banned_users = ('pete', 'jack', ...
...
if username.lower() in banned_users:
Extracting banned users to a list is good. A problem could arise here if you add one with a capital letter to the forbidden list. You will not be prevented from commenting! In general, there are two ways to avoid this problem.
- Make a comparison between upper and lower case. In Java, this would be done with username.equalsIgnoreCase (banned_user)
- Normalize both chains and then compare. This is the recommended method in Python.
Therefore, I would suggest making sure that all prohibited users have been lowercase
banned_users = ('pete', 'jack', ...
banned_users = (banned_user.lower() for banned_user in banned_users)
If you think you will ever need to deal with international usernames, you may find the case fold interesting. It will normalize the strings more aggressively than the lower will.
One result of lowering names is that it reduces the number of available user names. If Ben Dor and B. Endor try to register as BenDor and BEndor respectively, only one can get the name. That's fine? Will this ever be a problem?
repeat = input(continue_prompt)
if repeat == 'no':
This is a lowercase place as an answer of "NO" or "No" to clearly indicate that the user has finished.
user_comments = {}
...
user_comments(username) = comment
Since comments are stored in a dictionary, each username will only have one comment, the last one they have made. For example, if the chat was meant to be
A: where is the coal?
B: walk west for 2 mins then
B: north for 1 min
Comments appear as
A: where is the coal?
B: north for 1 min
Is that intentional? If not, you can change to use a list that contains the username and comment.
comments = ()
...
comments.append((username, comment))
and display_comments seems almost identical
for username, comment in comments:
...
