สถานที่ที่เหมาะสมในการเก็บไฟล์ Signal.py ของฉันในโครงการ Django


88

จากเอกสารของ Django ที่ฉันกำลังอ่านดูเหมือนว่าsignals.pyในโฟลเดอร์แอพจะเป็นจุดเริ่มต้นที่ดี แต่ปัญหาที่ฉันพบคือเมื่อฉันสร้างสัญญาณpre_saveและฉันพยายามนำเข้าคลาสจากโมเดลมันขัดแย้งกับimportในแบบจำลองของฉัน

# models.py

from django.contrib.auth.models import User
from django.db import models
from django.utils.translation import gettext as _
from signals import *

class Comm_Queue(CommunicatorAbstract):
    queue_statuses = (
        ('P', _('Pending')),
        ('S', _('Sent')),
        ('E', _('Error')),
        ('R', _('Rejected')),
    )
    status          = models.CharField(max_length=10, db_index=True, default='P')
    is_html         = models.BooleanField(default=False)
    language        = models.CharField(max_length=6, choices=settings.LANGUAGES)
    sender_email    = models.EmailField()
    recipient_email = models.EmailField()
    subject         = models.CharField(max_length=100)
    content         = models.TextField()

# signals.py

from django.conf import settings
from django.db.models.signals import pre_save
from django.dispatch import receiver
from models import Comm_Queue

@receiver(pre_save, sender=Comm_Queue)
def get_sender_email_from_settings(sender, **kwargs):
    obj=kwargs['instance']
    if not obj.sender_email:
        obj.sender_email='%s' % settings.ADMINS[0][1]

รหัสนี้จะไม่ทำงานเนื่องจากฉันนำเข้าComm_Queueภายในsignals.pyและฉันก็นำเข้าสัญญาณข้างในmodels.pyด้วย

ใครช่วยแนะนำว่าฉันจะแก้ไขปัญหานี้ได้อย่างไร

ความนับถือ


คำตอบ:


65

คำตอบเดิมสำหรับ Django <1.7:

คุณสามารถลงทะเบียนสัญญาณโดยการนำเข้าsignals.pyใน__init__.pyไฟล์ของแอพ:

# __init__.py
import signals

วิธีนี้จะอนุญาตให้นำเข้าmodels.pyจากsignals.pyโดยไม่มีข้อผิดพลาดในการนำเข้าแบบวงกลม

ปัญหาอย่างหนึ่งของแนวทางนี้คือทำให้ผลลัพธ์ความครอบคลุมยุ่งเหยิงหากคุณใช้ coverage.py

การอภิปรายที่เกี่ยวข้อง

แก้ไข: สำหรับ Django> = 1.7:

ตั้งแต่เปิดตัว AppConfig วิธีการนำเข้าสัญญาณที่แนะนำอยู่ในinit()ฟังก์ชัน ดู คำตอบของ Eric Marcosสำหรับรายละเอียดเพิ่มเติม


6
ใช้สัญญาณใน Django 1.9 ใช้วิธีการด้านล่าง (แนะนำโดย django) วิธีนี้ใช้ไม่ได้ผลAppRegistryNotReady("Apps aren't loaded yet.")
s0nskar

1
Eric Marcos คำตอบของเขาควรเป็นคำตอบที่ยอมรับได้: stackoverflow.com/a/21612050/3202958ตั้งแต่ Django> = 1.7 โดยใช้การกำหนดค่าแอป
Nrzonline

1
เห็นด้วย ฉันจะแก้ไขคำตอบเพื่อชี้ไปที่คำตอบของ Eric Marcos สำหรับ Django
1.7+

197

หากคุณใช้ Django <= 1.6 ฉันขอแนะนำโซลูชัน Kamagatos: เพียงนำเข้าสัญญาณของคุณที่ส่วนท้ายของโมดูลโมเดลของคุณ

สำหรับ Django เวอร์ชันอนาคต (> = 1.7) วิธีที่แนะนำคือการนำเข้าโมดูลสัญญาณของคุณในฟังก์ชันconfig ready ()ของแอป:

my_app/apps.py

from django.apps import AppConfig

class MyAppConfig(AppConfig):
    name = 'my_app'

    def ready(self):
        import my_app.signals

my_app/__init__.py

default_app_config = 'my_app.apps.MyAppConfig'

7
นอกจากนี้ยังกล่าวถึงในเอกสารประกอบ 1.7 ว่าบางครั้งสามารถเรียกพร้อมใช้งานได้หลายครั้งดังนั้นเพื่อหลีกเลี่ยงสัญญาณที่ซ้ำกันให้แนบตัวระบุที่ไม่ซ้ำกันกับการเรียกตัวเชื่อมต่อสัญญาณของคุณ: request_finished.connect (my_callback, dispatch_uid = "my_unique_identifier") โดยปกติ dispatch_uid จะเป็นสตริง แต่สามารถเป็นวัตถุที่ล้างทำความสะอาดได้ docs.djangoproject.com/th/1.7/topics/signals/…
Emeka

13
นี่ควรเป็นคำตอบที่ได้รับการยอมรับ! คำตอบที่ยอมรับข้างต้นทำให้เกิดข้อผิดพลาดเมื่อปรับใช้โดยใช้ uwsgi
Patrick

2
อืมไม่ได้ผลกับ django 2 ถ้าฉันนำเข้าโมเดลโดยตรงพร้อม - ดีทั้งหมด หากฉันนำเข้าโมเดลในสัญญาณและนำเข้าสัญญาณพร้อมฉันได้รับข้อผิดพลาดdoesn't declare an explicit app_label..
Aldarund

@Aldarun คุณสามารถลองใส่ 'my_app.apps.MyAppConfig' ไว้ใน INSTALLED_APPS
Ramil Aglyautdinov

26

ในการแก้ปัญหาของคุณคุณต้องนำเข้า signal.py หลังจากนิยามโมเดลของคุณ นั่นคือทั้งหมด


2
นี่เป็นวิธีที่ง่ายที่สุดและฉันไม่รู้เลยว่ามันจะทำงานได้โดยไม่ต้องพึ่งพาวงจร ขอบคุณ!
bradenm

2
ยอดเยี่ยม. ชอบคนนี้ดีกว่าคำตอบของฉัน. แม้ว่าฉันจะไม่ค่อยเข้าใจว่าทำไมมันถึงไม่ทำให้เกิดการนำเข้าแบบวงกลม ...
yprez

โซลูชันไม่ทำงานกับปลั๊กอิน autopep8 ที่เปิดใช้งานใน Eclipse
ramusus

5

ฉันยังใส่สัญญาณในไฟล์ signal.py และยังมีข้อมูลโค้ดนี้ที่โหลดสัญญาณทั้งหมด:

# import this in url.py file !

import logging

from importlib import import_module

from django.conf import settings

logger = logging.getLogger(__name__)

signal_modules = {}

for app in settings.INSTALLED_APPS:
    signals_module = '%s.signals' % app
    try:
        logger.debug('loading "%s" ..' % signals_module)
        signal_modules[app] = import_module(signals_module)
    except ImportError as e:
        logger.warning(
            'failed to import "%s", reason: %s' % (signals_module, str(e)))

นี่สำหรับโปรเจ็กต์ฉันไม่แน่ใจว่ามันใช้ได้ในระดับแอพหรือเปล่า


นี่คือวิธีการแก้ปัญหาที่ชื่นชอบเพื่อให้ไกลที่สุดเท่าที่มันเหมาะกับรูปแบบอื่น ๆ (เช่น tasks.py)
dalore

1
พบปัญหากับสิ่งนี้หากคุณเริ่มเชลล์ urls.py ไม่ได้รับการนำเข้าและสัญญาณของคุณจะไม่แนบ
dalore

ใช่คำตอบของฉันล้าสมัยดูเหมือนว่า django จะมีคลาส AppConfig ในทุกวันนี้ ครั้งล่าสุดที่ฉันใช้ django มันเป็นเวอร์ชัน 1.3 แนะนำให้สอบสวนรอบด้าน.
aisbaa

1
เรายังคงเป็น 1.6 ดังนั้นฉันจึงต้องย้ายสัญญาณทั้งหมดของเราไปยังโมเดลไม่เช่นนั้นคำสั่งขึ้นฉ่ายและการจัดการจะไม่ถูกหยิบขึ้นมา
dalore

5

ใน Django เวอร์ชันเก่าจะเป็นการดีที่จะใส่สัญญาณ__init__.pyหรืออาจจะอยู่ในmodels.py(แม้ว่าในตอนท้ายจะมีขนาดใหญ่สำหรับรสนิยมของฉันก็ตาม)

ด้วย Django 1.9 จะเป็นการดีกว่าที่ฉันคิดว่าจะวางสัญญาณบนsignals.pyไฟล์และนำเข้าด้วยapps.pyตำแหน่งที่จะโหลดหลังจากโหลดโมเดล

apps.py:

from django.apps import AppConfig


class PollsConfig(AppConfig):
    name = 'polls'

    def ready(self):
        from . import signals  # NOQA

คุณยังสามารถแบ่งสัญญาณของคุณในsignals.pyและhandlers.pyในโฟลเดอร์อื่นภายในโมเดลของคุณที่ตั้งชื่อได้signalsเช่นกัน แต่สำหรับฉันนั่นเป็นเพียงเรื่องวิศวกรรม ดูที่การวางสัญญาณ


3

ฉันเดาว่าคุณกำลังทำเช่นนั้นเพื่อให้สัญญาณของคุณได้รับการลงทะเบียนเพื่อให้พวกเขาพบที่ไหนสักแห่ง ฉันแค่ใส่สัญญาณของฉันลงในไฟล์ models.py ตามปกติ


ใช่เมื่อฉันย้ายสัญญาณภายในไฟล์โมเดลมันจะช่วยแก้ปัญหาได้ แต่ไฟล์ model.py ของฉันค่อนข้างใหญ่เมื่อมีคลาสผู้จัดการและกฎโมเดลทั้งหมด
Mo J. Mughrabi

1
ผู้จัดการค่อนข้างง่ายกว่าที่จะดึงประสบการณ์ของฉันออกมา Managers.py ftw.
Issac Kelly

3

สิ่งนี้ใช้ได้เฉพาะเมื่อคุณมีสัญญาณของคุณในsignals.pyไฟล์แยกต่างหาก

เห็นด้วยอย่างยิ่งกับคำตอบของ @EricMarcos แต่ควรระบุว่า django docsแนะนำอย่างชัดเจนว่าอย่าใช้ตัวแปร default_app_config (แม้ว่าจะไม่ผิดก็ตาม) สำหรับเวอร์ชันปัจจุบันวิธีที่ถูกต้องคือ:

my_app / apps.py

from django.apps import AppConfig

class MyAppConfig(AppConfig):
    name = 'my_app'

    def ready(self):
        import my_app.signals

settings.py

(ตรวจสอบให้แน่ใจว่าคุณไม่ได้มีเพียงชื่อแอพของคุณในแอพที่ติดตั้ง แต่เป็นพา ธ สัมพัทธ์ไปยัง AppConfig ของคุณแทน)

INSTALLED_APPS = [
    'my_app.apps.MyAppConfig',
    # ...
]

1

อีกทางเลือกหนึ่งคือการนำเข้าฟังก์ชันการโทรกลับจากsignals.pyและเชื่อมต่อในmodels.py:

Signal.py

def pre_save_callback_function(sender, instance, **kwargs):
    # Do stuff here

model.py

# Your imports here
from django.db.models.signals import pre_save
from yourapp.signals import pre_save_callback_function

class YourModel:
    # Model stuff here
pre_save.connect(pre_save_callback_function, sender=YourModel)

PS: การนำเข้าYourModelในsignals.pyจะสร้างการเรียกซ้ำนั้น ใช้senderแทน

Ps2: การบันทึกอินสแตนซ์อีกครั้งในฟังก์ชันการเรียกกลับจะสร้างการเรียกซ้ำ คุณสามารถสร้างอาร์กิวเมนต์การควบคุมใน.saveวิธีการเพื่อควบคุมได้

โดยการใช้ไซต์ของเรา หมายความว่าคุณได้อ่านและทำความเข้าใจนโยบายคุกกี้และนโยบายความเป็นส่วนตัวของเราแล้ว
Licensed under cc by-sa 3.0 with attribution required.