MyWackoSite : Письмо1

Решение задачи требует доработки, см. /SpeckursCxx/Zadachi/ObshhieTrebovanija. Все пункты: Оформление, Сборка, Исполнение.

Некоторые дополнительные замечания (что успел просмотреть):

• Используйте std::stringstream или boost::lexical_cast вместо sprintf, atof и т.п.
• Используйте умные указатели (это из области безопасности исключений, о чем я планирую много говорить :) :

ConfigData* result = new ConfigData(); while(!ifs.eof()) { ifs.getline(buf, BUF_SIZE); processLine(buf, &currentTag, result); } ifs.close(); // если до этого момента произойдет исключение, ConfigData не будет уничтожен. // можно использовать std::auto_ptr или boost::shared_ptr return result;

• Здесь можно положиться на деструктор:

ofstream ofs; ofs.open(fileName.c_str()); // ... ofs.flush(); ofs.close(); // не нужно

• Старайтесь использовать std::string и его интерфейс вместо строк в стиле C:

char * pos = strchr(line, ']'); // лучше std::string::find_first_of ifs.getline(buf, BUF_SIZE); // что будет если строка будет длинее BUF_SIZE? void conf::INIConfigReader::processLine(const char * line, string* currentTag, ...) // string& currentTag

• Использование typedef вам бы облегчило жизнь при написании ConfigData.

Антон Дедов