First of all, cursors are typically not a good performer.  If you can get away with not using a cursor, definitely do it.  Try to stick to set based logic...