use strict; use warnings; will catch that $i is undefined as you enter the c-style for loop, It will catch that @old is undefined, too.

It will not catch that you prepare a DBI statement handle with placeholders each time through the loop. Move it outside to reduce overhead.

Could you clarifiy what you are trying to do?

After Compline,
Zaxo