DJANGO FORM もう少しスマートに書きたいのですが

Question

DJANGOを触り続けて、約一カ月たち
ようやくフォームを使った処理を書けるようになりました。

ガントチャートをセレクタでフィルタリング処理を書いています。

form入力後の条件分岐が、構造プログラマの
悪習でif文続きになってしまい、どうもすっきりしません

もう少しスマートに書く方法は、ないものでしょうか?

    if title == "" and action=="" and order != "None":
        target = Schedules.objects.all().order_by(order)[:100]     
    elif order != "None":
        target = Schedules.objects.filter(Q(title=title)|Q(action=action)).order_by(order)
    else:
        target = Schedules.objects.filter(Q(title=title)|Q(action=action))

class Chartconfig(forms.Form):    
    title = ModelChoiceField(
          queryset = Schedules.objects.all().values_list("title",flat=True).distinct(),
          label='item',      to_field_name="title", required=False)

    action = ModelChoiceField(
          queryset = Schedules.objects.all().values_list("action",flat=True).distinct(),
          label='action',    to_field_name="action", required=False)

    order = forms.ChoiceField(
          choices = (
    ('None', 'なし'),
    ('start_plan', '予定'),
    ('start_plan', '実績'),
),
          label = "Sort",required= True)


def Gantchart(request):
    results = {}
    ret = ''
    
    if request.method == 'POST':
        ret = 'OK'
        form = Chartconfig(request.POST)
        is_valid = form.is_valid()

        title =  form.data["title"]
        action =  form.data["action"]
        order = form.data["order"]

        if title == "" and action=="" and order != "None":
            target = Schedules.objects.all().order_by(order)[:100]     
        elif order != "None":
            target = Schedules.objects.filter(Q(title=title)|Q(action=action)).order_by(order)
        else:
            target = Schedules.objects.filter(Q(title=title)|Q(action=action))
  
    else:    
        target = Schedules.objects.all()[:100]
        form = Chartconfig()

    context = {'form': form}
    context.update(csrf(request))
    context.update({'context': gt.markdown(target)})
    return TemplateResponse(request, 'faispc/gantchart.html', context = context)

Answer 1

コードを拝見すると title と action の両方が入力されているときは OR 条件で検索するようにされているようですが、これは意図されているとおりの挙動ですか？

目指されている形がどういうものかよくわかっていませんが、私が Queryset を使用してモデルオブジェクトの一覧を返す検索フォームを作るときには以下のようなパターンを使うことが多いです。直接の回答にはなっていませんがご参考になればと思います。

# どこかで定義しておく
VALID_ORDERS = ['start_plan', ...]

...

# 以下で入力に応じて Queryset の絞り込みを行う
queryset = Schedules.objects.all()

if order in VALID_ORDERS:
    queryset = queryset.order_by(order=order)

# ↓ OR 条件になっていません。あくまでも参考例です。
if title:
    queryset = queryset.filter(title=title)

if action:
    queryset = queryset.filter(action=action)

target = queryset[:100]

おそらく最後の [:100] のところは Django のページネーション機能を使ったりすればこの書き方にはしないと思いますので、これはあくまでもイメージです。

Comments

[yuujiMotoki]

返答が遅くなりました。アドバイスありがとうございました。