C++で練習用にドラクエモドキ作っているんですが

class PlayChara{
int hp;
int mp;
…
public:
void setAttack(ATTACK_TYPE atk_type, ATTCK_POINT atk_point);
}

プレイキャラクラス自身に敵からのダメージを与える為のメソッドで
（攻撃のタイプと攻撃力の引数を受け取る）setAttackってのを考えたんですけど名前変でしょうか？
なんか自分自身が攻撃コマンドを実行しようとしているみたいで紛らわしい気が。
設計そのものがおかしいのかもしれないですけど…