如何烘乾我的模特

Question

我是一個製作Q &的網站。目前，我有這樣的模型

class Question(models.Model): 
    title = models.CharField(max_length=150) 
    detail = models.TextField() 
    submitter = models.ForeignKey(User) 
    date_added = models.DateTimeField(auto_now_add = True) 
    ...# some additional fields such as tags 

class Answer(models.Model): 
    detail = models.TextField() 
    submitter = models.ForeignKey(User) 
    date_added = models.DateTimeField(auto_now_add = True) 
    ... 

class QuestionVote(models.Model): 
    voter = models.ForeignKey(User) 
    question = models.ForeignKey(Question) 

#replicating what I did for QuestionVote 
class AnswerVote(models.Model): 
     voter = models.ForeignKey(User) 
     question = models.ForeignKey(Question) 

問題和答案模型是一樣的，除了標題和標籤。要爲Answers添加投票功能，我必須將QuestionVote模型複製爲AnswerVote，並重復我在視圖中爲問題投票所做的所有操作。我看了一下模型繼承，但如果我聲明一個抽象基類並繼承問題和答案模型，那麼我不能使用外鍵。那麼避免這種重複的最佳方法是什麼？

Answer 1

您可以使用一個一對一關係的其他周圍方式：

class Vote(models.Model): 
    voter = models.ForeignKey(User) 
    ...# some additional fields 

class Question(models.Model): 
    title = models.CharField(max_length=150) 
    detail = models.TextField() 
    submitter = models.ForeignKey(User) 
    date_added = models.DateTimeField(auto_now_add = True) 
    vote = models.OneToOneField(Vote) 
    ...# some additional fields such as tags 

class Answer(models.Model): 
    detail = models.TextField() 
    submitter = models.ForeignKey(User) 
    date_added = models.DateTimeField(auto_now_add = True) 
    vote = models.OneToOneField(Vote) 
    ... 

Answer 2

看一看abstract base classes

您可以把常用的領域/功能在裏面（submitter，detail等），然後在你的Question和Answer車型繼承它

編輯：

根據其他答案和我看你的問題多一點我改變我的答案。您目前正走在「DRY」和「良好的可讀性/可維護性」之間，不同的開發人員會在不同的網站上走下坡路，所以您可能只需要選擇一個您最喜歡的那個。

關於這一點，這裏就是我會一五一十地出來：

############################## 
# Question and Answer Models # 
############################## 

class QuestionAnswerBase(models.Model): # Choose a better name :) 
    detail = models.TextField() 
    submitter = models.ForeignKey(User) 
    date_added = models.DateTimeField(auto_now_add = True) 
    ... # More common fields here 

    class Meta: 
     abstract = True 


class Question(QuestionAnswerBase): 
    title = models.CharField(max_length=150) 
    ...# some additional fields such as tags etc 


class Answer(QuestionAnswerBase): 
    ... # No extra fields needed but you'll prob want a custom __unicode__ method etc 


############### 
# Vote Models # 
############### 

class VoteBase(models.Model): 
    voter = models.ForeignKey(User) 
    ... # Other shared fields etc 

    class Meta: 
     abstract = True 

class AnswerVote(VoteBase): 
    answer = models.ForeignKey(Answer) 


class QuestionVote(VoteBase): 
    question = models.ForeignKey(Question) 

（註釋塊是在這種情況下的可讀性。）

「但現在我有更多的車型！」

是你做的，但也有其中的任何沒有重複領域，抽象意味着你必須添加不同的行爲TO-的示例 - 一個QuestionVote和AnswerVote電源（說你想要的能力如果更好的Answer出現，請收回AnswerVote s，但不希望能夠收回QuestionVote s），而不需要有「這是X是Y還是Z？如果是Z是否這樣做」你寫的每種模型方法。

「看起來很醜！

它的讀取和明確的，這意味着它beautiful :)

「我想要做的另一種方式！」

這很好，現在已經有其他好的答案顯示如何做到這一點。這只是我的建議，因爲對最佳實踐和可讀代碼非常熱衷的人。

Answer 3

我個人覺得DRY很重要，所以我願意犧牲一些優雅來實現它。例如，在這種情況下，我可能做到這一點：

class QuestionOrAnswer(models.Model): 
    is_question = models.BooleanField() 
    title = models.CharField(max_length=150) 
    detail = models.TextField() 
    submitter = models.ForeignKey(User) 
    date_added = models.DateTimeField(auto_now_add = True) 
    question_specific_field = ... 

    def clean(self): 
     # Make sure that question_specific_field is set only if is_question is true. 

class Vote(models.Model): 
    target = models.ForeignKey(QuestionOrAnswer) 
    ...