switchの中のどこで数値を引いてるって?

>JHP = JHP -da;

の部分か?
元の変数JHPを書き換えるのをやめればいいのでは?