TK
Блин, знаю что уже говорили, но давай всё же перейдём на бота?
+++
Size: a a a
2019 October 07
💭П
@k3k43k и @kostandyan в чем проблема? Это опен соурс, где ваш пулреквест?
TK
так я предлагал
TK
сказали и так норм
TK
и на этом тему закрыли
💭П
У меня сейчас нет времени писать "теплый и ламповый" бот )
TK
Сюда кто-то кидал даже готовый скрипт
P
У меня сейчас нет времени писать "теплый и ламповый" бот )
Намёк чтобы я написал бота, который это будет выполнять за тебя ?)
💭П
# Check time. About 120 seconds (2 minutes) should elapse between the first and second measurements
if countdown_time * 0.9 <= elapsed_time or elapsed_time >= countdown_time * 1.1:
step_status = "Pass"
actual_result = self.get_replace(verify_that)
else:
raise Exception(f"Time between measurements is: {elapsed_time}")
Это нормальный код? Что-то меня в нем напрягает, а что не могу понять (
ЕЧ
Всё?)
💭П
Всё?)
Нет. Мне не нравиться как вяглядит условие. Можно написать свой метод который будет принимать значение, область сравнения и возвращать реузльтат или ошибку. Но нужно ли это делать? Может не всё так плохо и можно оставить это условие )
ЕЧ
Требуется помощь зала
ЕЧ
Не знаю как делают в питоне, я бы условие где три и больше частей вытащил в метод/функцию и написал красивое название. Можно даже зарифмовать с другими методами, чтобы потом читать с выражением))
NK
# Check time. About 120 seconds (2 minutes) should elapse between the first and second measurements
if countdown_time * 0.9 <= elapsed_time or elapsed_time >= countdown_time * 1.1:
step_status = "Pass"
actual_result = self.get_replace(verify_that)
else:
raise Exception(f"Time between measurements is: {elapsed_time}")
Это нормальный код? Что-то меня в нем напрягает, а что не могу понять (
Мне magic numbers не нравятся в первую очередь
NK
Да и статус должен из списка допустимых браться
💭П
Мне magic numbers не нравятся в первую очередь
это условия теста. Я вынес ее в отлеьную переменную. Но как избавитсья от 0,9 и 1,1 не знаю.
💭П
Да и статус должен из списка допустимых браться
У меня только два статуса. Если все хороша то файл, а если ошибка то сразу Фэйл.
NK
else не нужен.
Нужно рэйзить ошибку на инвертированное условие
Нужно рэйзить ошибку на инвертированное условие
💭П
else не нужен.
Нужно рэйзить ошибку на инвертированное условие
Нужно рэйзить ошибку на инвертированное условие
т.е. делать свой метод для проверки, и уже на выходе ставить if?
NK
Нет.
raise Error if wrong_condition
step_status = 'Pass'
...other logic
raise Error if wrong_condition
step_status = 'Pass'
...other logic