Правильно ли я составил код хранимой процедуры?

Нужно всегда стараться упаковать все в один select или DML, таким образом ты сократишь обращения к БД и ускоришь время работы процедуры.