今天在做一个项目的时候,发现了这样一个问题,为了让大家看得更直接明了,我直接放代码:
public void InsertObjectToList(){ List<NewsProtetype> list = new List<NewsProtetype>(); NewsProtetype info = new NewsProtetype(); DataTable dt = new DataTable(); KeywordUrlManage kum = new KeywordUrlManage(); dt = kum.GetKeywords(); if (dt.Rows.Count > 0) { for (int i = 0; i < dt.Rows.Count; i++) { info.keyword = dt.Rows[i][0].ToString(); info.url = dt.Rows[i][1].ToString(); list.Add(info); } } }
当你断点调试的时候就会发现一个很严重的问题,那就是后一个info实体会不断的覆盖前一个实体,最后导致list集合里面的内容都是一样的!当时我一时还不知道为什么会有这个结果,于是我又在for循环里面添加另一个集合记录i的值,确有意思的发现这个是没有被覆盖的!一时间我是百思不得其解;后来我在网上查了,看到有人说是数据类型的区别:int是值类型,而上面是实体info是引用类型!这两者的区别博客园的大牛们都写了很多文章,也很详细,其中我就注意到了这两句:引用类型变量的赋值只复制对象的引用,而不复制对象本身。而将一个值类型变量赋给另一个值类型变量时,将复制包含的值。
我这里的实体是引用类型,也就是说我在for循环赋值的时候,一直都在同一个对象赋值add给List集合。所以不断的给List添加同一个对象,自然会被不断覆盖前有的值,而值类型却没有
所以我就意识到解决这个bug其实只需要把for循环外面定义的实体类转移到for循环内部就可以了,每添加一个实体对象就new一个出来保证不同;其正确的代码如下:
public void InsertObjectToList(){ List<NewsProtetype> list = new List<NewsProtetype>(); DataTable dt = new DataTable(); KeywordUrlManage kum = new KeywordUrlManage(); dt = kum.GetKeywords(); if (dt.Rows.Count > 0) { for (int i = 0; i < dt.Rows.Count; i++) { NewsProtetype info = new NewsProtetype(); info.keyword = dt.Rows[i][0].ToString(); info.url = dt.Rows[i][1].ToString(); list.Add(info); } } }
PS:因为本人理论知识不强,不知道我理解对不对,或者说还不够深层次!希望各位朋友指出来,让我学习!谢谢了!
这是我第一篇技术博客,说来惭愧啊!毕业一年了,以前只是有问题的时候来博客园查资料......今后也会有时间更新博客的,这也是为了提高自己!!好了,就写到这吧