Advertisement
lllumineux

Untitled

Nov 17th, 2020
123
0
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
text 3.10 KB | None | 0 0
  1. Тупые ошибки:
  2.  
  3. Cottage в первой строке начинается с русской "С"... (прикол или чего?)
  4.  
  5. Cottage(), Customer() - не нужны скобки
  6.  
  7. 2 строка слишком длинная (21 и 53 тоже лучше сократить)
  8.  
  9. на 86 строчке нет двоеточия
  10.  
  11. на 64 строке неинициализированная переменная, которая, видимо должна была быть инициализирована на 55 строке
  12.  
  13. неициниализированны глобальные константы!
  14.  
  15. нет тайп-хинтов и комментариев, поэтому, чтобы понять задумку, нужно подумать
  16.  
  17. во всех функциях, кроме двух, неиспользуемые переменные (у класса Customer даже в init'е)
  18.  
  19. непонятно, зачем нужны все методы, кроме get_food_price() и get_drink_price() - они тупо возвращают значение (так ещё и зачем-то сначала его спрашивают)...
  20.  
  21. однако сами get_food_price() и get_drink_price() никак не связаны с классом и непонятно, что в нём делают (they should be static)
  22.  
  23. --------------------------------------------------------------------------------
  24.  
  25. Умные ошибки:
  26.  
  27. наследуются классы AdditionalServices от Cottage и Food, Drinks от Customer - ок, в первом случае просто назвать надо было нормально и добавить super().__init__(), а вот во втором случае вообще нет смысла делать наследование, ибо ни Food, ни Drinks не являются подвидом класса Customer (иными словами - у Customer нет значения snacks (к примеру), а у Drinks тем более нет значений full_name, budget, sleep_place_number и тд)
  28.  
  29. если я правильно понял, что планировалось сделать, то каждый из additional services должен быть отдельным классом, наследованным от AdditionalService и у каждого должны быть свои свойства
  30.  
  31. зачем называть методы get_drinks_price() и get_food_price(), если они итак находятся внутри класса Drinks и Food соответственно (нужно было просто назвать get_price() в обоих случаях)
  32.  
  33. --------------------------------------------------------------------------------
  34.  
  35. ну и получается, что тут нужно теперь добавить свойст и методов к классам из main_new.py, чтобы подходило под условие задания
  36.  
  37. НО хочется респект выразить за нормальные названия переменных (почти везде)
Advertisement
Add Comment
Please, Sign In to add comment
Advertisement