Question

I have the following code:

    if (pick == null || pick == "active")
    {
       var Proj = db.usp_Train(idnbr)
                 .Where(a => a.Inactive == false).ToList();
     }        
     else 
     {
       var Proj = db.usp_Train(idnbr).ToList();

     }

return PartialView(Proj);

For where I have return PartialView(Projs) I get a message saying that Proj does not exist in the current Any idea on how I can fix this.

Answer 1

You’re declaring Proj within the scopes of the if and else blocks – so it doesn’t exist outside those blocks. You want something like:

// Or whatever the type is...
List<Train> proj;

if (pick == null || pick == "active")
{
    proj = db.usp_Train(idnbr)
             .Where(a => a.Inactive == false).ToList();
}        
else 
{
    proj = db.usp_Train(idnbr).ToList();
}
return PartialView(proj);

Alternatively, you could use the conditional operator:

bool pickActive = pick == null || pick == "active";
var proj = pickActive ? db.usp_Train(idnbr).Where(a => !a.Inactive).ToList()
                      : db.usp_Train(idnbr).ToList();
return PartialView(proj);

Or selectively apply the condition, which removes all the redundancy:

IQueryable<Train> proj = db.usp_Train(idnbr);
if (pick == null || pick == "active")
{
    proj = proj.Where(a => !a.Inactive);
}
return PartialView(proj.ToList());