Мой код слишком громоздкий?

Question

[TimofeyShapovalov]

Вот задача:

Допустим, что сосиски упакованы в пакеты по
10 штук, а булочки - в пакеты по 8 штук. Напишите программу, которая вычисляет
количество упаковок с сосисками и количество упаковок с булочками, необходимых для
пикника с минимальными остатками. Программа должна запросить у пользователя количество участников пикника и количество хот-догов, которые будут предложены каждому
участнику. Программа должна показать приведенные ниже подробности:
• минимально необходимое количество упаковок с сосисками;
• минимально необходимое количество упаковок с булочками;
• количество оставшихся сосисок;
• количество оставшихся булочек.

Вот мой код, или мне кажется или он слишком громоздкий?

number_of_packages_with_buns = int(input("Введите количество булочек которые будут предложены участникам пикника: "))
number_of_packages_with_sausages = int(input("Введите количество сосисок которые будут предложены участникам пикника: "))
number_of_picnic_participants = int(input("Введите число участников пикника: "))
pinic = number_of_packages_with_buns * number_of_picnic_participants
pinic1 = number_of_packages_with_sausages * number_of_picnic_participants
pinic3 = pinic / 8
pinic4 = pinic1 / 10
pinic5 = pinic % 8
pinic6 = pinic1 % 10
print(f"Минимальное необходимое количество упаковок с сосисками: {pinic4:.0f}\nМинимальное необходимое количество упаковок с булочками: {pinic3:.0f}\nКоличество оставшихся сосисок {pinic6}\nКоличество оставшихся булочек: {pinic5}")

Может его можно как-то улучшить, или так оставить?

Comments

[res2001]

Такое впечатление создают имена переменных. Не стоит увлекаться максимализмом - ищите золотую середину.
Имена переменных не обязательно должны быть длинной в предложение, полностью описывающее все детали использования переменной. Достаточно дать намек именем переменной о содержимом переменной читающему программисту, знакомому с задачей. Важно, что читающий должен быть знаком с задачей - тогда он легко поймет назначение переменной по сокращенному имени. А не знакомый, не будет вникать в логику кода, он сюда случайно зашел, и ему тем более будет наплевать на имена переменных.
Например я бы ограничился вариантом num_buns и т.п.

Как правильно заметил mayton2019, результаты промежуточных вычислений не обязательно сохранять. Разве что эти результаты используются в нескольких местах ниже по коду (pinic, pinic1)

Магические константы 8 и 10 надо именовать. Сейчас у вас простая задача и вопросов к ним не возникает, но если задача будет объемней, то все подобные константы автоматически порождают вопросы.

Answer 1

[kalapanga]

Задача решена неверно. Тут нужно не улучшать, а исправлять! У Вас что, люди будут есть отдельно булки и отдельно сосиски? Нет! Так зачем Вы просите ввести количество булок и отдельно количество сосисок? В условии же ясно написано - запросить количество участников и количество хотдогов, которые будут предложены каждому. Соответственно и дальнейший расчёт должен отталкиваться именно от этих значений.

Comments

[mayton2019]

Подумалось что если в топик зайдет физик - то он будет строить уравнение вселенной с балансом
сосисок людей и хлеба.

Если дата инженер - то он тут-же сделает pipeline который стримит статистику по сьеднным хот-догам
в разрезе генедров или времен года или будет искать прогнозирование на следующий год.

Зайдет диетолог - и тут-же _бнет нам формулу сбалансированного питания чтоб ни один студент
не переел мяса и хлеба.

Веб-кодер будет тоже что-то свое делать. Формочку ввода людей и булок с UX. Вобщем ... все профессии
важны. Выбирай на вкус.

[kalapanga]

mayton2019, не понял к чему это?
Вы отрицаете, что решение автора неверное? На любом тестировании разбор решения до вычислений даже не дойдёт. Так как остановится на том, что программа вместо двух чисел ожидает три. Автор абсолютно не понял условие задачи и написал ерунду.
Просто признайте, что не обратили на это внимание. Вот про проверки ввода у Вас замечание правильное.

[mayton2019]

kalapanga, чего я там где отрицаю? Я-ж вроде не на суде? А?

Звучит блин как из полицейских протоколов.... Подсудимый вину отрицает!!... Ахах

[TimofeyShapovalov]

kalapanga, я еще более запутался.....

[kalapanga]

TimofeyShapovalov, Вы просто не совсем поняли условие задачи. Участники поглощают хотдоги - это пара булка+сосиска. Т.е. требуемое количество булок и сосисок одинаковое. А у Вас получается, что оно может быть разным.
Спрашиваете пользователя количество участников (N) и сколько каждому участнику полагается хотдогов (K). Т.е. все гости съедят N*K хотдогов - а это N*K булок и столько же сосисок. А вот сколько минимально нужно купить упаковок того и другого уже считаете.
В самом расчёте у Вас тоже ошибка. Вы "недокармливаете" гостей :). Если требуемое число не делится нацело на количество в упаковке, то результат необходимо округлить в большую сторону.
На пальцах:
Допустим, требуется 36 хотдогов. Вы посчитали для сосисок 36/10=3,6 упаковки. Но это не ответ. Вам же в магазине так не продадут. Правильный ответ будет 4 упаковки! И лишними останутся 4*10-36=4 сосиски. Аналогично посчитать для булок (вместо 10 везде ставим 8).
Вот что требуется в этой задачке.

[TimofeyShapovalov]

kalapanga, аааа теперь понял спасибо

Answer 2

[mayton2019]

1. pinic3..pinic6 можно не вычислять а вставить прямо в printf выражение.
2. Обычно во всех пользовательских вводах делают больше проверок. Например проверяют что введено
неотрицательное число сосикок. Казалось-бы пустяк. Но если подумать что это торговый автомат и дело
касается денег - то такие баги вовсе не пустяк.

Comments

[TimofeyShapovalov]

Согласен насчет второго, подумаю. Спасибо!