Advertisement
Guest User

Untitled

a guest
Apr 20th, 2018
207
0
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
text 1.81 KB | None | 0 0
  1. Такие вещи лучше в константы:
  2. username='Anonymous',
  3. email='anonymous',
  4. password='b67f71a782accc6e99740fb4d0295572d81c9a15f8e9e24174e0d1a2a1cee7435d1a99833490983eaba65c68022122bcea002e29fb8d76716e97db79741819dc'
  5.  
  6. ######################################################
  7.  
  8. Это:
  9. k = self.__db.get(key)
  10. if k:
  11. return k.decode()
  12. return k
  13.  
  14. я бы сделал так:
  15. k = self.__db.get(key)
  16. return k.decode() if k else k
  17.  
  18. ######################################################
  19.  
  20. return True if self[item] else False --> return bool(self[item])
  21.  
  22. ######################################################
  23.  
  24. В декоратор:
  25. if token not in r:
  26. return {'error': 'Not authorized'}, 401
  27.  
  28. ######################################################
  29.  
  30. Варианты ответов подобного вида:
  31. {
  32. 'id': re.id,
  33. 'expression': re.expression,
  34. 'explanation': re.explanation,
  35. 'author': user.username
  36. }
  37.  
  38. Встречаются тоже не раз, мб есть смысл какойто namedtuple сделать
  39.  
  40. ######################################################
  41.  
  42. if re.author_id != author_id:
  43. abort(404)
  44.  
  45. мб есть смысл отдавать 403, т.к. это права
  46.  
  47. ######################################################
  48.  
  49. Сомнительная хрень перехватывать все ошибки базы:
  50. except (IntegrityError, sqlalchemy.exc.IntegrityError):
  51.  
  52. Лучше конкретизировать
  53.  
  54. ######################################################
  55.  
  56. Такие вещи:
  57. 0 + limit_by * offset, limit_by * offset + limit_by
  58.  
  59. Лучше выносить в отдельные переменные с осмысленным именем
  60.  
  61. ######################################################
Advertisement
Add Comment
Please, Sign In to add comment
Advertisement