Skip to content

Luca A Full project#8

Open
lucafungo wants to merge 22 commits intoalexnaylor99:mainfrom
lucafungo:main
Open

Luca A Full project#8
lucafungo wants to merge 22 commits intoalexnaylor99:mainfrom
lucafungo:main

Conversation

@lucafungo
Copy link

No description provided.

main.py Outdated
# making sure the input is an integer
while True:
num = int(input("How many name example you want me to create for you?\n"))
if num is not int:

Choose a reason for hiding this comment

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

Please can you recheck the 'if' statement? I think it's 'if num is int'

main.py Outdated
if len(name) >= 2 and isinstance(name, str):
break
else:
print("don't be a fool a give me a real name!")

Choose a reason for hiding this comment

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

It would be great if you could recheck the minor grammatical errors and punctuations!! Just a suggestion for this outstanding code.

@Zainu-123
Copy link

Excellent thinking and a totally different approach worth experimenting!!

@lucafungo lucafungo changed the title Full project Luca AvFull project Mar 15, 2023
@lucafungo lucafungo changed the title Luca AvFull project Luca A Full project Mar 15, 2023

If you ask me, my favourite is John Chirpy
```
Currently the script generate all surnames in Italian, this can be changed modifying the last two letters of the `url variable` to `en` for English
Copy link
Owner

Choose a reason for hiding this comment

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

Hey Luca, I like the detail you put in your README. Nice work!

@@ -0,0 +1,67 @@
"""
Copy link
Owner

Choose a reason for hiding this comment

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

Good use of the docstring at the start of the file, I really like how you integrated the API call into the script.

# generated_names list
madeup_name = f"\n{name.title()} {nickname.title()}"
print(madeup_name)
generated_names.append(madeup_name)
Copy link
Owner

Choose a reason for hiding this comment

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

Good use of comments in the script! It's clear to me what you're implementing.

# and check is an acceptabe input
while True:
name = str(input("Please provide a first name of your choice:\n"))
if len(name) >= 2 and re.match("^[a-zA-Z]+$", name):
Copy link
Owner

Choose a reason for hiding this comment

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

Nice use of regular expressions!

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