Почему не срабатывает MemberwiseClone?

@netBool · Регистрация: 16.11.2010

Студворк — интернет-сервис помощи студентам

Доброго времени!

Я понимаю, что это поверхностное копирование. По msdn этот метод вроде как создает новый объект и копирует в него значения полей типа значения и ссылки ссылочных полей. У меня есть есть такой метод у класса DSlist

C#
1
2
3
4
5
6
7
8
9
        public DSlist Clone()
        {
            var vr = (DSlist)this.MemberwiseClone();
            for (int i = 0; i < this.Count; i++) 
            {
                vr[i] = this[i].CloneToReciept();
            }
            return vr;
        }

DSlist - это потомок List типа DString, в котором в свою очередь есть метод CloneToReciept:

C#
1
2
3
4
        public DString CloneToReciept()
        { 
            return (DString)this.MemberwiseClone(); 
        }

Сам тип DString - простой, как пробка, и насквозь состоит из типов значения. В частности есть одно из его полей, по которому я ориентируюсь:

C#
1
2
3
4
5
6
7
8
9
10
11
12
        int count;
        public int theCount
        {
            get
            {
                return count;
            }
            set
            {
                count = value;
            }
        }

На выходе метода Clone() объекта DSlist по идее должен получить полную копию DSlist, включающую копию каждого элемента List<DString>. Но!
В этом коде:

C#
1
2
3
dslist[0].theCount = 1;
var copiedlist = dslist.Clone();
copiedlist[0].theCount = 2;

Я должен получить dslist[0].theCount = 1 и copiedlist[0].theCount = 2. Но на выходе при смене значения в copiedlist[0].theCount меняется значение и в dslist[0].theCount.

Пробовал на простом примере:

C#
1
2
3
DS.theCount = 1;
var dsclone = DS.CloneToReciept();
dsclone.theCount = 2;

Все работает прекрасно. Но не пойму, в чем ошибка у меня. Может быть такое, что в строке

C#
1
vr[i] = this[i].CloneToReciept();

this[i].CloneToReciept() не присваивает в vr[i] ссылку на копию this[i], а оставляет старую? Это у меня как-то не усваивается в голове

Не по теме:

И еще вопрос не по теме. Никто не знает, почему в мсдн в некоторых методах только декларации без тела метода?

В частности у MemberwiseClone:

C#
1
2
3
4
5
6
7
8
9
    // Returns a new object instance that is a memberwise copy of this 
    // object.  This is always a shallow copy of the instance. The method is protected
    // so that other object may only call this method on themselves.  It is entended to
    // support the ICloneable interface.
    // 
    [System.Security.SecuritySafeCritical]  // auto-generated
    [ResourceExposure(ResourceScope.None)]
    [MethodImplAttribute(MethodImplOptions.InternalCall)]
    protected extern Object MemberwiseClone();

@insite2012 · 18.07.2018, 21:37

netBool, дайте полный код вашего теста.

@netBool · 18.07.2018, 21:46 **[ТС]**

Сейчас мне пришло в голову, что это может быть из-за того, что ссылка на сам List<> остается старой и копируется ссылкой. Но у меня соотношение не через композицию или агрегацию, типа такой:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
public class DSlist
{
       List<ForReady.DString> list;
       public DString this[int index]
       {
              get
              {
                      return list[index];
              }
              set
              {
                      list[index] = value;
              }
       }
}

а через наследование:

C#
1
public class DSlist : List<ForReady.DString>

Мне кажется, для такого случая MemberwiseClone должен создать новый экземпляр единый, как для предка, так и для потомка. Или он оставляет где-то там в глубине ссылку на старого нескопированного предка? Может быть такое?

Добавлено через 7 минут
insite2012, вот такой тестик получился:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
 
namespace TestClone
{
    class DSlist : List<DString>
    {
        public DSlist Clone()
        {
            var vr = (DSlist)this.MemberwiseClone();
            for (int i = 0; i < this.Count; i++)
            {
                vr[i] = this[i].CloneToReciept();
            }
            return vr;
        }
    }
 
    class DString
    {
        public DString CloneToReciept()
        {
            return (DString)this.MemberwiseClone();
        }
 
        int count;
        public int theCount
        {
            get
            {
                return count;
            }
            set
            {
                count = value;
            }
        }
    }
 
    class Program
    {
        static void Main(string[] args)
        {
            var ds = new DSlist();
            ds.Add(new DString() { theCount = 1 });
            var dsclone = ds.Clone();
            dsclone[0].theCount = 2;
 
            Console.WriteLine(ds[0].theCount);
            Console.WriteLine(dsclone[0].theCount);
            
            
            Console.Read();
        }
    }
}

на выходе получается 2 2. Хотя по задумке вроде как должно быть 1 2

@insite2012 · 18.07.2018, 22:01

netBool, такой вариант?

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
 
namespace TestClone {
    class DSlist : List<DString> {
        public DSlist Clone() {
            var vr = new DSlist();
            for (int i = 0; i < this.Count; i++) {
                vr.Add(this[i].CloneToReciept());
            }
            return vr;
        }
    }
 
    class DString {
        public DString CloneToReciept() {
            DString dstr = new DString();
            dstr.theCount = this.theCount;
            return dstr;
        }
 
        int count;
        public int theCount {
            get {
                return count;
            }
            set {
                count = value;
            }
        }
    }
 
    class Program {
        static void Main(string[] args) {
            var ds = new DSlist();
            ds.Add(new DString() { theCount = 1 });
            var dsclone = ds.Clone();
            dsclone[0].theCount = 2;
 
            Console.WriteLine(ds[0].theCount);
            Console.WriteLine(dsclone[0].theCount);
 
 
            Console.Read();
        }
    }
}

@kolorotur · 19.07.2018, 02:03

netBool, ну так вы же сами сказали, что у вас поверхностное клонирование - вот MemberwiseClone и клонирует ссылку на тот же массив, который используется в кишках List<T>, к которому вы потом обращаетесь через this[]

@netBool · 20.07.2018, 14:47 **[ТС]**

Сообщение от insite2012

netBool, такой вариант?

У вашего варианта нет шансов не работать)) Раньше так и делал. Но решил поискать более универсальное решение. Дело в том, что в классе DSList у меня есть еще с полдюжины полей, которых тоже надо копировать (я просто не стал загружать лишней информацией тестовый пример, оставил только необходимое). И эти поля могут добавляться/удаляться. А значит и контруктор копирования пришлось бы каждый раз модифицировать:

C#
1
2
3
4
5
6
7
        public DSlist Clone() {
            var vr = new DSlist(){поле1=this.поле1,поле2=this.поле2,и тд};
            for (int i = 0; i < this.Count; i++) {
                vr.Add(this[i].CloneToReciept());
            }
            return vr;
        }

А MemberwiseClone как раз делает всю лишнюю работу по копированию полей

С другой стороны мне было любопытно, почему он не срабатывал в моем слуачае

kolorotur, ага, верно. Свои поля проверил, а про те, что List инкапсулирует, забыл

В общем не придумал ничего лучше, чем вызвать конструктор для внутреннего массива List в копированном объекте:

C#
1
2
            typeof(DSlist).BaseType.GetField("_items",
                BindingFlags.NonPublic | BindingFlags.Instance).SetValue(vr, new DString[this.Count]);//0

К сожалению, не смог обойтись без Reflection(

Так работает. Может, кому пригодится такой подход

Не по теме:

вроде теперь понял, в чем преимущество ассоциации перед наследованием. При ассоциации (агрегации или композиции) можно было бы обойтись без Рефлекшн

Новые блоги и статьи Все статьи Все блоги /
Кому нужен AOT? DevAlt 26.03.2026 Решил сделать простой ланчер Написал заготовку: dotnet new console --aot -o UrlHandler var items = args. Split(":"); var tag = items; var id = items; var executable = args;. . .	Отправка уведомления на почту при изменении наименования справочника Maks 24.03.2026 Программная отправка письма электронной почты на примере изменения наименования типового справочника "Склады" в конфигурации БП3. Перед реализацией необходимо выполнить настройку системной учетной. . .	модель ЗдравоСохранения 5. Меньше увольнений- больше дохода! anaschu 24.03.2026 Теперь система здравосохранения уменьшает количество увольнений. 9TO2GP2bpX4 a42b81fb172ffc12ca589c7898261ccb/ https:/ / rutube. ru/ video/ a42b81fb172ffc12ca589c7898261ccb/ Слева синяя линия -. . .	Midnight Chicago Blues kumehtar 24.03.2026 Такой Midnight Chicago Blues, знаешь?. . Когда вечерние улицы становятся ночными, а ты не можешь уснуть. Ты идёшь в любимый старый бар, и бармен наливает тебе виски. Ты смотришь на пролетающие. . .
SDL3 для Desktop (MinGW): Вывод текста со шрифтом TTF с помощью библиотеки SDL3_ttf на Си и C++ 8Observer8 24.03.2026 Содержание блога Финальные проекты на Си и на C++: finish-text-sdl3-c. zip finish-text-sdl3-cpp. zip	Жизнь в неопределённости kumehtar 23.03.2026 Жизнь — это постоянное существование в неопределённости. Например, даже если у тебя есть список дел, невозможно дойти до точки, где всё окончательно завершено и больше ничего не осталось. В принципе,. . .	Модель здравоСохранения: работники работают быстрее после её введения. anaschu 23.03.2026 geJalZw1fLo Корпорация до введения программа здравоохранения имела много невыполненных работниками заданий, после введения программы количество заданий выросло. Но на выплатах по больничным это. . .	Контроль уникальности заводского номера Maks 23.03.2026 Алгоритм контроля уникальности заводского (или серийного) номера на примере нетипового документа выдачи шин для спецтехники с табличной частью, разработанного в конфигурации КА2. Номеклатура. . .

@insite2012 5548 / 4311 / 1218 Регистрация: 12.10.2013 Сообщений: 12,371 Записей в блоге: 2
	18.07.2018, 21:37
	netBool, дайте полный код вашего теста. 0

@kolorotur 17823 / 12973 / 3382 Регистрация: 17.09.2011 Сообщений: 21,261
	19.07.2018, 02:03
	Сообщение было отмечено netBool как решение Решение netBool, ну так вы же сами сказали, что у вас поверхностное клонирование - вот MemberwiseClone и клонирует ссылку на тот же массив, который используется в кишках List<T>, к которому вы потом обращаетесь через this[] 1

Почему не срабатывает MemberwiseClone?

Решение