IS
А ошибку свою гуглил?
Size: a a a
2021 September 29
EB
Старайся впредь скидывать код или скриншот или ссылку на gist GitHub давать, чтобы были видны номера строчек кода. И весь стек трейс с кодом ошибки где упало
BW
EB
Потому что где-то у тебя присваивается в коде что-то типа
a, b = iterable # None
А ты ожидаешь что у тебя a,b = (1, 2)
a, b = iterable # None
А ты ожидаешь что у тебя a,b = (1, 2)
EB
Это может быть ошибка как я указал выше или в случае с анпакингом аргументов функции, как вариант
IS
инфа сотка анпакинг
EB
Ну анпакингом это называется и при multiple assignment
EB
В обоих случаях это анпакинг о чем и говорит текст ошибки, только я в коде выше не увидел это место
А
если иф не отработал - у тя ничего нет в ретурне (None). бест практис - не делать ретурны посреди кода, а назначать резалт в переменную (с дефолтным значением), а в конце всегда возвращать ее
А
def are_elements_present(self, elm, param):
result = None, f"Param length {param} not present"
try:
elem = self.get_page_elements(elm)
if len(elem) == param:
result = True, "It's good!"
except NoSuchElementException:
result = False, f'Нет такого объкета {elm.name}!
return result
result = None, f"Param length {param} not present"
try:
elem = self.get_page_elements(elm)
if len(elem) == param:
result = True, "It's good!"
except NoSuchElementException:
result = False, f'Нет такого объкета {elm.name}!
return result
BW
Ок, спасибо за совет
EB
Чёт какой-то странный совет. Когда есть возможность вернуть то, что нужно, полезно short circuit сделать прямо там , где-то имеет смысл
EB
Таким образом например куча методов в руби и питоне в веб фреймворках рендерят ошибки обработки данных. Любой обход коллекции на truthiness тоже может иметь быстрый return если нет необходимости обходить всю коллекцию
IS
Это не бест практис
IS
Бест практис возвращать где можно, а где не можно - рейзить эксепшн
EB
А
ну и хорошим тоном считается называть переменные адекватно их функциональности, а не рандомные парамы. ЧТобы люди могли читать твой код быстрее.
def are_elements_present(self, element_type, element_length): - так намного понятнее
def are_elements_present(self, element_type, element_length): - так намного понятнее
IS
Расскажи ему еще про тайпхинты и у него кукуха отлеттит. Ему похуй
А
вонтам выше short circuit приводит к багу. Его смысл - оптимизация по выполнению. В случае тестов же важнее стабильность кода