ข้อผิดพลาด:“ ไม่สามารถแก้ไขค่าที่ส่งคืนได้” c #

ฉันใช้คุณสมบัติที่ใช้งานอัตโนมัติ ฉันเดาวิธีที่เร็วที่สุดในการแก้ไขปัญหาต่อไปนี้คือการประกาศตัวแปรสำรองของฉันเอง

public Point Origin { get; set; }

Origin.X = 10; // fails with CS1612

ข้อความแสดงข้อผิดพลาด: ไม่สามารถแก้ไขค่าที่ส่งคืนของ 'expression' ได้เนื่องจากไม่ใช่ตัวแปร

มีความพยายามในการแก้ไขชนิดของค่าที่เป็นผลลัพธ์ของนิพจน์กลาง เนื่องจากค่าไม่คงอยู่ค่าจะไม่เปลี่ยนแปลง

เมื่อต้องการแก้ไขข้อผิดพลาดนี้ให้เก็บผลลัพธ์ของนิพจน์ในค่าระดับกลางหรือใช้ประเภทการอ้างอิงสำหรับนิพจน์กลาง

นี่เป็นเพราะPointเป็นประเภทค่า ( struct)

ด้วยเหตุนี้เมื่อคุณเข้าถึงOriginคุณสมบัติที่คุณกำลังเข้าถึงสำเนาของค่าที่เก็บไว้ในชั้นเรียนไม่ได้ค่าตัวเองเช่นเดียวกับที่คุณมีประเภทอ้างอิง ( class) ดังนั้นหากคุณตั้งค่าXคุณสมบัติในนั้นคุณจะตั้งค่า คุณสมบัติบนสำเนาแล้วละทิ้งทิ้งค่าเดิมไม่เปลี่ยนแปลง นี่อาจไม่ใช่สิ่งที่คุณตั้งใจซึ่งเป็นสาเหตุที่คอมไพเลอร์เตือนคุณเกี่ยวกับเรื่องนี้

หากคุณต้องการเปลี่ยนเพียงXค่าคุณต้องทำสิ่งนี้:

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

การใช้ตัวแปรสำรองจะไม่ช่วย Pointประเภทเป็นชนิดราคา

คุณต้องกำหนดค่าคะแนนทั้งหมดให้กับคุณสมบัติ Origin: -

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

ปัญหาคือเมื่อคุณเข้าถึงคุณสมบัติ Origin สิ่งที่ส่งคืนโดยgetคือสำเนาของโครงสร้างจุดในฟิลด์คุณสมบัติที่สร้างขึ้นโดยอัตโนมัติ ดังนั้นการแก้ไขฟิลด์ X สำเนาของคุณนี้จะไม่ส่งผลกระทบกับฟิลด์ต้นแบบ คอมไพเลอร์ตรวจจับสิ่งนี้และแจ้งข้อผิดพลาดให้คุณเนื่องจากการดำเนินการนี้ไม่มีประโยชน์อย่างสิ้นเชิง

แม้ว่าคุณจะใช้ตัวแปรสำรองของคุณเองคุณgetก็จะมีลักษณะดังนี้: -

get { return myOrigin; }

คุณยังคงส่งคืนสำเนาของโครงสร้างจุดและคุณจะได้รับข้อผิดพลาดเดียวกัน

อืม ... เมื่อได้อ่านคำถามของคุณอย่างระมัดระวังบางทีคุณอาจต้องการปรับเปลี่ยนตัวแปรสำรองโดยตรงจากในชั้นเรียนของคุณ: -

myOrigin.X = 10;

ใช่ว่าจะเป็นสิ่งที่คุณต้องการ

ถึงตอนนี้คุณก็รู้แล้วว่าแหล่งที่มาของข้อผิดพลาดคืออะไร ในกรณีที่คอนสตรัคเตอร์ไม่ได้มีการโอเวอร์โหลดเพื่อใช้คุณสมบัติของคุณ (ในกรณีนี้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;

ดูเหมือนว่าจะเป็นสิ่งที่แปลกมากที่จะไม่แนะนำเลย เพียงแค่แสดงรายการวิธีอื่น วิธีที่ดีกว่าที่จะทำคือทำให้ struct ไม่เปลี่ยนแปลงและจัดให้มี Constructor ที่เหมาะสม

นอกเหนือจากการอภิปรายข้อดีข้อเสียของ structs กับคลาสฉันมักจะมองไปที่เป้าหมายและเข้าถึงปัญหาจากมุมมองนั้น

ที่ถูกกล่าวว่าถ้าคุณไม่จำเป็นต้องเขียนโค้ดด้านหลังคุณสมบัติรับและกำหนดวิธีการ (เช่นในตัวอย่างของคุณ) แล้วมันจะไม่ง่ายกว่าที่จะประกาศ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

ปัญหาคือคุณชี้ไปที่ค่าที่ตั้งอยู่บนสแต็กและค่าจะไม่ถูก relfected กลับไปที่คุณสมบัติ orignal ดังนั้น C # ไม่อนุญาตให้คุณส่งคืนการอ้างอิงถึงชนิดของค่า ฉันคิดว่าคุณสามารถแก้ปัญหานี้ได้โดยการลบคุณสมบัติ Origin และใช้ไฟล์สาธารณะแทนใช่ฉันรู้ว่าไม่ใช่วิธีแก้ปัญหาที่ดี โซลูชันอื่นคือไม่ใช้ Point และสร้างประเภท Point ของคุณเองเป็นวัตถุแทน

ฉันเดาว่าที่นี่คือคุณกำลังพยายามกำหนดค่าย่อยของวัตถุในคำสั่งแทนที่จะกำหนดวัตถุเอง คุณจำเป็นต้องกำหนดออบเจ็กต์ Point ทั้งหมดในกรณีนี้เนื่องจากประเภทคุณสมบัติคือ Point

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

หวังว่าฉันจะทำให้รู้สึกมี

เพียงลบคุณสมบัติ "รับการตั้งค่า" ดังต่อไปนี้จากนั้นทุกอย่างจะทำงานเหมือนเดิม

ในกรณีที่ชนิดดั้งเดิมนั้นใช้ 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]

    }
}      

ผมคิดว่าคนจำนวนมากจะได้รับการสับสนที่นี่ปัญหานี้โดยเฉพาะอย่างยิ่งที่เกี่ยวข้องกับการทำความเข้าใจว่าประเภทค่าคุณสมบัติกลับสำเนาของประเภทค่าที่ (เช่นเดียวกับวิธีการและ indexers) และประเภทค่าเขตข้อมูลที่มีการเข้าถึงโดยตรง รหัสต่อไปนี้ทำในสิ่งที่คุณพยายามจะทำโดยการเข้าถึงเขตข้อมูลสำรองของทรัพย์สินโดยตรง (หมายเหตุ: การแสดงคุณสมบัติในรูปแบบ verbose ด้วยเขตข้อมูลสำรองนั้นเทียบเท่ากับคุณสมบัติอัตโนมัติ แต่มีข้อได้เปรียบในรหัสของเราที่เราสามารถทำได้ เข้าถึงข้อมูลสำรองโดยตรง):

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