Błąd:" nie można zmienić wartości zwracanej " c#

użycie zmiennej pomocniczej nie pomoże. typ Point jest typem wartości.

Musisz przypisać wartość całego punktu do Właściwości Origin:-

Origin = new Point(10, Origin.Y);

Problem polega na tym, że gdy uzyskujesz dostęp do Właściwości Origin, to co jest zwracane przez {[4] } jest kopią struktury punktu w automatycznie utworzonym polu Właściwości Origin. Stąd twoja modyfikacja Pola X ta kopia nie wpłynie na pole bazowe. Kompilator wykrywa to i daje błąd, ponieważ to operacja jest całkowicie bezużyteczna.

Nawet jeśli użyjesz własnej zmiennej nośnej, Twoje get wyglądałoby tak:-

get { return myOrigin; }

Nadal zwracałbyś kopię struktury punktowej i otrzymałbyś ten sam błąd.

Hmm... po dokładniejszym zapoznaniu się z twoim pytaniem, być może chcesz zmodyfikować zmienną podkładową bezpośrednio z poziomu swojej klasy: -

myOrigin.X = 10;

Już wiesz, co jest źródłem błędu. W przypadku, gdy konstruktor nie istnieje z przeciążeniem, aby zabrać twoją własność( w tym przypadku X), możesz użyć inicjalizatora obiektu (który wykona całą magię za kulisami). nie dlatego, że nie musisz zmieniać swoich struktur , ale po prostu podając dodatkowe informacje:

struct Point
{
    public int X { get; set; }
    public int Y { get; set; }
}

class MyClass
{
    public Point Origin { get; set; }
}

MyClass c = new MyClass();
c.Origin.X = 23; //fails.

//but you could do:
c.Origin = new Point { X = 23, Y = c.Origin.Y }; //though you are invoking default constructor

//instead of
c.Origin = new Point(23, c.Origin.Y); //in case there is no constructor like this.

Jest to możliwe, ponieważ dzieje się to za kulisami:

Point tmp = new Point();
tmp.X = 23;
tmp.Y = Origin.Y;
c.Origin = tmp;

Poza dyskusją o zaletach i wadach struktur kontra klas, mam tendencję do patrzenia na cel i podejścia do problemu z tej perspektywy.

Jeśli nie musisz pisać kodu za właściwością get I set methods (jak w twoim przykładzie), to czy nie byłoby łatwiej zadeklarować Origin jako pole klasy, a nie Właściwość? Myślę, że to pozwoli Ci osiągnąć swój cel.

struct Point
{
    public int X { get; set; }
    public int Y { get; set; }
}

class MyClass
{
    public Point Origin;
}

MyClass c = new MyClass();
c.Origin.X = 23;   // No error.  Sets X just fine

Problem polega na tym, że wskazujesz na wartość znajdującą się na stosie i nie będzie ona przenoszona z powrotem do właściwości orignal, więc C# nie pozwala zwracać referencji do typu wartości. Myślę, że możesz to rozwiązać usuwając właściwość Origin i zamiast tego użyć publicznego pliku, tak Wiem, że nie jest to miłe rozwiązanie. Innym rozwiązaniem jest nie używać punktu, a zamiast tego utworzyć własny typ punktu jako obiekt.

Myślę, że haczyk polega na tym, że próbujesz przypisać podrzędne wartości obiektu w instrukcji, zamiast przypisywać sam obiekt. W tym przypadku należy przypisać cały obiekt Point jako typ właściwości Point.

Point newOrigin = new Point(10, 10);
Origin = newOrigin;

Hope I made sense there

Po prostu usuń właściwość "get set" w następujący sposób, a potem wszystko działa jak zawsze.

W przypadku typów prymitywnych instread użyj get; set;...

using Microsoft.Xna.Framework;
using System;

namespace DL
{
    [Serializable()]
    public class CameraProperty
    {
        #region [READONLY PROPERTIES]
        public static readonly string CameraPropertyVersion = "v1.00";
        #endregion [READONLY PROPERTIES]


        /// <summary>
        /// CONSTRUCTOR
        /// </summary>
        public CameraProperty() {
            // INIT
            Scrolling               = 0f;
            CameraPos               = new Vector2(0f, 0f);
        }
        #region [PROPERTIES]   

        /// <summary>
        /// Scrolling
        /// </summary>
        public float Scrolling { get; set; }

        /// <summary>
        /// Position of the camera
        /// </summary>
        public Vector2 CameraPos;
        // instead of: public Vector2 CameraPos { get; set; }

        #endregion [PROPERTIES]

    }
}      

Myślę, że wiele osób się tutaj myli, ten szczególny problem jest związany ze zrozumieniem, że właściwości typu wartości zwracają kopię typu wartości (tak jak w przypadku metod i indekserów), a pola typu wartości są dostępne bezpośrednio. Poniższy kod robi dokładnie to, co próbujesz osiągnąć, uzyskując bezpośredni dostęp do pola zapasowego właściwości (Uwaga: wyrażenie właściwości w jej wyrazistej formie z polem zapasowym jest odpowiednikiem właściwości auto, ale ma tę zaletę, że w naszym kodzie możemy uzyskać bezpośredni dostęp do zaplecza):

class Program
{
    static void Main(string[] args)
    {
        var myClass = new MyClass();
        myClass.SetOrigin();
        Debug.Assert(myClass.Origin.X == 10); //succeeds
    }
}

class MyClass
{
    private Point _origin;
    public Point Origin
    { 
        get => _origin; 
        set => _origin = value; 
    }

    public void SetOrigin()
    {
        _origin.X = 10; //this works
        //Origin.X = 10; // fails with CS1612;
    }
}

Błąd, który otrzymujesz, jest pośrednią konsekwencją nie zrozumienia, że właściwość zwraca kopię typu wartości. Jeśli zostanie zwrócona Kopia typu wartości i nie przypiszesz jej do zmiennej lokalnej, to wszelkie zmiany wprowadzone w tej kopii nigdy nie zostaną odczytane i dlatego kompilator zgłosi to jako błąd, ponieważ nie może to być zamierzone. Jeśli przypiszemy kopię do zmiennej lokalnej wtedy możemy zmienić wartość X, ale będzie ona zmieniana tylko na lokalnej kopii, co naprawia błąd czasu kompilacji, ale nie będzie miało pożądanego efektu modyfikowania właściwości Origin. Poniższy kod ilustruje to, ponieważ błąd kompilacji zniknął, ale twierdzenie debug nie powiedzie się:

class Program
{
    static void Main(string[] args)
    {
        var myClass = new MyClass();
        myClass.SetOrigin();
        Debug.Assert(myClass.Origin.X == 10); //throws error
    }
}

class MyClass
{
    private Point _origin;
    public Point Origin
    { 
        get => _origin; 
        set => _origin = value; 
    }

    public void SetOrigin()
    {
        var origin = Origin;
        origin.X = 10; //this is only changing the value of the local copy
    }
}