Conversation
Add adding function
Add division and multiply
Add function difference
Correct adding function, delete braces
Edited README add photo
iam28th
left a comment
iam28th
left a comment
There was a problem hiding this comment.
приветствую!
хорошая работа;
с организацией репозитория всё здорово.
По сообшениям коммитов будто бы есть дублирование ("Add image with our team on briefing", "Add meeting image"), и по-возможности старайтесь всегда использовать git вместо загрузки файлов через браузер.
Коммит "Add meeting image" делает не совсем это: https://github.com/BelyashMatvey/HW2_Belyakov/pull/4/files
Также есть небольшие замечания по коду.
Баллы:
readme 10/10 + 10/10 форки + 65/80 код и коммиты + 10 доп.баллы
итого у вас 85 + 10/100 баллов за это дз.
| - Платонова Евгения | ||
|
|
||
| ## Безликое фото команды! | ||
|  |
There was a problem hiding this comment.
readme хорошее, всё по делу
немного тоскливо только что фото безликое)
|
|
||
|
|
||
| def division(a, b): | ||
| return(a/b) |
There was a problem hiding this comment.
здесь где-то скобки есть, где-то нет, с пробелами то же самое
по-хорошему надо единообразно, без скобок и с пробелами вокруг оператора:
| return(a/b) | |
| return a / b |
|
|
||
|
|
||
| def division(a, b): | ||
| return(a/b) |
There was a problem hiding this comment.
про названия функций - лучше тоже было однообразно: add, diff, mult, div вместо adding, division...
(к слову, вычитание обычно пишут как sub от substraction)
| expression = input().split() | ||
| a = int(expression[0]) | ||
| oper = expression[1] | ||
| b = int(expression[2]) |
There was a problem hiding this comment.
по условию калькулятор должен ещё уметь работать с float
т.е. обрабатывть входы типа 3.14 * 2
забегая несколько вперёд, в Питоне проще извиниться, чем просить разрешения;
в соответствии с этим принципом код будет такой:
try:
a = int(expression[0])
except ValueError:
a = float(expression[0])| if oper == '*': | ||
| print(mult(a, b)) | ||
| else: | ||
| print(division(a, b)) |
There was a problem hiding this comment.
лучше сперва сохранить результат в переменную, и напечатать её после всех if-ов
получится короче, и будет проще изменить программу, если вы захотите вместо вывода на печать делать с результатом что-то иное
5 participants
No description provided.