Question

以下是我更新记录的存储过程：

ALTER
PROCEDURE [dbo].[sp_UpdatetoShipped]
(
@Date datetime,
@SerialNumber
varchar(50),
@User
varchar(50),
@WorkWeek
varchar(50) 
)
AS
BEGIN

UPDATE dbo.FG_FILLIN SET Status='SHIPPED',DateModified=@Date,ModifiedBy=@User,WorkWeek=@WorkWeek where (Status='KITTED')and SerialNumber=@SerialNumber

END

然后这是我的DAL：

public int UpdatetoShipped(FillinEntity fin) 
{
SqlConnection conn = new SqlConnection(connStr); 
conn.Open();
SqlCommand cmd = new SqlCommand("sp_UpdatetoShipped", conn); 
cmd.CommandType =CommandType.StoredProcedure; 
try
{
cmd.Parameters.Add("@SerialNumber", SqlDbType.VarChar,50).Value = fin.SerialNumber; 
cmd.Parameters.Add("@WorkWeek", SqlDbType.VarChar, 50).Value = fin.WorkWeek; 
cmd.Parameters.Add("@Date", SqlDbType.DateTime).Value = DateTime.Now.ToString(); 
cmd.Parameters.AddWithValue("@User", fin.ModifiedBy); 
return cmd.ExecuteNonQuery(); 
}
catch
{
throw; 
}
finally
{
cmd.Dispose();
conn.Close();
conn.Dispose();
}
}

我的BLL：

public int UpdatetoShipped(FillinEntity fin) 
{
DAL pDAL = new DAL(); 
try
{
return pDAL.UpdatetoShipped(fin); 
}
catch
{
throw; 
}
finally
{
pDAL = null; 
}
}

和我的用户界面：

string filepath2 = txtPath2.Text; 
Stream stream2 = new FileStream(filepath2, FileMode.Open, FileAccess.Read, FileShare.Read); 
ExcelMapper<FillinEntity> exceltoshipped = new ExcelMapper<FillinEntity>(); 
IExcelParser excelParser2 = new ExcelReaderExcelParser(stream2); 
IExcelRowMapper<FillinEntity> mapper2 = new ShippedRowMapper(); 
IEnumerable<FillinEntity> fillin2 = exceltoshipped.ListAll(excelParser2, mapper2);
int intResult = 0; 
BAL pBAL = new BAL();
try
{
foreach (FillinEntity fin in fillin2) 
{
fin.ModifiedBy = loggedUser;
intResult = pBAL.UpdatetoShipped(fin);
}
if (intResult > 0) 
MessageBox.Show("Record Updated Successfully."); 
else
MessageBox.Show("Record couldn't Updated Check Serial"); 
}
catch (Exception ee) 
{
MessageBox.Show(ee.Message.ToString()); 
}
finally
{
pBAL =null; 
}

我的问题是它总是说得更新得多。但如果我再次更新它作为重复更新我想显示序列已经更新。

Answer 1

您需要进行的密钥更改是从存储过程中执行以下SQL行：

UPDATE dbo.FG_FILLIN 
SET   Status='SHIPPED',
      DateModified=@Date,
      ModifiedBy=@User,
      WorkWeek=@WorkWeek 
WHERE (Status='KITTED')
AND   SerialNumber=@SerialNumber

您需要返回一个值，以便您确定此UPDATE是否已经发生，例如：

DECLARE @iUpdateAlreadyComplete INT
SET @iUpdateAlreadyComplete = 0;
IF EXISTS 
 (
    SELECT 1 
    FROM   dbo.FG_FILLIN 
    WHERE  Status='SHIPPED' 
    AND    SerialNumber=@SerialNumber
  )
BEGIN
  SET @iUpdateAlreadyComplete = 1
END
ELSE
BEGIN
    UPDATE dbo.FG_FILLIN 
    SET   Status='SHIPPED',
          DateModified=@Date,
          ModifiedBy=@User,
          WorkWeek=@WorkWeek 
    WHERE (Status='KITTED')
    AND   SerialNumber=@SerialNumber 
END

SELECT @iUpdateAlreadyComplete AS Result

然后，您可以将DAL从return cmd.ExecuteNonQuery();更改为：

var result = Convert.ToInt32(cmd.ExecuteScalar());

return result;

对于已更新的记录，返回值现在为0;对于已经处理过的记录，返回值为1。

其他笔记

您应该考虑改变其他一些事项：

sp_UpdatetoShipped是存储过程的错误名称。 Do not use the sp_ prefix。
你的DAL故意catch es和re - throw是一个例外（诚然是以“最佳”方式），你真的需要吗？
而不是显式调用Dipose()，而是使用using() {}语法，因为这样可以确保调用Dispose()，即使出现异常也是如此。

using语法：

using(SqlConnection conn = new SqlConnection(connStr))
{
    conn.Open();
    using (SqlCommand cmd = new SqlCommand("sp_UpdatetoShipped", conn))
    {
    }
}

Answer 2

这似乎更像是一个业务规则问题而不是与错误有关的问题。您可能想要做的是创建一个字典来保存已经更新的序列号。

e.g。

Dictoinary<string,string> updatedSerialNumbers = new Dictionary<string, string>();

foreach (FillinEntity fin in fillin2) 
{
fin.ModifiedBy = loggedUser;
if (updatedSerialNumbers.Contains(fin.SerialNumber) == false)
{ 
  intResult = pBAL.UpdatetoShipped(fin);
  updatedSerialNumbers.Add(fin.SerialNumber,fin.SerialNumber);
}

这样的事情应该解决你的问题。

存储语句中的问题错误处理？

2 个答案: