Если не выносить в отдельный класс, то в GetModuleManager будет много методов

Нет)

Это говно собачье

Такое только для мелких задач можно делать

Ты вот сказал, что небезопасно поля отдавать, тебе твои сеттеры как вообще безопасность дадут?

Это не только небезопасно, но и нечитабельно

Это не обсуждается, я вопрос о другом ставил

У меня всегда вопросов больше становится от того, что я твой код вижу

Не могу удержаться

Моя реализация может быть выполнена аж 3мя способами: первый - это как сейчас я это реализовал, второе - через наследование, а третье - это класс Directories имеет статические методы и поля

Вот вопрос, когда какой случай выбирать?

Наследование вообще неуместно в данном случае, но сам факт такой реализаций возможен и поэтому оставил

В С++ некоторые вещи реализовать можно более чем одним способом, но как выбрать правильный? На чем нужно основываться? Стиль? Архитектура? Производительность?

1) Список директорий представляет собой аморфную сущность, которая может произвольно меняться в рантайме и требует сложной логики для синхронизации доступа.
2) Не используй блядь наследование вместо композиции, это рак мозга и делает из твоего кода лапшу.
3) Значения не изменяются в рантайме вовсе, либо задаются один раз при инициализации до того, как код начнет их использовать (важно задокументировать момент инициализации).

Только это самое

про пункт 3, класс там вообще выкинуть надо тогда

Понятие класса подразумевает создание его экземпляров

Статические поля предназначены, чтобы делиться данными между экземплярами не нарушая инкапсуляцию.