Обход каталогов

[egoroff]

А все таки, в языке Си есть свое очарование. Да, пускай это высокоуровневый ассемблер, да на нем приходится писать много кода для простых вещей, да это не оружие массового поражения, вроде C++, но в его простоте и заключается красота. Это как меч, - им достаточно сложно научиться пользоваться, в отличие например от пистолета или автомата, но если

Comments

[chemodax]

О, APR pools это мега круто. Хочешь въедливое ревью?

[egoroff]

:) давай, буду рад!

[chemodax]

Для начала на высоком уровне:

1. Знающие собаководы говорят что обход дерева директорий в ширину намного эффективнее чем в глубину. Т.е. сначала лучше обойти все файлы в директории, а потом зайти в каждую директорию -- так лучше дисковый кэш будет работать. Хотя это потребует сохранения всех поддиректорий в памяти и изменит поведение.


2. Для большей гибкости я бы сделал Include/Exclude pattern тоже через callback или перенес на ответственность PfnFileHandler. Я уж если оставлять такую возможность то на паттерн надо делить на токены один раз, а не при каждой проверке.


3. Непонятен вопрос в обработкой ошибок. Я бы вынес печать сообщений об ошибке на более высокий уровень, а в функциях возвращал apr_status_t. Как минимум сделал бы callback.
4. У PfnFileHandler нет возможности прервать процесс по ошибке. Я бы добавил apr_status_t

Теперь по мелочи:

1. void (* PfnFileHandler)(apr_pool_t* pool, const char* pathToFile, DataContext* ctx);
   ctx должен быть типа void *, иначе нарушается инкапсуляция.
   

[egoroff]

:) о! круто! спасибо большое! все по делу. Поправлю, разве что обработку ошибок оставлю в таком виде, и возможно пока не буду обходить дерево в ширину.