Модераторы: Daevaorn

Поиск:

Ответ в темуСоздание новой темы Создание опроса
> Парсер ini файлов 
V
    Опции темы
Cheloveck
Дата 23.6.2009, 21:41 (ссылка) |    (голосов:4) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
***


Профиль
Группа: Завсегдатай
Сообщений: 1578
Регистрация: 26.7.2008
Где: Тула

Репутация: 3
Всего: 32



Надоело, что виндовые функции семейства *PrivateProfile* каждый раз открывают и читают файл. К тому же функционал очень сомнительный. Написал свою кроссплатформенную версию парсера ini файлов. В инете видел один примерчик, написанный на boost.spirit, но он меня не впечатлил, болше ничего хорошего не находил. Мой парсер использует объектную модель документа (DOM) и способен загружать файл как целиком, так и по частям (равно как и записывать). Это мой первый опыт написания открытого кода, так что необессудте. Сообщайте о любых багах и недочётах в ЛС по e-mail или на jabber, можно и в эту тему.


P.S. Тему можно удалить, исходники убрал

Это сообщение отредактировал(а) Cheloveck - 25.6.2009, 15:01


--------------------
user posted image
PM Jabber   Вверх
zim22
Дата 23.6.2009, 22:05 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


depict1
****


Профиль
Группа: Завсегдатай
Сообщений: 2682
Регистрация: 15.1.2009
Где: Украина

Репутация: 24
Всего: 69



Цитата(Cheloveck @  23.6.2009,  21:41 Найти цитируемый пост)
 Сообщайте о любых багах и недочётах... можно и в эту тему.

мне в операторе присваивания не понятно несколько моментов:
1) почему возвращается константная ссылка?
2) будет ли ваш код работать правильно, если я отключу assert с помощью #define NDEBUG

Выделить всёкод C++
1:
2:
3:
4:
5:
6:
7:
8:
9:
10:
const section & section::operator=( const section & rhs )
{
    ifthis == &rhs )
        return *this;
    delete records_;
    assert( records_ = new std::vector<record>( *rhs.records_ ) ); // ???
    name_ = rhs.name_;
    return *this;
}




--------------------
PM MAIL   Вверх
Cheloveck
Дата 23.6.2009, 22:11 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
***


Профиль
Группа: Завсегдатай
Сообщений: 1578
Регистрация: 26.7.2008
Где: Тула

Репутация: 3
Всего: 32



Цитата(zim22 @  23.6.2009,  23:05 Найти цитируемый пост)
1) почему возвращается константная ссылка?

да, интересно, почему?)) Исправлю.

Цитата(zim22 @  23.6.2009,  23:05 Найти цитируемый пост)
2) будет ли ваш код работать правильно, если я отключу assert с помощью #define NDEBUG

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


--------------------
user posted image
PM Jabber   Вверх
zim22
Дата 23.6.2009, 22:13 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


depict1
****


Профиль
Группа: Завсегдатай
Сообщений: 2682
Регистрация: 15.1.2009
Где: Украина

Репутация: 24
Всего: 69



Цитата(Cheloveck @  23.6.2009,  22:11 Найти цитируемый пост)
ну, почему бы он и не будет работать? assert - это макрос, и его отключение ни на что не повлияет

повлияет. если логика кода находилась внутри этого макроса. код будет выглядеть так:
Выделить всёкод C++
1:
2:
3:
4:
5:
6:
7:
8:
9:
10:
const section & section::operator=( const section & rhs )
{
    ifthis == &rhs )
        return *this;
    delete records_;
    //assert( records_ = new std::vector<record>( *rhs.records_ ) ); 
    name_ = rhs.name_;
    return *this;
}



--------------------
PM MAIL   Вверх
Cheloveck
Дата 23.6.2009, 22:18 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
***


Профиль
Группа: Завсегдатай
Сообщений: 1578
Регистрация: 26.7.2008
Где: Тула

Репутация: 3
Всего: 32



zim22, спасибо, поправлю. Да, действительно, что-то такое было у майерса

Добавлено через 5 минут и 20 секунд
Поправил


--------------------
user posted image
PM Jabber   Вверх
Lazin
Дата 23.6.2009, 22:26 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
****


Профиль
Группа: Завсегдатай
Сообщений: 3820
Регистрация: 11.12.2006
Где: paranoid oil empi re

Репутация: 41
Всего: 154



Цитата(Cheloveck @  23.6.2009,  21:41 Найти цитируемый пост)
 В инете видел один примерчик, написанный на boost.spirit, но он меня не впечатлил, болше ничего хорошего не находил.

а зря, если нужно что-то распарсить, то лучше использовать spirit smile 
PM MAIL Skype GTalk   Вверх
Cheloveck
Дата 23.6.2009, 22:29 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
***


Профиль
Группа: Завсегдатай
Сообщений: 1578
Регистрация: 26.7.2008
Где: Тула

Репутация: 3
Всего: 32



Lazin, не впечатлил не spirit, а примерчик.


--------------------
user posted image
PM Jabber   Вверх
nerezus
Дата 23.6.2009, 22:36 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Вселенский отказник
****


Профиль
Группа: Участник
Сообщений: 3330
Регистрация: 15.6.2005

Репутация: 3
Всего: 43



Цитата

а зря, если нужно что-то распарсить, то лучше использовать spirit 
 На него выливали кучу вонючего коричневого по отношению к flex/bison из-за compile-time всего.


--------------------
Сообщество художников Artsociety.ru
PM MAIL WWW   Вверх
azesmcar
Дата 23.6.2009, 22:39 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


uploading...
****


Профиль
Группа: Участник Клуба
Сообщений: 6291
Регистрация: 12.11.2004
Где: Армения

Репутация: 81
Всего: 211



Cheloveck

Меня больше всего вот этот момент забеспокоил
Выделить всёкод C++
1:
2:
3:
4:
5:
6:
7:
8:
const std::string iniparser::loadRecord( const std::string & filename,
                const std::string & section_name,
                const std::string & key )
{
    std::ifstream input( filename.c_str() );
    if( !input.is_open() )
        return NULL;
...

return NULL;
как это работает. Вызывается конструктор принимающий const char*.
Каким образом будете проверять что функция сработала нормально? Ни одного удовлетворяющего стандарту метода я не вижу.
Да и вообще, скорее всего еще при возврате значения вылетит.

Это сообщение отредактировал(а) azesmcar - 23.6.2009, 22:43
PM   Вверх
Cheloveck
Дата 23.6.2009, 22:42 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
***


Профиль
Группа: Завсегдатай
Сообщений: 1578
Регистрация: 26.7.2008
Где: Тула

Репутация: 3
Всего: 32



azesmcar, мне представляется, что создастся объект std::string, метод empty() которого вернёт true, но если ты точно знаешь, что будет бяка, то я переделаю на return "";


--------------------
user posted image
PM Jabber   Вверх
azesmcar
Дата 23.6.2009, 22:44 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


uploading...
****


Профиль
Группа: Участник Клуба
Сообщений: 6291
Регистрация: 12.11.2004
Где: Армения

Репутация: 81
Всего: 211



Цитата(Cheloveck @  23.6.2009,  22:42 Найти цитируемый пост)
azesmcar, мне представляется, что создастся объект std::string, метод empty() которого вернёт true

 smile вылетет при первой же возможности smile 

Цитата(Cheloveck @  23.6.2009,  22:42 Найти цитируемый пост)
то я переделаю на return ""; 

Я бы использовал исключения, возвращать "" как-то некрасиво.

Добавлено через 4 минуты и 5 секунд
Цитата

basic_string(const charT* s, const Allocator& a = Allocator());
9 Requires: s shall not be a null pointer.
10 Effects: Constructs an object of class basic_string and determines its initial string value from the
array of charT of length traits::length(s) whose first element is designated by s, as indicated
in Table 41:

PM   Вверх
Cheloveck
Дата 23.6.2009, 22:52 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
***


Профиль
Группа: Завсегдатай
Сообщений: 1578
Регистрация: 26.7.2008
Где: Тула

Репутация: 3
Всего: 32



azesmcar, я уже протестил. Исправил на return ""; Исключения тут не нужны ИМХО, строка пустая, если не найдена и всё, кажется логично.


--------------------
user posted image
PM Jabber   Вверх
azesmcar
Дата 23.6.2009, 22:55 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


uploading...
****


Профиль
Группа: Участник Клуба
Сообщений: 6291
Регистрация: 12.11.2004
Где: Армения

Репутация: 81
Всего: 211



Цитата(Cheloveck @  23.6.2009,  22:52 Найти цитируемый пост)
Исключения тут не нужны ИМХО, строка пустая, если не найдена и всё, кажется логично. 

Строка пустая если
1. Невозможно открыть файл
2. файл открыт, но в 
Выделить всёкод C++
1:
2:
3:
4:
5:
if( first == key )
{
    input.close();
    return second;
}

ни разу не зашел. А если там в самом деле пустая строка?
Исключениями красиво, но не мне решать конечно smile я только идею предлагаю.


Это сообщение отредактировал(а) azesmcar - 23.6.2009, 22:56
PM   Вверх
Cheloveck
Дата 23.6.2009, 22:57 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
***


Профиль
Группа: Завсегдатай
Сообщений: 1578
Регистрация: 26.7.2008
Где: Тула

Репутация: 3
Всего: 32



azesmcar, хорошо, я подумаю


--------------------
user posted image
PM Jabber   Вверх
Lazin
Дата 24.6.2009, 05:46 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
****


Профиль
Группа: Завсегдатай
Сообщений: 3820
Регистрация: 11.12.2006
Где: paranoid oil empi re

Репутация: 41
Всего: 154



Цитата(nerezus @  23.6.2009,  22:36 Найти цитируемый пост)
На него выливали кучу вонючего коричневого по отношению к flex/bison из-за compile-time всего.

если честно, не понимаю что в этом плохого, лично я использую spirit даже для разбора параметров командной строки, использовать flex/bison ради этого я бы не стал smile 
PM MAIL Skype GTalk   Вверх
Ответ в темуСоздание новой темы Создание опроса
Правила форума "С++:Общие вопросы"
Earnest Daevaorn

Добро пожаловать!

  • Черновик стандарта C++ (за октябрь 2005) можно скачать с этого сайта. Прямая ссылка на файл черновика(4.4мб).
  • Черновик стандарта C (за сентябрь 2005) можно скачать с этого сайта. Прямая ссылка на файл черновика (3.4мб).
  • Прежде чем задать вопрос, прочтите это и/или это!
  • Здесь хранится весь мировой запас ссылок на документы, связанные с C++ :)
  • Не брезгуйте пользоваться тегами [code=cpp][/code].
  • Пожалуйста, не просите написать за вас программы в этом разделе - для этого существует "Центр Помощи".
  • C++ FAQ

Если Вам понравилась атмосфера форума, заходите к нам чаще! С уважением, Earnest Daevaorn

 
1 Пользователей читают эту тему (1 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема | C/C++: Общие вопросы | Следующая тема »


 




[ Время генерации скрипта: 0.0974 ]   [ Использовано запросов: 21 ]   [ GZIP включён ]


Реклама на сайте     Информационное спонсорство

 
По вопросам размещения рекламы пишите на vladimir(sobaka)vingrad.ru
Отказ от ответственности     Powered by Invision Power Board(R) 1.3 © 2003  IPS, Inc.