Conversation
Add plus function
Add decimal and minus functions
Add product function
Add devision operation
eksytnik
left a comment
eksytnik
left a comment
There was a problem hiding this comment.
Хорошая работа! Отлично выглядят коммиты, прекрасные сообщения к ним. Есть небольшие замечания по коду в комментариях, пока это просто на заметку, но в будущих заданиях уже может стоить баллов, обратите внимание.
Баллы: 10/10 (README) + 10/10 (ветки) + 80/80 (код и коммиты) + 10 доп. =
= 100/100 + 10 доп.
(тимлид, не забудь показать ревью всей команде)
| return a * b | ||
|
|
||
| def divide(a, b): | ||
| if b!=0: |
There was a problem hiding this comment.
Не забывайте про отступы!
| if b!=0: | |
| if b != 0: |
| if b!=0: | ||
| return(a / b) | ||
| else: | ||
| return('делить на 0 нельзя!') |
There was a problem hiding this comment.
Здорово, что об этом подумали, но это не оптимальное решение. Пока это не ошибка, мы еще поговорим позже, что правильно с этим делать, но как базовый принцип - в большинстве случаев стоит придерживаться однородности типа возвращаемых данных. Это не 100% правило, но база, если функция отдаёт число - она должна отдавать число, иногда может быть None - но не строка.
Но да, что с этим делать - еще будет впереди :)
| a = Decimal(elements[0]) | ||
| b = Decimal(elements[2]) |
There was a problem hiding this comment.
a, b - однобуквенные переменные, так лучше не делать. Тут бы хорошо подошло num1, num2 или что-то подобное на свой вкус.
| result = product(a, b) | ||
| elif (elements[1] == '/'): | ||
| result = divide(a, b) | ||
| return result |
There was a problem hiding this comment.
получать результат и печатать его на экран
Внимательнее к тексту задания, тут это не принципиально, но мелочи бывают очень важны!
5 participants
No description provided.