Well the first is okay, aside from that you don't want it to move him to map center.
As for second, you are dropping the item from a null unit(call UnitRemoveItemFromSlotSwapped( GetRandomInt(1, 6), null ))

JASS:
function UnitRemoveItemFromSlotSwapped takes integer itemSlot, unit whichHero returns item
__________________