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

Skip to content

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