djangoのserializerで、バリデーションと変換で同じクエリを二回発行したくない場合の手法について

Question

例えばユーザのメールアドレスの存在確認とuser_idへの変換を行いたい場合に、

class BlogSearchSerializer(serializers.Serializer):
    user_email = serializers.CharField(required=false)
    user_id = serializers.SerializerMethodField()

    def validate_user_email(self, value):
        user = User.get(email=value)  # ←これと
        if not user:
            raise serializers.ValidationError('user not found')
        return value

    def get_user_id(self, instance):
        user = User.get(email=value)  # ←これ
        return user.id

バリデーションとID変換で、二回User.getを発行するのが気になり、悩んでいます。
このような場合のベストプラクティスはあるのでしょうか。

Comments

[belgh]

普通？はメアドが間違ってるから検索結果が0件とするところですが、
「メアドが間違っています。該当ユーザは存在しません」というエラーを返す必要がある想定です。

Answer 1

ベストプラクティスかどうか自信がないですが、現実装はこうしてみました。

class BlogSearchSerializer(serializers.Serializer):
    user_email = serializers.CharField(required=false)
    user_id = serializers.SerializerMethodField()

    def __init__(self, *args, **kwargs):
        super().__init__(*args, **kwargs)
        try:
            self.filtered_user = User.get(email=self.init_data.get('email')) if email else None
        except:
            self.filtered_user = None

    def validate_user_email(self, value):
        if value and self.filtered_user is None:
            raise serializers.ValidationError('user not found')
        return value

    def get_user_id(self, instance):
        if self.filtered_user:
            return self.filtered_user.id
        return None

他に方法ありましたら、ぜひご教示お願い致します。