kghackers / kgparser

Statistics for https://klavogonki.ru/. Also contains code for legacy competitions parser.
https://www.klavostat.com
4 stars 3 forks source link

Конфигурации топов словарей должны читаться из файла, а не из кучи классов #98

Open dmitry-weirdo opened 1 week ago

dmitry-weirdo commented 1 week ago

Конфигурация вроде OneHundredRussianTopExporter — это избыточный оверкилл. Логично читать подобные конфигурации из json-файла, чтобы настраивать словари можно было более-менее динамически, не меняя код.

Также надо подумать, как включать/выключать словари в конфиге вроде statistics-generator-config-generate-all.json (класс StatisticsGeneratorConfig).

Аналогично для NonStandardDictionary. NonStandardDictionary нужно оставить тем же объектом, но читать из файла вместо хардкодинга. Получать из кэша по коду словаря (числовому айдишнику)

Более того, полный список словарей потом можно будет загрузить из csv-файла Переборыча. Но там нет названия в родительном падеже. Но их можно загружать отдельно, уже из внешнего файла (необходимо только для словарей, которые используются в топах).

Также, это избавит от необходимости придумывать имя для значения энума, как сейчас в NonStandardDictionary.