Badriev A. (learn-homework-1) #141
Conversation
krepysh
left a comment
krepysh
left a comment
There was a problem hiding this comment.
Отлично, но иногда переусложняешь простые вещи. Всегда держи в голове вопрос, а могу ли я это сделать проще?
1_if1.py
Outdated
| dict_have_to_do = {tuple(range(1, 3)): 'Вам нужно учиться в детском саду', | ||
| tuple(range(3, 18)): 'Вам нужно учиться в школе', | ||
| tuple(range(18, 21)): 'Вам нужно учиться в ВУЗе', | ||
| tuple(range(21, 80)): 'Вам нужно работать'} |
There was a problem hiding this comment.
когда ты оборачиваешь range в tuple, ты заставляешь программу хранить в память все 60 чисел, в несортированном тупле. Это и памяти много занимает, и перебор он делает последовательно по всем числам в списках.
По крайней мере оставь просто range, должно работать.
Но вообще задача была про сделать цепочку if.
C одной стороны у тебя классный, расширяемый подход. С другой стороны цепочка if понятнее, и работает быстрее. Если тебе не нужно вытаскивать произвольный список правил из БД, то я бы сказал что эта абстрактность тут ни к чему - она только усложняет код.
2_if2.py
Outdated
|
|
||
|
|
||
| def strings(str1, str2): | ||
| if isinstance(str1, str) and isinstance(str2, str): |
There was a problem hiding this comment.
А можно сделать код менее вложенным? Вложенный условия сложно понимать. Совершенно не вижу причины для трех уровней вложенности тут.
3_for.py
Outdated
|
|
||
| def sum_sales_for_each_product(lsts): | ||
| for lst in lsts: | ||
| print(f'Cуммарное количество продаж товара: "{lst["product"]}" составляет {sum(lst["items_sold"])} шт.') |
There was a problem hiding this comment.
print наружу из функции, тут делаем return
3_for.py
Outdated
|
|
||
| def avg_sales_for_each_product(lsts): | ||
| for lst in lsts: | ||
| print(f'Среднее количество продаж товара: "{lst["product"]}" составляет {round(np.average(lst["items_sold"]))} шт.') |
There was a problem hiding this comment.
Попробуй решить без numpy. Использование нампи не дает большой выгоды тут, и в продакшен коде * такое использовать не принято
3_for.py
Outdated
|
|
||
|
|
||
| def avg_sales_for_all_product(lsts): | ||
| return f'Среднее количество продаж товаров: "{", ".join([lst["product"] for lst in lsts])}" составляет {round(np.average([np.average(lst["items_sold"]) for lst in lsts]))} шт.' |
There was a problem hiding this comment.
Кажется среднее средних не тоже самое что просили посчитать в задаче.
| """ | ||
| pass | ||
|
|
||
| while True: |
There was a problem hiding this comment.
В этом цикле все смешалось в кучу, логика, ввод-вывод, задержки.
Попробуй упростить это.
7_exception2.py
Outdated
| try: | ||
| price = abs(float(price)) | ||
| except (ValueError, TypeError): | ||
| return f'Произошла ошибка {sys.exc_info()[0]}. Проверьте корректность введенной цены товара {price=}.' |
There was a problem hiding this comment.
Можно проще получать текущее исключение, в except блоке добавляешь as ex, и используешь переменную ex
2 participants
No description provided.