Как можно улучшить этот код(ревью)?

Question

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

Вот код, что-то типо сборки пк. Сейчас таких скрипта 2, они похожи, но мне кажется это плохо. Как это можно улучшить?
Можно ли как то избавиться от тегов?

Код материнской платы

using UnityEngine;

public class MotherBoardPlace : MonoBehaviour
{

    Collision collision;

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

    private void Place(string PlaceName, Vector3 Angle)
    {
        collision.rigidbody.isKinematic = true;
        collision.rigidbody.detectCollisions = false;
        collision.rigidbody.useGravity = false;
        collision.transform.parent = transform.Find(PlaceName);
        collision.transform.localPosition = Vector3.zero;
        collision.transform.localEulerAngles = Vector3.zero + Angle;
        if (!Grabbing.GrabObj.collider.CompareTag("Motherboard"))
        {
            Grabbing.grab = false;
        }

    }
}

Код корпуса компа

using UnityEngine;

public class Place : MonoBehaviour
{
    Collision collision;
    private void OnCollisionEnter(Collision collision)
    {
        this.collision = collision;
        switch(collision.collider.tag)
        {
            case "Motherboard":
                place("MotherboardPlace", Vector3.zero);
                break;

            case "Power":
                place("PowerPlace", Vector3.zero);
                break;
            case "Hdd":
                place("HddPlace", Vector3.zero);
                break;
            case "Ssd":
                place("SsdPlace", Vector3.zero);
                break;
            case "PcDoor":
                place("PcDoorPlace", Vector3.zero);
                break;
        }
    }

    private void place(string PlaceName, Vector3 Angle)
    {
        collision.rigidbody.isKinematic = true;
        collision.rigidbody.detectCollisions = false;
        collision.rigidbody.useGravity = false;
        collision.transform.parent = transform.Find(PlaceName);
        collision.transform.localPosition = Vector3.zero;
        collision.transform.localEulerAngles = Vector3.zero + Angle;
    }
}

Answer 1

[GavriKos]

- не используйте Find и прочее
- нет констант - переименование тегов влечет редактирование кода
- именование и кодстайл так себе (метод place в классе Place)
- с архитекторуй да, так себе

Comments

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

А как избавиться от Find? У меня нету идей как избавиться от изменения в коде.

Answer 2

[profesor08]

Строчки в коде замени на enum
https://docs.microsoft.com/en-us/dotnet/csharp/lan...

Именование путает, непонятное.

Плохая архитектура привела к дублированию кода. Тебе надо взять объект и расположить в другом объекте, это можно вынести в отдельный код, который будет заниматься расположением одного объекта внутри другого. Сами объект должны предоставлять всю необходимую информацию, возможно их можно сделать инстансом одного класса, который характеризовал некий объект с именем и внутренними слотами, если они есть. Таким образом можно будет организовать любую вложенность и компоновку.

Comments

[GavriKos]

На енам не очень получится - отвалится CompareTag - юнитевая штука. Но ХОТЯ БЫ на константы - точно стоит.

Точнее так - на енам тоже можно - отказавшись вообще от тегов - но тогда весь код надо выкинуть и писать совсем по другому.

Answer 3

[Виктор П.]

Привет,
Насчёт архитектуры сложно, посмотрите паттерны, рекомендую сайт https://refactoring.guru/ru.
Дублированного кода не должно быть. Вынести можно несколькими способами, после ознакомления с паттернами, подберёте наиболее вам подходящий.
Магические текстовки - плохая практика. Замените хотя бы на константы, но лучше работайте с энумами.
Насчёт code style так же верно заметили. У вас один метод именуется с большой буквы, в другом классе метод с маленькой буквы - это бросается в глаза и оставляет отпечаток, что сделано "тяп-ляп", лабораторная работа номер два.
Есть спецификации языка, в которых описывается, как правильно внедрять новые фишки. На основе этого (или через боль поиска багов) внутри компаний формируются свои code style документы. Могу скинуть вам на почту свой документ, это очень полезная вещь

Comments

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

[e-mail]

[Виктор П.]

Дядя Богдан, отправил

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

Виктор П., Кстати не пришло

[Виктор П.]

Дядя Богдан, в отправленных висит. Кинул ещё раз

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

Виктор П., Редко почту смотрю, даже не заметил что она почему-то в самом низу.

[Виктор П.]

Дядя Богдан, ну как, полезно? )