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

Question

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

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

因此，

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

在设计我的类时，我在以下之间做出决定：

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

很多时候，意外事件的空间太大，特别是在这种情况下，您必须根据输入确定如何采取行动，然后进一步进行验证处理（即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
}

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

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

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