Question

我有以下两种方法：

class Debug
{
    static bool OutputToConsole = true;

    public static void Log(string Type, string URL, StringBuilder Parameters)
    {
        string Output = Type + ":" + new string(' ', 9 - Type.Length) + URL + " { " + Parameters.ToString() + " }";
        Trace.WriteLine(Output);
        if(OutputToConsole) Console.WriteLine(Output);
    }

    public static void Log(string Data)
    {
        string Output = "Response: " + Data;
        Trace.WriteLine(Output);
        if(OutputToConsole) Console.WriteLine(Output);
    }
}

如果你注意到，只有string Output发生变化。

两种方法之后的两行相同。

我只是想知道是否有办法保留DRY principle并结合这两种方法？

Answer 1

将公共代码重构为私有帮助函数:( Visual Studio将通过突出显示代码的相关部分，右键单击，并选择Refactor - ＆gt; extract方法...来完成所有这些...）

private static void LogHelper(string text)
{
    Trace.WriteLine(text);
    if(OutputToConsole) Console.WriteLine(text);
}

（注意名称更改以确保其与Log(string Data)具有不同的签名。）

然后只需从其他人那里调用该功能。

Answer 2

不确定这是多少改进...

class Debug
{
    static bool OutputToConsole = true;

    public static void LogRequest(string type, string url, StringBuilder params)
    {
        log(type + ":" + new string(' ', 9 - type.Length) + url + " { " + params.ToString() + " }");
    }

    public static void LogResponse(string data)
    {
        log("Response: " + data);
    }

    private static void log(string msg)
    {
        Trace.WriteLine(msg);
        if(OutputToConsole) Console.WriteLine(msg);
    }
}

Answer 3

可能无法合并两种方法并从中获取任何真正的值，因为生成的Output不同，但您可以创建执行实际跟踪的方法：

public static void Trace(string Output)
{
    Trace.WriteLine(Output);
    if(OutputToConsole) Console.WriteLine(Output);
}

然后从那些方法调用它。

Answer 4

你可以使用可选参数，这有点不寻常，但有趣：

public static void Log(string Data = null, string Type = null, string URL = null, StringBuilder Parameters = null)
{
    string Output = "";
    if (Data != null)
    {
        Output = "Response: " + Data;
    }
    else if (Type != null && URL != null && Parameters != null)
    {
        Output = Type + ":" + new string(' ', 9 - Type.Length) + URL + " { " + Parameters.ToString() + " }";
    }
    else
    {
        throw new ArgumentException("Provide yada yada arguments lala");
    }
    Trace.WriteLine(Output);
    if (OutputToConsole) Console.WriteLine(Output);
}

像这样使用：

Log(Data: "Test");
StringBuilder sb = new StringBuilder();
Log(Type: "myType", URL: "www.bla", Parameters: sb);

有没有办法结合这两种方法？

4 个答案: