ตรวจสอบว่า OneToOneField ไม่มีใน Django


85

ฉันมีสองรุ่นดังนี้:

class Type1Profile(models.Model):
    user = models.OneToOneField(User, unique=True)
    ...


class Type2Profile(models.Model):
    user = models.OneToOneField(User, unique=True)
    ...

ฉันต้องทำอะไรบางอย่างหากผู้ใช้มีโปรไฟล์ Type1 หรือ Type2:

if request.user.type1profile != None:
    # do something
elif request.user.type2profile != None:
    # do something else
else:
    # do something else

แต่สำหรับผู้ใช้ที่ไม่มีโปรไฟล์ type1 หรือ type2 การรันโค้ดแบบนี้จะทำให้เกิดข้อผิดพลาดต่อไปนี้:

Type1Profile matching query does not exist.

ฉันจะตรวจสอบประเภทของโปรไฟล์ที่ผู้ใช้ได้อย่างไร?

ขอบคุณ

คำตอบ:


92

ในการตรวจสอบว่าความสัมพันธ์ (OneToOne) มีอยู่หรือไม่คุณสามารถใช้hasattrฟังก์ชัน:

if hasattr(request.user, 'type1profile'):
    # do something
elif hasattr(request.user, 'type2profile'):
    # do something else
else:
    # do something else

4
ขอบคุณสำหรับการแก้ปัญหานี้ น่าเสียดายที่การดำเนินการนี้ใช้ไม่ได้ตลอดเวลา ในกรณีที่คุณต้องการร่วมงานด้วยในselect_related()ตอนนี้หรือในอนาคต - หรืออาจจะเพื่อให้แน่ใจว่าคุณได้จัดการกับเวทมนตร์ประเภทอื่น ๆ ที่อาจเกิดขึ้นที่อื่น - คุณต้องขยายการทดสอบดังนี้:if hasattr(object, 'onetoonerevrelattr') and object.onetoonerevrelattr != None
คลาส stacker

7
โปรดทราบว่าในหลาม <3.2 hasattrจะกลืนทุกDoesNotExistข้อยกเว้นที่เกิดขึ้นในระหว่างการค้นหาฐานข้อมูลและไม่ได้เป็นเพียง นี่อาจจะเสียไม่ใช่สิ่งที่คุณต้องการ
Pi Delport

ไม่ทำงานกับ python 2.7 แม้ว่า OneToOne จะไม่มีอยู่ แต่จะส่งคืนอ็อบเจ็กต์ django.db.models.fields.related.RelatedManager
alexpirine

@alartur คุณใช้ django เวอร์ชั่นอะไร?
joctee

Django 1.5. แต่ฉันแก้ไขปัญหาเฉพาะของฉันด้วยการใช้สิ่งที่ฉันต้องการทำในวิธีที่แตกต่างออกไป
alexpirine

48

เป็นไปได้ที่จะดูว่าความสัมพันธ์แบบหนึ่งต่อหนึ่งที่เป็นโมฆะนั้นเป็นโมฆะสำหรับโมเดลหนึ่ง ๆ หรือไม่เพียงแค่ทดสอบฟิลด์ที่เกี่ยวข้องบนโมเดลสำหรับNoneness แต่ต่อเมื่อคุณทดสอบโมเดลที่ความสัมพันธ์แบบหนึ่งต่อหนึ่งเกิดขึ้น ตัวอย่างเช่นให้สองคลาสนี้ ...

class Place(models.Model):
    name = models.CharField(max_length=50)
    address = models.CharField(max_length=80)

class Restaurant(models.Model):  # The class where the one-to-one originates
    place = models.OneToOneField(Place, blank=True, null=True)
    serves_hot_dogs = models.BooleanField()
    serves_pizza = models.BooleanField()

…เพื่อดูว่า Restaurantมี a Placeหรือไม่เราสามารถใช้รหัสต่อไปนี้:

>>> r = Restaurant(serves_hot_dogs=True, serves_pizza=False)
>>> r.save()
>>> if r.place is None:
>>>    print "Restaurant has no place!"
Restaurant has no place!

หากต้องการดูว่าPlaceมีRestaurantหรือไม่สิ่งสำคัญคือต้องเข้าใจว่าการอ้างถึงrestaurantสถานที่ให้บริการในกรณีที่PlaceมีRestaurant.DoesNotExistข้อยกเว้นหากไม่มีร้านอาหารที่เกี่ยวข้อง นี้เกิดขึ้นเพราะ Django QuerySet.get()ทำการค้นหาภายในโดยใช้ ตัวอย่างเช่น:

>>> p2 = Place(name='Ace Hardware', address='1013 N. Ashland')
>>> p2.save()
>>> p2.restaurant
Traceback (most recent call last):
    ...
DoesNotExist: Restaurant matching query does not exist.

ในสถานการณ์นี้มีดโกนของ Occam มีชัยและแนวทางที่ดีที่สุดในการตัดสินใจว่าสิ่งที่PlaceมีRestautrantจะเป็นมาตรฐานtry/ exceptโครงสร้างตามที่อธิบายไว้ที่นี่หรือไม่

>>> try:
>>>     restaurant = p2.restaurant
>>> except Restaurant.DoesNotExist:
>>>     print "Place has no restaurant!"
>>> else:
>>>     # Do something with p2's restaurant here.

ในขณะที่ข้อเสนอแนะของ joctee ในการใช้hasattrงานในทางปฏิบัติมันใช้ได้ผลโดยบังเอิญเท่านั้นเนื่องจากจะhasattrระงับข้อยกเว้นทั้งหมด (รวมถึงDoesNotExist) เมื่อเทียบกับ just AttributeErrors เช่นที่ควร ในฐานะที่เป็นพี่ Delportชี้ให้เห็นพฤติกรรมนี้ได้รับการแก้ไขจริงในหลาม 3.2 ต่อตั๋วต่อไปนี้: http://bugs.python.org/issue9666 นอกจากนี้ - และเสี่ยงต่อการแสดงความคิดเห็น - ฉันเชื่อว่าtry/ exceptโครงสร้างข้างต้นเป็นตัวแทนของการทำงานของ Django ในขณะที่ใช้hasattrสามารถทำให้ปัญหาสำหรับมือใหม่ซึ่งอาจสร้าง FUD และแพร่กระจายนิสัยที่ไม่ดี

แก้ไข การประนีประนอมอย่างสมเหตุสมผลของ Don Kirkbyก็ดูสมเหตุสมผลสำหรับฉันเช่นกัน


19

ฉันชอบคำตอบของ jocteeเพราะมันง่ายมาก

if hasattr(request.user, 'type1profile'):
    # do something
elif hasattr(request.user, 'type2profile'):
    # do something else
else:
    # do something else

ผู้แสดงความคิดเห็นรายอื่นได้แจ้งข้อกังวลว่าอาจใช้ไม่ได้กับ Python หรือ Django บางเวอร์ชัน แต่เอกสารของ Djangoแสดงเทคนิคนี้เป็นหนึ่งในตัวเลือก:

คุณยังสามารถใช้ hasattr เพื่อหลีกเลี่ยงความจำเป็นในการจับข้อยกเว้น:

>>> hasattr(p2, 'restaurant')
False

แน่นอนเอกสารยังแสดงเทคนิคการจับข้อยกเว้น:

p2 ไม่มีร้านอาหารที่เกี่ยวข้อง:

>>> from django.core.exceptions import ObjectDoesNotExist
>>> try:
>>>     p2.restaurant
>>> except ObjectDoesNotExist:
>>>     print("There is no restaurant here.")
There is no restaurant here.

ฉันเห็นด้วยกับโจชัวว่าการจับข้อยกเว้นทำให้ชัดเจนขึ้นว่าเกิดอะไรขึ้น แต่ดูเหมือนจะยุ่งกว่าสำหรับฉัน บางทีนี่อาจเป็นการประนีประนอมที่สมเหตุสมผล?

>>> print(Restaurant.objects.filter(place=p2).first())
None

นี่เป็นเพียงการสืบค้นRestaurantวัตถุตามสถานที่ มันกลับมาNoneหากสถานที่นั้นไม่มีร้านอาหาร

นี่คือตัวอย่างข้อมูลปฏิบัติการเพื่อให้คุณเล่นกับตัวเลือกต่างๆ หากคุณติดตั้ง Python, Django และ SQLite3 ไว้ก็ควรรัน ฉันทดสอบด้วย Python 2.7, Python 3.4, Django 1.9.2 และ SQLite3 3.8.2

# Tested with Django 1.9.2
import sys

import django
from django.apps import apps
from django.apps.config import AppConfig
from django.conf import settings
from django.core.exceptions import ObjectDoesNotExist
from django.db import connections, models, DEFAULT_DB_ALIAS
from django.db.models.base import ModelBase

NAME = 'udjango'


def main():
    setup()

    class Place(models.Model):
        name = models.CharField(max_length=50)
        address = models.CharField(max_length=80)

        def __str__(self):              # __unicode__ on Python 2
            return "%s the place" % self.name

    class Restaurant(models.Model):
        place = models.OneToOneField(Place, primary_key=True)
        serves_hot_dogs = models.BooleanField(default=False)
        serves_pizza = models.BooleanField(default=False)

        def __str__(self):              # __unicode__ on Python 2
            return "%s the restaurant" % self.place.name

    class Waiter(models.Model):
        restaurant = models.ForeignKey(Restaurant)
        name = models.CharField(max_length=50)

        def __str__(self):              # __unicode__ on Python 2
            return "%s the waiter at %s" % (self.name, self.restaurant)

    syncdb(Place)
    syncdb(Restaurant)
    syncdb(Waiter)

    p1 = Place(name='Demon Dogs', address='944 W. Fullerton')
    p1.save()
    p2 = Place(name='Ace Hardware', address='1013 N. Ashland')
    p2.save()
    r = Restaurant(place=p1, serves_hot_dogs=True, serves_pizza=False)
    r.save()

    print(r.place)
    print(p1.restaurant)

    # Option 1: try/except
    try:
        print(p2.restaurant)
    except ObjectDoesNotExist:
        print("There is no restaurant here.")

    # Option 2: getattr and hasattr
    print(getattr(p2, 'restaurant', 'There is no restaurant attribute.'))
    if hasattr(p2, 'restaurant'):
        print('Restaurant found by hasattr().')
    else:
        print('Restaurant not found by hasattr().')

    # Option 3: a query
    print(Restaurant.objects.filter(place=p2).first())


def setup():
    DB_FILE = NAME + '.db'
    with open(DB_FILE, 'w'):
        pass  # wipe the database
    settings.configure(
        DEBUG=True,
        DATABASES={
            DEFAULT_DB_ALIAS: {
                'ENGINE': 'django.db.backends.sqlite3',
                'NAME': DB_FILE}},
        LOGGING={'version': 1,
                 'disable_existing_loggers': False,
                 'formatters': {
                    'debug': {
                        'format': '%(asctime)s[%(levelname)s]'
                                  '%(name)s.%(funcName)s(): %(message)s',
                        'datefmt': '%Y-%m-%d %H:%M:%S'}},
                 'handlers': {
                    'console': {
                        'level': 'DEBUG',
                        'class': 'logging.StreamHandler',
                        'formatter': 'debug'}},
                 'root': {
                    'handlers': ['console'],
                    'level': 'WARN'},
                 'loggers': {
                    "django.db": {"level": "WARN"}}})
    app_config = AppConfig(NAME, sys.modules['__main__'])
    apps.populate([app_config])
    django.setup()
    original_new_func = ModelBase.__new__

    @staticmethod
    def patched_new(cls, name, bases, attrs):
        if 'Meta' not in attrs:
            class Meta:
                app_label = NAME
            attrs['Meta'] = Meta
        return original_new_func(cls, name, bases, attrs)
    ModelBase.__new__ = patched_new


def syncdb(model):
    """ Standard syncdb expects models to be in reliable locations.

    Based on https://github.com/django/django/blob/1.9.3
    /django/core/management/commands/migrate.py#L285
    """
    connection = connections[DEFAULT_DB_ALIAS]
    with connection.schema_editor() as editor:
        editor.create_model(model)

main()

10

ลองใช้ / ยกเว้นบล็อกเป็นอย่างไร?

def get_profile_or_none(user, profile_cls):

    try:
        profile = getattr(user, profile_cls.__name__.lower())
    except profile_cls.DoesNotExist:
        profile = None

    return profile

จากนั้นใช้แบบนี้!

u = request.user
if get_profile_or_none(u, Type1Profile) is not None:
    # do something
elif get_profile_or_none(u, Type2Profile) is not None:
    # do something else
else:
    # d'oh!

ฉันคิดว่าคุณสามารถใช้สิ่งนี้เป็นฟังก์ชันทั่วไปเพื่อรับอินสแตนซ์ OneToOne แบบย้อนกลับโดยให้คลาสเริ่มต้น (ที่นี่: คลาสโปรไฟล์ของคุณ) และอินสแตนซ์ที่เกี่ยวข้อง (ที่นี่: request.user)


3

ใช้select_related!

>>> user = User.objects.select_related('type1profile').get(pk=111)
>>> user.type1profile
None

2
ฉันรู้ว่ามันทำงานแบบนี้ แต่พฤติกรรมของ select_related นี้มีการบันทึกไว้จริงหรือไม่
Kos

3
ฉันแค่พยายามนี้ใน Django 1.9.2 RelatedObjectDoesNotExistและมันก็เกิด
Don Kirkby

1

ในกรณีที่คุณมี Model

class UserProfile(models.Model):
    user = models.OneToOneField(User, unique=True)

และคุณเพียงแค่ต้องทราบสำหรับผู้ใช้ใด ๆ ว่า UserProfile มีอยู่ / หรือไม่ - วิธีที่มีประสิทธิภาพที่สุดจากมุมมองฐานข้อมูลเพื่อใช้แบบสอบถามที่มีอยู่ที่มีอยู่แบบสอบถาม

การสืบค้นที่มีอยู่จะส่งคืนเพียงบูลีนแทนที่จะเป็นการเข้าถึงแอตทริบิวต์ย้อนกลับเช่นhasattr(request.user, 'type1profile')- ซึ่งจะสร้างการสืบค้นและส่งคืนการแสดงวัตถุแบบเต็ม

ในการดำเนินการ - คุณต้องเพิ่มคุณสมบัติให้กับโมเดลผู้ใช้

class User(AbstractBaseUser)

@property
def has_profile():
    return UserProfile.objects.filter(user=self.pk).exists()

0

ฉันใช้ชุดค่าผสมของ has_attr และไม่มี:

class DriverLocation(models.Model):
    driver = models.OneToOneField(Driver, related_name='location', on_delete=models.CASCADE)

class Driver(models.Model):
    pass

    @property
    def has_location(self):
        return not hasattr(self, "location") or self.location is None

0

หนึ่งในวิธีการอันชาญฉลาดคือการเพิ่มช่องOneToOneOrNoneField แบบกำหนดเองและใช้ [ใช้ได้กับ Django> = 1.9]

from django.db.models.fields.related_descriptors import ReverseOneToOneDescriptor
from django.core.exceptions import ObjectDoesNotExist
from django.db import models


class SingleRelatedObjectDescriptorReturnsNone(ReverseOneToOneDescriptor):
    def __get__(self, *args, **kwargs):
        try:
            return super().__get__(*args, **kwargs)
        except ObjectDoesNotExist:
            return None


class OneToOneOrNoneField(models.OneToOneField):
    """A OneToOneField that returns None if the related object doesn't exist"""
    related_accessor_class = SingleRelatedObjectDescriptorReturnsNone

    def __init__(self, *args, **kwargs):
        kwargs.setdefault('null', True)
        kwargs.setdefault('blank', True)
        super().__init__(*args, **kwargs)

การนำไปใช้

class Restaurant(models.Model):  # The class where the one-to-one originates
    place = OneToOneOrNoneField(Place)
    serves_hot_dogs = models.BooleanField()
    serves_pizza = models.BooleanField()

การใช้งาน

r = Restaurant(serves_hot_dogs=True, serves_pizza=False)
r.place  # will return None

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