Почему при добавление объекта класса в list один объект перенимает значение другого?

Question

[xXNullXx]

Доброго времени суток!

Делаю приложение для накладывания фильтров на фото(чисто потренироваться).

Нужно сделать список фильтров с предпросмотром как в инстаграме.
Решил это сделать так: добавляю обработанные объекты класса "Picture"

public class Picture
    {
        public uint Width { get; set; }
        public uint Height { get; set; }

        public Pixel[,] Pixels { get; set; }
    }

в list Pictures

public class FilterExplorer
    {
        private readonly OriginalPicture originalPicture = OriginalPicture.GetInstance(); // Здесь один из ОБ класса "Picture"  каторые не должен меняться в данном классе
        private readonly IFilter halftoneFilter = HalftoneFilter.GetInstance(); // Это об одного из фиитров

        public List<Picture> Pictures { get; private set; } = new List<Picture>(); // Здесь хранятся обработанные картинки и одна НЕ обработанная 

        private FilterExplorer()
        {
            Initialization();
        }
            
        private void Initialization()
        {
            Pictures.Add(originalPicture.Picture); // вот та самая не обработанная картинка 
            Pictures.Add(halftoneFilter.Filtration(originalPicture.Picture)); // и обработанная
        }
    }

Проблема собственно в чем когда я добавляю то первая картинка перенимает значение второго. Причем это происходит до возвращения значения функции "Filtration"

public Picture Filtration(Picture picture)
        {
            for (uint x = 0; x < picture.Width; x++)
            {
                for (uint y = 0; y < picture.Height; y++)
                {
                    double convertedPoint =
                        0.3 * picture.Pixels[x, y].R
                        + 0.59 * picture.Pixels[x, y].G
                        + 0.11 * picture.Pixels[x, y].B;

                    picture.Pixels[x, y].R = convertedPoint;
                    picture.Pixels[x, y].G = convertedPoint;
                    picture.Pixels[x, y].B = convertedPoint;
                }
            }
       
            return picture;
        }

Answer 1

[#]

в методе public Picture Filtration(Picture picture) вы принимаете экземпляр картинки по ссылке (ибо это экземпляр класса, не какой то там int)

обрабатываете его, и его же и возвращаете. вот и весь ответ

что бы реализовать задумку, как вы ее описываете, вам нужно клонировать исходное изображение (создать новый экземпляр того же класса, с таким же содержимым). и только после этого применить обработку, и вернуть клон а не оригинал. это можно сделать в том же методе Filtration. логично создать клон в самом начале метода

ps подробнее тут https://docs.microsoft.com/ru-ru/dotnet/csharp/pro...

Comments

[xXNullXx]

Если я правильно понял то я это сделал, но результат тот же

public Picture Filtration(Picture originalPicture)
        {
            //Picture picture = (Picture)originalPicture.Clone();

            Picture picture = new Picture()
            {
                Width = originalPicture.Width,
                Height = originalPicture.Height,

                Pixels = originalPicture.Pixels
            };

            for (uint x = 0; x < picture.Width; x++)
            {
                for (uint y = 0; y < picture.Height; y++)
                {
                    double convertedPoint =
                        0.3 * picture.Pixels[x, y].R
                        + 0.59 * picture.Pixels[x, y].G
                        + 0.11 * picture.Pixels[x, y].B;

                    picture.Pixels[x, y].R = convertedPoint;
                    picture.Pixels[x, y].G = convertedPoint;
                    picture.Pixels[x, y].B = convertedPoint;
                }
            }
        
            return picture;
        }

[#]

xXNullXx, с виду все верно. ошибка должна пропасть. но раз говорите результат тот же - значит есть еще ошибки (ибо чудес не бывает)

весь проект в студию, плз

[#]

xXNullXx, хотя не, и так видно. массив тоже ссылочный тип. вы содали класс, но скопировали ссылку на оригинальный массив

подробнее тут
https://docs.microsoft.com/ru-ru/dotnet/csharp/pro...

[#]

xXNullXx, еще пара ссылок, хотя там и не сказано про массивы явно, тем не менее это ссылочный тип
https://docs.microsoft.com/ru-ru/dotnet/csharp/lan...
https://docs.microsoft.com/ru-ru/dotnet/csharp/lan...

еще может запутать, что в C#, .Net вообще, все есть object. тем не менее есть ряд примитивных обьектов, а так же структуры, которые не являются ссылочными, и по дефолту передаются по значению. все остальное - по ссылке. значимые типы так же могут передаваться по ссылке ключевым словом ref.

но в вашей ситуации надо решить обратную задачу - клонировать ссылочные объекты.
начните с отдельного метода Clone(), но не поленитесь upd создать массив и выписать циклы копирования. что бы расставить все на свои места.

потом уже можно поиграться с Array.Copy() это компактнее запись как минимум, и даже Buffer.BlockCopy() а это точно быстрее. но оба метода требуют большего уровня владения языком, чем у вас на данный момент

[#]

xXNullXx, ну тогда уж еще важные моменты:
- значимые и ссылочные типы ведут себя соответственно названию, не только при передаче данных, но и при присвоении (ваш начальный вариант клонирования именно об это и спотыкается)
- если есть структура, которая в себе содержит значимые поля (int, long, double) и ссылочные (массив, класс, строка строки кстати надо проверять, самый хитрый тип, он и ссылочный, но при передаче в параметры и единичном присвоении - точно по значению ))) при передаче параметром или присвоении, будет создана новая структура, где все члены будут копиями, но значимые будут иметь копии значений, а ссылочные указывать на оригинал

подобное разделение есть во всех языках программирования, хотя могут быть и нюансы, и даже весьма хитрые. но в целом, к этому надо просто привыкнуть. со временем будете на автомате все это разруливать

[#]

xXNullXx, вот архив с очень упрощенным примером. вместо фильтра просто инверсия знака. и два варианта клонирования good и bad - https://www.sendspace.com/file/r3gj11

возможно вам пригодятся некоторые приемы сделать код компактнее и читаемее

[xXNullXx]

#, Спасибо большое за ответы, очень помогло и много новое узнал. Вот реализация:

public struct Pixel : System.ICloneable
    {
       ...
 public object Clone()
        {
            return new Pixel { R = this.R, G = this.G, B = this.B };
        }
    }

public class Picture : System.ICloneable
    {

public object Clone()
        {
            ...
            Pixel[,] ClonePixels = new Pixel[Width, Height];
            for (uint x = 0; x < Width; x++)
            {
                for (uint y = 0; y < Height; y++)
                {
                    ClonePixels[x, y] = (Pixel)Pixels[x, y].Clone();
                }
            }

            return new Picture() { Width = this.Width, Height = this.Height, Pixels = ClonePixels };
        }
    }

public Picture Filtration(Picture originalPicture)
        {
            Picture picture = (Picture)originalPicture.Clone();
            ...
        }

[#]

xXNullXx,

Спасибо большое за ответы, очень помогло и много новое узнал.

а отметочку ответом вернуть? ;))

про реализацию - тут уже сами, а то если найду время - опять попрошу весь проект, и перекрою со словами "типа так кошернее", возможно до неузнаваемости ;)))

[xXNullXx]

#, Вот если интересно: https://github.com/reydr/PhotoshopOnMinimal

[xXNullXx]

#,

опять попрошу весь проект, и перекрою со словами "типа так кошернее",

Было бы интересно посмотреть на более оптимальное и лаконичное решение)

[#]

xXNullXx, ну пугать то мы все горазды ;)), ключевая фраза

если найду время

но в закладки положу ))

[xXNullXx]

#,

если найду время

Ну это понятно, все понимаю)

Answer 2

[freeExec]

когда я добавляю то первая картинка перенимает значение второго.

Фраза не понятна.
Но попробую угадать вашу проблему.
Ваш фильтр не создаёт новую картинку, он меняет значения в исходной. Поэтому в итоге у вас списке два раза один и тот же объект.