Как сделать из этого не говнокод?

Question

[Дядя Богдан]

Вот код чтобы типо комп собирался. как его улучшить?

Говнокод

using UnityEngine;

public class MotherBoardPlace : MonoBehaviour
{
    private void OnCollisionEnter(Collision collision)
    {
        switch(collision.collider.tag)
        {
            case "Processor":

                collision.rigidbody.isKinematic = true;
                collision.rigidbody.detectCollisions = false;
                collision.rigidbody.useGravity = false;
                collision.transform.parent = transform;
                collision.transform.eulerAngles = new Vector3(0, 0, 0);
                collision.transform.localPosition = transform.Find("ProcessorPlace").localPosition;
                break;
            case "ram":

                collision.rigidbody.isKinematic = true;
                collision.rigidbody.detectCollisions = false;
                collision.rigidbody.useGravity = false;
                collision.transform.parent = transform;
                collision.transform.eulerAngles = new Vector3(-90, 0, 90);
                collision.transform.localPosition = transform.Find("RamPlace").localPosition;

                break;
        }
    }
}

Comments

[Василий Банников]

Начать можно с замены тегов на компоненты

[Александр Синицын]

Если кто-то сказал, что это говнокод... забей? Если сам видишь, то сам должен понимать, что именно напрягает... берешь и делаешь рефакторинг напрягов )

[i__egor]

А чего тебе не нравится здесь? Ну я бы в юнити функциях не стал бы писать кучу всего а выносил бы в отдельные функции

Answer 1

[cicatrix]

DRY = Don't Repeat Yourself (не повторяйся)
Как минимум вынеси в отдельный модуль присваивание свойств. Настрой умолчания, чтобы каждый раз не присваивать. Замени строковые тэги строковыми константами.

Comments

[Дядя Богдан]

Типо так?

Текущий вариант

using UnityEngine;

public class MotherBoardPlace : MonoBehaviour
{

    Collision collision;

    private void OnCollisionEnter(Collision collision)
    {
        this.collision = collision;
        switch(collision.collider.tag)
        {
            case "Processor":
                Place("ProcessorPlace", new Vector3(0, 0, 0));
                break;
            case "ram":
                Place("RamPlace", new Vector3(-90, 0, 90));
                break;
        }
    }

    private void Place(string TagName, Vector3 Angle)
    {
        collision.rigidbody.isKinematic = true;
        collision.rigidbody.detectCollisions = false;
        collision.rigidbody.useGravity = false;
        collision.transform.parent = transform;
        collision.transform.eulerAngles = Angle;
        collision.transform.localPosition = transform.Find(TagName).localPosition;
    }
}