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

多くの引数を取ることができる関数を処理する 1 つの方法は、その関数の引数を保持することだけを目的とする新しいオブジェクト型を作成することです。次に、そのタイプの新しいオブジェクトを作成し、必要に応じてプロパティを設定してから、その 1 つのオブジェクト参照を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

すべてのブール値パラメーターを、フラグとしてマークされた列挙型の単一のパラメーターに切り替えることができます。宣言の例を次に示します。

' 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 に答える