c# - 除了命名约定外，我没有发现这种方法有任何问题，是吗？

Question

我试图在下面的代码中找到异常，但除了错误的命名约定之外我找不到任何东西。

public static Response DoIt(ADUser user)
{
    string search = "(cn=" + user.name.Trim() + ")";
    // More code lines
}

Answer 1

它不返回Response类型对象，除此之外它很好。"(cn="如果您在多个地方使用它，则可以将双引号 ( ) 中的字符串替换为常量。如果您担心字符串在一行中的连接，那没关系。但是，如果您需要在循环中连接字符串search或需要大量连接的东西，那么您可以考虑使用StringBuilder来构建您的字符串。

Answer 2

没有错，但您可以使用以下命令使其更具可读性string.Format：

string search = string.Format("(cn={0})", user.name.Trim());