错误：“无法修改返回值”c#

Question

我正在使用自动实现的属性。 我想解决以下问题的最快方法是声明我自己的支持变量？

public Point Origin { get; set; }

Origin.X = 10; // fails with CS1612

错误消息：无法修改“表达式”的返回值，因为 它不是变量

试图修改一个值类型，该值类型是 中间表达。因为值没有持久化，所以值 将保持不变。

要解决此错误，请将表达式的结果存储在 中间值，或使用中间值的引用类型 表达。

Answer 1

这是因为Point 是一个值类型（struct）。

因此，当您访问 Origin 属性时，您访问的是类所持有值的副本，而不是引用类型 (@987654325) 中的值本身@)，因此如果您在其上设置了 X 属性，那么您将在副本上设置该属性，然后将其丢弃，保持原始值不变。这可能不是您想要的，这就是编译器警告您的原因。

如果您只想更改 X 值，则需要执行以下操作：

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

Answer 2

使用支持变量没有帮助。Point 类型是值类型。

您需要将整个 Point 值分配给 Origin 属性：-

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

问题是当您访问 Origin 属性时，get 返回的是 Origin 属性自动创建字段中 Point 结构的副本。因此，您对该副本的 X 字段的修改不会影响基础字段。编译器检测到这一点并给你一个错误，因为这个操作完全没用。

即使您使用自己的支持变量，您的 get 也会如下所示：-

get { return myOrigin; }

您仍然会返回 Point 结构的副本，并且会收到相同的错误。

嗯...更仔细地阅读了您的问题，也许您实际上的意思是直接从您的类中修改支持变量：-

myOrigin.X = 10;

是的，这正是您所需要的。

Answer 3

现在您已经知道错误的根源是什么。如果构造函数不存在并带有重载来获取您的属性（在本例中为X），您可以使用对象初始化器（它将在幕后完成所有魔术）。 并不是说你不需要让你的结构不可变，而只是提供额外的信息：

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.

这是可能的，因为这是在幕后发生的：

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

这看起来很奇怪，完全不推荐。只是列出另一种方式。更好的方法是使结构不可变并提供适当的构造函数。

Answer 4

除了辩论结构与类的优缺点之外，我倾向于从这个角度看待目标并解决问题。

话虽如此，如果您不需要在属性 get 和 set 方法后面编写代码（如您的示例中所示），那么简单地将 Origin 声明为类的字段不是更容易吗？比财产？我应该认为这会让你实现你的目标。

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

Answer 5

我认为很多人在这里感到困惑，这个特殊问题与理解值类型属性返回值类型的副本（与方法和索引器一样）有关，和值类型字段直接访问。以下代码通过直接访问属性的支持字段来完成您尝试实现的目标（注意：使用支持字段以详细形式表示属性相当于自动属性，但具有在我们的代码中我们可以直接访问支持字段）：

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;
    }
}

您遇到的错误是不了解属性返回值类型副本的间接后果。如果您返回一个值类型的副本并且您没有将其分配给局部变量，那么您对该副本所做的任何更改都无法读取，因此编译器将此作为错误提出，因为这不是故意的。如果我们将副本分配给局部变量，那么我们可以更改 X 的值，但它只会在本地副本上更改，这修复了编译时错误，但不会达到修改 Origin 属性的预期效果。下面的代码说明了这一点，因为编译错误消失了，但调试断言会失败：

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
    }
}

Answer 6

问题是您指向位于堆栈上的值，并且该值不会被重新反射回原始属性，因此 C# 不允许您返回对值类型的引用。我认为您可以通过删除 Origin 属性来解决这个问题，而是使用公共文件，是的，我知道这不是一个好的解决方案。另一种解决方案是不使用 Point，而是创建自己的 Point 类型作为对象。

Answer 7

我想这里的问题是您试图在语句中分配对象的子值，而不是分配对象本身。在这种情况下，您需要分配整个 Point 对象，因为属性类型是 Point。

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

希望我说得有道理

Answer 8

只需按如下方式删除“get set”属性，然后一切正常。

如果是原始类型，请使用 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]

    }
}