-
> Погоди, я не понял. Ты имеешь ввиду, что лучше В ЭТОЙ ФУНКЦИИ использовать внешнюю текстовую константу с запросом вместо того чтобы переопределять > свойство SQL у компоненты, так? Я тебя понял так, что нужно вместо сервисной функции везде делать явный вызов метода. И именно с этим был не согласен.
вот изза этой фигни > ' + IntToStr(aNum) + ' запрос становиться динамическим и перекомпиляется сервером каждый раз перед выполнением, правильно статически прописать запрос, а меняемую часть сделать параметром. причем твоя функция прямо таки навязывает стиль с изменением запроса, вместо параметров (передается строка запроса). не показан способ формирования первого запроса > procedure DoDBUpdate(aNum: integer; aSQL: string); но могу "забиться" на то что там тот же стиль, с переменными сделанными частью запроса... вот это вот я и называю идиотским подходом. неважно с чем ты там несогласен.
> Я не собираюсь изобретать велосипед и переписывать компонент. поздравляю, хоть и не хотел, но начал.
> а так достаточно внести изменения в сервисную функцию. вот вот. привычка к своему велосипеду уже сказывается. вместо того чтобы изыскать методы при нормальной работе.
-
> > вот изза этой фигни > > ' + IntToStr(aNum) + ' > запрос становиться динамическим и перекомпиляется сервером > каждый раз перед выполнением, > правильно статически прописать запрос, а меняемую часть > сделать параметром. причем твоя функция прямо таки навязывает > стиль с изменением запроса, вместо параметров (передается > строка запроса). > не показан способ формирования первого запроса > > procedure DoDBUpdate(aNum: integer; aSQL: string); > но могу "забиться" на то что там тот же стиль, с переменными > сделанными частью запроса... > вот это вот я и называю идиотским подходом. неважно с чем > ты там несогласен.
Ага, дошло. Спасибо, буду думать.
> > > Я не собираюсь изобретать велосипед и переписывать компонент. > > поздравляю, хоть и не хотел, но начал.
И в чем это выражается?
> > а так достаточно внести изменения в сервисную функцию. > > вот вот. привычка к своему велосипеду уже сказывается. вместо > того чтобы изыскать методы при нормальной работе.
Что есть "методы при нормальной работе"? Писать одинаковый код в сотне мест, а затем для внесения изменений перелопачивать весь проект? Чувствую, все-таки я чего-то не понял :)
-
> Писать одинаковый код в сотне мест, а затем для внесения > изменений перелопачивать весь проект? Чувствую, все-таки > я чего-то не понял :)
Если выносишь какой то код в процедуру/метод, то надо вынести его так, чтобы было правильно. А правильно - работать с параметрами, а не генерить на кажный чих новый запрос.
|