8000 Merge pull request #2 from shoshan/master · mayusGomez/Clean-Code-in-Python@d8aea69 · GitHub
[go: up one dir, main page]

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Commit d8aea69

Browse files
authored
Merge pull request PacktPublishing#2 from shoshan/master
Fix USERS f-string for last name.
2 parents 6e58d52 + 8f43622 commit d8aea69

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

Chapter03/packing_1.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
"""
55

66

7-
USERS = [(i, f"first_name_{i}", "last_name_{i}") for i in range(1_000)]
7+
USERS = [(i, f"first_name_{i}", f"last_name_{i}") for i in range(1_000)]
88

99

1010
class User:
@@ -13,6 +13,9 @@ def __init__(self, user_id, first_name, last_name):
1313
self.first_name = first_name
1414
self.last_name = last_name
1515

16+
def __repr__(self):
17+
return f"{self.__class__.__name__}({self.user_id!r}, {self.first_name!r}, {self.last_name!r})"
18+
1619

1720
def bad_users_from_rows(dbrows) -> list:
1821
"""A bad case (non-pythonic) of creating ``User``s from DB rows."""

0 commit comments

Comments
 (0)
0