Skip to content

hw_7 done#6

Open
bobereek wants to merge 3 commits intomainfrom
hw_7
Open

hw_7 done#6
bobereek wants to merge 3 commits intomainfrom
hw_7

Conversation

@bobereek
Copy link
Copy Markdown
Owner

No description provided.

@rmusbuss
Copy link
Copy Markdown

Привет! Спасибо большое за работу! Не хватает примеров, тестов: например, что будет в результате выполнения примера для корги из задания

По критериям (-1 балл):
[-1] Отсутствие if name == 'main'

DoD (-7 баллов)
при обращении к вложенным атрибутам программа выводит репр объекта <issue_1.Advert object ...>, но не его содержимое! (то есть при обращении к example.location вместо example.location.address)
Не выбрасывается исключение на отрицательный атрибут price, когда цена сидит в json'е. Вероятно ошибка в том, что property применяется не всегда (то есть для любых is_root)
Пример:

    iphone_str_init = '{"title": "iPhone", "price": -100}'
    iphone_ad_init = Advert(json.loads(iphone_str_init))
    print(iphone_ad_init.price)
## out: -100

Код программы падает при попытке получить атрибут price, когда его нет в изначальном json'е, а мы пытаемся посмотреть его значение (оно должно быть 0): Exception: 'Advert' object has no attribute 'price_'
Когда печатаешь пример из задания с ColorizedAdvert, то вложенные атрибуты json'а выводятся в виде repr'а

Обрати внимание (тут замечание, без снятия баллов):
по критериям [-3] В PR лишние файлы - .рус; .idea; файлы прошлых домашек
Как вариант - создавай новую ветку для каждой домашки отдельно

Итого: 10 - 1 - 7 = 2 балла

@bobereek
Copy link
Copy Markdown
Owner Author

не совсем понял проблему с лишними файлами, тк для каждой домашки создаю отдельные ветки и вроде ничего не должно было просочиться

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants