Вопрос Проверка на то, вошла уже сущность в стрим или нет

[E-Exception]

Всем привет, написал для себя небольшие удобные методы расширения для класса Entity. Суть кода в том, что можно локально привязать лямбда - функции для мгновенного выполнения при входе в стрим сущности. (Кому мб пригодится может даже )

Код:

public static void OnStreamEvent_Add(this Entity e, Action handler)
{
        List<Action> Actions = e.HasData("OnStreamEvents") ? e.GetData<List<Action>>("OnStreamEvents") : new List<Action>();
        Actions.Add(handler);
        e.SetData<List<Action>>("OnStreamEvents", Actions);
  }
//Another class
RAGE.Events.OnEntityStreamIn += (e) =>
{
          if (e.HasData("OnStreamEvents"))
          {
                e.GetData<List<Action>>("OnStreamEvents").ForEach((action) => { action?.Invoke(); });
          }
  };

Вот так используется в коде

Код:

var ped = new Ped(hash, veh.Position, 0, veh.Dimension);

ped .OnStreamEvent_Add(() =>
{
    ped .SetAlpha(1, false);
}

Но такая проблема есть, даже смешно. Необходимо в методе OnStreamEvent_Add добавить проверку на то, что сущность УЖЕ существует в стриме и вызвать хранимые функции.
Сейчас получается так, то сначала вызывается событие OnEntityStreamIn вызывая пустые функции, а затем только они добавляются... странно. Какими методами это сделать?

 

[E-Exception]

Самый тупой способ это проверка на содержание сущности в RAGE.Game.Entities. ... .Streamed, но не очень хочется захламлять код тупыми проверками на тип сущности..

 

[MoonFusion]

E-Exception написал(а):

Всем привет, написал для себя небольшие удобные методы расширения для класса Entity. Суть кода в том, что можно локально привязать лямбда - функции для мгновенного выполнения при входе в стрим сущности. (Кому мб пригодится может даже )

Код:

public static void OnStreamEvent_Add(this Entity e, Action handler)
{
        List<Action> Actions = e.HasData("OnStreamEvents") ? e.GetData<List<Action>>("OnStreamEvents") : new List<Action>();
        Actions.Add(handler);
        e.SetData<List<Action>>("OnStreamEvents", Actions);
  }
//Another class
RAGE.Events.OnEntityStreamIn += (e) =>
{
          if (e.HasData("OnStreamEvents"))
          {
                e.GetData<List<Action>>("OnStreamEvents").ForEach((action) => { action?.Invoke(); });
          }
  };

Вот так используется в коде

Код:

var ped = new Ped(hash, veh.Position, 0, veh.Dimension);

ped .OnStreamEvent_Add(() =>
{
    ped .SetAlpha(1, false);
}

Но такая проблема есть, даже смешно. Необходимо в методе OnStreamEvent_Add добавить проверку на то, что сущность УЖЕ существует в стриме и вызвать хранимые функции.
Сейчас получается так, то сначала вызывается событие OnEntityStreamIn вызывая пустые функции, а затем только они добавляются... странно)

Нажмите для раскрытия...

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

 

[MoonFusion]

Ну и плюс не e в лямбда функции, а полный нейм entity, что бы было понятно с чем ты взаимодействуешь

 

[E-Exception]

Без оффтопа мб?

 

[Mr.Scalise]

E-Exception написал(а):

Без оффтопа мб?

Нажмите для раскрытия...

А где он заоффтопил?

 

[E-Exception]

Я ответа на вопрос не получил. Получил только советы для улучшения качества написания кода.

Mr.Scalise написал(а):

А где он заоффтопил?

Нажмите для раскрытия...

.

 

[E-Exception]

Я, конечно, благодарен ему, но по-сути вопрос не был решен)

UPD: Возможно вопрос не очевидный, поправил немного, мой косяк

 

[MoonFusion]

E-Exception написал(а):

Я, конечно, благодарен ему, но по-сути вопрос не был решен)

UPD: Возможно вопрос не очевидный, поправил немного, мой косяк

Нажмите для раскрытия...

Да изначально и не понял что тут был вопрос, но кстати все ещё не до конца понял что ты хочешь сделать, можешь перефразировать вопрос?

 

[E-Exception]

MoonFusion написал(а):

Да изначально и не понял что тут был вопрос, но кстати все ещё не до конца понял что ты хочешь сделать, можешь перефразировать вопрос?

Нажмите для раскрытия...

Короче, как проверить сущность в данный момент в зоне стрима или нет?

 

[XDeveluxe]

E-Exception написал(а):

Короче, как проверить сущность в данный момент в зоне стрима или нет?

Нажмите для раскрытия...

Сущность в зоне стрима имеет Handle != 0. Вне стрима она всегда будет = 0.

 

[E-Exception]

xDeveluxe написал(а):

Сущность в зоне стрима имеет Handle != 0. Вне стрима она всегда будет = 0.

Нажмите для раскрытия...

Беда в том, что у типа Enitity нет свойства Handle :c

 

[XDeveluxe]

E-Exception написал(а):

Беда в том, что у типа Enitity нет свойства Handle :c

Нажмите для раскрытия...

А у Player? Если у Player есть и тебе нужно проверять только игроков, то проверяй на entity.Type, если != Player, return. А после сделай каст:

C#:

Player target = (Player)entity;
// или
Player target = entity as Player;

 

[E-Exception]

xDeveluxe написал(а):

А у Player? Если у Player есть и тебе нужно проверять только игроков, то проверяй на entity.Type, если != Player, return. А после сделай каст:

C#:

Player target = (Player)entity;
// или
Player target = entity as Player;

Нажмите для раскрытия...

Я планировал сделать эти расширения общими для Entity. Но видимо прийдётся делать проверки на тип(

 

[MoonFusion]

А в чем проблема ручками на клиенте у игрока создать лист entity которые у него в стриме?

 

[MiLT]

MoonFusion написал(а):

А в чем проблема ручками на клиенте у игрока создать лист entity которые у него в стриме?

Нажмите для раскрытия...

И после этого у тебя еще роль high coder

Я планировал сделать эти расширения общими для Entity. Но видимо прийдётся делать проверки на тип(

Нажмите для раскрытия...

Не нужны проверки. То, что у Entity в шарпах нет .handle, это скорее всего просто недоработка шарповского API, на жсовском клиенте у всех есть .handle. Не припомню конечно, чтобы в шарпах были возможности расширить класс пропертями извне, поэтому можешь просто спокойно делать проверку на (entity as PlayerMp).Handle != 0. Выглядит костыльно, но других вариантов особо нет

UPD: Загуглил, оказывается в шарпах as нельзя делать с несовместимыми типами (ещё один pros в пользу тс)

 

[MoonFusion]

Ну и уже от этого реализовать проверку

MiLT написал(а):

И после этого у тебя еще роль high coder



Не нужны проверки. То, что у Entity в шарпах нет .handle, это скорее всего просто недоработка шарповского API, на жсовском клиенте у всех есть .handle. Не припомню конечно, чтобы в шарпах были возможности расширить класс пропертями извне, поэтому можешь просто спокойно делать проверку на (entity as PlayerMp).Handle != 0. Выглядит костыльно, но других вариантов особо нет

UPD: Загуглил, оказывается в шарпах as нельзя делать с несовместимыми типами (ещё один pros в пользу тс)

Нажмите для раскрытия...

пососи лох безработный

 

[XDeveluxe]

MiLT написал(а):

Загуглил, оказывается в шарпах as нельзя делать с несовместимыми типами (ещё один pros в пользу тс)

Нажмите для раскрытия...

Для этого есть "is".

C#:

if (entity is not PlayerMp target) return;
// или
if (!(entity is PlayerMp target)) return;

 

[MiLT]

xDeveluxe написал(а):

Для этого есть "is".

C#:

if (entity is not PlayerMp target) return;
// или
if (!(entity is PlayerMp target)) return;

Нажмите для раскрытия...

is для проверки типов в рантайме, не понимаю к чему ты это написал. Да и к тому же as эквивалентно выражению E is T ? (T)E : (T)null
А если и проверять типы у ентити, то лучше через entity.type

 

[MoonFusion]

MiLT написал(а):

is для проверки типов в рантайме, не понимаю к чему ты это написал. Да и к тому же as эквивалентно выражению E is T ? (T)E : (T)null
А если и проверять типы у ентити, то лучше через entity.type

Нажмите для раскрытия...

Трахнул