2011-08-18 133 views
-1

我对下面的代码有困惑,即为什么我们需要将布尔值传递给方法以及它返回什么?为什么我们需要将布尔值传递给方法

RefreshLookupList(false); 
private void RefreshLookupList(bool forExport) 

    { 
     grdLists.Rows.Clear(); 
     grdLists.DataSource = Lists.GetListItems(cboListType.SelectedValue); 
     grdLists.DataBind(); 

     lblCount.InnerText = "Lookups: " + grdLists.Rows.Count.ToString(); 
    } 
+0

你有没有试过在CodeReview上问这个问题? – GregC

回答

0

看起来像陈旧的旧参数。删除它。

该方法不返回任何内容。

+0

谢谢大家......但是如果我们给方法提供bool值,那么它的意思是什么? – shaker

+0

@shaker - 这取决于。很难说出原来的意图是什么。 –

0

从您发布的代码开始,它根本不需要布尔参数,您可以将它从方法签名中移除,并查看您的代码是否仍然编译。

0

嗯,你需要传递的布尔值,因为该方法的签名要求一个

,并返回“无中生有”,因为它宣布为无效,这是一个方法,而不是功能/特性

2

在这种特殊情况下,参数不被使用。但是将其从方法签名中删除可能并不安全。它可能专门用于更改签名以避免与另一种方法发生命名冲突。

真的不好,但有时是必要的。

就我个人而言,我会尝试删除它。如果由于与名称类似的方法发生冲突而将其删除时,它不会编译,我会将它保留在那里,但使参数成为默认值(如果您使用的是C#4)。我说让它留在那里,因为有人可能会用反射来引用它。

如果您知道在应用程序中未使用反射,请右键单击重构Visual Studio中的方法名称并将其重命名,这也将重命名所有引用(但不是用于反射的文字引用)。然后删除参数。

0

forExport未使用,但较早的实现可能已完成。 IE浏览器。 “由于历史原因”。

在版本控制中检查函数的历史记录。

0

它看起来像它从来没有使用过,方法不会返回任何东西。

0

那么,如果这是整体方法,然后传递forExport的值意味着什么。

该方法不使用任何参数。

而且void方法的返回值为null。

0

forExport变量从不使用,因此实际上是多余的。此方法可能已被重新考虑并且方法变量未被删除。

0

那么如果这是整个代码块,那么你根本不需要传递它。你可以删除它。

在方法的代码上进行搜索,找出还有什么叫它。如果没有,那么你可以安全地删除它。

试一试,并尝试重新编译代码。祝你好运

相关问题