如何创建列表集合的深层副本

假设我有以下课程：

public class Author
{
    public int ID {get; private set;}
    public string firstName {get; private set;}
    public string lastName {get; private set; }

    public Author(int id, string firstname, string lastname)
    {
        this.ID = ID;
        this.firstName = firstname;
        this.lastName = lastName;
    }

    public static Author Clone(Author clone)
    {
        Author copyAuth = new Author(clone.ID, clone.firstName, clone.lastName);
        return copyAuth;
    }
}

and

public class Book
{
    public string bookTitle {get; private set;}
    private List<Author> authors;
    private List<Author> copyofAuthors;
    public string ISBN {get; private set; }

    public Book(string bookTitle, List<Author> authors, string ISBN)
    {
        copyofAuthors = new List<Author>();
        this.bookTitle = bookTitle;
        this.ISBN = ISBN;
        //How do I create a deep copy of my authors List?

        foreach(Author copy in authors)
        {
            Author addAuthors = Author.Clone(copy);
            copyofAuthors.Add(addAuthors);
        }
    }
}

我将如何创建我的深层副本List<Authors>收藏？我读过 StackOverFlow 上建议序列化的其他页面，以及我不熟悉且似乎令人困惑的建议。

我跟着这个链接 https://stackoverflow.com/questions/35470863/create-a-defensive-copy-in-the-constructor/35471044#35471044创建我的克隆方法。

问题一：

上述实现是否被视为深复制？如果是这样的话，这样做可以吗？我的意思是，构造函数中的 foreach 循环将作者复制到新的列表集合中。

问题2：

如果我修改 copyofAuthor 集合中的任何内容，它不再参考原集正确吗？所以original集合应该保持不变吗？

更新#1：

    public List<Author> Authors
    {
        get
        {
            return returnAuthors(authors);
        }

    }

    private List<Author> returnAuthors(List<Author> copyList)
    {
        List<Author> getAuthors = new List<Author>();

        foreach(Author copy in copyList){
            getAuthors.Add(Author.Clone(copy));
        }

        return getAuthors;
    }

我是否正确实现了 getter 集合，以便当它返回 List 集合时，它独立于原始集合？因此，对 getter 返回的集合所做的任何更改不会被反映在原始集合中正确吗？

更新#2：

使用 ReadOnlyCollection

    public class Book
    {

        public string bookTitle {get; private set;}
        private ReadOnlyCollection<Author> authors;
        public string ISBN {get; private set; }


        public Book(string bookTitle, ReadOnlyCollection<Author> authors, string ISBN)
        {

            this.bookTitle = bookTitle;
            this.ISBN = ISBN;
            //Is it okay to do this? 
            this.authors = authors;

        }

        public List<Author> Authors
        {
            get
            {   //Create a shallow copy
                return new ReadOnlyCollection<Author>(authors);
            }

        }

    }

在评论中解决这个问题变得太难了。

• 从您的 Author 类中删除 Clone 方法。这是没有用的。

在你的读书课上，你有两个问题需要解决。

• 构造函数获取作者列表，但传入它的调用者可能会更改它。如果我们只是复制参考文献，那么调用者可能会意外更改书籍所保存的列表。

• 这本书返回了作者名单。如果调用者再次向该列表添加某些内容，则他们已经更改了该书。

您可以使用不可变集合解决这两个问题。如果您尚未下载不可变集合库，请使用 NuGet 下载它。

using System.Collections.Immutable;
...
public class Book
{
  public string bookTitle {get; private set;}
  private ImmutableList<Author> authors;
  public IReadOnlyList<Author> Authors { get { return authors; } }
  public string ISBN {get; private set; }

  public Book(string bookTitle, IEnumerable<Author> authors, string ISBN)
  {
    this.authors = ImmutableList<Author>.Empty.AddRange(authors);
    this.bookTitle = bookTitle;
    this.ISBN = ISBN;
  }
}

那里。现在你做一个copy of the sequence作者，所以如果调用者更改了该顺序，不用担心，您有一个副本。并且您分发一个由不可变集合实现的 IReadOnlyList，因此没有人可以更改它。

结合更多的东西。你问“这是对的吗？”

public class Book
{
    private ReadOnlyCollection<Author> authors;
    public Book(ReadOnlyCollection<Author> authors)
    {
        //Is it okay to do this? 
        this.authors = authors;
    }

    public List<Author> Authors
    {
        get
        {   //Create a shallow copy
            return new ReadOnlyCollection<Author>(authors);
        }
    }

（去除多余的东西）。

不，这并不完全正确，有几个原因。首先，只读集合只是一个wrapper围绕可变集合。您仍然处于调用者控制底层集合的情况，因此可以更改它。

其次，打字不太好；您无法将 ReadOnlyCollection 转换为 List。

我知道这很令人困惑。这里有一个微妙的区别。只读集合就是：you只能读它。这并不意味着其他人写不出来！这样的合集还是mutable，它只是不可变的by you。不可变集合确实是不可变的；没有人可以改变它。

接下来：通过使作者和书都是不可变的，你做得很好。但如果你想改变它怎么办？正如您所注意到的，更改一本不可变的书意味着制作一本新书。但你已经有一本旧书了；你怎样才能有效地做到这一点？常见的模式是：

public class Book
{
  public string Title {get; private set;}
  private ImmutableList<Author> authors;
  public IReadOnlyList<Author> Authors { get { return authors; } }
  public string ISBN {get; private set; }

  public Book(string title, IEnumerable<Author> authors, string ISBN) : this(
    title, 
    ImmutableList<Author>.Empty.AddRange(authors),
    ISBN) {}

  public Book(string title, ImmutableList<Authors> authors, string ISBN) 
  {
    this.Title = title;
    this.Authors = authors;
    this.ISBN = ISBN;
  }
  public Book WithTitle(string newTitle)
  {
    return new Book(newTitle, authors, ISBN); 
  }
  public Book WithISBN(string newISBN)
  {
    return new Book(Title, authors, newISBN);
  }
  public Book WithAuthor(Author author)
  {
    return new Book(Title, authors.Add(author), ISBN);
  }
  public static readonly Empty = new Book("", ImmutableList<Author>.Empty, "");
}

现在你可以这样做：

Book tlotr = Book.Empty.WithAuthor("JRRT").WithTitle("The Lord Of The Rings");

等等。