Best Practice in real life: SysPolicy form as a bad example

[Владимир Максимов]

Т.е. алгоритм такой

1. Сделали кастомизацию
2. Программный код создания записи не исправили по этой кастомизации, т.е. он стал не корректным
3. Тестировать "как положено" не стали
4. В релизе ValidateWrite отловит эту ошибку. Пользователь работать не сможет. Выставит bug-report разработчику

Т.е. сам по себе код создания не корректный. А ValidateWrite нужен для того, чтобы переложить бремя тестирования на пользователя. Ну, тоже стратегия. Все как у Microsoft. С чего, собственно, я и начал...

[trud]

Цитата:

Сообщение от Владимир Максимов

3. Тестировать "как положено" не стали

Я написал, тестируем RSAT, в моем варианте он поймает исключение, в вашем варианте вставится некорректное значение, никто этого не заметит

error('something went wrong');

Это вообще страшный код. тут запись одна, а если транзакция? вставляем хорошие, не вставляем плохие? Вариант Владимира - когда вставляем хоть что-то, в этом случае мне кажется даже лучшим

[wojzeh]

Цитата:

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

error('something went wrong');

Это вообще страшный код. тут запись одна, а если транзакция? вставляем хорошие, не вставляем плохие? Вариант Владимира - когда вставляем хоть что-то, в этом случае мне кажется даже лучшим

Страшный он или нет, это определяется каждым конкретным сценарием. Я говорю о best practice вообще. Следование им самим же рекомендателем было бы логичным и полезным для разработчиков, вынужденных работать в парадигме extensions.

Твой же пример со статическим вызовом буферного метода и генерацией исключительной ситуации противоречит данной рекомендации, но было бы интересно взглянуть, как ты его используешь на конкретном примере.

[Владимир Максимов]

Цитата:

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

Я написал, тестируем RSAT, в моем варианте он поймает исключение, в вашем варианте вставится некорректное значение, никто этого не заметит

Как это не заметит? Вы что, не проверяете, что в результате было создано? Я про это и говорил, что тестирование не корректно - не посмотрели результат теста не в смысле лога ошибок, а сам результат. Что получилось-то?

PS: Что такое RSAT? Автотесты что-ли?