高分求查询的一个思路

[Shine_Moon]

在考虑如何做一个查询页面(Web)
当然上面有很多文本框，列表框，根据用户的输入决定查询的条件

现在的问题是
1.在cs文件中根据用户在某个控件的是否输入与选择来拼SQL语句好，还是索性把它们都仍到存储过程里，在存储过程里根据参数是否为null来拼接SQL语句好？理由？项目不大。

2.如果要用存储过程，为了让参数在存储过程里是null，是否必须在程序中赋值System.DBNull.Value

3.
if (txtStart.Text.Trim() == string.Empty)
cmd.Parameters.Add("@StartTime", SqlDbType.DateTime).Value = System.DBNull.Value;
else
cmd.Parameters.Add("@StartTime", SqlDbType.DateTime).Value = DateTime.Parse(txtStart.Text.Trim());

能否给个能通过的?:号表达式来完成，如果N个语句都这样些，实在太难看了

谢谢！

[amandag]

和你的项目的形式统一，如果别的地方没有用存储过程，这里用就会很突兀，如果别的地方用了，这里当然用最好

个人推荐使用存储过程，传递空值进去也无妨，可以在存储过程里判断

[flyingfz]

mark

[daishengs]

同意高歌

[bydwater]

拼ＳＱＬ语句好

[为梦去奋斗]

UP

[anncesky]

还是用存储过程的好

[hanguoji84]

这种多条件查询的sql在存储过程中处理要好一些，因为可以随意改动不需要调整程序。并且存储过程是通过数据库服务器来调用速度比较快，给你个小例子可以参考：
CREATE PROCEDURE SendLog_GetAll
(
@LogType varchar(10),
@SendTimeStart varchar(50),
@SendTimeEnd varchar(50),
@SendState int
)
AS

declare @strsql varchar(3000) --临时sql语句
declare @StrSqlWhere varchar(1500) --临时sql的条件语句

set @StrSqlWhere = ''

if @LogType <> 'A'
BEGIN
set @StrSqlWhere = @StrSqlWhere + ' and SendLog.LogType = ''' + @LogType + ''''
END

if @SendTimeStart <> ''
BEGIN
set @StrSqlWhere = @StrSqlWhere + ' and CONVERT(varchar(10), SendLog.SendTime, 121) >= ''' + @SendTimeStart +''''
END

if @SendTimeEnd <> ''
BEGIN
set @StrSqlWhere = @StrSqlWhere + ' and CONVERT(varchar(10), SendLog.SendTime, 121) <= ''' + @SendTimeEnd + ''''
END

if @SendState <> -1
begin
set @StrSqlWhere = @StrSqlWhere + ' and '+ ' SendLog.State =' + CAST(@SendState AS varchar(12))
END

IF @StrSqlWhere <>''
BEGIN
SET @StrSqlWhere =' where ' + right(@StrSqlWhere, len(@StrSqlWhere)-4)
end
最后用你的 set @strsql = @strsql + @StrSqlWhere
然后exec(@strsql)

[绝代坏坏]

拼接sql语句没什么好办法

[photoplan]

存储过程里这样写：
where (f1 = p1 or p1 is null)
and (f2 = p2 or p2 is null)
f 是字段，p是参数，这样可以写很长的条件而不显得乱。

传参时要给 DBNull.Value，
string.IsNullOrEmpty(str) ? DBNull.Value : str;

可以写一个实用工具类，封装个方法，签名如下：
object Uitl.GetDBValue(string)
实现起来很简单

[photoplan]

写得有点急，存储过程里参数的前缀落下了，有点影响效果，凑合看吧

[super_shu]

可以在每个条件的前面加个复选框，不过这你得先加个恒成立的条件比如1=1这样比较好判断的

[chuxue1342]

3.
if (txtStart.Text.Trim() == string.Empty)
cmd.Parameters.Add("@StartTime", SqlDbType.DateTime).Value = System.DBNull.Value;
else
cmd.Parameters.Add("@StartTime", SqlDbType.DateTime).Value = DateTime.Parse(txtStart.Text.Trim());

能否给个能通过的?:号表达式来完成，如果N个语句都这样些，实在太难看了
----------------------------------------------------------------------------
试试
cmd.Parameters.Add("@StartTime", SqlDbType.DateTime).Value =txtStart.Text.Trim() == string.Empty?System.DBNull.Value:DateTime.Parse(txtStart.Text.Trim());

[hy_lihuan]

肯定是拼接方便啊:
一、逻辑清楚
二、修改简单

[bydxyj]

个人觉得拼接条件好一些！如楼上所说：逻辑清楚，修改方便

[whoo529]

存储过程写好点，程序就简单多了

txtStart.Text.Trim() == string.Empty ？cmd.Parameters.Add("@StartTime", SqlDbType.DateTime).Value = System.DBNull.Value
： cmd.Parameters.Add("@StartTime", SqlDbType.DateTime).Value = DateTime.Parse(txtStart.Text.Trim())

[WangWei19871227]

我觉得 拼写sql查询的条件比较好

[waikey]

我认为在传参数时不用去判断吧，到过程里面去处理可能会好点吧，不然只会添加工作量而已吧

[vvsamln]

up

[ycxiangyun]

拼SQL语句肯定要好，写存储过程会让程序对数据库的依赖性太强，而且修改不方便

[bb1210]

不管是拼接SQL语句还是在存储过程内拼接，都要防止SQL注入

参数都设为Varchar更好处理