Skip to content

test commit#1

Open
pmalibor wants to merge 7 commits into
mainfrom
lesson1
Open

test commit#1
pmalibor wants to merge 7 commits into
mainfrom
lesson1

Conversation

@pmalibor
Copy link
Copy Markdown
Owner

Тестирую сдачу домашнего задания по 1-му уроку

Comment thread 2_seconds.py
m = (ss%3600)//60
# вычисление секунд
s = 0
s = (ss%3600)%60
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

здесь можно просто найти остаток деления на %60 - это и будут оставшиеся секунды


str_n = input('Введите целое положительное число : ')

max = 0
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

имя max зарезервировано в Питоне под встроенную функцию расчета максимального значения. Зарезервированными именами лучше не называть свои переменные.

Comment thread 5_profit.py
# проверка ввода через try -except

try:

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

большие разрывы между конструкциями лучше не оставлять

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.

3 participants