ifステートメントのコードをクリーンアップ

Question

このような記述がありますが、どうすればこのロジックを最適化できますか？私は約1時間だと思うが、私は見つけられない。

は、どのように私はこのロジックをsimplfyことができますか？

if (model.WaitingChecked) 
{ 
    if (model.selectedStatus != 0) 
    { 
     model.DataList = data.Where(x => x.status != 6 && x.status == model.selectedStatus); 
     return View(model); 
    } 

    model.DataList = data.Where(x => x.status != 6); 
    return View(model); 
} 

if (model.SelectedStatus != 0) 
{ 
    model.DataList = data.Cast<DataDetailedList>().Where(x => x.status == model.selectedStatus); 
    return View(model); 
} 

Answer 1

さまざまな条件でパターンマッチングを使用できます。パターンマッチングは網羅的である必要があることを忘れないでください。意味するすべての値の組み合わせを定義する必要があります。

(model.WaitingChecked, model.selectedStatus) match{ 
    case (true, 0) => 
    model.DataList = data.Where(x => x.status != 6) 
    View(model) 
    case (true, _) => 
    model.DataList = data.Where(x => x.status != 6 && x.status == model.selectedStatus) 
    View(model) 
    case (_, 0) => 
    //missing logic for model.selectedStatus is zero return 
    case (_, _) => 
    model.DataList = data.Cast<DataDetailedList>().Where(x => x.status == model.selectedStatus) 
    View(model) 
}