Как сделать из такой системы что-то нормальное?

Question

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

Создал GridRepository:

public class StatisticRepository
{
    public int Coins;
}

И GridInteractor:

public class StatisticInteractor
{
    StatisticRepository _repository = new StatisticRepository();
    public int Coins => _repository.Coins;

    public delegate void OnCoinsValue();
    public event OnCoinsValue OnCoinsValueChanged;

    public void AddCoins(int value)
    {
        _repository.Coins += value;

        OnCoinsValueChanged?.Invoke();
    }

    public void SpentCoins(int value)
    {
        _repository.Coins -= value;

        OnCoinsValueChanged?.Invoke();
    }
}

Потом создал монетку:

using UnityEngine;

public class Coin : MonoBehaviour
{
    [SerializeField] private int _value;
    [SerializeField] private ParticleSystem _effect;

    private StatisticInteractor _statisticInteractor = new StatisticInteractor();

    private void OnTriggerEnter(Collider other)
    {
        if(other.GetComponent<PlayerMovement>())
        {
            _statisticInteractor.AddCoins(_value);
            Instantiate(_effect, transform.position, Quaternion.identity);

            Destroy(gameObject);
        }
    }
}

И проблема в том что у каждой монеты свое количество монет. Как сделать чтобы прибавлялся к "общему" экземпляру?

Потом в CoinsRender ничего не отображается.

using UnityEngine;
using TMPro;

public class CoinsRender : MonoBehaviour
{
    [SerializeField] private TMP_Text _coinsText;

    StatisticInteractor _statisticInteractor = new StatisticInteractor();

    private void OnEnable()
    {
        _statisticInteractor.OnCoinsValueChanged += RenderText;
    }

    private void OnDisable()
    {
        _statisticInteractor.OnCoinsValueChanged -= RenderText;
    }

    public void RenderText()
    {
        _coinsText.text = "Монетки: " + _statisticInteractor.Coins;
    }
}

Answer 1

[freeExec]

Создавать StatisticInteractor единожды, а не при каждом случае. Например как статический объект. Загуглите Singleton под Unity

Comments

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

Я слышал что статика это очень плохо и ей пользоватся ВООБЩЕ нельзя.

[freeExec]

Дядя Богдан, Ну не хочешь, не пользуйся, ни кто же не заставляет, пусть у тебя в каждой монетке по кошельку остаётся.

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

freeExec, Ок. Но из-за этого все равно есть проблема. Как мне поменять значение? Например я сейчас создал свойство Health и поле _health.

Но этот код выдает ошику, так как в статичное нельзя занести не статичное.

Что делать в этом случае, если должно быть начальное значение?

public class StatisticRepository : MonoBehaviour
{
    [SerializeField] private float _health;
    [SerializeField] private int _coins;

    public static float Coins
    {
        get { return _coins; }
        set
        {
            Coins -= value;

            if (Coins < 0)
            {
                Coins = 0;
            }
        }
    }

    public static float Health { 
        get { return _health; } 
        set { 
            Health -= value;

            if(Health < 0)
            {
                Health = 0;
            }
    }   }
}