存储过程优化

[boboluping]

最近，遇到一个项目，每秒大概有2000次，对2000张表的插入操作。每1次的数据量是200个字节，并且表的数量是不确定的，是随着设备的新增而增加的。我写了如下一个存储过程：
create proc add_GPSInfo
@tableName varchar(30),
@sLongitude varchar(50),
@sLatitude varchar(50),
@sStatus int,
@sDataType int,
@sHeading real,
@sSpeed real,
@sGpsTime varchar(30),
@sMiles varchar(10)
AS
if exists (select * from dbo.sysobjects where id = object_id(N'[dbo].['+@TableName+']') and OBJECTPROPERTY(id, N'IsUserTable') = 1)
begin
exec('INSERT INTO ' +@tableName+ ' (Longitude,Latitude,Status,DataType,Heading,Speed,GpsTime,Miles) VALUES ('''+@sLongitude+''','''+@sLatitude+''','+@sStatus+','+@sDataType+','+@sHeading+','+@sSpeed+','''+@sGpsTime+''','+@sMiles+')' );
end
else
begin exec(
'CREATE TABLE [dbo].['+@tableName+'] (
[Longitude] [float] NULL ,
[Latitude] [float] NULL ,
[Status] [int] NULL ,
[DataType] [int] NULL ,
[Heading] [real] NULL ,
[Speed] [real] NULL ,
[GpsTime] [datetime] NULL ,
[Miles] [varchar] (10) COLLATE Chinese_PRC_CI_AS NULL ) ON [PRIMARY]'
);
exec('INSERT INTO ' +@tableName+ ' (Longitude,Latitude,Status,DataType,Heading,Speed,GpsTime,Miles) VALUES ('''+@sLongitude+''','''+@sLatitude+''','+@sStatus+','+@sDataType+','+@sHeading+','+@sSpeed+','''+@sGpsTime+''','+@sMiles+')' );
end

GO

在这里，请教一下各位，如果不该表思路，在SQL语句层面上，这个存储过程还可以怎么优化一下呢？

[boboluping]

这位仁兄，别败啊，SQL我不了解，这是临时抱佛脚。而且现在这个有个问题是什么呢？会有一段时间某个表插不进去数据，导致一部分的GPS数据丢失了。我在设置了locktimeout 0之后，截止到目前，运行良好。没有再丢过数据了，可能会有几个丢的，但是不会出现一段时间没数据了。
[Quote=引用 6 楼 szx1999 的回复:]
尽管不太合理，居然真的没报错。。。
我败了。。。

else部分可以考虑用select ... into ...的方式，将建表和插入数据合二为一。
[/Quote]

[让你望见影子的墙]

都是简单的sql，貌似没办法优化了

[等不到来世]

你的存储过程应该报错。
@sStatus等字段是int型，你未作任何转换而进行了字符串拼接。

[等不到来世]

存储过程问题还很多，不想写了

给个小提示，新建一个表可以直接这么写：

select '小笨笨' as colname into newtable

[duanzhi1984]

语句并不复杂,已经是最简化的了

[boboluping]

过程并没有报错，因为已经实际使用起来了。只是感觉，这个写出来好繁琐。

[等不到来世]

尽管不太合理，居然真的没报错。。。
我败了。。。

else部分可以考虑用select ... into ...的方式，将建表和插入数据合二为一。

[Q315054403]

设计腐化啊。。。既然结构相同，一个表就OK喽

[IAmXirour]

呵呵，我对这句比较感兴趣：
select '小笨笨' as colname into newtable

[fcuandy]

[Quote=引用 2 楼 szx1999 的回复:]
你的存储过程应该报错。
@sStatus等字段是int型，你未作任何转换而进行了字符串拼接。
[/Quote]


理论是死的,不能照搬.

exec(....)这里跟的任何拼接变量会尝试转varchar,nvarhcar,char,...,因为它只接收这样的参数, 所以这里不对int显式转换为字串是不错的,反而显式写了转换倒会报错.

其实用select into和insert select 还是有区别的, 并不是说select into因为避免了一些检查效率高些就一定适用的.

select into 会占用系统表资源,可能会造成一些操作阻塞.

[boboluping]

看来是个高手。
[Quote=引用 10 楼 fcuandy 的回复:]
引用 2 楼 szx1999 的回复:
你的存储过程应该报错。
@sStatus等字段是int型，你未作任何转换而进行了字符串拼接。



理论是死的,不能照搬.

exec(....)这里跟的任何拼接变量会尝试转varchar,nvarhcar,char,...,因为它只接收这样的参数, 所以这里不对int显式转换为字串是不错的,反而显式写了转换倒会报错.

其实用select into和insert select 还是有区别的, 并不是说select into因为避免了一些检查效率高些就一定…
[/Quote]