C#でList<MyClass>のコピーをしたい（値渡し）

Question

c#でList<MyClass>のディープコピーができないのですがどうすればいいのでしょうか？

// 自前で用意したclass
class MyClass {
	public int val {get; set;}
	public string text {get; set;}
}

// MyClassのListをプロパティに持つclass
class HogeClass {
	public List<MyClass> mycl {get; set;}

	public void deepcopy(HogeClass mc) {
		this.mycl = new List<MyClass>(mc.mycl);
	}
}

public partial class Form_main : Form {
	public main(){
		// MyClassのListを作る
		List<MyClass> mclist = new List<MyClass>();
		mclist.Add(new MyClass(){val = 1, text = "aaa"});
		mclist.Add(new MyClass(){val = 2, text = "bbb"});
		mclist.Add(new MyClass(){val = 3, text = "ccc"});
		
		// インスタンスhcAを作成
		HogeClass hcA = new HogeClass();
		hcA.mycl = mclist;

		// インスタンスhcBを作成
		HogeClass hcB = new HogeClass();

		// ★ここでhcB.myclにhcA.myclのコピーをしたい
		hcB.deepcopy(hcA);	// ←これだと参照渡しになってしまいます
	}
}

この場合どのようなコードにすると値渡しでコピーができるのでしょうか？
どうぞよろしくお願い致します。

Windows10 + VisualStudio 2015

Answer 1

こういうことでしょうか？

public void deepcopy(HogeClass mc) {
    this.mycl = new List<MyClass>();
    this.mycl.AddRange(mc.mycl.Select(i => (MyClass)i.Clone()));
}

ただし、MyClass に Clone() メソッドが実装されていないので、このままではコンパイルエラーになります。
Clone() メソッドを追加してください。

class MyClass: ICloneable
{
    public object Clone()
    {
        return new MyClass() { val = this.val, text = this.text }
    }
}

今回のように簡単なクラスなら Clone() を使わなくてもいいかもしれませんが、メンテナンスを考えると、最初期から Clone() を実装するのがいいと思います。

public void deepcopy(HogeClass mc) {
    this.mycl = new List<MyClass>();
    this.mycl.AddRange(mc.mycl.Select(i => new MyClass() { val = i.val, text = i.text }));
}

Comments

[退会済みユーザー]

ありがとうございます！
最初自分でもICloneableを継承してCloneメソッドを作ってみてうまくいきませんでしたが、ご提示のコードでバッチリうまく動作しました！