Цитата:
Изначально опубликовано bagyr
Удалил полностьб все датасорсы и сделал 3 программируемых секции.
Во-первых, пользователи вашего отчета потеряли возможность вводить критерии поиска.
Во-вторых, они потеряли возможность управлять сортировкой
В-третьих, они потеряли возможность выводить только итоги и пользоваться другими опциями (очень полезными кстати)
В-четвертых, вы используете неокругленное значение mmQty = cpp2.Quantity / cpp1.Quantity
В-пятых, вы очень странно выполняете группировку.
В-шестых, вы наверняка для строки s1 использовали тип str, что приведет к чудовищным последствиям при смене выравнивания в name (случай гипотетический, но последствия будут неприятными
)
В-седьмых, не используйте функцию str2date("01/10/2004", 123), пишите литерал 01\10\2004;
В-восьмых, что это за магическая константа?
И т.п.
Самым главным недостатком приведенного кода считаю то, что пользователя лишили очень удобного инструмента для ввода критериев, сортировки и управления параметрами отчета. Не говоря уже о пакетной обработке и прочей внутренней хренотени.
Ребяты! Не программируйте.
А если уж программируете, то делайте это хорошо.
Пожалуйста.