ID:268575
 
=/
I don't see anything wrong, but, you can equip tons of weapons(not the same one)...
            Equip()
set category = null
if(!src.Righthand)
Righthand = src
usr << "You equip the [src.Righthand]!"
src.suffix += "Equiped"
usr.Str += src.PwrUp
usr.Spd += src.DexUp
usr.Str -= src.PwrDwn
usr.Spd -= src.DexDwn
else
usr << "You already have a [src.Righthand] equiped!"

I don't see ANYTHING wrong with that. =/. Well, thanks for helping if you do.
Since you didn't paste the object type, I'm not sure if its mob/verb/Equip(), or whatever.

I'm assuming its the object's verb, and that isn't the way you should be doing equips.

You should be assigning the mob a variable of what you equiped.

Example:
mob/var
/obj/weapon/RightHand


In the equip proc, you only need to compare the variable with the type.

Example:
obj/weapon
Equip()
set category = null
if(usr.RightHand==src)
usr << "Already equiped!"
else
usr.RightHand = src
// add effects here


Hoped I helped,
~~> Dragon Lord
In response to Unknown Person
You did, thanks DL. :P