the 'as obj' and 'as mob' are unnecessary, and you should change 'obj/item/weapon/W' to 'obj/item/I' so that future peoples don't get confused if they can't add items to the attackby.