Skip to content

Homework, № 1-7#1

Open
analogina wants to merge 3 commits intomasterfrom
homework
Open

Homework, № 1-7#1
analogina wants to merge 3 commits intomasterfrom
homework

Conversation

@analogina
Copy link
Owner

Выполнена домашняя работа, номера 1 - 7.

Copy link

@LadyCaster LadyCaster left a comment

Choose a reason for hiding this comment

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

Отличная работа! Ты молодец! Если доделаешь работу с ботом и модулем ephem, дай знать, я еще раз посмотрю работу

except ValueError:
result = 'Вы ввели не число.'

else:

Choose a reason for hiding this comment

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

отличное использование try/except/else, но обычно мы стараемся ограничится обычным try/except, потому что код либо отработает и переменная, которую ты заполняешь внутри, будет с правильным значением использоваться дальше, либо программа упадет
Блок else включается очень опционально. Это действительно действия, которые мы хотим выполнить, если исключение не было поднято. Но в большинстве случаев мы просто продолжим писать код вне блока try/except
За мою практику почти никогда выход из блока не использовался через else, поэтому можешь смело оставлять вот такой вариант
Например:

try:
     user_age = int(user_input)
except ValueError:
     result = 'Вы ввели не число.'
if user_age <= 6:
    ...

И проверь, пожалуйста, сколько пробелов по умолчанию в py файлах выставляет твой редактор, так как сейчас в коде у тебя по 2 пробела вместо 4

def check_func(str1, str2):
if type(str1) != str or type(str2) != str:
return 0
elif (str1 == str2):

Choose a reason for hiding this comment

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

круглые скобки вокруг выражения сравнения совершенно не обязательны
они нужны в других языках, но не в python)


for phone in sold:
product_name = phone['product']
total_sold = sum(phone['items_sold']) #сумма продаж

Choose a reason for hiding this comment

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

отличное использование sum, лайк!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments