Нормально ли разбивать все на такое количество методов?

Question

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

Есть вот такой код:

Код

using UnityEngine;

public class CharactersSelector : MonoBehaviour
{
    [SerializeField] private Transform _leftCharacterParent;
    [SerializeField] private Transform _rightCharacterParent;

    [SerializeField] private UICharacter _startUICharacter;

    private void Awake()
    {
        SelectCharacter(_leftCharacterParent, _startUICharacter);
        SelectCharacter(_rightCharacterParent, _startUICharacter);
    }

    public void SelectFirstCharacter(UICharacter UICharacter) => SelectCharacter(_leftCharacterParent, UICharacter);
    public void SelectSecondCharacter(UICharacter UICharacter) => SelectCharacter(_rightCharacterParent, UICharacter);

    private void SelectCharacter(Transform parent, UICharacter UICharacter)
    {
        ClearParentChilds(parent);

        InstantiateCharacterOnParent(parent, UICharacter.gameObject);
    }

    private void ClearParentChilds(Transform parent)
    {
        foreach(Transform child in parent)
        {
            Destroy(child.gameObject);
        }
    }

    private void InstantiateCharacterOnParent(Transform parent, GameObject UICharacterGameObject) => Instantiate(UICharacterGameObject, parent);
}

Плохо ли что тут большое количество методов? Или так наоборот лучше? Один мне говорит что нужно чтобы методов было мало, а другой что много.

Я понимаю что лучше много, но у меня методы внутри которых 1 строчка кода. Можно написать и без метода?

Answer 1

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

выглядит хорошо, за исключением неиспользуемых методов.
Лучше так, чем нагромождение кода.

Один мне говорит что нужно чтобы методов было мало, а другой что много.

А почему тогда у этих же людей не спросили, почему они так говорят?

Comments

[edward_freedom]

тебе точно ревьювить чужие коды не стоит https://qna.habr.com/answer?answer_id=2049444#answ...

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

edward_freedom, а что именно не так в коде по ссылке? Он вполне рабочий, делает ровно то, что от него требовалось, покрыт тестами, и в принципе не нарушает каких-то общепринятых вещей.
Самое большое зло - то что он реализован на листах, а не на ienumerable, и что имя параметра не очень.

[edward_freedom]

Василий Банников, на какой версии фрейма, string.Join(' ', x)) принимает первым параметром чар?
Я проверил у себя на последней версии доступной у меня (4.8), все перегрузки джойна со стрингом

[ComVisible(false)]
        public static String Join<T>(String separator, IEnumerable<T> values);
        [ComVisible(false)]
        public static String Join(String separator, IEnumerable<String> values);
        [SecuritySafeCritical]
        public static String Join(String separator, String[] value, int startIndex, int count);
        public static String Join(String separator, params String[] value);
        [ComVisible(false)]
        public static String Join(String separator, params object[] values);

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

edward_freedom, тестил на .net 5 - есть перегрузки для char

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

edward_freedom,
вот ссылка на доку: https://docs.microsoft.com/ru-ru/dotnet/api/system...

[Иван Спасов]

edward_freedom, не обижайте Василия Банникова,
вот ваш ответ, выглядит как примитив спагетти, словно не существует MinMax методов https://qna.habr.com/q/1031454#answer_1999844

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

Неиспользуемые методы - это методы с которыми я работаю в юнити. Поэтому они публичные. А метод Awake встроенный в Unity и вызывается при старте игры. Поэтому лишних методов тут нет)

[edward_freedom]

Иван Спасов, я хотел именно донести логику решения задачи в том случае, а не дать уже готовое решение

Answer 2

[Некит Медведев]

я обычно разделяю на классы и делю их на слои, нижний слой расписываю на объекты по максимуму, слои выше, группируют объекты(используют нижние,как библиотеки) по итогу создав единожды нижний слой, можно на него не отвлекаться и использовать методы классов верхнего уровня.

Comments

[Некит Медведев]

но в этом случае, стоит подумать об одиночках

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

Некит Медведев, Вся статика - это говнокод. Лучше для этого использовать Zenject и архитектуру SCA