О проверках в классе ComExcelDocument_RU

[Ruff]

Цитата:

Сообщение от Gustav

P.S. Проверки в этом классе напоминают мне примерно следующую ситуацию:

Допустим, я иду по адресу внутри города:
- вот эта улица (document),
- вот этот дом (sheet),
- вот эта квартира (range),
- заношу палец над звонком в дверь...
И тут всплывает вопрос "А тот ли это город?!"

Вот один товарищ не задался этим вопросом... Подробнее см. в х/ф "Ирония судьбы или с легким паром"

А что касается проверок... ИМХО, это один из признаков хорошего тона - типа "семь раз отмерь". Согласен, можно, конечно, найти в данном конкретном случае избыточность, но наличие привычки все перепроверять само по себе не лишнее...

Расскажу страшилку в тему Лично я, будучи еще зеленым студентом, из-за отсутствия этой привычки провалил тестовое задание при трудоустройстве в солидную контору Как сейчас помню - сгенерил (в смысле new) некий объект, проверил его сразу же на != null и передал указатель в метод. А в теле метода сразу стал обращаться к объекту, как заведомо существующему.
Проверяющий мою работу программист назвал это грубой ошибкой (кстати, спасибо ему за это). С тех пор стараюсь следовать правилу "если что-то пришло извне, сначала проверь, потом используй"

[ivas]

в методе
\Classes\ComWordDocument_RU\FindField

есть такие строки:

            if (substr(m_comApplication.version(), 1 ,1) >= '9') // Word 2000 and later
            {
                field = fields.item(_bookMark);
            }
            else
            {
                   ,,,
            }

проверка на то что версия офиса начинается с символа большего либо равного 9ти
у меня например версия "10.0" условие не выполняется и работает код предназначенный для более старых версий соответственно более медленный )