LifeEXE | Unreal Engine | CG

👨‍💻 Всем вечерний привет! 👨‍💻

В новый Unreal Engine 5.7 завезли ИИ ассистента — плагин AIAssistant, который находится по адресу: Engine\Plugins\Experimental\AIAssistant

Плагин элементарный: он просто открывает веб-морду Epic Developer Assistant в отдельном окне редактора движка, и, кстати, работает без логина:

🔗

https://dev.epicgames.com/community/assistant/embedded

😐 Но код конечно интересный — пробежался по исходникам.

Всё выглядит так, что разработчики, создавая плагин для интеллекта искусственного решили пренебречь интеллектом естественным.

Лапидарно — несколько моментов, которые прямо бросаются в глаза 👀

🚨 Структура — всё в кучу, как в гостях у Облонских: и бизнес-логика, и Slate UI, и конфиги, и тесты.

💡 Один из способов организации кода плагина можно посмотреть в моём open-source UnrealOpenAIPlugin — простейшее разделение исходников на 3 модуля:

🔤Core классы — ядро
🔤Пользовательский интерфейс — Slate/UMG
🔤Тесты

🔴 Использование TSet в простейшем случае для 14 символов? Возможно, конечно автор изучал структуры данных на данном коде — кто знает:

FString FAIAssistantConfig::GetMainUrlAsRegexString() const
{
  static const TSet<FString::ElementType> CharactersToEscape{
    '.', '*', '+', '?', '(', ')', '[', ']', '{', '}', '^', '$', '|', '\\',
  };
  FString Escaped;
  Escaped.Reserve((MainUrl.Len() * 2) + 2 /* Allow for regex anchors */);
  Escaped += FString::ElementType('^');
  for (const FString::ElementType Character : MainUrl)
  {
    if (CharactersToEscape.Contains(Character))
    {
      Escaped += FString::ElementType('\\');
    }
    Escaped += Character;
  }
  Escaped += FString::ElementType('$');
  Escaped.Shrink();
  return Escaped;
}

💡

TSet тут избыточен — создаётся хеш-таблица с бакетами и метаданными для всего нескольких статических символов. Хватит switch/if или простейшей строки-набора:

static const FString CharsToEsc = TEXT(".+*?()[]{}^$|\\");

💡

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

🔴

FString::ElementType в данном случае только усложняет читаемость — можно смело использовать TCHAR.

🟢 Напоминаю, что тип FString имеет методы Append и AppendChar, которые аккуратно избавят от громоздких конструкций вида += FString::ElementType

⬇️

Escaped += FString::ElementType('^');

⬇️

Escaped.AppendChar('^');

💡 Сама функция GetMainUrlAsRegexString привязана к классу. Очевидно, её можно освободить или оформить как часть библиотеки утилит, передавая MainUrl параметром. Это разблокирует (помимо славы выдающегося архитектора ПО):

🔤Переиспользование
🔤Простоту тестирования — изолированные unit-тесты
🔤Композицию — можно комбинировать функции

🔤

Single Responsibility Principe (SRP) — принцип единственной ответственности: функция делает ровно свою работу, а не «висит» на конфиг-классе, к которому относится лишь косвенно

🔴 Тесты написаны в старых нотациях IMPLEMENT_SIMPLE_AUTOMATION_TEST, хотя Automation Spec уже давно доступен. Ничего критичного, но зачем тянуть старое в новом плагине.

💡

В курсе по тестированию на канале мы рассматриванием оба подхода.

🔤

⏺

🔤

🤝

Поддержать:
Patreon | Boosty | PayPal

⏫

Please open Telegram to view this post