传递null参数

Question

我目前在解决这个问题时遇到了一些麻烦。我有一个方法，不需要任何参数，现在我需要添加一个参数，但我不想在调用该方法的所有不同位置添加参数。这是我目前的方法：

private IEnumerable<SearchItems> GetItems(ItemDescriptionFormViewModel viewModel = null) 
    { 
     IOrderedQueryable<ItemDescription> items= _itemDescriptionRepository.FindAll().OrderBy(
      c => c.Sort == null).ThenBy(
       c => c.Sort).ThenBy(c => c.Description); 

     if(items.Count()==0) 
      ModelState.AddModelError("", string.Format("No active {0} entered.", Kids.Resources.Entities.ItemDescription.EntityNamePlural)); 
     return 
      _itemDescriptionRepository.FindAll().OrderBy(c => c.Description).Where(a=>a.IsActive == true || viewModel == null || a.ItemDescriptionId == viewModel.ItemDescriptionId).Select(
       c => new SearchItems {Text = c.Description, Value = c.ItemDescriptionId.ToString()}); 
    } 

我试着传递null作为其他地方的参数调用此方法，但我得到一个错误。什么方式来超载这个问题？

目的：添加viewModel的目的是因为我有一个下拉列表与活动项目可供选择。一旦用户选择活动项目，然后出于某种原因该项目变为非活动状态并且用户去编辑他们的选择。在下拉列表中应该有活动项目的列表以及他们之前选择的项目现在不活动。我正在使用ViewModel来检查先前选择的项目的ID。

感谢

Answer 1

大家...感谢您尝试解决此问题的所有帮助和努力。我终于能够解决它，这里是我解决了我的问题的人：Checking List Item Id

Answer 2

声明它是这样的：

private IEnumerable<SearchItems> GetItems(ItemDescriptionFormViewModel viewModel = null) 

，并确保有一个null检查在你的方法，这样的事情：

return 
     _itemDescriptionRepository.FindAll().OrderBy(c => c.Description).Where(a=> viewModel == null || (a.IsActive == true || a.ItemDescriptionId == viewModel.ItemDescriptionId)).Select(
      c => new SearchItems {Text = c.Description, Value = c.ItemDescriptionId.ToString()}); 

这将允许你打电话使用GetItems()和GetItems(ItemDescriptionFormViewModel viewModel)的方法

编辑：无视？因为ItemDescriptionFormViewModel是一类

Answer 3

你在这里提领一空引用：

_itemDescriptionRepository.FindAll() 
    .OrderBy(c => c.Description) 
    .Where(a=>a.IsActive == true || 
     a.ItemDescriptionId == viewModel.ItemDescriptionId) // if viewModel null, this throws 
    .Select(c => new SearchItems 
    { 
     Text = c.Description, Value = c.ItemDescriptionId.ToString() 
    }); 

所以你可以只更新Where()条款添加通如果视图模型为空：

... 
.Where(a => a.IsActive || viewModel == null || viewModel.ItemDescriptionId == a.ItemDescriptionId) 

然后，您可以安全地通过null作为参数，甚至可以设置默认参数值，因此如果没有参数传递，它将采用null。

这是当然的，假设ItemDescriptionFormViewModel是一类，如果它是一个结构，那么你就必须使其可为空ItemDescriptionFormViewModel?

Answer 4

如果你想保留这两种方法（我的意思是GetItems（）和GetItems （viewModel））它们会自动超载。

如果你想在第二个方法，你可以传递参数这样

private IEnumerable<SearchItems> GetItems(ItemDescriptionFormViewModel? viewModel) 

现在，你可以把null作为它的参数。

Answer 5

您可以将默认值赋给viewModel：

private IEnumerable<SearchItems> GetItems(
    ItemDescriptionFormViewModel viewModel = null) 
{ 
    if (viewModel == null) 
     viewModel = new ItemDescriptionFormViewModel(); 

这样一来，如果你只需调用GetItems()，它将把viewModel作为new ItemDescriptionFormViewModel()。