Check tests #2
Check tests #2
Conversation
There was a problem hiding this comment.
Почти все хорошо.
В целом, print-ы для отладки можно оставить, но в рабочем коде они обычно не нужны - во-первых, их некому читать, во-вторых, print - не очень гибкий инструмент для этой цели. Когда у тебя есть задача сделать функцию, которая выводит данные на экран - print подходит, а когда задача написать фукнцию, которая вычисляет данные - print внутри не нужен.
Ну и в 3 решении небольшая бага. p1 и p2 берутся не оттуда.
homework/tests.py
Outdated
| @@ -5,7 +5,9 @@ def test_even_fucntion(): | |||
| """ | |||
|
|
|||
| def even_filter(*args): | |||
| pass | |||
| even = [arg for arg in args if arg % 2 == 0] | |||
| print(even) | |||
There was a problem hiding this comment.
print() здесь лишний. В рабочем коде у тебя функция будет печатать на консоль какие-то данные, хотя должна только возвращать значения (return).
homework/tests.py
Outdated
| @@ -44,7 +51,9 @@ def __init__(self, p1, p2): | |||
| pass | |||
|
|
|||
| def length(self): | |||
| return 0 | |||
| dist = math.hypot(p2.x - p1.x, p2.y - p1.y) | |||
There was a problem hiding this comment.
Здесь идет работа с переменными снаружи класса Segment (т.е. у тебя класс сломается, как только их переименуют в point1 и point2). Нужно честно записать их в self в __init__-методе, и работать с ними также через self.
No description provided.