User settings #47
User settings #47
Conversation
user_settings.py
Outdated
| if __name__ == '__main__': | ||
| ... |
There was a problem hiding this comment.
Tutaj if _name__ == '__main__' nie jest potrzebne. Stosujemy to tylko, jeśli mamy jakiś kod, który nie jest zawarty w funkcjach/klasach, bo on nam się wtedy odpali od razu przy pobieraniu pliku do innego modułu.
There was a problem hiding this comment.
Do tego przypominam jeszcze raz o pustej linijce na końcu pliku.
user_settings.py
Outdated
| if len(json_data) == 0: | ||
| return "There are no users" | ||
| else: | ||
| return [(self.user["user_id"], self.user["name"] for self.user in json_data["users"]] |
There was a problem hiding this comment.
Sprawdź dokładnie, jak działają comprehension lists. Tu niepotrzebne jest to self, bo zobacz że wtedy Python będzie Ci szukał w tym self, które przekazujesz do tej metody (czyli tak naprawdę do Twojego obiektu klasy UserSettings atrybutu user, którego nie masz. Ty chcesz po prostu przejść po elementach listy json_data["users"].
user_settings.py
Outdated
| print("User not found") | ||
|
|
||
| # Finding user | ||
| def find_user(self, user_id: int, name: str): |
user_settings.py
Outdated
|
|
||
| # Querying list of users | ||
| def get_users_list(self): | ||
| json_data = self.load_json_user_file() |
There was a problem hiding this comment.
Część tych wszystkich funkcji będziemy chcieli w przyszłości reużyć, a nie będziemy już mieli jsona. Poza tym i tak te nazwy zmiennych powinny bardziej opisywać, co dokładnie pobierasz, niż tylko json_data. Proszę zmień te wszystkie nazwy na coś bardziej intuicyjnego, np users.
User seetings:
https://jira.is-academy.pl/browse/JPYDZR6SCD-40