Skip to content

Breakout Room 1 - Implementing Mad Libs Generator#2

Open
anselrognlie wants to merge 1 commit intomainfrom
Breakout-Room-1
Open

Breakout Room 1 - Implementing Mad Libs Generator#2
anselrognlie wants to merge 1 commit intomainfrom
Breakout-Room-1

Conversation

@anselrognlie
Copy link

No description provided.

Copy link
Collaborator

@johendrickson johendrickson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Finished the in-class activity for practicing code reviews.

third_noun = input("Choose a noun: ")
// Displays the story based on the users input
print ("------------------------------------------")
print ("Be kind to your",noun,"- footed", plural_noun)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
print ("Be kind to your",noun,"- footed", plural_noun)
print(f"Be kind to your {noun}-footed {plural_noun}")

Consider using string interpolation instead of string concatenation. This makes it less likely to make formatting mistakes when combining strings.

// Displays the story based on the users input
print ("------------------------------------------")
print ("Be kind to your",noun,"- footed", plural_noun)
print ("For a duck may be somebody's", seond_noun,",")
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
print ("For a duck may be somebody's", seond_noun,",")
print ("For a duck may be somebody's", second_noun,",")

It appears there's a typo here with seond_noun, it should match the variable second_noun or else this will raise a NameError.

@@ -1 +1,23 @@
#Mad Libs Generator Project
//Loop back to this point once code finishes
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
//Loop back to this point once code finishes
#Loop back to this point once code finishes

On lines 2, 5, 12, 22 the incorrect syntax for comments is being used. For Python, we use # for commenting, this looks like JavaScript.

Comment on lines +3 to +4
loop = 1
while (loop < 9):
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
loop = 1
while (loop < 9):
while loop < 9:

This while loop is incorrectly formatted to have parentheses. This is JavaScript syntax and this should be in Python. Another thing is that it looks like this loop only runs a total of 8 times, but the project requires generating 10 stories. In this case, a for loop might be better since we know the exact number of times we need to loop.

Suggested change
loop = 1
while (loop < 9):
while for i in range(10):

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.

3 participants