Muitas vezes eu uso cursores desta maneira:

for rec in (select * from MY_TABLE where MY_COND = ITION) loop
    if rec.FIELD1 = 'something' then
        do_something();
    end if;

    if rec.FIELD2 <> 'somethingelse' then
        blabla();
    end if;
end loop;

O líder da minha equipe me disse para não usarselect * porque é uma programação ruim, mas não entendo o porquê (neste contexto