8.
Dnki
4
05.03.11 00:38
Сейчас в теме
Сделал человек - спасибо человеку. Лично я возьму за основу, с позволения автора облагорожу и буду пользоваться.
Теперь суровая критика:
* за отсутствие скриншота народ укоряет справедливо. Дай им БОЛЬШУЮ картинку.
* кнопка "Получить типы документов" какой имеет смысл? Можно пользоваться программой и ее НЕ нажимать? Вывод: заполнять список при открытии, кнопку убрать.
* кнопку "Выбрать документы" расположить после дат (правее/ниже).
* порядок обхода полей чуть продуманнее.
* Все документы по-умолчанию отмечать. Заметил, что женщины любят раз пять поочередно клацнуть перед нажатием главной кнопки. А вот нашему брату чем быстрей...
* На эту же тему быстроты:
- неплохо бы даты какие-то предлагать? С начала года/месяца.
- а если не нажимать "Выбрать док-ты"? Может программа бы все документы обработала?
* при первом открытии окно широковатое, но сплющенное по высоте. На какой экран ориентируешься?
* В остальном дизайн нормальный.
* По тексту модуля (почему программисты 1С плюют на него? Может они-ж "для себя" пишут?) :
- между процедурами пропуск в 1 строку, а внутри процедур пустые поля по 2-3 строки. Это специально?
- процедура получитьДокументыНажатие() лишняя. А след-но функ-ии "добавУсловиеСостДокум()", "ОтборПоДате()" нет нужды выделять в отдельные процедуры. Чем более линейный текст - тем более читаемый (к типовым конфигурациям 1С этот принцип уже не применим в силу безнадежности).
- так же проц-ра посчитатьМаксЗнчИнд() вызывается 1 раз. В чем красота? Просто вставь кусок текста вместо отдельной процедуры.
- все знают, что 1С - регистронезависимый язык. Но не стоит эти злоупотреблять:
"Для каждого стр из СписокДокум Цикл". К именам процедур это тоже относится.
- кнопки и др. объекты тоже надо называть с душой, я бы сказал с любовью. Например, 2 одинаковых по виду и функ-ям [V] называюются: 1)"Кнопка1", 2) "v".
- комментов нет. Ну да Аллах с ними! А информативность "// -- Выполняем необходимые действия --" бесспорна.
* И самое главное: каждая из процедур обработки документов похожа друг на друга: цикл по документам, внутри него пару строк собственно операции. Не красивше ли сделать 1 головной цикл? Внутри него вызывать маленькие процедуры.
* Фраза "добавь произвольный код для обработки перед/вместо/после обработки и тогда будет уже интересно" подразумевает след.: создай окно ввода текста, где пользователь напишет несколько строк кода 1С. По команде "Выполнить" программа над каждым документом выполнит эти строки.
Пожелания:
* Убрать пункт "Удалить непосредственно". Много слез он может породить.
* Сделать операцию "Заменить реквизит". Естественно должны быть поля "Старое значение", "Новое".
* Сделать вторую закладку "Справочники"
P.S.
Извините, если комментарий к программе оказался длиннее самой программы. Вечером есть время побездельничать.
