如何重构需要大量参数的函数？

Question

我为 COM 对象编写了一个包装器，仅将字符串作为输入除外，因此在良好的 OOP 实践中，我将字符串包装在函数中，以便更容易构建和调用。

我只是想知道是否有人能想出更好的方法来执行以下代码。

   Public Function OpenTable(ByVal TablePath As String, Optional ByVal OpenAs As String = Nothing, _
            Optional ByVal Hide As Boolean = False, Optional ByVal AsReadOnly As Boolean = False, _
            Optional ByVal Interactive As Boolean = True, Optional ByVal Password As String = Nothing, _
            Optional ByVal NoIndex As Boolean = False, Optional ByVal ViewAutomatic As Boolean = True) As TableInfo

            If String.IsNullOrEmpty(TablePath) Then
                Throw New ArgumentNullException("TablePath", "TablePath cannot be null or empty")
            End If

            Dim Builder = New StringBuilder("Open Table ")
            Builder.AppendFormat("{0}{1}{2}", ControlChars.Quote, TablePath, ControlChars.Quote)

            If (Not String.IsNullOrEmpty(OpenAs)) Then Builder.AppendFormat(" as {0} ", OpenAs)
            If (Hide) Then Builder.Append(" Hide ")
            If (AsReadOnly) Then Builder.Append(" ReadOnly ")
            If (Interactive) Then Builder.Append(" Interactive ")
            If (Not String.IsNullOrEmpty(Password)) Then Builder.AppendFormat(" Password {0} ", Password)
            If (NoIndex) Then Builder.Append(" NoIndex ")
            If (ViewAutomatic) Then Builder.Append(" View Automatic ")

            MyComApp.Do(Builder.ToString)

            Dim FileInfo = New IO.FileInfo(TablePath)
            Return New TableInfo(FileInfo.Name.Substring(0, InStrRev(FileInfo.Name, ".") - 1))
        End Function

该函数必须接受的参数数量是我最担心的。 这个还不错，但是将来我可能需要制作一些其他函数，这些函数将需要更多参数，所以我主要寻找更好的方法来构建大参数函数。

Answer 1

在这种情况下，许多参数似乎只是“配置值”（最终是字符串），您可以修改它以接受您在调用之前准备的所有配置的单个类，并且相应地返回字符串。

就像是

class COMConfiguration {
    private bool Hide = false;
    private bool AsReadOnly = false;
    //and so on...

    public void setHide(bool v) { Hide = v; }

    //only setters

    public string getConfigString() {
        StringBuilder sb = new StringBuilder();
        if (Hide) { sb.Append(" Hide "); }
        if (AsReadOnly) { sb.Append(" ReadOnly "); }
        //and so on
        return sb.ToString()
    }
}

Answer 2

处理可能带有大量参数的函数的一种方法是创建一个新的对象类型，其唯一目的是保存该函数的参数。 然后，您创建该类型的一个新对象，根据需要设置属性，然后将该一个对象引用传递给您的 OpenTable 函数。

Answer 3

由于我不知道你的编程语言，我将把它保留为伪代码，但我的一般答案是使用 ann 数组作为单个参数：

function OpenTable( options As array) {
    if (options is not array or options is empty) {
        Throw exception
    }
    return_string = "";
    if ( key is set ('readOnly', options) and is not empty) {
        return_string = return_string + ' readonly';
    }
    // repeat last 3 lines for all your params
}

好吧，你的函数的最后一部分对我来说没有意义，但参数数组的想法应该我想遇到过。 祝你好运。

Answer 4

您可以将所有布尔参数切换为 的单个参数enum 类型，标记为 Flags。 这是一个声明示例：

' Define an Enum with FlagsAttribute.
<FlagsAttribute( )> _
Enum TableOptions as Short
    Hide = 1
    AsReadOnly = 2
    Interactive = 4
    NoIndex = 8
    ViewAutomatic = 16
End Enum