Nekrasova's team pull-request #6

nekrasovadasha22 · 2023-09-14T18:01:15Z

Finished calculator.py project

This reverts commit 32381db.

Hw2 sapozhnikov pull request approved

Add substruct function pull request approved

Add def divide in calculator.py pull request approved

Adding addition function to calculator.py fixed merge conflict, pull request approved

Added team photo

pavlovanadia · 2023-09-19T22:33:13Z

HW2_nekrasova/calculator.py

Не очень хорошо, что main() определяется до определения подфункций, вызываемых main(). Лучше соблюдать логику кода: сначала определение подфункций, а уже потом больших функций.

pavlovanadia · 2023-09-19T22:33:19Z

HW2_nekrasova/calculator.py

@@ -0,0 +1,62 @@
+def main(expression: str):


Ого, аннотация типов!👍

pavlovanadia · 2023-09-19T22:33:23Z

HW2_nekrasova/calculator.py

@@ -0,0 +1,62 @@
+def main(expression: str):
+    expression_array = expression.split(' ')


Suggested change

expression_array = expression.split(' ')

expression_array = expression.split()

Если хочется сделать сплит по пробельным символам, то советую такую конструкцию. В случае введения двойного пробела ваш вариант вставит " " в список, а предложенная - нет. 😌

pavlovanadia · 2023-09-19T22:33:26Z

HW2_nekrasova/calculator.py

+    expression_array = expression.split(' ')
+    array_of_operands = []
+
+    try:


Очень уместное и лаконичное использование конструкции try-except для определения типов👍

pavlovanadia · 2023-09-19T22:33:29Z

HW2_nekrasova/calculator.py

+
+    try:
+        left_int = int(expression_array[0])
+        array_of_operands.append(left_int)


Suggested change

array_of_operands.append(left_int)

nums.append(left_int)

Логичнее было бы назвать переменную списком чисел, т.к. операнд - это +, -, /, *, и т.п.

pavlovanadia · 2023-09-19T22:33:36Z

HW2_nekrasova/calculator.py

+       left_float = float(expression_array[0])
+       array_of_operands.append(left_float)


Suggested change

left_float = float(expression_array[0])

array_of_operands.append(left_float)

left_float = float(expression_array[0])

array_of_operands.append(left_float)

Куда потерялся пробел?😢

pavlovanadia · 2023-09-19T22:33:41Z

HW2_nekrasova/calculator.py

+       right_float = float(expression_array[2])
+       array_of_operands.append(right_float)


Suggested change

right_float = float(expression_array[2])

array_of_operands.append(right_float)

right_float = float(expression_array[2])

array_of_operands.append(right_float)

Снова пробел...

pavlovanadia · 2023-09-19T22:33:45Z

HW2_nekrasova/calculator.py

+
+    answer = None
+
+    if expression_array[1] == '+':


Уместнее было бы записать операнд в переменную и сравнивать её значение, но это так, перфекционизм

pavlovanadia · 2023-09-19T22:33:52Z

HW2_nekrasova/calculator.py

+
+
+def add(arr_of_operands: list):
+    return arr_of_operands[0]+arr_of_operands[1]


Suggested change

return arr_of_operands[0]+arr_of_operands[1]

return arr_of_operands[0] + arr_of_operands[1]

По PEP8 тут нужны пробелы)) Сейчас на оценку не влияет, но в будущем обращайте внимание)

pavlovanadia · 2023-09-19T22:33:55Z

HW2_nekrasova/calculator.py

+def add(arr_of_operands: list):
+    return arr_of_operands[0]+arr_of_operands[1]
+
+def substruct(arr_of_operands: list):


Suggested change

def substruct(arr_of_operands: list):

def substract(arr_of_operands: list):

Нейминг😌

pavlovanadia · 2023-09-19T22:33:59Z

HW2_nekrasova/calculator.py

+    return arr_of_operands[0]+arr_of_operands[1]
+
+def substruct(arr_of_operands: list):
+    # substruct


Такие комментарии лучше удалять из кода, если они не несут дополнительной информации 😌

pavlovanadia · 2023-09-19T22:34:01Z

HW2_nekrasova/calculator.py

+
+def substruct(arr_of_operands: list):
+    # substruct
+    return arr_of_operands[0]-arr_of_operands[1]


Suggested change

return arr_of_operands[0]-arr_of_operands[1]

return arr_of_operands[0] - arr_of_operands[1]

pavlovanadia · 2023-09-19T22:34:04Z

HW2_nekrasova/calculator.py

+
+def divide(arr_of_operands: list):
+    if arr_of_operands[1] != 0:
+        answer = arr_of_operands[0]/arr_of_operands[1]


Suggested change

answer = arr_of_operands[0]/arr_of_operands[1]

answer = arr_of_operands[0] / arr_of_operands[1]

pavlovanadia · 2023-09-19T22:34:12Z

HW2_nekrasova/calculator.py

+    else:
+        answer = 'Деление на 0!'


Хороший вариант сказать пользователю о делении на ноль!)

pavlovanadia · 2023-09-19T22:34:15Z

HW2_nekrasova/calculator.py

+
+
+current_expression = input()
+main(current_expression)


Suggested change

main(current_expression)

if __name__ == "__main__":

current_expression = input()

main(current_expression)

pavlovanadia · 2023-09-19T22:34:54Z

HW2_nekrasova/README.md

+This project is leaded by nekrasovadasha22 with contributions from the following people: 
+- grishchenkoira
+- anisssum
+- NSapozhnikov
+- Komarov I.


README малоинформативный. Не указан вклад каждого из членов команды, нет примеров использования программы: что она получает на вход и что выдает на выходе.
Здорово, что дана команда для вызова из терминала!

pavlovanadia · 2023-09-19T22:47:58Z

HW2_nekrasova/calculator.py

Хорошая работа!
Основное задание - работа с гитом в команде - выполнено отлично; небольшие комментарии по стилю и логике кода, PEP8, отступам и неймингу переменных - это советы на будущее (в следующих дз на это будем обращать внимание).

Так что молодцы!

Итог:
Функции: 1.6 * 5 = 8
Ветки и пулл-реквесты: 1 балл
README: 0.5
Доп. балл за фото: 1

Итог: 10.5

Daria and others added 21 commits September 13, 2023 18:31

Created calculator.py with main function

f7ba5dc

Update calculator.py

e9fc7ba

Update calculator.py

32381db

Revert "Update calculator.py"

d3ca03d

This reverts commit 32381db.

Add multiple function to calculator.py

b672eb9

Minor code review

4f7eae8

Add substruct function

22d6854

Add def divide in calculator.py

98ee437

Add modification in def divide

9c2723d

Add new modification in def divide

1b1799a

Create README.md

aa0d07d

Update README.md

477c539

Add addition function

beb1684

Fix string in add function

a94dcb2

Merge pull request #5 from NSapozhnikov/HW2_Sapozhnikov

268db30

Hw2 sapozhnikov pull request approved

Merge pull request #6 from anisssum/HW2_chesnokova

6cc9aea

Add substruct function pull request approved

Merge pull request #7 from grishchenkoira/HW2_grishchenko

7f59f19

Add def divide in calculator.py pull request approved

Merge branch 'HW2_nekrasova' into dev_branch

a35c2d6

Merge pull request #8 from IlyaKomarkov/dev_branch

61d8bb7

Adding addition function to calculator.py fixed merge conflict, pull request approved

fix little bugs

d2d7380

Update README.md

977549a

Added team photo

pavlovanadia reviewed Sep 19, 2023

View reviewed changes

HW2_nekrasova/calculator.py

@@ -0,0 +1,62 @@

def main(expression: str):

Copy link

pavlovanadia Sep 19, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ого, аннотация типов!👍

pavlovanadia reviewed Sep 19, 2023

View reviewed changes

		@@ -0,0 +1,62 @@
		def main(expression: str):
		expression_array = expression.split(' ')

	expression_array = expression.split(' ')
	expression_array = expression.split()

		left_float = float(expression_array[0])
		array_of_operands.append(left_float)

		right_float = float(expression_array[2])
		array_of_operands.append(right_float)



		def add(arr_of_operands: list):
		return arr_of_operands[0]+arr_of_operands[1]

	return arr_of_operands[0]+arr_of_operands[1]
	return arr_of_operands[0] + arr_of_operands[1]

	def substruct(arr_of_operands: list):
	def substract(arr_of_operands: list):

	return arr_of_operands[0]-arr_of_operands[1]
	return arr_of_operands[0] - arr_of_operands[1]

	answer = arr_of_operands[0]/arr_of_operands[1]
	answer = arr_of_operands[0] / arr_of_operands[1]

Nekrasova's team pull-request #6

Are you sure you want to change the base?

Nekrasova's team pull-request #6

Uh oh!

Conversation

nekrasovadasha22 commented Sep 14, 2023

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!