Посоветуйте как оптимизировать

[Karmadon]

Переписываю чужой код и заодно учусь
встретил такое

SettingsFile ReadSettingsFile(const char *readingfile)
{
	ReadConfigFile cfg(readingfile);
	SettingsFile Config;
			
		if (cfg.keyExists("RunDay"))
			Config.iRunDay = cfg.getValueOfKey<int>("RunDay");
		else cout << "value Config.iRunDay: not found \n";
		if (cfg.keyExists("AgentDirSize"))
			Config.iAgentDirSize = cfg.getValueOfKey<int>("AgentDirSize");
		else cout << "value Config.iAgentDirSize: not found \n";
		if (cfg.keyExists("ExternalDiskCopyNumber"))
			Config.iExternalDiskCopyNumber = cfg.getValueOfKey<int>("ExternalDiskCopyNumber");
		else cout << "value Config.iExternalDiskCopyNumber: not found \n";

И так еще дофига копипастовых строчек
Это процедура чтения фала настроек
все данные сохраняются в структуре

struct SettingsFile
  { 
	bool bEncryptOutput; // Шифровать выходные файлы

    int iRunDay; //Запуск через iRunDay дней
	int iAgentDirSize; // Макс размер папки агента
	int iExternalDiskCopyNumber; // Колво копий на внешнем диске
	int iInternalDiskCopyNumber; // кол-во копий на внутр диск
	int iAlertLogMaximumSize; // Максимальный размер алертлога
	int iAlertLogWarningSize; // Критический размер алертлога
	int iListenerLogMaximumSize; // Максимальный размер листенера
	int iListenerLogWarningSize; // Критический размер листенера

И еще много строчек структуры
Нет ли возможности оптимизировать этот индусский код
поможет ли создание класса в этом случае?
Заранее спасибо

[Пепел Феникса]

int readSetting(ReadConfigFile &cfg, const char *paramName)
{
    if(cfg.keyExists(paramName))
        return cfg.getValueOfKey<int>(paramName);
    else std::cout<<"Value Config."<<paramName"<<": not found"<<std::endl;
    return -1;
}

SettingsFile ReadSettingsFile(const char *readingfile)
{
	ReadConfigFile cfg(readingfile);
	SettingsFile Config;
	Config.iRunDay = readSetting(cfg, "RunDay");
	Config.iAgentDirSize = readSetting(cfg, "AgentDirSize");
	Config.iExternalDiskCopyNumber = readSetting(cfg, "ExternalDiskCopyNumber");

так как значение обязано быть всегда(в переменной всмысле)
я принял -1 за пустое значение.

думаю намек ясен.

если там есть разные типы в структуре то можно еще и шаблонизировать.

[Karmadon]

Сообщение от Пепел Феникса

int readSetting(ReadConfigFile &cfg, const char *paramName)
{
    if(cfg.keyExists(paramName))
        return cfg.getValueOfKey<int>(paramName);
    else std::cout<<"Value Config."<<paramName"<<": not found"<<std::endl;
    return -1;
}

SettingsFile ReadSettingsFile(const char *readingfile)
{
	ReadConfigFile cfg(readingfile);
	SettingsFile Config;
	Config.iRunDay = readSetting(cfg, "RunDay");
	Config.iAgentDirSize = readSetting(cfg, "AgentDirSize");
	Config.iExternalDiskCopyNumber = readSetting(cfg, "ExternalDiskCopyNumber");

так как значение обязано быть всегда(в переменной всмысле)
я принял -1 за пустое значение.

думаю намек ясен.

если там есть разные типы в структуре то можно еще и шаблонизировать.

Приятное решение примерно такая идя была, но решение Ваше интереснее. да - там несколько типов данных string, int, bool, double , как шаблон написать не в курсе
Можно ли в масиве например описать все переменные
и потом в цикле подставить разом все из масива?

PS Вышеприведенные код уже переписан мною до этого имена переменных были var1,var2 итд коментов небыло вообще. Я вообще пхпист а тут нате на работе дали задачку перелопатить. спасибо за помощь

[Пепел Феникса]

template<typename T>
void readSetting(T &val, ReadConfigFile &cfg, const char *paramName)
{
    if(cfg.keyExists(paramName))
        val = cfg.getValueOfKey<T>(paramName);
    else std::cout<<"Value Config."<<paramName"<<": not found"<<std::endl;
}

SettingsFile ReadSettingsFile(const char *readingfile)
{
	ReadConfigFile cfg(readingfile);
	SettingsFile Config;
	readSetting(Config.iRunDay, cfg, "RunDay");
	readSetting(Config.iAgentDirSize, cfg, "AgentDirSize");
	readSetting(Config.iExternalDiskCopyNumber, cfg, "ExternalDiskCopyNumber");

поидее так.

Можно ли в масиве например описать все переменные
и потом в цикле подставить разом все из масива?

нечто подобное конечно можно сотворить, но это надо сразу делать(подобное видел в boost::fusion)

[Karmadon]

Спасибо, буду разбирать последний с шаблоном, ибо пока не все понятно как оно работает.

И небольшой оффтоп чтобы тему новую не создавать

cout << getenv("TEMP") << endl;
	cout << getenv("USERDOMAIN") << endl;

Есть ли проще варианты? и как ее(переменную окружения) в переменную запхнуть

[Пепел Феникса]

Есть ли проще варианты?

куда проще то?
итак одним вызовом получаете.

и как ее(переменную окружения) в переменную запхнуть

char *pTemp=getenv("TEMP");
//или
std::string pTemp(getenv("TEMP"));

только помним, что в первом случае её изменять нельзя.

[Karmadon]

Сообщение от Пепел Феникса

куда проще то?
итак одним вызовом получаете.

Это первая функция которую нарыл и она оказалась самой простой, просто может что еще более удобное есть))

Сообщение от Пепел Феникса

char *pTemp=getenv("TEMP");
//или
std::string pTemp(getenv("TEMP"));

только помним, что в первом случае её изменять нельзя.

ну вот а я по первому примеру чуть голову себе не проломил...
Второй вариант - то что надо огромное спасибо!!!
Но по шаблонам пошел маны читать))

[Karmadon]

Если не затруднит, опишите в чем различие реализаций описанных выше - через функцию и с помощью шаблона
Что-то никак допетрить не могу

PS - Код функции написанные с помощью шаблона работает прекрасно спасибо, но хотелось бы понять как)))

[Пепел Феникса]

в данном случае компилятор способен понять(без явного указания) какой тип мне нужен.
то есть вместо T ставиться тип переменной что передаете(в моих примерах это int)

[Karmadon]

Сообщение от Пепел Феникса

в данном случае компилятор способен понять(без явного указания) какой тип мне нужен.
то есть вместо T ставиться тип переменной что передаете(в моих примерах это int)

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