C++っぽい書き方とは

Question

先生にC++で書いたゲームのコードを見せたところ、Javaっぽい書き方していると言われました。
おそらく、１つのファイルにまとめすぎているのが原因だとは思います。
現在は先生が書いたコードを真似して直そうとしているのですが、C++の良い書き方、悪い書き方について教えていただけないでしょうか?

Comments

[ozwk]

javaっぽいと言われたコードがどんなものか気になるので一部分貼ってもらえるとありがたいです。

[albacrow]

すみません、コードを貼るのは遠慮したいので要点だけ書きます。
・if文のブロックが3重４重になっている
・文が横に長い
・コピペが多い
・関数を分割しきれていない
などが自分でも悪いとは考えています。

Answer 1

C++でJavaっぽいコードというとむしろ褒め言葉のように感じるのは私だけなんでしょうか…

・if文のブロックが3重４重になっている
・文が横に長い
・コピペが多い
・関数を分割しきれていない

このあたりはJavaの特徴と合致するものではありません。
先生が何か先生の思うフィーリングみたいなものを言っただけかもしれません。
例えば昔、似たソースコードを見た…とか。

C++ のコーディングスタイルの話に関して言えば、私は以下のスライドで紹介されているJeff Bayの手法がかなり好きです。

http://www.slideshare.net/MoriharuOhzu/ss-14083300

全部実践する必要性は無いし、そもそも実践しようとしてもとてもしんどくて全部は無理なのですが、Jeff BayのObject Calisthenicsは私にコードの理想像を一つ教えてくれた気がします。

後は書籍リーダブルコードは良書です。
全部ここで伝えようとしても伝えきれないほどソースコードを綺麗に書くコツというのはたくさんあります。

まずはこういうところから見てみてもいいかもしれませんね。

Comments

[albacrow]

ありがとうございます。精進します。

Answer 2

1つのファイルにまとめすぎる、という点で言うとC++の場合、ヘッダファイル(.h)にクラスの宣言(定義も兼ねる)を書きますが、メンバ関数(Javaでいうメソッド)は宣言だけ書いておいて、実際の定義は別のファイル(.cppなど)に書きます。
ただ、セッター・ゲッターなどはクラス宣言の方に書くことが多いです。

ヘッダファイル(MyClass.h)に記述

class MyClass {
public:
    // コンストラクタ
    MyClass();
    // デストラクタ
    ~MyClass();

    // セッター・ゲッター
    void SetCount(int value) {
        Count = value;
    }
    int GetCount(void) const {
        return Count;
    }
    // 計算
  int Calc(int x, int y);
private:
    // メンバ変数
    int Count;
    
}

MyClass.cppに記述

#include MyClass.h
MyClass::MyClass()
{
    // 初期化処理
}
MyClass::~MyClass()
{
    // 後片付け
}
int MyClass::Calc(int x, int y)
{
    int Ret = 0;
    // xとYを使って計算
  ...
    return Ret;
}

こんな感じで書くと体裁はJavaっぽくなくなります。

あとは、メンバ関数内でなにがしかオブジェクトを使う場合

MyObject obj1;   // 1
MyObject* obj2 = new MyObject();  // 2

と2種類の書き方ができます。
よほどサイズの大きいオブジェクトでなければ 1 の書き方で構わないと思いますが、サイズが大きい場合、2の書き方にしないと、メモリが不足してしまいビルドは成功しても実行時に不正終了したりします。
じゃあ2の書き方の方がいいか、というと、C++はnewしたものはどこかで確実にdeleteしないとメモリリークになりますので、スコープを抜ける際に自動で破棄してくれる1の書き方が楽ではあります。
逆に1の書き方をしている場合、スコープ外に出ると破棄されてしまうので、そうなっては困る場合には2の書き方をする必要があります。

このあたりの使い分けは慣れるしかないと思いますが、Javaとはだいぶ違うと思える部分です。
こんな感じでしょうか。

Comments

[albacrow]

ありがとうございます。コード見直して、直せる部分は直したいと思います。

Answer 3

他の方からも回答されているので別の情報を。
GoogleのC++スタイルガイドは、C++でどう書くべきか理由も含めて詳しく書いてありますので一読を勧めます。
http://www.textdrop.net/google-styleguide-ja/cppguide.xml

Comments

[albacrow]

ありがとうございます!
今日、先生からソースコードもらえたので、見比べながら勉強します。

Answer 4

他の方の回答にもありますが、

１つのファイルにまとめすぎている

という特徴は「Javaっぽい」には合致しないと思います。
C++でもJavaでも、1ファイルが数千行というコードは何度も見たことがあります。

Comments

[albacrow]

ありがとうございます。ただ、数千行は見づらいと先生によく言われたので、適度な行数に押さえたいと思います。

[argius]

すみません、肝心なことを書き忘れていました。
悪い意味で1ファイルが数千行というコードはC++でもJavaでもあるということです。先生のおっしゃるとおり、適度な行数に抑えたほうが良いです。

Answer 5

C++とJavaは兄弟みたいなものなのでそんなに違いが出るとは思わないのですが
Javaっぽい書き方ってなんなんでしょうね。
クラス名が無駄に長いとか・・・

ゲームだと、効率を重視するためにあんまりインスタンスを作らなかったり
newしないことがあるので、もしかしたらnewしまくってるとか。

Comments

[albacrow]

ゲームのキャラクターを破棄する際にdelete、新たに作成するたびにnewしている事があるので、それが原因かもしれません。
ありがとうございます。