1

我为只排除字符串作为输入的 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

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

4

4 回答 4

4

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

就像是

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()
    }
}
于 2008-10-08T07:09:11.517 回答
2

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

于 2008-10-08T07:08:48.367 回答
0

由于我不了解您的编程语言，因此我会将其保留为伪代码，但我的一般答案是使用 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
}

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

于 2008-10-08T07:08:34.497 回答
0

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

' Define an Enum with FlagsAttribute.
<FlagsAttribute( )> _
Enum TableOptions as Short
    Hide = 1
    AsReadOnly = 2
    Interactive = 4
    NoIndex = 8
    ViewAutomatic = 16
End Enum
于 2008-10-08T07:18:34.507 回答