addParameterValue(cmd,id);

{

if(id != null)
当然，这个 addParameterValue() 封装函数，是可有可无的。多写几个 DbParameter p = cmd.CreateParameter() 并没有什么大问题。

sql += " and name=?";
if(id>0 && string.IsNullOrEmpty(name))
DbParameter p = cmd.CreateParameter();

command.Parameters.Add(id);
cmd.Parameters.Add(p);
if(id != null)
2. 有几个动态参数，代码的重复量就成了这些参数的不同数量的组合数，动态参数越多，重复量越大。
command.CommandText = "select * from t1 where id=?";
"where 1=1" 虽是教科书中没有的小技巧，却很管用。
}
addParameterValue(cmd,name);
在数据库的理论中，没有指定的数据，是用 null 来表示的，不论是 string 还是 int/long。

sql += " and id=?";
command.CommandText = "select * from t1 where id=? and name=?";
command.CommandText = sql;

复制代码 代码如下:

{

if(!string.IsNullOrEmpty(name))

command.Parameters.Add(name);
问题在于，0 是否是不正常的业务数值，代码中看不出来。不排除程序员随意指定一个数值，作为“用户无录入的默认值”，如果不巧这个默认值，实际上是有其他意义的，那就造成问题。
p.ParameterName = "@XXX";
}

这里的技巧在于，使用了一个 "where 1=1"， 巧妙解决了后续 sql 拼接中，每行开头是否要有 "and" 的问题。而这个 "where 1=1"，并不会对数据库的索引执行，造成性能上的影响。

常见的误解有：

string sql ="select * from t1 where 1=1";
对参数进行排列组合，然后写各种组合的 SQL，这个思路很奇怪。问题是，很多初学者，都有这个思维习惯。本人不是计算机科班出身，不知道是否哪本教科书，就是如此教导的。但很不幸的是，这个思维习惯是错误的。
if(id<=0 && !string.IsNullOrEmpty(name))
p.Value = TTT;
addParameterValue(cmd,name); 是一个简单封装的函数，用来封装如下一小段代码，目的是让最后的代码，较为简捷直观：
command.Parameters.Add(name);
}
对于第二个问题，以下的错误代码为其证据：
}

if(id<=0 && !string.IsNullOrEmpty(name))
另外，在程序中，一般会在用户界面上让使用用户录入数字，这个数字的数值，在代码中会自动变成 string，然后尝试 string 转换成 int/long，最后送到 sql 函数里。这里需要特别注意的是，很多人把某个特殊的数值，作为“用户无录入的默认值”，正如本文开头所写的错误代码那样：

复制代码 代码如下:

复制代码 代码如下:

复制代码 代码如下:

复制代码 代码如下: