Как написать хороший код? Оптимизирован ли мой код? [закрыто]

Иногда я задумываюсь, много ли я написал LOC, чтобы изменить свою логику в программировании. Если мой код не так уж сильно оптимизирован. Чтобы поделиться этим, у меня есть две картинки, где я должен передать некоторые контекстные данные в Django Templates, и я передал много контекстных данных. Другая картинка содержит логику для получения 13 месяцев, чтобы я мог отфильтровать мои модели QuerySet по месяцам. Потому что мне нужны данные для создания графика, в котором я должен показать динамические колебания в данных за последние 12 месяцев. Мой вопрос заключается в следующем:: Являются ли приведенные коды оптимизированными? Есть ли лучший способ написать их? Если есть, пожалуйста, постарайтесь направить меня. Ваша помощь будет очень важна для меня. Спасибо

Код для предоставления 12 месяцев

код для передачи контекстных данных в django

#код для контекстных данных, которые я передал

            context["monthly_sold_data"] = monthly_sold_data
            context["labelmonths"] = labelmonths
            context ["loss"] = loss
            context["data"] = data_data 
            context["labels"] = datalabel
            context["profit"] = data_profit
            context["date_today"] = seven_days[0]
            context["date_before_seven"]= seven_days[6]
            context["products_added_today"] = num_prod_today
            context["percentage_change"] = percentage_change_in_number_added_prod
            context["most_sold_item"] = most_sold_item
            context["most_sold_item_amount"] = most_sold_item_amount
            context["total_sales_change"] = total_sales_change
            context["total_profit_change"] = total_profit_change
            context["total_sold"] = '{:,.2f}'.format(total_sales_data[0])
            context["total_profit"] = '{:,.2f}'.format(total_profit_data[0])
            context["total_item_sold"] = hp.total_item(SoldProducts,"product_amount_sold",seven_days[0])
        
            context["total_remainig_item"] = hp.total_item(Products, "product_amount")
            context["total_remaining_cost"] = '{:,.2f}'.format(hp.total_rem_cost(Products))
            
            context["total_item_sold_till_now"] = hp.total_item(SoldProducts, "product_amount_sold")
             
            total_sold_cost, total_profit_earned = hp.week_day_sales(SoldProducts)
            context["total_sold_cost_till_now"],context["total_profit_earned_till_now"] = 'Rs {:,.2f}'.format(total_sold_cost),'Rs {:,.2f}'.format(total_profit_earned)
            
            

            item,amount = hp.find_item_n_amount(SoldProducts)
            context["most_sold_item_till_now"],context["most_sold_item_amount_till_now"] = item,amount
            context["most_sold_item_cost"] = "Rs {:,.2f}".format(item.product_cost_price)
            

            most_profit_given_item,profit_given_by_item = hp.most_profit_given()
            context["most_profit_given_item"],context["profit_given_by_item"] = most_profit_given_item,"Rs {:,.2f}".format(profit_given_by_item)
            
 
            expensive_item,costly_item = hp.most_expensive_costly()
            context["most_expensive_item"],context["most_costly_item"] = expensive_item,costly_item
            context["most_expensive_item_cost"],context["most_costly_item_cost"] = "Rs {:,.2f}".format(expensive_item.product_cost_price),"Rs {:,.2f}".format(costly_item.product_cost_price)
            
            sagar = hp.find_item_n_amount(SoldProducts,list_return=5)

            context["top_five_items"] = sagar
            context["date"] = self.datetime.today() 

#код для генератора месяцев

from calendar import monthrange
from datetime import date,timedelta
from .models import *

def year_month(sth):
    year_today = sth.year
    month_today = sth.month
    return year_today,month_today

def month_days(year,month):
    month_day = monthrange(year,month)
    return month_day[1]


def one_month_ago(date_today,month_days):
    date_one_month_ago = date_today - timedelta(month_days)
    return date_one_month_ago

def months_generator():
    twelve_months = []
    month_short = []
    date_today = date.today()
    twelve_months.append(date_today)
    month_short.append(date_today.strftime("%b"))
    for i in range(12):
        year_today,month_today = year_month(date_today)
        month_day = month_days(year_today, month_today)
        date_today = one_month_ago(date_today, month_day)
        month_short.append(date_today.strftime("%b"))
        twelve_months.append(date_today)
        
    return twelve_months,month_short

Вернуться на верх