写法：插入表除X之外的所有内容，if语句

Question

这个社区有史以来的第二个问题！我是一个菜鸟，我的弱点是循环内或循环之间的 if 语句以及其他 if 语句。

这是我的场景。此方法将任何内容插入数据库，但我想验证某些内容。我不希望在数据库中添加任何内容，而是不希望输入任何以“LIFT”开头的内容，我希望该方法跳过该行并继续下一行。有什么办法可以将其编程到这个方法中吗？或者我需要写一个新方法吗？非常感谢！

      public bool BatchInsert(string table, string[] values)
{
    string statement = "INSERT INTO " + table + " VALUES(";
    for (var i = 0; i < values.Length - 1; i++)
    {
        if(values[i].Contains("'")){
            values[i] = values[i].Replace("'", "''");
        }
        statement += "'"+values[i]+"', ";
    }
    statement += "'" + values[values.Length - 1] + "');";

    SqlCommand comm = new SqlCommand(statement, connectionPCICUSTOM);
    try
    {
        comm.Connection.Open();
        comm.ExecuteNonQuery();
    }
    catch (Exception e)
    {
        KaplanFTP.errorMsg = "Database error: " + e.Message;
    }
    finally
    {
        comm.Connection.Close();
    }

    return true;

}

Answer 1

一些提示。不要 += 字符串类型，因为它会降低性能。我也更喜欢 foreach 循环，因为代码更干净、更容易阅读/不太可能弄乱索引。还可以使用 using 语句来确保正确处置。

假设您有对 System.Linq 的引用，您可以使用以下内容。我没有测试它，但它应该可以工作：

public bool BatchInsert(string table, IEnumerable<string> values)
    {
        var sql = new StringBuilder();
        sql.Append("INSERT INTO " + table + " VALUES(");

        var newValues = values.Where(x => !x.StartsWith("LIFT")).Select(x => string.Format("'{0}'", x.Replace("'", "''")));
        sql.Append(string.Join("","", newValues.ToArray()));
        sql.Append(")");

        using (var comm = new SqlCommand(statement, connectionPCICUSTOM))
        {
            try
            {
                comm.Connection.Open();
                comm.ExecuteNonQuery();
            }
            catch (Exception e)
            {
                KaplanFTP.errorMsg = "Database error: " + e.Message;
            }
            finally
            {
                comm.Connection.Close();
            }
        }
        return true;
    }

Answer 2

如果您的目标是迭代“值”集合，保持以“lift”开头的值及其相应的列不变，则可能需要修改 INSERT 查询的构建方式。您将根据需要添加列，而不是假设每个值都会被考虑在内。基本上，您需要使用以下形式：

INSERT INTO tablename (col1, col2...) VALUES (val1, val2...)

例如：

string statement = "INSERT INTO tablename ";
string columns = "(";
string values = "(";
for (var i = 0; i < values.Length - 1; i++)
{
     //if values doesn't contain lift, add it to the statement
     if(!values[i].contains("LIFT")){
          //columnName is a collection of your db column names
          columns += "'"+columnName[i]+"'";
          values += "'"+values[i]+"'";
     }
 }
 columns += ")";
 values += ")";
 statement += columns +" VALUES " + values;

就像一些评论所述，这种方法会导致 SQL 注入。谨慎使用。

编辑：抱歉，我错过了您所说的“以“LIFT”开头”。将 .contains() 行修改为以下内容：

if(!values[i].StartsWith("LIFT")){