Nekrasova's team pull-request by nekrasovadasha22 · Pull Request #6 · Python-BI-2023/HW2_Git_and_python

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

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

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

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


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

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

@@ -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

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


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

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

+
+    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

+       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

+       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

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


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

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

+
+
+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

+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

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


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

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

+
+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

+
+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

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


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

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

+
+
+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

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


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

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

Хорошая работа!
Основное задание - работа с гитом в команде - выполнено отлично; небольшие комментарии по стилю и логике кода, 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

		@@ -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]

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!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants