Всем привет! Знакомился с ScriptableObject по этой ( https://habr.com/ru/post/421523/ )

статье и там меня не устроила система ивентов (с которой я сталкивался тоже впервые), поскольку была очень уже непрактичная, решил переделать. Может кто сделать код ревью?
Репозиторий работы: https://github.com/Dima34/ScriptableObjecs-Refactored
Скрипты ,которые писал:
https://github.com/Dima34/ScriptableObjecs-Refactored/blob/main/Assets/RW/Scripts/GameEventListener.cs
https://github.com/Dima34/ScriptableObjecs-Refactored/blob/main/Assets/RW/Scripts/MerchantUI.cs
https://github.com/Dima34/ScriptableObjecs-Refactored/blob/main/Assets/RW/Scripts/Sword.cs
https://github.com/Dima34/ScriptableObjecs-Refactored/blob/main/Assets/RW/Scripts/ScriptableObjects/GameEvent.cs

4 ответов

6 просмотров

От того что ты заинкапсулировал некоторые поля, ты не избавился от самаой системы. А вся проблема щаключается что там используется UnityEvent через инспектор.

Dima- Автор вопроса
Mikhail Suvorov
От того что ты заинкапсулировал некоторые поля, ты...

Это считается плохо? Какие есть альтернативы?

Dima- Автор вопроса

Асtion?*

Dima
Асtion?*

да UnityAction менее производительнее чем Action/Func/Delegate

Похожие вопросы

Обсуждают сегодня

Добрый вечер, Пока не совсем понимаю как наладить общение между телеграм ботом и ПО для работы с сим боксом. По самому боту так понял: - Нужен некий баланс, который можно поп...
Magic
6
сделал сайт, прикрутил в боте сайт, и виджет логина. как автоматически логинить пользователя в аккаунт(телеграм), при входе с бота?
Александра Чернивецкая
5
Объясните, пожалуйста, почему компилятор ругается на использование в условии неинициализированной переменной: int x; Task.Run(async () => { x = await somefunc(); }).Wait...
Александр
5
Ребят, подскажите, пожалуйста, почему в префиксе к ассетам, которые генерируются через фильтр | theme в шаблоне, стал вдруг появляться index.php? Вот так выглядит ссылка на а...
Виталий
1
Всем привет. Ребята, подскажите, пожалуйста. у ботов есть ограничение на отправку сообщений - 30 сообщений в секунду, эти ограничения накладываются на все сообщения? или на со...
Artem Stormageddon
4
Блин, ребята, сори за тупые вопросы. А можно ли как-то открыть вебапку по нажатию на кнопку в меню(которое появляется слева, команды)?
Artem Stormageddon
3
а плаксы из-под питона умеют только в комфортных условиях что-то выдавить из себя?)
Lencore
9
Но, может, есть уже проверенная? Наши требования такие: 1. Сообщения должны приходить из Инста в CRM оду 2. Должна быть возможность подключить несколько экаунтов Инстаграм. Р...
Alexander Sharoiko MSE / Александр Шаройко
13
Это может быть все-таки не флудвейт? у меня ботфазер принимает изменения и отображает даже что они изменились, на видео видно что он прислал якобы уже измененное описание, н...
OVERLINK
13
Коллеги, может знает кто, можно ли цвет бейджа счётчика в BackendMenu менять без бубнов?
Alex Blaze
3
Карта сайта