2.
RailMen
823
26.05.10 13:00
Сейчас в теме
1) Справка
Нет ее, впрочем, как и у большинства таких работ.
2) Удобство работы с формой.
Все логично и удобно, что редкость.
Немного придерусь к: привязки полей ввода «Организация» и «Сотрудник» и кнопки «ПодготовкаДанных» некорректны. Поля ввода «НачПериода» и «КонПериода» надо уменьшить. Почему реквизит обработки называется «Сотрудник», а тип у него «Физ. лицо» - почему бы не назвать вещи своими именами? По большому счету кнопка «ПодготовкаДанных» не нужна: как только меняются реквизиты шапки – сразу заполняем табличные поля с начислениями и удержаниями. Даешь меньше кнопок! Кнопку «Сформировать» делать доступной только после заполнения табличных полей, тогда не нужна будет проверка: Если Начисления.Количество() = 0 и Удержания.Количество() = 0 Тогда
Предупреждение("Сначала подготовьте данные!");
3) Модуль основной формы
Комментариев (или же тегов) в коде нет совсем.
4 процедуры (УстФлажкиНажатие, УдалФлажкиНажатие, УстФлажкиУНажатие, УдалФлажкиУНажатие), которые делают одно и тоже: включают и отключают флажки в табличном поле. Создал бы лучше одну с 2 параметрами (ТабПоле, ВарантДействия), поместил в модуль обработки со словом ЭКСПОРТ и вызывал бы ее.
Запрос в Функция СформироватьЗапросДляПечати() не идеален по производительности в случае, когда физ. лиц в базе очень много.
А теперь о серьезном.
Запрос в Процедура КнопкаСформироватьНажатие(Кнопка)
У тебя параметр запроса «Физ лицо», а поле ты выбираешь из запроса «Сотрудник». Но у одного физ. лица может быть несколько сотрудников (штатный, совместитель внешний/внутренний или даже договорник). В общем случае может вернуться несколько записей по физ. лицу. Особенно меня насторожила Функция ПолучитьДолжность(Сотруд)
в которой ты ищешь должность сотрудника, хотя в параметрах обработки есть только реквизит типа «Физ. лицо». Да и наличие отдельно стоящей такой функции в коде – дурной тон.
4) Модуль объекта
А он пустой! Кучу кода можно из модуля формы перенести сюда, особенно формирование запросов.
Резюме. При условии, что сотрудники принимаются только в штат и не работают по совместительству или как договорники, а так же при условии, что отчет будет использоваться в небольшой или средней фирме – работа зачетная.