Best Practices

[Eldar9x]

Что означает сообщение

Цитата:

TwC: проверка доступа к значению, возвращаемому из метода отображения или изменения.

?

[kashperuk]

Это значит, что edit методы теперь ругаются - типа не стоит так делать.
Надо вверху метода, до объявления написатб

//BP Deviation Documented

поищите на стандартных методах - кучу примеров найдете

[Eldar9x]

Надо же, хотел, но решил, что шаманство. Огромное спасибо!

[Hyper]

Цитата:

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

Это значит, что edit методы теперь ругаются - типа не стоит так делать.

А почему не стоит, кстати? Если явно пишешь display или edit, наверное ведь, подумал об этом заранее.

[kashperuk]

Вот как раз этот самый //BP Deviation Documented и говорит, что ты подумал и в теле метода позаботился о безопасности его исполнения.
Чтобы кто-то не мог в него параметром передать строку с runBuf, к примеру, которая будет ему ЗП поднимать раз в день

[Hyper]

Цитата:

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

Чтобы кто-то не мог в него параметром передать строку с runBuf, к примеру, которая будет ему ЗП поднимать

Где можно посмотреть на примеры таких дыр?

[miklenew]

Цитата:

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

Где можно посмотреть на примеры таких дыр?

Мужики не палите. С-с-с

[avf]

Данное сообщение означает, что, возможно, необходима проверка на то, имеет ли пользователь необходимые права доступа на просмотр или редактирование значения, "возвращаемого из метода отображения или изменения." Данная рекомендация BestPractice какбэ намекает о том, что в метод, возможно, следует добавить проверку hasSecuritykeyAccess или комментарий //BP Deviation Documented, в случае, если проверка будет излишней.

display AmountMST accountBalance(LedgerTable _ledgerTable)
{
    AmountMST   balance;
    ;
    if (!hasSecuritykeyAccess(securitykeynum(LedgerMisc), AccessType::View))       
        return balance;
    ...

Т.е. если нет прав на просмотр, то возвращаем 0.

Цитата:

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

Это значит, что edit методы теперь ругаются - типа не стоит так делать.
Надо вверху метода, до объявления написатб

//BP Deviation Documented

поищите на стандартных методах - кучу примеров найдете

я правильно понял, что теперь комменты обрабатываются специальным образом, фактически стали исполняемым кодом ?

[S.Kuskov]

Цитата:

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

я правильно понял, что теперь комменты обрабатываются специальным образом, фактически стали исполняемым кодом ?

Я бы сравнил этот подход с атрибутами C# нежели с исполняемым кодом.

[glibs]

Цитата:

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

...
комменты ... фактически стали исполняемым кодом
...

С кодом работает компилятор. Проверка ВР — это не компитлятор, а проверка соответствия кода и свойств ряда объектов в АОТ некому своду рекомендаций.

[kornix]

а ToDo обрабатывает компиллятор?

[glibs]

Если под компиляцией понимать перевод программного кода в машинный, т.е. команды процессора (см. http://ru.wikipedia.org/wiki/%D0%9A%...82%D0%BE%D1%80), то нет.

Признаю что сформулировал свою мысль плохо. Но изначально речь шла о том что комментарии стали исполняемым кодом. Я имел ввиду, что не стали. А ВР, // TODO — это скорее сервисные функции среды разработки. В моем понимании.