使用委托、匿名 (lambda) 函数和函数指针

Question

我已经阅读了一些关于匿名 (lambda) 函数和委托的内容。我相信我正在处理我的部分功能可以/应该使用它们的情况。我不确定我的假设是否正确。

当前代码：

fDoSave(fGetSqlCheckEmpJob(), fGetSqlUpdateEmpJob(), fGetSqlInsertEmpJob());
fDoSave(fGetSqlCheckEmpPayrl(), fGetSqlUpdateEmpPayrl(), fGetSqlInsertEmpPayrl());
fDoSave(fGetSqlCheckEEO(), fGetSqlUpdateEEO(), fGetSqlInsertEEO());
fDoSave(fGetSqlCheckEmpPhone(), fGetSqlUpdateEmpPhone(), fGetSqlInsertEmpPhone());

fGetSqlCheck...() - 将 sql 语句作为字符串返回，该字符串返回具有特定 ID 的所有行的 count() fGetSqlUpdate...() 将 sql 语句作为执行更新的字符串返回。 fGetSqlInsert...() 将 sql 语句作为执行插入的字符串返回。 fDoSave() 根据fGetCheck...() 返回的值进行更新或插入

fGetSql 函数如下所示：

private string fGetSql...()
{
   StringBuilder sb = new StringBuilder();
   //Create sql statement
   return sb.ToString();
}

fDoSave 函数如下所示：

private void fDoSave(string sSql_Check, string sSql_Update, sSql_Insert)
{
   OracleDataReader dr = ERPDB.sqlGetDataReader(sSql_Check);
   while (dr.Read())
   {
        if(fCheckIfRecrodExists(dr) > 0) //if fGetSqlCheck...() found a row with a specific ID
            //do update using sSql_Update
        else
            //do insert using sSql_Insert
    }
}

这可以使用 lambda 函数或委托重写吗？应该吗？应该怎么改写？

Answer 1

你的问题还是含糊的，不过我还是这么说吧。

案例：

1：可重用和“静态”
如果您重用 SQL 语句并且它们有些静态，请将它们放在属性中。并考虑更好的名字。

2：可重复使用，但“变量”虽然简单
如果你重用 SQL 语句并且它们是可变的但不占用太多 CPU，这意味着它们会随着不同的状态而变化，并且创建和构建的速度非常快，那么就让它保持原样吧。

3：可重用但“可变”且复杂
如果您重用 SQL 语句并且它们是可变的但非常复杂并且需要大量 CPU 能力，请将它们放在方法中，但将它们作为委托调用，不要让它们匿名。

4：不可重用但“可变”且复杂
如果您永远不会重用 SQL 语句（可能永远不会如此）并且它们是可变的且非常复杂并且需要大量 CPU 能力，请将它们放在匿名函数中。

在所有情况下
使用更好的名称。

我的建议
我更喜欢案例 1 和 2，因为其余的似乎是对可能不存在的问题的过于复杂的解决方案。
另外，我不知道你的整个代码库，但我不喜欢应该保存的对象没有提供给 fDoSave()。

我会这样做：

// Also often called Upsert short for "Update or Insert"
public int Save(EmpObj employeeObj) 
{
     if(CheckIfEmployeeExists(employeeObj))
     {
         return Update(employeeObj); // returns rows affected
     }
     else
     {
         return Insert(employeeObj); // Returns new Id of the employee.
     }
}

// Other methods, where the select, update and insert statements lies 
or gets called    and build
public bool CheckIfEmployeeExists(employeeObj) // Check If Employee Exists
public int Update(employeeObj); // Updates the employee
public int Insert(employeeObj); // Inserts the employee