SQL Server存储过程正在插入两条记录

时间:2014-07-22 15:44:39

标签: sql sql-server stored-procedures

我的存储过程应该克隆一次项目,但它会创建两个克隆记录。

我的C#程序有一个调用此方法的按钮。调试只激发一次并返回第二个克隆项目的neweventid

protected void cloneEvent(object sender, EventArgs e)
{
        using (SqlConnection myConnection = new SqlConnection(ConfigurationManager.ConnectionStrings["AppConnection"].ConnectionString))
        {
            using (SqlCommand myCommand = new SqlCommand("addRecycleEventAcceptedMaterialsClone"))
            {
                //object returnValue;

                myCommand.CommandType = CommandType.StoredProcedure;
                myCommand.Connection = myConnection;
                myCommand.Parameters.AddWithValue("@event_id", qsEventId);
                myConnection.Open();
                myCommand.ExecuteNonQuery();

                //returnValue = myCommand.ExecuteScalar();
                NewEventID = (int)myCommand.ExecuteScalar();
            }
        }
        Response.Redirect("eventDetail.aspx?eventid=" + NewEventID);
    }

我检查了没有重复的event_id,它是主键/标识列

ALTER PROCEDURE [dbo].[addRecycleEventAcceptedMaterialsClone] 
    -- Add the parameters for the stored procedure here
    --Pass in the original event_id
    @event_id int
    --@newEvent_id INT OUTPUT
AS
BEGIN
   -- SET NOCOUNT ON added to prevent extra result sets from
   -- interfering with SELECT statements.
   SET NOCOUNT ON;

   INSERT INTO RECYCLE_EVENT (event_nm, start_dt, end_dt, start_tm, end_tm,
                              website_ad, address_ad, city_nm, state_cd, zip_cd,
                              county_id, description_ds, moreinfo_ds, 
                              latitude, longitude, phone_nr)
     SELECT 
        event_nm, start_dt, end_dt, start_tm, end_tm,
        website_ad, address_ad, city_nm, state_cd, zip_cd,
        county_id,  description_ds, moreinfo_ds,
        latitude, longitude, phone_nr 
    FROM 
        RECYCLE_EVENT
    WHERE
        event_id = @event_id

    --SET @newEvent_id = IDENT_CURRENT('RECYCLE_EVENT');    
    --SELECT @newEvent_id = SCOPE_IDENTITY()
      SELECT CAST(scope_identity() AS int);
    --SELECT @newEvent_id AS newEventID

--NEW CLONED EVENT HAS BEEN ADDED AND A NEW ID (newEventID) has been generated.
--NOW INSERT all materials accepted for the @event_id and insert them into the RECYCLER_EVENT_MATERIALS_ACCEPTED table
--Give it the newEventID

INSERT INTO RECYCLER_EVENT_MATERIALS_ACCEPTED ( 
        event_id,
        material_type_id,
        acceptance_cd,
        residential_fl,
        commercial_fl,
        service_type_cd,
        end_dt,
        event_material_cloned_id
        )
        SELECT 
        IDENT_CURRENT('RECYCLE_EVENT'),
        material_type_id,
        acceptance_cd,
        residential_fl,
        commercial_fl,
        service_type_cd,
        end_dt,
        event_material_id
        FROM RECYCLER_EVENT_MATERIALS_ACCEPTED
        where event_id = @event_id

        --Grab all the records that have the old event_material_id
        --insert a new row using the same county id and the new event_material_id
        --eventid-76: material-id's: 18, 21, 22
        --eventid-124: material-id's: 24, 25, 26, 27, 28, 29, 30, 31, 32, 33

        INSERT INTO RECYCLER_EVENT_COUNTY_SERVED (
            county_id,
            event_material_id )
            SELECT s.county_id, a.event_material_id
            FROM RECYCLER_EVENT_COUNTY_SERVED s
            INNER JOIN RECYCLER_EVENT_MATERIALS_ACCEPTED a ON s.event_material_id = a.event_material_cloned_id

谢谢!

2 个答案:

答案 0 :(得分:5)

您正在从代码执行两次,首先通过ExecuteNonQuery然后通过ExecuteScalar执行,以便插入两次而不是一次。

答案 1 :(得分:0)

您正在执行存储过程两次,

 myCommand.ExecuteNonQuery();
 NewEventID = (int)myCommand.ExecuteScalar();

两者都调用插入,我会注释掉第一行。