2017-08-04 152 views
0

編輯:我改變了標題,以更好地代表我尋求的解決方案。如何將命名參數傳遞給另一個函數需要的函數?

我正在重複一段代碼,並且想寫一個函數來減少混亂。我無法通過kwarg密鑰hourly_id(原始代碼的最後一行)。

原始代碼

# Create Hourly data. 
hourly_data = validated_data.pop('hourly') 
hourly_points_data = hourly_data.pop('data') 
hourly = Hourly.objects.create(**hourly_data) 
for hourly_point_data in hourly_points_data: 
    hourly_point = HourlyPoint.objects.create(
     hourly_id=hourly.pk, **hourly_point_data) <-- This 

新功能

def create_data_block(self, data, block_str, DataBlock, DataPoint, id): 
    block_data = data.pop(block_str) 
    points_data = block_data.pop('data') 
    block = DataBlock.objects.create(**block_data) 
    for point_data in points_data: 
     point = DataPoint.objects.create(
      id=block.pk, **point_data) <-- This 

函數調用

self.create_data_block(validated_data, 'hourly', Hourly, HourlyPoint, 'hourly_id') 

所以你可以看到在這裏我試圖通過hourly_id作爲id使用字符串,但我得到一個數據庫錯誤,說hourly_id丟失,所以我顯然沒有正確傳遞它。

回溯

Traceback (most recent call last): 
    File "/home/cudb/.virtualenvs/otto/lib/python3.5/site-packages/django/core/handlers/exception.py", line 42, in inner 
    response = get_response(request) 
    File "/home/cudb/.virtualenvs/otto/lib/python3.5/site-packages/django/core/handlers/base.py", line 187, in _get_response 
    response = self.process_exception_by_middleware(e, request) 
    File "/home/cudb/.virtualenvs/otto/lib/python3.5/site-packages/django/core/handlers/base.py", line 185, in _get_response 
    response = wrapped_callback(request, *callback_args, **callback_kwargs) 
    File "/usr/lib/python3.5/contextlib.py", line 30, in inner 
    return func(*args, **kwds) 
    File "/home/cudb/.virtualenvs/otto/lib/python3.5/site-packages/django/views/decorators/csrf.py", line 58, in wrapped_view 
    return view_func(*args, **kwargs) 
    File "/home/cudb/.virtualenvs/otto/lib/python3.5/site-packages/django/views/generic/base.py", line 68, in view 
    return self.dispatch(request, *args, **kwargs) 
    File "/home/cudb/.virtualenvs/otto/lib/python3.5/site-packages/rest_framework/views.py", line 489, in dispatch 
    response = self.handle_exception(exc) 
    File "/home/cudb/.virtualenvs/otto/lib/python3.5/site-packages/rest_framework/views.py", line 449, in handle_exception 
    self.raise_uncaught_exception(exc) 
    File "/home/cudb/.virtualenvs/otto/lib/python3.5/site-packages/rest_framework/views.py", line 486, in dispatch 
    response = handler(request, *args, **kwargs) 
    File "/home/cudb/Projects/a/b/c/weather/views.py", line 36, in get 
    response = self.get_entry(latitude, longitude) 
    File "/home/cudb/Projects/a/b/c/weather/views.py", line 59, in get_entry 
    response = self.create_or_update_entry(latitude, longitude) 
    File "/home/cudb/Projects/a/b/c/weather/views.py", line 76, in create_or_update_entry 
    location_serializer.save() 
    File "/home/cudb/.virtualenvs/otto/lib/python3.5/site-packages/rest_framework/serializers.py", line 215, in save 
    self.instance = self.create(validated_data) 
    File "/home/cudb/Projects/a/b/c/weather/serializers.py", line 119, in create 
    validated_data, 'hourly', Hourly, HourlyPoint, 'hourly_id') 
    File "/home/cudb/Projects/a/b/c/weather/serializers.py", line 169, in create_data_block 
    id=block.pk, **point_data) 
    File "/home/cudb/.virtualenvs/otto/lib/python3.5/site-packages/django/db/models/manager.py", line 85, in manager_method 
    return getattr(self.get_queryset(), name)(*args, **kwargs) 
    File "/home/cudb/.virtualenvs/otto/lib/python3.5/site-packages/django/db/models/query.py", line 399, in create 
    obj.save(force_insert=True, using=self.db) 
    File "/home/cudb/.virtualenvs/otto/lib/python3.5/site-packages/django/db/models/base.py", line 796, in save 
    force_update=force_update, update_fields=update_fields) 
    File "/home/cudb/.virtualenvs/otto/lib/python3.5/site-packages/django/db/models/base.py", line 824, in save_base 
    updated = self._save_table(raw, cls, force_insert, force_update, using, update_fields) 
    File "/home/cudb/.virtualenvs/otto/lib/python3.5/site-packages/django/db/models/base.py", line 908, in _save_table 
    result = self._do_insert(cls._base_manager, using, fields, update_pk, raw) 
    File "/home/cudb/.virtualenvs/otto/lib/python3.5/site-packages/django/db/models/base.py", line 947, in _do_insert 
    using=using, raw=raw) 
    File "/home/cudb/.virtualenvs/otto/lib/python3.5/site-packages/django/db/models/manager.py", line 85, in manager_method 
    return getattr(self.get_queryset(), name)(*args, **kwargs) 
    File "/home/cudb/.virtualenvs/otto/lib/python3.5/site-packages/django/db/models/query.py", line 1045, in _insert 
    return query.get_compiler(using=using).execute_sql(return_id) 
    File "/home/cudb/.virtualenvs/otto/lib/python3.5/site-packages/django/db/models/sql/compiler.py", line 1054, in execute_sql 
    cursor.execute(sql, params) 
    File "/home/cudb/.virtualenvs/otto/lib/python3.5/site-packages/debug_toolbar/panels/sql/tracking.py", line 164, in execute 
    return self._record(self.cursor.execute, sql, params) 
    File "/home/cudb/.virtualenvs/otto/lib/python3.5/site-packages/debug_toolbar/panels/sql/tracking.py", line 106, in _record 
    return method(sql, params) 
    File "/home/cudb/.virtualenvs/otto/lib/python3.5/site-packages/django/db/backends/utils.py", line 79, in execute 
    return super(CursorDebugWrapper, self).execute(sql, params) 
    File "/home/cudb/.virtualenvs/otto/lib/python3.5/site-packages/django/db/backends/utils.py", line 64, in execute 
    return self.cursor.execute(sql, params) 
    File "/home/cudb/.virtualenvs/otto/lib/python3.5/site-packages/django/db/utils.py", line 94, in __exit__ 
    six.reraise(dj_exc_type, dj_exc_value, traceback) 
    File "/home/cudb/.virtualenvs/otto/lib/python3.5/site-packages/django/utils/six.py", line 685, in reraise 
    raise value.with_traceback(tb) 
    File "/home/cudb/.virtualenvs/otto/lib/python3.5/site-packages/django/db/backends/utils.py", line 64, in execute 
    return self.cursor.execute(sql, params) 
django.db.utils.IntegrityError: null value in column "hourly_id" violates not-null constraint 
DETAIL: Failing row contains (6, 76.22, 77.31, 0.80, 0.00, clear-night, Clear, null). 

[04/Aug/2017 12:05:30] "GET /weather/0,4 HTTP/1.1" 500 288106 
+0

包括你的全棧跟蹤。另外,鍵只是字符串。 – Marcin

+0

我附上了痕跡。 – CudB

+0

也發佈模型 –

回答

1

這地方我看到一個問題:

hourly_point = HourlyPoint.objects.create(hourly_id=hourly.pk, **hourly_point_data) 
point = DataPoint.objects.create(id=block.pk, **point_data) 

可以重命名變量是您認爲合適的更通用的,但hourly_id不是一個變量,它的一個命名參數。通過使用id來代替,您沒有傳入預期的hourly_id參數,而是傳遞了可能不需要的或不正確的參數。

除非您更改create()本身的定義,否則參數名稱仍爲hourly_id。解決這個問題的一個方法就是在可能的情況下使用參數,而不是名稱。 (即是它只是一個關鍵字參數或位置參數由關鍵字訪問?)

另外,如果你在'hourly_id'作爲id通過,則增加point_data與此鍵和值:

def create_data_block(self, data, block_str, DataBlock, DataPoint, id): 
    block_data = data.pop(block_str) 
    points_data = block_data.pop('data') 
    block = DataBlock.objects.create(**block_data) 
    for point_data in points_data: 
     point = DataPoint.objects.create(**{id: block.pk, **point_data}) 
相關問題