Как получить ссылку на Inventory (находится на префабе игрока) из InventoryGUI (висит на сцене в UI panel в Canvas)?

Question

[Vlad-1234567890]

На префабе игрока висит Inventory, а на сцене, в канвасе висит InventoryGUI. Как можно получить ссылку на Inventroy из InvantoryGUI или наоборот? Мне на ум пришло только сделать из канваса префаб и спавнить его из Inventory, есть ли другой способ?

Comments

[K0TlK]

Есть. Инициализировать все из одного места. Сделать точку входа в ней инициализировать всё (инвентарь, игрока и остальное) и прокинуть зависимости как вам угодно. Модель инвентаря (Inventory) вообще не должна быть MonoBehaviour и не должна зависеть ни от чего. Представление инвентаря(InventoryGUI) тоже не должно напрямую зависеть от модели, вместо этого создайте какой-нибуль класс, который будет связывать между собой модель и представление.

Answer 1

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

Доступ к статическому классу(или одиночке) можно получить из любой точки приложения, можно создать класс переходник в который ты запишешь нужную тебе информацию и из которого ты её получишь. Кстати статический класс и одиночка не одно и тоже. Статическими классами советуют не злоупотреблять, в юнити их не использовал, так что смотри сам.
Поиск объекта по ID или тегу и создание ссылки на него.
Это если ты хочешь сделать программно, в инспекторе можно просто перетащить объект с нужным компонентом в требуемое поле, если названия классов совпадает, то юнити сам всё сделает.

Comments

[K0TlK]

Что статический класс, что одиночка преследуют одну цель - дать доступ к объекту откуда угодно и пользовать его во все дыры. Синглтон в 99% случаев не нужен, достаточно нормально прокинуть зависимости. И этот случай не относится к 1%.
Поиск по тэгу и ID => использование строковых литералов => написать не "Inventory", а "Invetnory" => искать почему же у меня выбрасывает NullReferenceException.
Достаточно просто сделать точку входа, где будут инициализироваться все модули, компоненты и т.д. и нормально прокидывать зависимости, а не создавать синглтон-модель инвентаря, чтобы потом использовать ее из класса снежка, который можно подобрать в инвентарь.

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

K0TlK, Как сделать одну точку входа?

[K0TlK]

Элементарно.

public class Game : MonoBehaviour
{
    private SomeModule _someModule;
    private SomeModule2 _someModule2;

    private void Awake()
    {
        _someModule = new SomeModule();
        _someModule2 = new SomeModule2(_someModule);
    }
}

public class SomeModule2
{
    private SomeModule _someDependency;

    public SomeModule2(SomeModule _dependency)
    {
        _someDependency = _dependency;
    }
}

В коде сверху, класс Game будет точкой входа в приложение. Это единственный компонент на сцене, у которого есть метод Awake, остальные MonoBehaviour будут использовать метод Start. Поэтому инициализироваться он будет самым первым. Теперь можно изменять порядок загрузки модулей и прокидывать зависимости как захочется. В примере сверху, SomeModule2 зависит от SomeModule, поэтому SomeModule инициализируется первым, а SomeModule2 вторым и ему передается эта зависимость. Не надо плодить миллион Монобехов в качестве отдельных модулей, это приведет к тому, что вы будете делать их синглтонами, статиками и т.д. лишь бы был легкий доступ. А это, в свою очередь, приведет к тому, что компонент будет использоваться не так как предполагалось изначально. Зависимости в MonoBehaviour передавать либо через метод-инициализатор либо использовать инъекцию зависимостей через zenject.

[Vlad-1234567890]

я реализовал это как-то так

using UnityEngine;
using Photon.Pun;

public class SpawnPlayers : MonoBehaviour
{
    [SerializeField] private GameObject _playerPrefab;
    [SerializeField] private float _minX, _maxX, _minY, _maxY;
    [SerializeField] private InventoryGUI _playerInventoryGUI;
    private GameObject _player;

    void Start()
    {
        Vector2 _randomPosition = new Vector2(Random.Range(_minX, _maxX), Random.Range(_minY, _maxY));
        _player = PhotonNetwork.Instantiate(_playerPrefab.name, _randomPosition, Quaternion.identity);

        _playerInventoryGUI.targetInventory = _player.GetComponent<Inventory>();
    }
}

хотел спросить насколько это правильно/неправильно

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

K0TlK, Ваш комментарий является более правильным ответом чем мой, вы бы перенесли его в ответы)

[K0TlK]

Vlad-1234567890, Я не понимаю логики. Почему объект SpawnPlayers работает с UI игрока? Что он с этим ui делает? Этот класс не должен взаимодействовать с представлением инвентаря вообще никак. Никто об этом ui вообще знать не должен, кроме класса, который связывает модель инвентаря и сам ui. Взаимодействие происходит через этого посредника, он принимает команды, говорит модели сделать то что надо, модель обновляется, вызывает ивент, и представление обновляется. Что-то типа этого:

using System;
using System.Collections.Generic;

public class Model
{
    public event Action<int[]> ModelUpdated;

    private List<int> _integers = new List<int>();

    public void Add(int integer)
    {
        _integers.Add(integer);
        ModelUpdated?.Invoke(_integers.ToArray());
    }

    public void RemoveRandom()
    {
        var random = new Random();
        RemoveAt(random.Next(0, _integers.Count));
    }
    private void RemoveAt(int integer)
    {
        _integers.RemoveAt(integer);
        ModelUpdated?.Invoke(_integers.ToArray());
    }
}

Есть модель, которая ни от кого не зависит. Хранит в себе список цифр. Может добавлять или удалять цифру из этого списка.

public interface IIntegersView
{
    void UpdateView(int[] integers);
}

Есть контракт представления, чтобы можно было сделать несколько ui для этой модели.

using UnityEngine;
using UnityEngine.UI;

[RequireComponent(typeof(Text))]
public class ViewOne : MonoBehaviour, IIntegersView
{
    private Text _text;

    private void Start()
    {
        _text = GetComponent<Text>();
    }

    public void UpdateView(int[] integers)
    {
        var text = " ";
        foreach (var i in integers)
        {
            text += $"{i}, ";
        }
        _text.text = text;
    }
}

Есть одно представление, которое обязуется исполнить контракт. Представление - MonoBehaviour. Данное представление вешается на ui у которого есть компонент Text. Оно будет выводить на экран все цифры, когда модель обновляется.

using System.Linq;
using UnityEngine;
using UnityEngine.UI;

[RequireComponent(typeof(Text))]
public class ViewTwo : MonoBehaviour, IIntegersView
{
    private Text _text;

    private void Start()
    {
        _text = GetComponent<Text>();
    }

    public void UpdateView(int[] integers)
    {
        var group = integers.Where(x => x % 3 == 0).ToArray();
        var text = " ";
        foreach(var i in group)
        {
            text += $"{i}, ";
        }
        _text.text = text;
    }
}

Есть вторая вьюшка, которая выводит на экран цифры, которые делятся без остатка на 3.

public class Controller
{
    private Model _model;
    private IIntegersView _view1;
    private IIntegersView _view2;

    public Controller(IIntegersView view1, IIntegersView view2)
    {
        _view1 = view1;
        _view2 = view2;
        _model = new Model();
        _model.ModelUpdated += _view1.UpdateView;
        _model.ModelUpdated += _view2.UpdateView;
    }

    public void AddInteger(int integer) => _model.Add(integer);

    public void RemoveInteger() => _model.RemoveRandom();
}

Контроллер, который принимает вьюшки, инициализирует модель и подписывает методы обновления представлений на событие обновления модели. Позволяет обновлять модель посредством вызова ее методов.

using UnityEngine;
using Random = UnityEngine.Random;

public class Player : MonoBehaviour
{
    [SerializeField] private ViewOne _view1;
    [SerializeField] private ViewTwo _view2;

    private Controller _controller;

    private void Start()
    {
        _controller = new Controller(_view1, _view2);
    }
    private void Update()
    {
        if (Input.GetKeyDown(KeyCode.Space))
        {
            _controller.AddInteger(Random.Range(1, 100));
        }

        if (Input.GetKeyDown(KeyCode.LeftShift))
        {
            _controller.RemoveInteger();
        }
    }
}

И, собственно, игрок, которому нужно закинуть вьюшки в инспекторе. Инициализирует контроллер. Если нажать пробел, то цифра добавится, если нажать левый шифт, то удалится.
Как все это выглядит в игре

[Vlad-1234567890]

K0TlK, Спасибо огромное за такой развернутый ответ) Последний маленький вопрос, если игрок спавнится, то у нас нет возможности указать ему _view1 и _view2 из редактора. если передавать эти поля при спавне игроку, то получается что их надо будет сделать публичными(ну или сделать для них сеттер) и что класс, который их спавнит, должен будет знать о _view1 и _view2. Как быть?

[K0TlK]

Vlad-1234567890, Их надо передавать не игроку, а контроллеру, а контроллер уже передавать игроку. У меня в примере все через игрока, чтобы не генерировать еще больше текста.

public class GameCore : MonoBehaviour
{
    [SerializeField] private ViewOne _view1;
    [SerializeField] private ViewTwo _view2;
    [SerializeField] private Player _playerPrefab;

    private Controller _controller;

    private void Awake()
    {
        InitializeUI();
        InitializePlayer(_controller);
    }

    private void InitializePlayer(Controller controller)
    {
        var player = Instantiate(_playerPrefab);
        player.Initialize(controller);
    }

    private void InitializeUI()
    {
        _controller = new Controller(_view1, _view2);
    }
}

Компонент GameCore - точка входа, она инициализируется самой первой на сцене, поэтому у нее метод Awake, а не старт. Благодаря тому, что она инициализируется самой первой, мы можем контролировать порядок инициализации других модулей и прокидывать им зависимости. Допустим, здесь инициализируется игрок, но т.к. он зависит от контроллера, то контроллер инициализируется первым. У игрока, соответственно, должен быть метод Initialize который принимает зависимости в качестве параметров.

Пример с моделью, контроллером и представлением - это дефолтный MVC паттерн, можешь погуглить это семейство паттернов, помотреть на другие и выбрать тот, что удобнее тебе.