Base implementation #1

Merged
Dmitriy merged 9 commits from base-impl into master 2026-09-29 14:25:08 +00:00
3 changed files with 33 additions and 6 deletions
Showing only changes of commit e756bb1d73 - Show all commits
+2 -1
View File
@@ -22,7 +22,8 @@ public:
MscEngine(std::vector<std::shared_ptr<MscEngineModule>> modules);
MscEventBus &getBus() override;
UNIMPL getLoadedModules() const override;
std::unordered_map<std::string, std::uint16_t> getLoadedModules() const override;
MscEngineModule &getModule(const std::string &name) override;
};
#endif // __MCS_ENGINE_HPP__
+8 -3
View File
@@ -2,8 +2,10 @@
#define __MSC_ENGINE_MODULE_API_HPP__
#include <engine/core/engine_bus.hpp>
typedef void UNIMPL;
#include <engine/core/engine_module.hpp>
#include <cstdint>
#include <string>
#include <unordered_map>
class IMscEngineModuleAPI
{
@@ -14,7 +16,10 @@ public:
virtual MscEventBus &getBus() = 0;
Review

не слишком ли много ответственности для этого молодого человека? чет он дохера чего в себе делает и чувствую бесконечность не предел.

не слишком ли много ответственности для этого молодого человека? чет он дохера чего в себе делает и чувствую бесконечность не предел.
Review

Я вот думаю над тем, как правильнее реализовать. Конкретно этот класс - это интерфейс публичного API от движка к модулям. И пока архитектура у движка такая: движок реализует событийную шину и что-то типа фабрики БД-провайдеров, а всё остальное реализуется только модулями, которые подключаются в движок при его конструкте.

Пока не вижу идеи, как лучше сделать ;(

Я вот думаю над тем, как правильнее реализовать. Конкретно этот класс - это интерфейс публичного API от движка к модулям. И пока архитектура у движка такая: движок реализует событийную шину и что-то типа фабрики БД-провайдеров, а всё остальное реализуется только модулями, которые подключаются в движок при его конструкте. Пока не вижу идеи, как лучше сделать ;(
/// Get access to loaded module list
virtual UNIMPL getLoadedModules() const = 0;
virtual std::unordered_map<std::string, std::uint16_t> getLoadedModules() const = 0;
/// Access module by its name
virtual MscEngineModule &getModule(const std::string &name) = 0;
};
#endif // __MSC_ENGINE_MODULE_API_HPP__
+23 -2
View File
@@ -59,7 +59,28 @@ MscEventBus &MscEngine::getBus()
return _bus;
}
UNIMPL MscEngine::getLoadedModules() const
std::unordered_map<std::string, std::uint16_t> MscEngine::getLoadedModules() const
{
throw std::runtime_error("Unimplemented");
std::unordered_map<std::string, std::uint16_t> result;
for (const auto &module : _modules)
{
if (!module)
continue;
auto mod = module.get();
result[mod->getName()] = mod->getRev();
}
return result;
}
MscEngineModule &MscEngine::getModule(const std::string &name)
{
for (auto &module : _modules)
{
if (module.get()->getName() == name)
return *module.get();
}
throw std::runtime_error("Module not found: " + name);
}