Skip to content

Conversation

@andrey-sterkhov
Copy link

исправил конфигурирование проекта

Nyarstot added a commit to Nyarstot/NauEngine that referenced this pull request Dec 3, 2024
@Kostec
Copy link

Kostec commented Feb 27, 2025

Та же проблема решается в этом pull request. Однако тут ещё и добавлена проверка на nullptr в коде. Кто-то смотрит правки чтобы сделать мерж в main?

@Kostec
Copy link

Kostec commented Feb 27, 2025

Решает ещё и #19.
Может глупая идея, но что если при вызове getServiceProvider().find<ICoreGraphics>() будет вместо null возвращать NullObject. Для этого должен быть реализован класс NullableCoreGraphics от ICoreGraphics, который может быть затычкой на случай null. Это поможет повысить стабильность движка и избавить разработчиков от проверки на null.

А в целом решение в main идёт против рекомендаций оформления кода:

  1. Используется raw pointer, не уверен в происходящем (только узнал о проекте и решил внести немного активности), но есть ощущение что должен быть std::shared_pointer, в противном случае где-то гонка потоков и необходимо её устранить
  2. Используется auto. Так себе рекомендация, но раз code style просит избегать, то можно избежать

@kapc3d @K-Tone Вы тут единственные контрибьюторы, пожалуйста, подайте знак жив ли проект.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants