Почему когда эта функция получает текст "2/10" она возвращает False? def

Question

Почему когда эта функция получает текст "2/10" она возвращает False? def

check_message_for_trans_status(message_text: str):
if "/" in message_text:
message_text = message_text.split("/")
if message_text[0] <= message_text[1]:
if message_text[0].isdigit():
if message_text[1].isdigit():
return True
else:
return False
else:
return False
else:
return False
else:
return False

#backend #programming #python #russian

0

18.06.2022

15 ответов

31 просмотр

ロリが好きですマキシム (шипер Гани и Кэцы кароче) Автор вопроса

Черепаха Баранов
Так луче не писать

А как лучше?

0

18.06.2022

Черепаха Баранов

ロリが好きですマキシム (шипер Гани и Кэцы кароче)
А как лучше?

По крайне мере нужно избавиться от такого ада из else

0

18.06.2022

Черепаха Баранов

ロリが好きですマキシム (шипер Гани и Кэцы кароче)
А как лучше?

Чего хочешь добиться ?

0

18.06.2022

ロリが好きですマキシム (шипер Гани и Кэцы кароче) Автор вопроса

Черепаха Баранов
Чего хочешь добиться ?

Чтобы сообщение от пользователя имело структуру "a/b", например "5/8". При этом "a" должно быть меньше или равняться "b", "a" и "b" должны быть числами

0

18.06.2022

Черепаха Баранов

ロリが好きですマキシム (шипер Гани и Кэцы кароче)
Чтобы сообщение от пользователя имело структуру "a...

Я бы так : 0. Не валить все в одну функцию, раскидать ответственность между функциями. 1. Функция, ее работа: Разделить данные на пример в список, работой по сепоратору "/", вернуть значение 2. Функция которая проверит соответствия значения в списке, пример [1, 4] проверит что он из двух элемнетов и также организует проверку на число если да, и идем дальше. В противном случаи функции пускай возвращают False.

0

18.06.2022

ロリが好きですマキシム (шипер Гани и Кэцы кароче) Автор вопроса

Черепаха Баранов
Я бы так : 0. Не валить все в одну функцию, раскид...

Я тупо убрал .isdigit и всё заработало, проверка на то что это инт вообще не понадобилась, само фолс возвращает если это текст, а не число. def check_message_for_trans_status(message_text: str): if "/" in message_text: message_text = message_text.split("/") a = int(message_text[0]) b = int(message_text[1]) if a <= b: return True else: return False else: return False

0

18.06.2022

ロリが好きですマキシム (шипер Гани и Кэцы кароче) Автор вопроса

Тем что isdigit почему-то не работает когда message_text = "2/10", странно конечно, щас по другому попробую сделать - поменять местами <= с isdigit

0

18.06.2022

ロリが好きですマキシム (шипер Гани и Кэцы кароче) Автор вопроса

ロリが好きですマキシム (шипер Гани и Кэцы кароче)
Тем что isdigit почему-то не работает когда messag...

Я их тупо поменял местами и всё заработало.

0

18.06.2022

Черепаха Баранов