クリティカル判定とエフェクトを連動させたい場合は
エフェクトを単体処理から呼ぶのが確かに一番簡単かもしれない
パッと思いつくバグは「全体スキルでエフェクトが対象の数と同じだけ飛ぶ」ぐらいか
これは対処が難しいものではない、はず