Por questões de lógica não teriam problemas, mas dependendo do banco de dados utilizado pode perder um pouco em questão de performance. Não sei como o firebird trata este caso, mas em determinados bancos, quando a consulta é executada com parâmetros o banco já salva o plano de execução e a próxima vez que a consulta for executada o mesmo não precisa ser feito novamente.
@StandAlone, mas desta maneira você esta comparando uma string com uma data, o correto seria converter o campo varchar para date também. Se não for muito trabalhoso, é aconselhável você converter esta coluna para o tipo date, gravar data em formato string é abrir uma porta para mil problemas diferentes ...
Se suas datas estão no formato americano seu to_date esta 100% correto! Um possível teste seria substituir o to_date por um Sysdate apenas para verificar se o erro persiste, eliminando a formatação de data e etc. Outro detalhe, o campo Data_final é do tipo date correto?
Esta pergunta pode ter mil respostas diferente. Depende muito da estrutura utilizada (client > server > banco ou client > banco), componentes utilizados, qual banco de dados e lógica de conexão e etc ...
@lukkicode, você não precisa chamar este procedimento em nenhum evento, o próprio sistema operacional que vai dispara-lo através de mensagens. Basta fazer a declaração deste método na classe do form que você deseja fazer sua verificação e fazer as tratativas necessárias no mesmo método.