Список объектов класса

Здравствуйте)
Проблема такая. Есть класс PossibleStep, создается список объектов этого класса в другом классе
В классе Game есть метод
Далее в методе происходит изменение элементов списка CurrSit. Так вот, при изменении элементов списка CurrSit, элементы списка steps также меняются. В чем может быть ошибка?

0

они имеют одну ссылку по этому и меняются

1

здесь не происходит копирование списка, а лишь копируется ссылка на него(или как то так).
как вариант можно сделать так:

1

Спасибо, стало понятно в чем проблема, но решение не помогло, изменения по-прежнему происходит в обоих списках. Прямое копирование элементов одного списка в другой также не помогло

0

Lazy Mashka, вы все равно копируете всего лишь ссылку, а не весь объект.

1

должно быть нечто вроде

1

Вопрос решен, всем спасибо)

0

kolorotur, кстати, к вам вопрос, как к специалисту - почему в шарпе не сделали конструктор копирования? Очень удобная штука ведь. И часто её не хватает, как раз в таких случаях.


Lazy Mashka, ты можешь просто вместо class написать struct и тогда можно будет просто написать как в посте #3

0

Сообщение от Psilon почему в шарпе не сделали конструктор копирования?

Мы же вроде уже в какой-то теме это обсуждали
Ситуация та же, что с клонированием: основная причина — неоднозначность понятия "копирование" в полностью объектно-ориентированном языке. Подразумевается, что это задача разработчика — определять, что именно может являться полноценной копией разрабатываемого им класса.
Ну это так же, как с понятиями равнозначность и иммутабельность: два объекта могут быть равнозначными, хотя побайтовое сравнение занимаемой ими памяти может отличаться, и объект может быть иммутабельным, даже если какие-то из его полей (скрытых от внешних глаз, как правило) и изменяются — в ООП важно то, что видно извне, а не то, что творится в "кишках" объекта.
По этой же причине два объекта могут считаться точными копиями друг друга, даже если и память, ими занимаемая, различается.
Вот и получается, что невозможно сделать некий универсальный метод для создания копии объекта, так как само понятие "копия" в ООП определяется отдельно для каждого типа.

Кстати, разработчики всё-таки не забыли про тех, кто хочет реализовать клонирование/копирование "наивным" методом, но не хочет ручками набивать "this.a = other.a; this.b = other.b": имеется защищенный метод MemberwiseClone, который, по сути, тупо копирует память, занимаемую объектом. А всякие "нюансы" уже потом можно ручками подрихтовать.

1

kolorotur, для меня копирование - это копирование всех полей. Не уверен, но вроде нагуглил, что object.MemrerwhiseClone это и делает. Если так - то вопрос снимается: в шарпе есть конструктор копирования, просто он замаскирован под метод основного класса.

Добавлено через 1 минуту

Сообщение от kolorotur Кстати, разработчики всё-таки не забыли про тех, кто хочет реализовать клонирование/копирование "наивным" методом, но не хочет ручками набивать "this.a = other.a; this.b = other.b": имеется защищенный метод MemberwiseClone, который, по сути, тупо копирует память, занимаемую объектом. А всякие "нюансы" уже потом можно ручками подрихтовать.

Мда, чет я как-то сначала пишу, а потом читаю ответ. Прошу понять и простить - отхожу от наркоза, только после операции + расстался с девушкой

0

Сообщение от Psilon для меня копирование - это копирование всех полей.

Вот пишите вы, допустим, на шарпе какую-нибудь виртуальную машину наподобие CLR, где каждая из ваших базовых единиц (в CLR — объект) состоит из шапки, представленной несколькими полями. Одно из этих полей — блок синхронизации типа int, который при захвате увеличивается, а при освобождении уменьшается.
Если значение равно нулю, то объект свободен, если нет, то объект залочен количеством потоков, равным значению этого поля. Его значение при копировании тоже копировать будем? Ведь если в этот момент его значение не равно нулю, то любая попытка произвести на нем синхронизацию приведет к взаимной блокировке (она же deadlock), т.к. никакой другой поток эту новую копию объекта не захватывал, а следовательно никогда и не освободит.

Сообщение от Psilon отхожу от наркоза, только после операции + расстался с девушкой

Вот это жопа
Держитесь!

1

Сообщение от kolorotur Вот пишите вы, допустим, на шарпе какую-нибудь виртуальную машину наподобие CLR, где каждая из ваших базовых единиц (в CLR — объект) состоит из шапки, представленной несколькими полями. Одно из этих полей — блок синхронизации типа int, который при захвате увеличивается, а при освобождении уменьшается. Если значение равно нулю, то объект свободен, если нет, то объект залочен количеством потоков, равным значению этого поля. Его значение при копировании тоже копировать будем? Ведь если в этот момент его значение не равно нулю, то любая попытка произвести на нем синхронизацию приведет к взаимной блокировке (она же deadlock), т.к. никакой другой поток эту новую копию объекта не захватывал, а следовательно никогда и не освободит.

насколько я помню, блок синхронизации имеет смещение -4 и поэтому не учитывается при копировании, да и не входит в размер объекта.

Сообщение от kolorotur Вот это жопа Держитесь!

да ладно, бывает у людей и хуже)

0

Сообщение от Psilon насколько я помню, блок синхронизации имеет смещение -4 и поэтому не учитывается при копировании, да и не входит в размер объекта.

Дык я же не про объект CLR говорю, а про что-то вроде этого, но вашей собственной реализации на шарпе:

Сообщение от kolorotur Вот пишите вы, допустим, на шарпе какую-нибудь виртуальную машину наподобие CLR...

0

kolorotur, ну не учитывать естественно

0

Сообщение от Lazy Mashka Спасибо, стало понятно в чем проблема, но решение не помогло, изменения по-прежнему происходит в обоих списках. Прямое копирование элементов одного списка в другой также не помогло C# 1 2 3 4 foreach (var p in steps) { CurrSit.Add(p); }

Нужно так:
или я не прав?

0

da1z, steps - это List<T>, а лист реализует интерфейс IEnumerable<T>, а значит по нему можно пробегать с помощью foreach. Преобразование в массив излишне.

0

tezaurismosis, смотря что ему нужно.

0

Это для того чтобы элементы скопировались, а не ссылки.

0

da1z, тогда и в массиве будут ссылки, ведь массив - ссылочный тип.

0

Точно, что-то я ступил

0