帮我用 out 关键字清理这个疯狂的 lambda

Question

我的代码看起来很难看，而且我知道必须有更好的方法来完成我正在做的事情：

private delegate string doStuff(
    PasswordEncrypter encrypter, RSAPublicKey publicKey,
    string privateKey, out string salt
);

private bool tryEncryptPassword(
    doStuff encryptPassword,
    out string errorMessage
)
{
    ...get some variables...
    string encryptedPassword = encryptPassword(encrypter, publicKey,
        privateKey, out salt);
    ...
}

到目前为止，这些东西并没有困扰我。这就是我调用 tryEncryptPassword 的方式，它看起来很丑陋，并且有重复，因为我通过两种方法调用它：

public bool method1(out string errorMessage)
{
    string rawPassword = "foo";
    return tryEncryptPassword(
        (PasswordEncrypter encrypter, RSAPublicKey publicKey,
            string privateKey, out string salt) =>
            encrypter.EncryptPasswordAndDoStuff( // Overload 1
                rawPassword, publicKey, privateKey, out salt
            ),
        out errorMessage
    );
}

public bool method2(SecureString unencryptedPassword,
    out string errorMessage)
{
    return tryEncryptPassword(
       (PasswordEncrypter encrypter, RSAPublicKey publicKey,
           string privateKey, out string salt) =>
           encrypter.EncryptPasswordAndDoStuff( // Overload 2
               unencryptedPassword, publicKey, privateKey, out salt
           ),
       out errorMessage
   );
}

丑陋的两部分：

• 我必须显式列出 lambda 表达式中的所有参数类型，因为单个 out 参数。
• EncryptPasswordAndDoStuff 的两个重载采用除第一个参数之外的所有相同参数，该参数可以是 string 或 SecureString。因此，method1 和 method2 几乎相同，它们只是调用 EncryptPasswordAndDoStuff 的不同重载。

有什么建议吗？

编辑（解决方案）：我最终使用了 Jeff 的建议并更改了 EncryptPasswordAndDoStuff 的重载以返回 EncryptionResult 的实例。然后我不需要显式定义的委托，我使用了以下代码：

private bool tryEncryptPassword(KeysAndEncrypter keys,
    Func<EncryptionResult> encryptPassword,
    out string errorMessage
) { ... }

private class KeysAndEncrypter
{
    public RSAPublicKey PublicKey { get; set; }
    public string PrivateKey { get; set; }
    public PasswordEncrypter Encrypter { get; set; }
}

这是method1的内容，method2是非常相似：

string rawPassword = "foo";
KeysAndEncrypter keys = getEncryptionKeys();
return tryEncryptPassword(keys, () =>
    keys.Encrypter.EncryptPasswordAndDoStuff(
        rawPassword, keys.PublicKey, keys.PrivateKey
    ),
    out errorMessage
);

Answer 1

您可以引入一种新类型来表示委托的返回值：

 public class EncryptionResult {
     public string EncryptedValue { get; set; }
     public string Salt { get; set; }
 }

...并将委托更改为如下所示：

 private delegate EncryptionResult doStuff(
     PasswordEncrypter encrypter, 
     RSAPublicKey publicKey,
     string privateKey);

Answer 2

为什么要为疯狂的 lambda 烦恼呢？

public bool Method1(out string errorMessage)
{
    return TryEncryptPassword("foo", out errorMessage);
}

public bool Method2(SecureString password, out string errorMessage)
{
    return TryEncryptPassword(password, out errorMessage);
}

private bool TryEncryptPassword(string password, out string errorMessage)
{
    GetSomeVariables();
    string encryptedPassword = encrypter.EncryptPasswordAndDoStuff(
           password, publicKey, privateKey, out salt);
    DoMoreStuff();
    // ...
}

private bool TryEncryptPassword(SecureString password, out string errorMessage)
{
    GetSomeVariables();
    string encryptedPassword = encrypter.EncryptPasswordAndDoStuff(
           password, publicKey, privateKey, out salt);
    DoMoreStuff();
    // ...
}

private void GetSomeVariables() { /* ...get some variables... */ }

private void DoMoreStuff() { /* ... */ }

Answer 3

杰夫的建议是合理的。如果您不想每次需要输出参数时都定义一个新类型，您也可以使用 Tuple 。

另一个建议是传入另一个委托来检索错误，而不是使用输出参数。

private bool tryEncryptPassword(
     doStuff encryptPassword,
     Action<string> setError
)

public bool method1(Action<string> setError) {
    ...
    tryEncryptPassword(..., ..., ..., setError);
}

在调用 method1 之前，您可以将 setError 分配给局部变量。

string error;
Action<string> setError = str => error = str;
method1(setError);        

Answer 4

另外，如果您不想创建自定义类型，您可以在此处返回一个元组。

private delegate Tuple<string, string> doStuff(
   PasswordEncrypter encrypter, 
   RSAPublicKey publicKey,
   string privateKey);

在 doStuff 中，只需创建并返回一个元组：

Tuple<string, string> result = new Tuple<string, string>(EncryptedValue, Salt);
return result;