質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

87.61%

django modelでForeignKeyの設定にForm経由でデータを保存する方法

受付中

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 3,028

score 18

APIで飛んできたデータを二つのモデルに分けて(Head,Detail)DBへ保存しています。
この二つのモデルにリレーションをかけると保存が上手く進まず躓きました。

【目標】
二つのデータをリレーションをかけて保存すること。

APIで一度に飛んでくるデータの構造は、

transaction_send&params={ "data":[{
   "table_name":"TransactionHead", 
  "proc_detail_name":"register_transaction",
  "rows":[{
       "transactionHeadId":"113",
       "transactionDateTime":"2013-01-20 10:01:34",}]
  },{
  "table_name":"TransactionDetail", 
  "proc_detail_name":"register_transaction",
  "rows":[{
       "transactionHeadId":"113",
       "transactionDateTime":"2013-01-20 10:01:34", 
       "transactionDetailId":"0",]}
  }
]}


このデータを保存する為のモデルは、

class TransactionHead(models.Model):
    class Meta:
        db_table = 'transaction_head'
    transactionHeadId           = models.BigIntegerField(verbose_name="取引ID", null=True)
    transactionDateTime         = models.DateTimeField(verbose_name="取引日時", null=True)

class TransactionDetail(models.Model):
    class Meta:
        db_table = 'transaction_detail'
    transactionHead             = models.ForeignKey(TransactionHead, on_delete=models.CASCADE, default=1)
    transactionHeadId           = models.BigIntegerField(verbose_name="取引ID", null=True)
    transactionDateTime         = models.DateTimeField(verbose_name="取引日時", null=True)
    transactionDetailId         = models.IntegerField(verbose_name="取引明細ID", null=True)

データをDBに保存する為のコードが下記です。

#jsonデータをモデルに保存していくコード
     # TransactionHead
        for hd in head_data:
            head_form = HeadForm(hd)
            if head_form.is_valid():
                head_form.save()

        # TransactionDetail
        for dd in detail_data:
            detail_form = DetailForm(dd)
            if detail_form.is_valid():
                detail_form.save()
                detail_flag += 1

そして問題となっている箇所がFormなのですが、

class DetailForm(forms.ModelForm):
    class Meta:
        model = TransactionDetail
        fields = '__all__'
  #
    transactionHeadId = forms.IntegerField(label="取引ID", required=False)
    transactionDateTime = forms.DateTimeField(label="取引日時", required=False)


いくつか試したことは、
①上記のfrom(ForeignKeyの部分を無くす)場合、headデータは保存されましたが、detailデータは保存されませんでした。
②上記の#を入れた箇所にtransactionHead = forms.ModelChoiceField(models.TransactionHead.objects)を入れるとエラーになりました。
③上記の#にforms.IntegerField()を入れましたが、同じくエラーになりました。

リレーションを掛けなかった場合、それぞれにデータが格納されたのは確認しているので、Formのこの箇所に問題があるのは理解できたのですが、解決策が全く見当がつきません。

お知恵をお借りできると幸いです。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

0

具体的などういうエラーでお困りなのかわからないので、想像でしかないですが、transactionHead は transactionHeadIdがわかればモデルオブジェクトを取得できるので、あえて ModelFormの項目からは除外して(=validationの対象外になる)、最終的にsaveするタイミングで、自動的に transactionHeadをセットしてからsaveするのが良さそうに思いました。

class DetailForm(forms.ModelForm):
    class Meta:
        model = TransactionDetail
        # transactionHead は除外
        fields = ("transactionHeadId", "transactionDateTime", "transactionDetailId")

    def save(self, commit=True):
        detail = super().save(commit=False) # 意図的にモデルをcommitしない

        detail.transactionHead = TransactionHead.objects.get(
          transactionHeadId=detail.transactionHeadId
        )
        detail.save()
        return detail

save(commit=False) にすると DBには保存せずに、モデルオブジェクトだけを作ってreturnしてくれます。

参考: https://docs.djangoproject.com/ja/2.1/topics/forms/modelforms/#the-save-method

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/01/06 01:34

    ご丁寧にありがとうございます。
    実は、この方法も考えたのですが、モデルの項目はかなり絞って3項目にしていますが、実際は50項目くらりありまして。。全部書くのも大変かなーと思い試してはいませんでした。

    別のシステムからAPIでデータを飛ばして、それを保存する処理が最後まで行けなければ「失敗」で終わると言う事して確認できておらず、どのようなエラーとなったか確認できて(方法の知識がなく)いませんでした。送信が「成功」となった時にDBを確認するとデータが格納されていないと言う確認方法で進めてきました。

    モデルフォームを利用して、データを通すフィールドが無ければデータがセーブされないと言う仕様になっているのでしょうか?

    キャンセル

  • 2019/01/06 03:36

    はい。そうです。fields に 指定しなければ、そのフィールドはFormに存在しないことになります。Formに存在しない以上、モデルの該当フィールド(カラム)にも何もセットされません

    キャンセル

  • 2019/01/06 11:41

    説明不足で申し訳ありません。
    例えば、保存したいデータが10項目あったとします。
    しかし、formで指定したフィールドが__all__としているのにフィールド数が9項目だったとします。

    今回と同じ現象ではあるのですが、一つでもフィールドが足りなかったら、その足りないフィールドだけの値が保存されないだけではなく、全てのデータが保存されないという認識で大丈夫でしょうか?

    キャンセル

  • 2019/01/06 19:32

    > formで指定したフィールドが__all__としているのにフィールド数が9項目だったとします。
    fields = "__all__" と書いたら、フィールド数は10項目になるんじゃないでしょうか? 9項目になるという話がよくわかりませんでした。ModelForm は、フォームフィールドの定義を自分でもしなくても、裏で勝手にモデルに応じたフィールドを生成してくれます。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 87.61%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る