billiedark/LavaAPI

just doeb

Opened this issue · 0 comments

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