Conversation
Add multiplication function
Add multiplication function
eksytnik
left a comment
eksytnik
left a comment
There was a problem hiding this comment.
Привет!
В целом - всё отлично. Небольшие замечания по сообщениям к коммитам:
- Формулировка "main.py" - она всё-таки выглядит как добавление нового файла в проект, а не новой функции в один из файлов проекта, это сбивает с толку. Лучше писать Add multiplication function.
- Они должны быть с большой буквы.
Сам код очень аккуратно сделан, лайк! У меня там много комментариев, но они все про best practice, а не про ошибки.
Баллы: 10/10 (README) + 10/10 (Forks) + 78/80 (код и коммиты) + 10 доп. =
= 98/100 + 10 доп.
|
|
||
| *Эвелина Кудашева*, *Анастасия Шипунова*, *Валерия Афанасьева*, *Иван Пацкан* *(Team Leader)* | ||
|
|
||
|  |
| #Функция умножения: | ||
| def multiplication(a, b): | ||
| res = a * b | ||
| return res |
There was a problem hiding this comment.
В остальных функциях просто return a * b.
Не ошибка, но на будущее лучше сохранять единообразие в оформлении кода.
| #Функция деления | ||
| def division(a, b): | ||
| if b == 0: | ||
| return 'На ноль делить нельзя' |
There was a problem hiding this comment.
- Супер, что об этом кейсе подумали.
- Мы еще пройдём, как с этим правильно обращаться, на будущее - возвращать строку не стоит. Если функция возвращает результат деления и ожидается, что там будет число - получить в ответ какую-то строку это очень непредсказуемое поведение, которое в остальном коде очень сложно учесть. Если это никак не обрабатывать - будет ошибка деления на 0 и это хорошо, с ошибками прекрасно можно работать - про это чуть позже будет материал.
Пока мы не проходили эту тему, поэтому это просто замечание на будущее, на баллы не влияет!
| print(res) | ||
| elif elements[1] == '-': | ||
| res = subtraction(elements[0], elements[2]) | ||
| print(res) |
There was a problem hiding this comment.
Чуть лучше было бы печатать один раз в конце. Результат и так кладется в переменную, if-ы взаимоисключающие.
Так код получается чуть короче, чуть понятнее - и его проще изменить, если решили в какой-то момент не печатать, а что-то другое с результатом делать.
|
|
||
|
|
||
| print('''Добро пожаловать в калькулятор. Введите строку в таком виде: "5 + 9". | ||
| Чтобы закончить работу с калькулятором, введите "стоп".''') |
There was a problem hiding this comment.
👍 Супер, что сделали флаг выхода и повтор программы
|
|
||
|
|
||
| #Функция сложения | ||
| def addition(a, b): |
There was a problem hiding this comment.
Хорошая практика - использовать не a, b, а что-то в духе num1, num2. "Говорящие" сами за себя переменные помогают читабельности кода - вот эта штука у нас складывает два числа.
Еще аннотации помогают, но про них позже, и они не всегда есть - а названия переменных не выкинешь из кода с:
4 participants
No description provided.