08/02/22 14:30:33 ik+aPlRF
>>559
まぁ、>>1はC++初心者のようだからな。
C++の標準記法を知らなくても不思議じゃない。
補足しておこう。
>・コンストラクタで初期化じゃなく代入をしてる
多分>>1は初期化リストを知らない。
初期化リストのほうが分かりやすいし、constな変数、参照は
これでしか初期化できない(代入できないため)。
>・継承しててもデストラクタがvirtualじゃない
デストラクタがvirtualじゃないと、
基底クラスのポインタに派生クラスのポインタを代入して
ポリモ-フィズムをしているとき、
オブジェクトを破棄するときに派生クラスのデストラクタが呼ばれない。
>・enumを使わず#define使ってる
#defineは時代遅れ。コンパイラが処理していないので。
できればenumやconstを使う。
>・ローカル変数にポインタか参照おけばいいのに"character[ci]->getDs()->"のような記述が沢山並んでいる
ローカル変数にポインタか参照を代入して、
hoge = character[ci]->getDs();
hoge->hoge();
のように呼び出したほうが読みやすい(し、関数を不必要に呼ばないので高速)
ということを言っている。