将枚举结果插入一行代码 C#

Question

I'm making a simple color picking script for a game that I've been working on. My problem is that I have to repeat these three lines for every color scheme that I have:

Robot.GetComponent<SpriteRenderer>().sprite = cPicker.Salmon.Chenilles;

Robot.transform.Find("Middle").gameObject.GetComponent<SpriteRenderer>().sprite = cPicker.Salmon.Cover;

Robot.transform.Find("Corps").gameObject.GetComponent<SpriteRenderer>().sprite = cPicker.Salmon.Head;

I know that I should be able to not have to repeat myself, but I can't figure out how to insert my enum result into this line:

Robot.GetComponent<SpriteRenderer>().sprite = cPicker.Slime.Chenilles; 

I'd like to do something like this:

Robot.GetComponent().sprite = cPicker.(color.unit).Chenilles;

But obviously it doesn't work like that. Does anyone know how I could find a way to do this? Thank you in advance.

using System.Collections.Generic;
using UnityEngine;
using System;

public enum ColorSelection { Slime, Salmon, Marine, Dusk }

[Serializable] public class ColorSet { public ColorSelection unit; }

public class InputManager : MonoBehaviour
{
    public ColorPicker cPicker;

    public GameObject Robot;

    public ColorSet color;

    public void Update()
    {
        switch (color.unit)
        {
            case ColorSelection.Slime:
                Robot.GetComponent<SpriteRenderer>().sprite = cPicker.Slime.Chenilles;
                Robot.transform.Find("Middle").gameObject.GetComponent<SpriteRenderer>().sprite = cPicker.Slime.Cover;
                Robot.transform.Find("Corps").gameObject.GetComponent<SpriteRenderer>().sprite = cPicker.Slime.Head;
                break;

            case ColorSelection.Salmon:
                Robot.GetComponent<SpriteRenderer>().sprite = cPicker.Salmon.Chenilles;
                Robot.transform.Find("Middle").gameObject.GetComponent<SpriteRenderer>().sprite = cPicker.Salmon.Cover;
                Robot.transform.Find("Corps").gameObject.GetComponent<SpriteRenderer>().sprite = cPicker.Salmon.Head;
                break;

            case ColorSelection.Marine:
                Robot.GetComponent<SpriteRenderer>().sprite = cPicker.Marine.Chenilles;
                Robot.transform.Find("Middle").gameObject.GetComponent<SpriteRenderer>().sprite = cPicker.Marine.Cover;
                Robot.transform.Find("Corps").gameObject.GetComponent<SpriteRenderer>().sprite = cPicker.Marine.Head;
                break;

            case ColorSelection.Dusk:
                Robot.GetComponent<SpriteRenderer>().sprite = cPicker.Dusk.Chenilles;
                Robot.transform.Find("Middle").gameObject.GetComponent<SpriteRenderer>().sprite = cPicker.Dusk.Cover;
                Robot.transform.Find("Corps").gameObject.GetComponent<SpriteRenderer>().sprite = cPicker.Dusk.Head;
                break;

        }

    }```

Answer 1

You can reduce the complexity of the code by having the switch expression return then type that has the object with the values on it, and the code that sets the sprites. This helps reduce a good amount of duplication. I could not infer the type, so I've called it ColorType here, please rename to the type current of cPicker.Slime :

using System.Collections.Generic;
using UnityEngine;
using System;

public enum ColorSelection { Slime, Salmon, Marine, Dusk }

[Serializable] public class ColorSet { public ColorSelection unit; }

public class InputManager : MonoBehaviour
{
    public ColorPicker cPicker;

    public GameObject Robot;

    public ColorSet color;

    private ColorType GetColorType(ColorSelection colorSelection, ColorPicker colorPicker)
    {
        switch (colorSelection)
        {
            case ColorSelection.Slime:
                return colorPicker.Slime;
            case ColorSelection.Salmon:
                return colorPicker.Salmon;
            case ColorSelection.Marine:
                return colorPicker.Marine;
            case ColorSelection.Dusk:
                return colorPicker.Dusk;
            default:
                throw new ArgumentOutOfRangeException(nameof(colorSelection), colorSelection, null);
        }
    }

    public void Update()
    {
        var colorType = GetColorType(color.unit, cPicker);
        Robot.GetComponent<SpriteRenderer>().sprite = colorType.Chenilles;
        Robot.transform.Find("Middle").gameObject.GetComponent<SpriteRenderer>().sprite = colorType.Cover;
        Robot.transform.Find("Corps").gameObject.GetComponent<SpriteRenderer>().sprite = colorType.Head;
    }
}

as a bonus, in C# 8.0 this can be reduced to a switch expression, making it even more concise:

private ColorType GetColorType(ColorSelection colorSelection, ColorPicker colorPicker) =>
    colorSelection switch
    {
        ColorSelection.Slime => colorPicker.Slime,
        ColorSelection.Salmon => colorPicker.Salmon,
        ColorSelection.Marine => colorPicker.Marine,
        ColorSelection.Dusk => colorPicker.Dusk,
        _ => throw new ArgumentOutOfRangeException(nameof(colorSelection), colorSelection, null)
    };