task_2_flat by Yaroslav-GB · Pull Request #4 · smartiqaorg/python_code_reviewer

Yaroslav-GB · 2021-04-28T15:11:41Z

No description provided.

smartiqa · 2021-04-29T07:05:08Z

task_2_flat

+class_f = Flat('kitchen', 'bedroom', 'bathroom')
+class_f.print_rooms()
+class_f.print_flat_size(4, 5, 6)
+class_f.print_bathroom_size(6)
+
+class_k = Kitchen('kitchen', 'to eat')
+print(class_k)
+class_k.get_size()
+
+class_k = Bathroom('bathroom', 'to wash')
+print(class_k)
+class_k.get_size()
+
+class_bb = BigBedroom('big bedroom', 'to sleep')
+print(class_bb)
+
+class_gb = GuestBedroom('guest bedroom', 'to sleep')
+print(class_gb)


Вот так должен выглядеть пример использования наших классов:

# Создаем объект класса Kitchen, передаем ему размер 13 my_kitchen = Kitchen(13) # Создаем объект класса Bedroom, передаем ему размер 18 my_bedroom = Bedroom(18) # Создаем объект класса Bathroom, передаем ему размер 4 my_bathroom = Bathroom(4) # Создаем объект класса Flat, передаем ему созданные выше объекты my_flat = Flat(my_kitchen, my_bedroom, my_bathroom) # Обратите внимание, что методы ниже не принимают никаких входных данных, # так как все эти данные уже хранятся внутри объекта класса Flat (а именно внутри объектов my_kitchen, my_bedroom и my_bathroom, которые мы передали во время создания объекта my_flat) my_flat.print_rooms() my_flat.print_flat_size() my_flat.print_kitchen_size() my_flat.print_bedroom_size() my_flat.print_bathroom_size()

smartiqa · 2021-04-29T07:06:00Z

task_2_flat

+
+    def get_size(self):
+        user_size = input(f'Enter size of {self.name} ')
+        print(f'{self.name} is {user_size}(sq.m.)')


Этот класс пока убираем, пишем код так, как-будто доп заданий 1-2 к задаче не было

smartiqa · 2021-04-29T07:06:59Z

task_2_flat

+class BigBedroom(Room):
+
+    def __str__(self):
+        return f'{self.name} Спальня с двуспальной кроватью'
+
+class GuestBedroom(Room):
+
+    def __str__(self):
+        return f'{self.name} Гостевая спальня'


Эти классы тоже пока убираем

task_2_flat

smartiqa · 2021-04-29T07:15:05Z

task_2_flat

+class Kitchen(Room):
+    pass


Данный класс реализуем в таком виде:

class Kitchen: TITLE = 'Kitchen' def __init__(self, size): self.__size = size self.__purpose = 'To eat' def get_size(self): return self.__size

На что тут обратить внимание?

Есть два приватных динамических поля __size и __purpose. Поле __size заполняется значением из параметра.

Есть одно статическое поле для хранения названия комнаты TITLE. Зачем помещать название комнаты в константу? Чтобы защитить себя от возможных опечаток. И кроме того если необходимо будет делать изменения, то достаточно просто изменить содержимое константы в одном месте, а не делать правки по всей программе.

Метод get_size() возвращает содержимое приватного поля __size (обратите внимание, что он никуда ничего не печатает)

smartiqa · 2021-04-29T07:15:56Z

task_2_flat

+    def print_rooms(self):
+        print (f'{self.kitchen, self.bedroom, self.bedroom}')
+
+    def print_flat_size(self, kitchen_size, bedroom_size, bathroom_size):
+        size = self._calculate_flat_size(kitchen_size, bedroom_size, bathroom_size)
+        print(f'Flat {size}sq.m.')
+
+    def print_kitchen_size(self, size):
+        print(f'{self.kitchen} {size}sq.m.')
+
+    def print_bedroom_size(self, size):
+        print(f'{self.bedroom} {size}sq.m.')
+
+    def print_bathroom_size(self, size):
+        print(f'{self.bathroom} {size}sq.m.')
+
+    def _calculate_flat_size(self, kitchen_size, bedroom_size, bathroom_size):
+        return f'{kitchen_size + bathroom_size + bedroom_size}'


Эти методы можете пока закомментить - мы к ним вернемся, когда верно реализуем классы Kitchen, Bedroom, Bathroom

smartiqa · 2021-04-29T07:16:25Z

task_2_flat

+class Bedroom(Room):
+    pass
+
+class Bathroom(Room):
+    pass


Реализуйте данные классы по примеру выше для класса Kitchen

1 Этап.

task_2_flat

+    #    print(f'{self.name} is {user_size}(sq.m.)')
+
+
+class Kitchen(Room):


task_2_flat

smartiqa · 2021-05-12T12:18:33Z

task_2_flat

+    #def print_rooms(self):
+    #    print (f'{self.kitchen, self.bedroom, self.bedroom}')
+
+    #def print_flat_size(self, kitchen_size, bedroom_size, bathroom_size):
+    #    size = self._calculate_flat_size(kitchen_size, bedroom_size, bathroom_size)
+    #    print(f'Flat {size}sq.m.')
+
+    #def print_kitchen_size(self, size):
+    #    print(f'{self.kitchen} {size}sq.m.')
+
+    #def print_bedroom_size(self, size):
+    #    print(f'{self.bedroom} {size}sq.m.')
+
+    #def print_bathroom_size(self, size):
+    #    print(f'{self.bathroom} {size}sq.m.')
+
+    #def _calculate_flat_size(self, kitchen_size, bedroom_size, bathroom_size):
+    #    return f'{kitchen_size + bathroom_size + bedroom_size}'


Когда закончите с предыдущими двумя комментариями, можно переходить к этапу 4. Реализуем до конца класс Flat, а именно - добавляем все те методы, которые закомментили:

def print_rooms(self): def print_flat_size(self): def print_kitchen_size(self): def print_bedroom_size(self): def print_bathroom_size(self): def _calculate_flat_size(self):

На что обратить внимание?

Еще раз акцентирую внимание - делаем только после того, как сделаем изменения по первым двум пунктам (так как для реализации текущих методов нужен метод get_size() классов Kitchen, Bedroom, Bathroom)

Методы, которые начинаются с print выводят строки на печать. А метод _calculate_flat_size() возвращает размер квартиры, то есть возвращает число.

Для получения размеров каждой из комнат используйте соответствующий метод get_size(). Например:

def print_kitchen_size(self): print(f'Kitchen size: {self.kitchen.get_size()}')

Плюс добавляем примеры использования свежесозданных методов:

my_kitchen = Kitchen(13) my_bedroom = Bedroom(18) my_bathroom = Bathroom(4) my_flat = Flat(my_kitchen, my_bedroom, my_bathroom) my_flat.print_rooms() my_flat.print_flat_size() my_flat.print_kitchen_size() my_flat.print_bedroom_size() my_flat.print_bathroom_size()

smartiqa · 2021-05-13T10:31:56Z

task_2_flat

+    def _calculate_flat_size(self, kitchen_size, bedroom_size, bathroom_size):
+        return f'{self.kitchen.get_size() + self.bathroom.get_size() + self.bedroom.get_size()}'


В чем смысл переданных в метод параметров? Класс Flat хранит внутри себя все три объекта для каждой комнаты:

self.kitchen = kitchen self.bedroom = bedroom self.bathroom = bathroom

Поэтому ничего внутрь передавать не надо. Используйте то, что уже есть внутри класса. Собственно именно это вы и делаете

self.kitchen.get_size() + self.bathroom.get_size() + self.bedroom.get_size()

А параметры kitchen_size, bedroom_size, bathroom_size не нужны

smartiqa · 2021-05-13T10:34:09Z

task_2_flat

+    def print_flat_size(self, kitchen_size, bedroom_size, bathroom_size):
+        flat_size = self._calculate_flat_size(kitchen_size, bedroom_size, bathroom_size)
+        print(f'Flat size: {flat_size}')


Аналогично - параметры не нужны

smartiqa · 2021-05-13T10:36:05Z

task_2_flat

+    def print_rooms(self):
+        print(f'{self.kitchen, self.bedroom, self.bedroom}')


Когда вы передаете на print сами объекты, то на выходе получаете:

(<__main__.Kitchen object at 0x10a6f8490>, <__main__.Bedroom object at 0x10a6f8810>, <__main__.Bedroom object at 0x10a6f8810>)

Это не читабельно. Воспользуйтесь свойством TITLE для каждого объекта, например:

Kitchen.TITLE

Подзадание №1

smartiqa · 2021-06-05T10:57:10Z

task_2_flat

+        self.__purpose = 'To wash'
+
+
+class BigBedroom(Room):


А в чем выгода наличия классов BigBedroom и GuestBedroom, если они наследуются от Room? Ни в чем. Надо наследовать их от класса Bedroom, тогда можно унаследовать например поле __purpose - ведь хоть это разные виды спален, но все равно назначение To sleep не меняется, это все равно спальни.

Плюс каждому из этих классов можно добавить свой функционал. Например, у класса BigBedroom можно не передавать в конструктор размер, а задать его внутри(ведь большой не может считаться спальня с любым размером). Например так:

class BigBedroom(Bedroom): TITLE = 'BigBedroom' SIZE = 20 def __init__(self): super().__init__(SIZE)

Create task_2_flat

b64a802

smartiqa requested changes Apr 29, 2021

View reviewed changes

Update task_2_flat

49e2b96

1 Этап.

smartiqa requested changes May 12, 2021

View reviewed changes

Update task_2_flat

6fe3d91

smartiqa requested changes May 13, 2021

View reviewed changes

Yaroslav-GB added 3 commits May 18, 2021 10:14

Update task_2_flat

19dc5dd

Update task_2_flat

4aea48d

Подзадание №1

Update task_2_flat

068d237

smartiqa requested changes Jun 5, 2021

View reviewed changes

		# print(f'{self.name} is {user_size}(sq.m.)')


		class Kitchen(Room):

		def _calculate_flat_size(self, kitchen_size, bedroom_size, bathroom_size):
		return f'{self.kitchen.get_size() + self.bathroom.get_size() + self.bedroom.get_size()}'

		def print_rooms(self):
		print(f'{self.kitchen, self.bedroom, self.bedroom}')

Conversation

Yaroslav-GB commented Apr 28, 2021

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

This comment was marked as outdated.

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

smartiqa May 13, 2021 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

smartiqa May 13, 2021 •

edited

Loading