并发数量多时,造成.net网站数据错乱

stephen_deng 2014-06-24 05:47:27

网站近期日均PV有18000,用的是.net三层架构+mssql2005.订单量一多时，
偶尔产生三两条错乱的数据，A用户输入的的商品被B用户打上自己订单号,造成商品出货发给了B，两人操作的时间相差两个小时，不是同时操作.
A用户操作detail.aspx页面插入加入商品到表detail中,
用户B操作header.aspx 1>生成订单号并保存订单信息到表header中,2>更新对应的子商品的信息(加订单号,备注要求出货)到表detail,
A用户和B用户的操作时会用到数据层的方法execute_simple



//conn是经操作层到业务层传来的

public static int execute_simple(string sql,SqlConnection conn) 

        {

            try

            {

                SqlCommand com = new SqlCommand(sql, conn);

                com.ExecuteNonQuery();

                com.Dispose();

                return 1;

            }

            catch (Exception e)

            {

                //AppException errlog = new AppException();

                //errlog.ExceptionLog("execute_simple sqlmake层错误", e.Message);

                string fileName = DateTime.Now.ToString("yyyyMMddhhmmss") + ".txt";

                write_error_tolog.ErrorLog("sqlmaker_forcycle.execute_simple错误位置\r" + e.ToString(), fileName, "/ErrorLog/");

                return 0;

            }

        }

请教朋友们，是不是因为用了static造成并发问题还是因为没有加事务处理？整个网站系统都没有加事处理，
更奇怪的是A用户和B不是同时操作造成的问题，非常不解.

...全文

517 20 打赏收藏转发到动态举报

写回复

用AI写文章

20 条回复

切换为时间正序

请发表友善的回复…

发表回复

zglover 2016-10-09

打赏
举报

同意楼上的，楼主你应该把相关联的几个表的结构关系和值贴出来，这样才有助于分析

greatbody 2016-10-09

打赏
举报

使用guid没有。否则可能会有产生重复的id等等。

jmcooler 2016-10-09

打赏
举报

static 本身不会有问题，有问题的是，在同一个 SQL 连接上，同时并发执行 Insert 操作。这里有几点思路： 1、了解下，SQL 相关的类，是否是线程安全的，特别是 SQL 连接对象，是否是线程安全的 2、你自己的代码，是否是线程安全的。也即静态函数中，绝对不要使用全局变量 3、如果你的插入操作，需要涉及到多个表，那么必须使用事务。 4、如果对并发要求比较高，你可以建立一个 SQL 连接池，该池子中有若干数据库连接，每次请求，都从连接池中拿出一个来，执行完 SQL，再放进去。不过操作连接池时，需要加锁

圣殿骑士18 2016-10-09

打赏
举报

怎么这么多坟贴

以专业开发人员为伍 2016-10-08

打赏
举报

没看到你贴出任何有用的代码、结构。所谓的”商品、detail、head”的关键结构和值是如何设计的？所谓的“1>...... 2>...... 代码如何设计的”？

闭包客 2016-10-08

打赏
举报

我个人认为你的 sql 命令里面有获取最新的自增 id 这种操作。如果获取的自增 id 不正确，订单明细（detail）就记录在错误的订单头（head）中。

闭包客 2016-10-08

打赏
举报

定位这种问题最好是有日志。记录下传入的 sql 这个变量，有可能在组合 sql 字符串的时候引用了静态变量。

闭包客 2016-10-08

打赏
举报

偶发的问题只看代码是较难定位的。不过至少可以确定这个静态方法没有问题，里面没有引用静态变量，可以排除这个。如果时间相差两小时也不可能是事务的问题，问题是你怎么确定相差两小时？因为用户会新建订单，也会修改订单。

wolf198 2016-10-08

打赏
举报

参数 SqlConnection 是否有共用?

stephen_deng 2014-06-25

打赏
举报

感谢各位回复，问题出在哪里仍没答案.

stephen_deng 2014-06-24

打赏
举报

回复请叫我官人我一楼的是方法，不是变量哦，呵

stephen_deng 2014-06-24

打赏
举报

回复bwangel 我的业务逻辑层都是简单的逻辑,


using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Data;
using System.Data.SqlClient;
using DAL;
namespace BLL
{
    public class receivedlist_bll
    {
        public DataTable getexcomitem()
        {
            receivedlist_dal receivelistdal = new receivedlist_dal();
            return receivelistdal.getexcomitem();
        }

        public DataTable getcutorderitems(string weblogin)
        {
            receivedlist_dal receivelistdal = new receivedlist_dal();
            return receivelistdal.getcutorderitems(weblogin);
        }
        public bool getoneitem(string tmpkey)
        {
            receivedlist_dal sysdal = new receivedlist_dal();
            return sysdal.getoneitem(tmpkey);
        }
        public string getoneitem2(string tmpkey)
        {
            receivedlist_dal sysdal = new receivedlist_dal();
            return sysdal.getoneitem2(tmpkey);
        }
        public bool getoneitem3(string tmpkey)
        {
            receivedlist_dal sysdal = new receivedlist_dal();
            return sysdal.getoneitem3(tmpkey);
        }
        public int retreccount(string weblogin)
        {
            receivedlist_dal receivelistdal = new receivedlist_dal();
            return receivelistdal.retreccount(weblogin);
        }
        public DataTable getdataset_curentpage(string weblogin, int pageIndex, int pageSize)
        {
            receivedlist_dal rettable = new receivedlist_dal();
            return rettable.getdataset_curentpage(weblogin, pageIndex, pageSize);
        }
        public bool addexpress(string tmpexnum,string tmpexcom,string weblogin)
        {
            receivedlist_dal dal = new receivedlist_dal();
            return dal.addexpress(tmpexnum, tmpexcom,weblogin);
        }
        public bool delorderitem(int itemid)
        {
            receivedlist_dal dal = new receivedlist_dal();
            return dal.delorderitem(itemid);
        }
        public bool updateitem(string cutremark,int itemid)
        {
            receivedlist_dal dal = new receivedlist_dal();
            return dal.updateitem(cutremark,itemid);
        }
        
    }
}