Question

C# 用GridView的RowDeletint事件，为什么点击一次删除，会执行2次删除操作，并删除2行记录。

如果删除的是最后一条记录，则会报错，int id = int.Parse(Grid_CodeFile.DataKeys[e.RowIndex].Value.ToString());这里的e.RowIndex会超出索引范围。大侠们，这是为什么么，在线等……
前台：
<asp:GridView ID="Grid_CodeFile" runat="server" BorderWidth="0px"
CellSpacing="1" Width="780px" AutoGenerateColumns="False"
CellPadding="4" CaptionAlign="Left" HorizontalAlign="Left"
DataKeyNames="F_ID" EnableModelValidation="True"
onrowdeleting="Grid_CodeFile_RowDeleting"
onrowediting="Grid_CodeFile_RowEditing">
<EmptyDataTemplate></EmptyDataTemplate><Columns>
<asp:CommandField ButtonType="Image" CancelImageUrl="~/Images/web_icon_001.gif" DeleteImageUrl="~/Images/web_icon_009.gif"
EditImageUrl="~/Images/web_icon_018.gif" ShowDeleteButton="True" ShowEditButton="True" UpdateImageUrl="~/Images/web_icon_025.gif" CancelText="Cancel" DeleteText="Delete" EditText="Edit" UpdateText="Save">
<HeaderStyle BackColor="#006699" Font-Bold="False" HorizontalAlign="Left" Width="40px" /><ItemStyle VerticalAlign="Top" BackColor="#F2F2F2" Width="40px" />
<FooterStyle VerticalAlign="Top" BackColor="#F2F2F2" /> </asp:CommandField>
</Columns></asp:GridView>

Answer 1

打个断点跟一下试试，要不代码贴出来

gridview：关键---DataKeyNames="Id"
<asp:GridView ID="gdv_Announce" runat="server" AutoGenerateColumns="False" CellPadding="4"
GridLines="None" PageSize="5" Style="font-size: 12px" Width="100%" OnRowDataBound="gdv_Announce_RowDataBound"
OnRowDeleting="gdv_Announce_RowDeleting" DataKeyNames="Id" ForeColor="Black"
AllowPaging="True">

模版删除按钮：关键---CommandName="delete"
<asp:TemplateField HeaderText="删除">
<EditItemTemplate>
<asp:TextBox ID="TextBox3" runat="server"></asp:TextBox>
</EditItemTemplate>
<HeaderStyle BackColor="#99CCFF" ForeColor="Black" Wrap="False" />
<ItemTemplate>
<asp:LinkButton ID="LinkButton1" runat="server" CommandName="delete">删除</asp:LinkButton> 
</ItemTemplate>
</asp:TemplateField>

cs：
//gdv_Announce行中的删除
protected void gdv_Announce_RowDeleting(object sender, GridViewDeleteEventArgs e)
{
string keyword = gdv_Announce.DataKeys[e.RowIndex].Value.ToString();
string sql_del = "delete from tsmannouncementmanage where id=" + keyword;
dbCon.deleteData(sql_del);
DataBindTogdv_Announce("order by id desc");
}

Answer 2

你的代码一定有逻辑问题，还是贴下后台代码吧

追问

protected void Grid_CodeFile_RowDeleting(object sender, GridViewDeleteEventArgs e)
{
int id = int.Parse(Grid_CodeFile.DataKeys[e.RowIndex].Value.ToString());
bool ifTrue = CodeFileBLL.Instance.DelCodeFile(id);
if (ifTrue)
{
TempDataTabel.Clear();
Grid_CodeFileBind(TempDataTabel);
}
}
}

追答

这段代码应该没问题，PageLoad里面绑定数据是否加了 if(!IsPostBack)的判断，也贴下吧

追问

恩，是的，PageLoad里面是加载事件了，但是是绑定GridView的事件啊。
if (!IsPostBack)
{
CreateDataColumn();
CreateEmptyRow();
Grid_CodeFileBind(TempDataTabel);
}
else
{
TempDataTabel.Clear();
CreateDataColumn();
CreateEmptyRow();
Grid_CodeFileBind(TempDataTabel);
}

追答

else里面的语句去掉看下

追问

不行

追答

你的代码是肯定有问题的，有三句是逻辑重复的，改成下面这样肯定是功能相同的

protected void Page_Load(object sender, EventArgs e)
{
if (!IsPostBack)
{
TempDataTabel.Clear();
}

CreateDataColumn();
CreateEmptyRow();
Grid_CodeFileBind(TempDataTabel);
}

这两句的代码无论如何都执行了，而在大多数情况下回传页无需在Page_Load里绑定数据（可以由viewstate恢复）。

TempDataTabel是个什么，莫非每次加载要先把它从数据库填充一次？

不知道CreateDataColumn(); 和 CreateEmptyRow();做了什么，但从方法的名字来看你是担心如果GridView没有数据，就绑定一个空列和空行，建议你把后台代码完整地帖出来

追问

CreateDataColumn(); CreateEmptyRow();
TempDataTabel是DataTable上面两个方法是为TempDataTabel添加一个空行、列，在GeridView的第一行我是绑定了其他控件的，要进行添加操作的，页面样式就是这样的，

追答

不看你的详细代码并调试怕很难找到问题所在

追问

已经找到了造成这个问题的原因了，之所以这个事件会加载2次，是因为在PageLoad里面动态初始化了一个空行，和动态添加了第一行的控件。现在正在找办法解决这个问题

追答

asp.net中动态添加的控件很难控制，尽量不用

Answer 3

前台要设置DataKeyNames="Id" 设置主键Grid_CodeFile.DataKeys[e.RowIndex].Value 才能使用

追问

设置了的啊
前台：
DataKeyNames="F_ID"
后台：
Grid_CodeFile.DataSource = dt;
Grid_CodeFile.DataKeyNames = new string[] { "F_ID" };
Grid_CodeFile.DataBind();

追答

是不是你那里调用了两次删除事件 看下你的RowCommod事件是否也调用删除 不行的话F11单步调试看看

追问

我没有加载Command事件，不存在这样的问题，
他是执行一次完成之后，按理应该刷新这个页面的，但是他直接回传到deleteing事件的地方继续取下一条记录的id

追答

你的刷新是不是有问题 删除完 你可以调用Bind(通常把gridview绑定逻辑卸载这里)事件重新绑定一次数据 就可以了不要刷新页面 强制刷新页面就会在执行一次删除的

追问

是的啊，我是在删除完了之后在重新绑定的啊
TempDataTabel.Clear();
Grid_CodeFileBind(TempDataTabel);

Answer 4

在前面源码删除里面加个属性DataKeys=true