Нет, это не плохая форма. Это немного основано на мнении, если вы спросите меня, хотя. Думаю, вам следует сделать так, чтобы ваш код представления также читался для других, которые присоединятся к вам в проекте.
Например, я бы посоветовал вам создать новое свойство с именем NetPay или что-то в этом роде. где вы будете вычислять значения в бэкэнде (или там, где определены ваши бизнес-логики c) и будете использовать их в своем представлении вместо создания собственных вычислений в представлении.
Вы сможете просто позвоните в собственность, и все поймут, что означает эта часть на ваш взгляд. Кроме того, всякий раз, когда необходимо изменить вашу логику c, вам нужно будет только один раз изменить деталь в бэкэнде вместо того, чтобы при использовании этой логи c проходить через несколько представлений несколько раз. И самое главное, это то, что вы можете создать запасную логику c на случай, если ваше значение (не уверенное, возможно ли это, потому что я не вижу вашу модель) равно нулю.
Надеюсь, что имеет смысл и удачи!