传递空参数

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

你可以给viewModel分配一个默认值：

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

这样，如果您只是调用GetItems()，它会将viewModel 视为new ItemDescriptionFormViewModel()。

Answer 3

这样声明：

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 4

您在此处取消引用空引用：

_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() 子句以添加 pass：

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

然后你可以安全地传递null作为参数，你甚至可以设置一个默认参数值，所以如果没有传递参数，它假定null。

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

Answer 5

如果你想保留这两个方法（我的意思是 GetItems() 和 GetItems(viewModel)），它们会自动重载。

如果你想要第二种方法，你可以像这样传递参数

 private IEnumerable<SearchItems> GetItems(ItemDescriptionFormViewModel? viewModel)

现在您可以将 null 作为其参数传递。