Fork me on GitHub
代码改变世界

添加List集合覆盖问题

2015-09-11 18:22  沉睡的木木夕  阅读(9133)  评论(7编辑  收藏  举报

  今天在做一个项目的时候,发现了这样一个问题,为了让大家看得更直接明了,我直接放代码:

  

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:因为本人理论知识不强,不知道我理解对不对,或者说还不够深层次!希望各位朋友指出来,让我学习!谢谢了!

  这是我第一篇技术博客,说来惭愧啊!毕业一年了,以前只是有问题的时候来博客园查资料......今后也会有时间更新博客的,这也是为了提高自己!!好了,就写到这吧