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

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

Да с чего Вы взяли, что запись не валидная? Вот можете внятно объяснить?

У вас запись из кода и дат. Код вставили. Даты вставили. Зачем еще раз проверять факт наличия этого самого кода и дат?

Кастомизация? А Вы что, не будете править код создания если, скажем, новое поле добавите? Переложите бремя тестирования на пользователя?

Вы программно, подчеркиваю еще раз - программно(!), готовите данные для новой записи. Если Вы "вменяемый" разработчик, то вы обязаны подготовить эти данные корректно. С тем, чтобы validateWrite() прошел без ошибок. Но если Вы изначально готовите данные корректно, то зачем Вам себе перепроверять? Или Вы предполагаете, что можете подготовить не корректные данные. А зачем это делать?

[wojzeh]

Цитата:

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

Да с чего Вы взяли, что запись не валидная? Вот можете внятно объяснить?

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

простой пример:
программно пишем дату, которую получили через параметры. она из прошлого, а мы по логике допускаем только даты из будущего. ну и так далее.

в моём конкретном случае, мне нужно написать extension для проверки дупликатов. было бы написано, как доктор прописал, щас бы уже всё работало, так - делаем выкрутасы.

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

Цитата:

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

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

Может я плохо искал, но в документации по Best Practices я такой рекомендации не нашел

Цитата:

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

простой пример:
программно пишем дату, которую получили через параметры. она из прошлого, а мы по логике допускаем только даты из будущего. ну и так далее.

Это принципиально другой пример. Параметр - это данные, пришедшие из-вне. Не созданные программистом. Такое, разумеется, надо проверять. Причем не факт, что в validateWrite. Возможно, непосредственно в коде создания записи

Но в примере, который послужил основанием для дискуссии, "внешних" данных нет вообще. Нечего проверять-то...

Цитата:

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

в моём конкретном случае, мне нужно написать extension для проверки дупликатов. было бы написано, как доктор прописал, щас бы уже всё работало, так - делаем выкрутасы.

Ну, если не рассматривать вопрос о том, что программный контроль уникальности - заведомо не надежен, то выбранный способ решения сам по себе не однозначен. Зависит от постановки задачи. Тут есть разные варианты и использование validateWrite - одно из многих. Не для обсуждения, а просто чтобы показать другие варианты

- Делать проверку на возможный дубликат до вызова метода создания
- Если есть дубль, то менять данные с тем, чтобы дубля не было при создании новой записи

Это так, первое, что в голову пришло. Но могут быть и другие варианты. Причем я не удивлюсь, если позже Вам придется добавлять в ValidateWrite() параметры для того, чтобы исключить ту или иную проверку...

[wojzeh]

Цитата:

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

Может я плохо искал, но в документации по Best Practices я такой рекомендации не нашел

да ты-то не нашёл - это ладно, а вот программисты микрософт не нашли (и я так понимаю, не искали) - это уже толкает меня опять сгонять на Кипр и посетить город Пафос.

кароче... видишь, что метод вызывается на форме? видишь в нём параметры? видишь, там ещё какая-то переменная используется, которая вообще из глобального контекста?



а ты вот этот плакат внимательно изучал?