just doeb
Opened this issue · 0 comments
Badsnus commented
- Не по пеп8
- if true - true, else false - false
Это делается в одну строчку
return post(self.api_url, headers=self.headers, data=data).json()["invoice"]["status"] == "success"
3)А если сервер пролагает и выкинет тебе не 200 с джсон ответом и у тебя все упадет? Таких мест кстати много
4)Стоит это вынести из инита и сделать "константой"
5)А какой сокральный смысл возвращать True?
- В таких ситуациях надо юзать словарь
- Тут кстати надо юзать tuple по-хорошему
- Ну ваще так-то инит ничего не возвращает
9)Советую юзать датаклассы, а не таскать по всюду словари