8年前いろいろと気に食わない部分があるので、プレイヤーのプログラムを書き換えようと思うのですが、せっかくなので、「ここも直しといた方がいいと思うよ」ってところがあったら教えてほしいです。長いかもなので、ヘッダだけコピペです。cppファイルは付加しときます。プレイヤーのヘッダです。PoseChange(POSE);//プレイヤーのポーズ(状態?)を安全に変更する。SetPoint(float,float);//座標をセットする。SetSpeed(float,float);//移動量をセットする。Movement(void);//座標を動かす。書き方ややめた方がいい場所など指摘していただければうれしいです。自分で書き換えたいなと思ってるのは、インクルードしないと逝けないものがおおいと言うことです。当たり判定とかを行おうと思うとインクルードしないと難しい気がします。(逆にマップの方でやろうと思うとマップの方でインクルードしなければならなくなります。)やはりその辺は専用の物で行うべきなのでしょうか?後は何カ所かロード化させるつもりです。(ここについては、ノーコメントでいいです。)いろいろ至らない部分もありますが、少し気になった所などでもいいのでお願いします。8年前ざっと見ですが、思った事を書いてみます。●関数の数クラス一つに収める関数が多すぎませんか?Playerは何かから継承出来ると思います。雑魚的やボス、プレイヤーなどは共通する部分があることでしょう。それを省けばPlayer自体の変数や関数も少なくなると思いますよ。見ても中で何が行われているか想像できないものがあります。「P_」と書いても何の事かわかりません。位置を表すPointのPかもしれません。パラメータなら、Paramなどと略すのが一般的でありましょう。3つをとったりすることがあります。ManagerならMgr等。略すにしても、単語が想像できるものが良いかと思います。●誤字ロードはLodeではなくLoadかと思います。・カウンタの組み合わせがいくつも現れているのは、役割分担が出来ていないあらわれになっている可能性が高いです。役割分担出来ないか考えてみてはどうでしょう。細かいところですが、スピードや移動量をPoint2Dという構造体で管理するよりも、Vector2D構造体とかを別に定義してそれで管理するほうがいいと思います。「Point」という単語は座標とか位置という意味を持ちますし、「Vector」はベクトルという意味を持ちます。ベクトルは移動を表すにはいいと思います。多いのかな?継承は実はまだちゃんと勉強してなかったり...この機会に使ってみたいと思います。>xとy必要になりそうなのはとりあえず返しとこう。アドレス放り込んで中でxとyの値を入れてもらえば十分だった気がしてきました...その場合は、構造体(Vectr)の値を返す子は必要なくなるのでしょうか...それとも、両方用意しといた方がいいのでしょうか?(どっちでも一回代入すれば使えますが...)どっちにしても、xとyを単体で返す子は必要なさそうですね。>命名規制コレはPlayerのPですね...自分でも途中で嫌になってほかのクラスでは付けてなかったり今度は、できるだけ気をつけたいと思います。>誤字ごめんなさいもう誤字は許してw(つづりが覚えられないんです)>役割り役割分担に付いてもっと考えてみます。>>tkmakwins15さんそうですね。参考にしたことのあるプログラムでも分けていましたし。次からは分けたいと思います。お早い解答ありがとうございました。(私の返信遅かったですが...)アドバイスを生かしてより良いプログラムに書き換えれるようにがんばります。解決にさせていただきます。(もっとあるという方がいれば書き込んでいただけるとうれしいです。)ありがとうございました。