发送 HTML 参数和文件路径参数？

Question

我正在创建一个需要打印 HTML 字符串和 HTML 文档的打印机类。所以基本上可以得到：

Printer.Print("<b>Hello world</b>");

和

Printer.Print(@"C:\hello.html");

因此，在设计我的课程时，我在以下之间做出了 Print 方法定义：

public static void Print(string inputString, string mode){
    if(mode=="htmlString"){//Print the string itself}
    else if(mode=="htmlFile"){//Print the document in the filepath}
}

或者

public static void Print(string inputString){
    if(file.Exists(inputString)){//Print the document in the filepath}
    else{//Print the string itself}
}

一般来说，哪种做法更好？第一个选项需要另一个不太好的参数，但是如果我们使用第二个选项，如果我们打算实际打印一个文件但使用了不正确的文件名，它将打印错误的东西。

Answer 1

我同意使用两种方法是最好的方法。但是，.Net 约定将具有以下方法名称：

public static void Print(string path) { ... }
public static void PrintHtml(string html) { ... }

Answer 2

我建议你采用失望先生建议的设计。

但是，如果出于某种原因您想保留最初的想法，我会稍作改动。与其将模式作为字符串传递，不如将其作为枚举传递。事实上，您也可以将失望先生的建议与此联系起来。例如

public enum PrintMode
{
  File,
  Raw
}

public static void Print(string printData, PrintMode mode)
{
  if(mode == PrintMode.Raw)
  {
    //Print the string itself
  }
  else if (mode == PrintMode.File)
  {
    //Print the document in the filepath
  }
  else
  {
    throw new ArgumentException("Invalid print mode specified");
  }
}

public static void PrintString(string input)
{
  Print(input, PrintMode.Raw);
}

public static void PrintFile(string input)
{
  Print(input, PrintMode.File);
}

您的第二个想法是个坏主意，因为每当用户打印原始字符串时，您都会执行不必​​要的文件系统检查。更重要的是，它可能会抛出异常，因为在打印原始字符串时，这将不是有效的文件路径。所以Exists 检查可能会失败。

Answer 3

很多时候，意外事件的空间太大，特别是在这种情况下，您必须根据输入确定如何采取行动，然后进一步进行验证处理（即 File.Exists )，它在为误报而哭泣。在我看来，改为这样做：

public static void PrintString(string input)
{
    //print the string, knowing precisely this is the intent,
    //and if not, it's what you're going to do anyway!
}

public static void PrintFile(string fileName)
{
    //no qualms here, you're going to print a file
}