Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Система пробивания препятствий пулями #26

Draft
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

FireNameFN
Copy link
Collaborator

Ссылка на ветку

https://discord.com/channels/919301044784226385/1303634507383181377

Технические детали

Изменён Server/PenetrationSystem
Добавлен компонент Penetratable
Добавлен PenetrationPower в Projectile

Список изменений

🆑

  • tweak: Теперь некоторые пули будут простреливать большинство объектов насквозь с некоторым шансом.

@@ -18,9 +19,14 @@ public sealed class ProjectileSystem : SharedProjectileSystem
[Dependency] private readonly GunSystem _guns = default!;
[Dependency] private readonly SharedCameraRecoilSystem _sharedCameraRecoil = default!;

private EntityQuery<PenetratableComponent> _penetratableQuery;

private Random _random = new();
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Почему не IRobustRandom, который предоставляется нам через зависимости в IoC

@@ -60,6 +60,8 @@
components:
- type: Damageable
damageContainer: Biological
- type: Penetratable
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Подобные изменение должны иметь комментарий, если вы меняете прототипы официального контента

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Этот компонент и так новый. Мы договорились не помечать в прототипах новые компоненты, так как и так понятно, что он не оффами добавлен.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Гадэм, в любом случае это будет на совести тех, кто делает апстртмы, чем я не занимаюсь

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Этот компонент и так новый. Мы договорились не помечать в прототипах новые компоненты, так как и так понятно, что он не оффами добавлен.

чзх, добавляй

@@ -67,9 +73,19 @@ private void OnStartCollide(EntityUid uid, ProjectileComponent component, ref St
_sharedCameraRecoil.KickCamera(target, direction);
}

component.DamagedEntity = true;
// Corvax-Next-Penetration
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Надо открыть и закрыть подобный комментарий, для удобства

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants