是否將臨時屬性應用於django模型以用於視圖不良練習？

Question

我有一個系統的工作，用於跟蹤員工工作時間，並且需要計算這些員工的年終信息。在我們的應用程序的單獨部分中，前一個開發人員通過具有數小時的員工/員工任務進行迭代，並創建字典以傳遞給視圖以進行顯示。我發現，我可以顯着運用「臨時屬性」的僱員削減必要做這個代碼量，因爲我遍歷它們，然後通過簡單地將員工的queryset到視圖，但是這是不好的做法，是有解決這個問題的更好方法是什麼？下面有一個例子的片段：

models.py

class Employee(models.Model): 
    user = models.ForeignKey(User) 

class EmployeeTask(models.Model): 
    emp = models.ForeignKey(Employee) 
    hours = models.FloatField() 

views.py

def employee_report(request): 
    employees = Employees.objects.all() 
    tasks = EmployeeTask.objects.all() 

    for employee in employees: 
     emp_tasks = [task for task in tasks 
          if task.employee == employee] 

     employee.total_hours = 0 

     for emp_task in emp_tasks: 
      employee.total_hours += emp_task.hours 

    context = { 
     'employees': employees 
    } 

    return render(request, 'app/employees.html', context) 

我試圖通過創建檢索的僱員的任務，做了方法放置在模型中的邏輯計算但導致大量的sql查詢。獲得並匹配員工與視圖中的任務是我可以找到的避免這種情況的唯一途徑。這種方法很有效，但我擔心它不一定是'pythonic'（或'djangonic'，因爲現在感覺就像是一件事情）的方式。

注意：我在實際視圖中做了比這更多的工作，例如根據類型過濾任務（支付時間等），並根據總公司小時數計算員工使用率，所有這些都是作爲臨時屬性的視圖，但爲簡潔起見，我不包括這裏的全部內容。

Answer 1

這不是不好的做法，你可以把它變得更好，把它在你的模型，像這樣的（未經測試，但應該給你的想法）：

class Employee(models.Model): 
    user = models.ForeignKey(User) 

    @property 
    def total_hours(self): 
     return self.task_set.all().aggregate(models.Sum('hours')) 


class EmployeeTask(models.Model): 
    emp = models.ForeignKey(Employee) 
    hours = models.FloatField()