parser

Написать ответ на текущее сообщение

 

 
   команды управления поиском

И еще...

Sumo 22.07.2006 21:46

Параметр docName надо вынести в единственное место - конструктор, а из остальных методов его убрать. Более того - этот параметр вобще не нужен: создали экземпляр класса для конкретного документа и работайте с ним...

Или Вы предполагали использовать класс исключиетельно как статический? Если так, то это большая ошибка - право на объединение в статический класс имеют только изолированные методы, которые объединяются в статический класс для создания логического пространства имен. Хороший пример - класс math в парсере.

p.s. Еще один момент: принято считать, что методы, имя которых начинается со слова get должны возвращать какое-то значение - у Вас это выполняетя далеко не везде.
И другой момент: вы везде перехватываете исключения и никак не анализируете причины их возникновения, да еще и не даете тем, кто будет использовать класс, внятного объяснения происходящего. В данном случае почти все блоки ^try из вашего кода можно убрать - пусть у пользователя будет возможность самому проанализировать ошибки ипонять, что происходит.