C#学习笔记--foreach问题

C#学习笔记--foreach问题一

• 先上原代码看
• 原因分析
• 解决方法
• 最后

小编第一次在CSDN这个大舞台写博客。今天小编在上课的时候碰到个问题，就是C#窗体程序中，从用离线数据操作读取到数据在dataGridView1中时，用foreach对dataGridView1内数据进行修改时，发现会出现如果是连续的两行数据进行修改，会出现第二行数据修改不了的情况。

先上原代码看

首先是程序页代码

string readerid = skinComboBox1.SelectedValue.ToString();
foreach(DataGridViewRow row in dataGridView1.Rows){
if (row.Cells[0].Value != null)
{
bis.BookLend((int)row.Cells[1].Value, readerid);
}
dataGridView2.DataSource = bis.showReaderLendInfo(readerid);
}

接下来是类的代码

public void BookLend(int x,string readerid)
{
DataRow[] bookRows = ds.Tables["TBookDetail"].Select("bookId='" + x + "'");
bookRows[0]["Status"] = "借出";

DataRow LendRow = ds.Tables["TLendInfo"].NewRow();
LendRow["BookID"] = x;
LendRow["ReaderID"] = readerid;
LendRow["LendDate"] = DateTime.Today;
ds.Tables["TLendInfo"].Rows.Add(LendRow);
}

public DataView showReaderLendInfo(string readerid)
{
DataView dvLend = new DataView(ds.Tables["TLendInfo"]);
dvLend.RowFilter = "ReaderID='" + readerid + "'";
return dvLend;
}

原因分析

原因在于foreach在检测到第一个打钩的项目后，它就把图书状态修改成“在库”和那个向TLendInfo表格添加数据同时进行了，所以会导致dataGridView1的数据在某一次foreach循环中都在刷新数据，所以读取到第一个打钩的行后，对那行进行操作，就导致表刷新，打钩行的下一行数据顶上来，而foreach却在下一行的位置，所以就导致会有读取不了的情况。

解决方法

方法一：用两个foreach将两个操作分开，使他们不同时进行

private void BookLend(){
string readerID = cbbReaderSelect.SelectedValue.ToString();
List<int> bookIdToLend=new List<int>();
//获取选中的书籍ID号
foreach(DataGridViewRow row in dgvBookList.Rows){
if(row.Cells["ColSelect"].Value!=null){
int bookId=(int)row.Cells[1].Value;//将BookIdJ加入泛型列表
bookIdToLend.Add(bookId);
}
}
//对每一本书，做两个数据操作：改TBookDetail,加TLendInfo
foreach(int bookId in bookIdToLend){
DataRow[] bookRows=ds.Tables["TBookDetail"].Select("bookId='"+bookId+"'");
bookRows[0]["Status"] = "借出";

DataRow LendRow = ds.Tables["TLendInfo"].NewRow();
LendRow["BookID"] = bookId;
LendRow["ReaderID"] = readerID;
LendRow["LendDate"] = DateTime.Today;
ds.Tables["TLendInfo"].Rows.Add(LendRow);
}
}

方法二：运用goto去实现foreach的再循环（但会导致效率不高）

string readerid = skinComboBox1.SelectedValue.ToString();

cc:foreach(DataGridViewRow row in dataGridView1.Rows){
if (row.Cells[0].Value != null)
{
bis.BookLend((int)row.Cells[1].Value, readerid);
goto cc;
}
}
dataGridView2.DataSource = bis.showReaderLendInfo(readerid);

方法三：不用foreach去循环，用for，控制循环遍历，就可以继续循环

for (int i = 0; i < dataGridView1.Rows.Count; i++ )
{
if (dataGridView1.Rows[i].Cells[0].Value != null)
{
bis.BookLend((int)dataGridView1.Rows[i].Cells[1].Value , readerid);
i--;
}
dataGridView2.DataSource = bis.showReaderLendInfo(readerid);
}

最后

感谢所有来阅读的人，也希望各位大佬留下你们的意见，或者小弟哪里表达不明的，可以留言，我必回复！！！