save（）保存されていない関連オブジェクトによるデータの損失を防ぐことが禁止されています。Django 1.10

Question

何か検索した後、関連する質問の中で私の特定の問題を解決できません。私は明らかに彼らが理解していないので、私は冗長のために謝罪します。

これは私のトレースバックです：

Traceback (most recent call last): File "csvimport.py", line 79, in f.save() File "/home/blake/django_/venv/lib/python3.5/site->packages/django/db/models/base.py", line 752, in save "unsaved related object '%s'." % field.name ValueError: save() prohibited to prevent data loss due to unsaved related >object 'aircraft'.

これが問題のスクリプトです：http://pastebin.com/Jks0HgEr

これらは私のモデルです：http://pastebin.com/uGUqckqS

私はなぜ関数assignAircraft（）ISNを言うことができませんAircraft.aircraft_typeオブジェクトが行56で呼び出されたときにそれを保存していない

奇妙なことに、最初のイン行[1]のタンスが航空機モデルに保存されています。

さらに、洗練された/ pep8の提案は大歓迎です。

Answer 1

私はassignAircraft

def assignAircraft(row_id): 
    if aircraft_queryset.filter(aircraft_type = row_id).exists(): 
     pass 
    else: 
     a = Aircraft(
      aircraft_type = row_id, 
     ) 
     a.save() 
    return Aircraft(aircraft_type = row_id) 

あなたの関数の戻りオブジェクトを指し示すしたいと思い、この機能は、保存されたインスタンスを返しますが、一時的なインスタンスAircraft(aircraft_type = row_id)しません。

保存したインスタンスaを返すか、既存のアイテムがない場合は何かを返す必要があります。

def assignAircraft(row_id): 
    if aircraft_queryset.filter(aircraft_type=row_id).exists(): 
     # DO SOMETHING AND RETURN SOMETHING 
    else: 
     a = Aircraft(aircraft_type=row_id) 
     a.save() 
     return a 

IMO、get_or_create manager functionが役に立ちます。