Hw2 bredov #2
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,11 @@ | ||
| Script `calculator.py` performs all basic operations: addition, subtraction, multiplication and division. | ||
|
|
||
| Credits: | ||
|
There was a problem hiding this comment. Круто что вы перечислили обязанности каждого члена команды 👍 |
||
|
|
||
| * Bobkov Gleb A. - implemented division; | ||
| * Bagrova Olga E. - implemented addition; | ||
| * Smertina Elena - implemented multiplication; | ||
| * Matach Dmitri A. - implemented subtraction; | ||
| * Bredov Denis V. - implemented `main()` function & team lead. | ||
|
|
||
|  | ||
|
|
||
| Original file line number | Diff line number | Diff line change | ||||||||
|---|---|---|---|---|---|---|---|---|---|---|
| @@ -0,0 +1,25 @@ | ||||||||||
| def divide(a, b): | ||||||||||
|
There was a problem hiding this comment. Здесь и далее лучше использовать более информативные названия переменных. Названия a и b не отражают содержимого. Лучше было бы, например, num1 и num2. Здесь за это баллы еще не снижаем, но с ДЗ 4 нейминг начнет влиять на оценку. |
||||||||||
| return a/b | ||||||||||
|
|
||||||||||
|
|
||||||||||
| def add(a,b): | ||||||||||
| return a+b | ||||||||||
|
Comment on lines
+4
to
+5
There was a problem hiding this comment. Здесь и в других местах не хватает оформления пробелами. Всё правильно, но на будущее вот так будет выглядеть чуть лучше:
Suggested change
|
||||||||||
|
|
||||||||||
| def multiply(x,y): | ||||||||||
| return(x*y) | ||||||||||
|
There was a problem hiding this comment. В таких случаях оформлять return скобками не принято. Так делают обычно если вы возвращаете несколько объектов |
||||||||||
|
|
||||||||||
| def subtract(eq1,eq2): | ||||||||||
| return(eq1-eq2) | ||||||||||
|
Comment on lines
+10
to
+11
There was a problem hiding this comment.
Suggested change
Что значит eq? 😄 Мы же не уравнения вычитаем |
||||||||||
|
|
||||||||||
| def main(): | ||||||||||
| # eq = [int(i) if i.isdigit() else i for i in input().split()] | ||||||||||
|
There was a problem hiding this comment. Обычно закомменченный код не очень хорошо оставлять в репозитории. |
||||||||||
| eq = input().split() | ||||||||||
|
There was a problem hiding this comment. Здесь было бы супер распаковать eq сразу в 3 элемента: |
||||||||||
| match eq[1]: | ||||||||||
|
There was a problem hiding this comment. Огонь что использовали pattern matching! |
||||||||||
| case "+": | ||||||||||
| return add(int(eq[0]), int(eq[2])) | ||||||||||
| case "-": | ||||||||||
| return subtract(int(eq[0]), int(eq[2])) | ||||||||||
| case "*": | ||||||||||
| return multiply(int(eq[0]), int(eq[2])) | ||||||||||
| case "/": | ||||||||||
| return divide(int(eq[0]), int(eq[2])) | ||||||||||
| main() | ||||||||||
|
There was a problem hiding this comment. Такие вещи еще зачастую пишут через if __name__ == '__main__':
main()На следующей лекции как раз разберем зачем это нужно |
||||||||||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Можно было бы сделать чуть более подробное описание вашего функционала. Пример использования какой-нибудь. Мало ли кто-то поставит ваш калькулятор себе но будет подавать выражение без пробелов ("2+2"). Но это уже совсем чтобы красоту навести.