Namespacing all of these with effect_ is unnecessary.  They're all going to be accessed via has_effect("hit_by_player") or set_effect("hit_by_player").