Skip to content

homework: lectures 2, practice 3#12

Open
iana-golovchenko wants to merge 4 commits into
mainfrom
iana.golovchenko
Open

homework: lectures 2, practice 3#12
iana-golovchenko wants to merge 4 commits into
mainfrom
iana.golovchenko

Conversation

@iana-golovchenko
Copy link
Copy Markdown
Collaborator

задание на проверку

Copy link
Copy Markdown
Owner

@IlyaOrlov IlyaOrlov left a comment

Choose a reason for hiding this comment

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

По коду - все супер! Только одно небольшое замечание по стилю.

Comment thread Practice/i.golovchenko/L2.Practice3.py
Copy link
Copy Markdown
Owner

@IlyaOrlov IlyaOrlov left a comment

Choose a reason for hiding this comment

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

Есть замечания!

Comment thread Practice/i.golovchenko/L3.Practice1.py Outdated
Comment thread Practice/i.golovchenko/L3.Practice1.py Outdated
Comment thread Practice/i.golovchenko/L3.Practice1.py Outdated
Comment thread Practice/i.golovchenko/L3.Practice2.py Outdated
Comment thread Practice/i.golovchenko/L3.Practice2.py Outdated
Comment thread Practice/i.golovchenko/L3.Practice2.py Outdated
Copy link
Copy Markdown
Owner

@IlyaOrlov IlyaOrlov left a comment

Choose a reason for hiding this comment

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

Постарался ответить, оставил комментарии.

word = input("Введите слово: ")

def palindrome(a):
if a.lower() == a[::-1].lower():
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Вот только двойной вызов lower() фактически для одних и тех же символов - это нехорошо. Лучше сначала a обратить в нижний регистр, а потом уже сравнивать с перевертышем.
А поскольку операция == и так возвращает логический тип (True/False), можно схлопнуть строки 4-7 в одну)

y = int(input("Введи 1 число диапазона: "))
z = int(input("Введи 2 число диапазона: "))
x = randint(y, z)
while 1: #опять это while 1: - без которого не работает, но я не до конца понимаю как работает с ним)
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Это бесконечный цикл. Условие цикла всегда истинно. Единственный вариант выйти из такого цикла - встретить где-нибудь в его теле оператор break.

z = int(input("Введи 2 число диапазона: "))
x = randint(y, z)
while 1: #опять это while 1: - без которого не работает, но я не до конца понимаю как работает с ним)
print(f"Загаданное число: {x}") #еще без этой строчки скрипт работает не верно - я понимаю, что без нее у меня цикл не понимает что за х, но как корректно передать не понимаю
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Эта строка на видимость x не влияет и вообще погоды не делает) Вероятно, без нее как-то форматирование едет (отступы/табуляции). Надо разобраться, что за ошибка без этой строки.

# не могу сообразить как мне прикрутить условие "или не введет нечисловой символ, продолжать опрос"
# i = int(input("Введи число: "))
# но если я убираю int то потом i строка не сравнивается с х = числом
# может сможете наводящими вопросами направить... No newline at end of file
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Сначала надо считать без int. Потом проверить, состоит ли строка только из цифр или нет (т.е. можно ли ее привести к типу число или нет) - для этого у строк есть аж три метода. Если строка состоит не только из цифр - продолжаем опрос (т.е. сразу переходим на следующую итерацию цикла - для этого есть соответствующий оператор), иначе превращаем строку в int и с этим int работаем дальше (по строкам 9-15).

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