如何在C#中优化含相似XPath定位器的代码？

你的思路是对的，但要注意语法细节，不同语言里字符串格式化的写法有区别，下面分情况说明：

1. C# 中的正确写法 #

你写的代码接近正确，但By.XPath方法不直接支持传入格式化参数，需要先完成字符串拼接：

// 定义通用模板
private const string ButtonTextXPathTemplate = ".//button[contains(text(), '{0}')]";

// 使用时格式化生成具体定位器
By viewDetailsBtn = By.XPath(string.Format(ButtonTextXPathTemplate, "View Details"));
By additionalDataBtn = By.XPath(string.Format(ButtonTextXPathTemplate, "Additional Data"));
By submitBtn = By.XPath(string.Format(ButtonTextXPathTemplate, "Submit"));

2. 进阶优化：封装成工具方法 #

为了更便捷复用，可以封装一个方法，省去每次写string.Format的重复操作：

private By GetButtonByText(string buttonText)
{
    string xPath = string.Format(".//button[contains(text(), '{0}')]", buttonText);
    return By.XPath(xPath);
}

// 调用示例
By viewDetailsBtn = GetButtonByText("View Details");

3. 避坑注意事项 #

• 如果按钮文本包含单引号，直接格式化会导致XPath语法报错，需要转义：把单引号替换成双单引号

  private By GetButtonByText(string buttonText)
  {
      string escapedText = buttonText.Replace("'", "''");
      string xPath = string.Format(".//button[contains(text(), '{0}')]", escapedText);
      return By.XPath(xPath);
  }
  

• 也可以改用XPath的concat函数避免引号问题，适配更复杂的文本场景：

  private By GetButtonByText(string buttonText)
  {
      string processedText = buttonText.Replace("'", "', \"'\", '");
      string xPath = $".//button[contains(text(), concat('{processedText}'))]";
      return By.XPath(xPath);
  }
  

内容的提问来源于stack exchange，提问作者Serhii Protsiv