• 締切済み

C++のクラス設計について

Webや書籍をあたってみたのですが、なかなか解決しないので こちらで相談させいただこうと思い立ちました。 C++で書かれたプログラムを解析しているのですが、 ドキュメントは外部仕様書のみであとはソースコードだけ、という状態です。 コメントもほとんど書かれていないコードについて、リバースエンジニアリングを する必要があり、手始めにクラス図を作成してみました。 すると、複数のクラスでメンバ変数が重複しているのです。 クラスAに、クラスB・C・Dの変数の一部が宣言されていて、 クラスAとクラスBの間で重複している変数はすべてどちらのクラスもprotectedのstatic、 クラスAとクラスC・Dの間で重複している変数はクラスAではprotectedのstatic、 C・Dではprivateです。 クラスBとC・Dは継承関係にありますが、メンバ変数の重複はありません。 クラスAと他クラスは継承関係がなく、メンバ変数が多数重複しています。 クラスAの変数は20個ほどありますが、9割以上がB・C・Dで宣言されている変数と 重複している状態で、かつstaticのprotectedです。 Cはわかるのですが、C++はほとんど書いたことがありません。 アンチパターンなどもあたってみたのですが、上記のような設計についての記述は 見つけられませんでした。こういったクラス間での変数重複は「あり」なのでしょうか？ これで情報が足りているかわかりませんが、何卒よろしくお願い致します。

回答 全件
それぞれのクラスがどのような機能を持っているかわからないしメンバ変数の…

みんなの回答

  • Tacosan
  • ベストアンサー率23% (3656/15482)
回答No.1

それぞれのクラスがどのような機能を持っているかわからないしメンバ変数の名前や意味も分からないようでは判断のしようがない.

bell1107
質問者

補足

やはりそうですか・・・ 重複しているメンバ変数は、ほとんどが入力データ項目を表しているものです。 実際には異なりますが、例えば、身長・体重・年齢など。 クラスC（仮称：CalcData）のコンストラクタの引数にクラスA（仮称：StockData）のポインタを渡していて、 コンストラクタの中でStockDataのメンバ変数の値をCalcDataのメンバ変数にコピーしています。 しかし、StockDataにはget系のメソッドしか定義されておらず、set系はCalcDataに定義されています。 入力データはクラスAの中で読み込んでいるようです。 なぜget系もCalcDataに定義しないのかがわかりませんし、入力データについても StockDataを経由せずCalcDataで読み込めば良いと思うのですが。 プログラムの概要をものすごく簡略化すると、1回exeを走らせる度に、 入力されたデータ（上記の例えで言うと一人分）がある条件を満たしているか否か いろいろ計算して判定する、というものです。 その入力データを表す変数が2つのクラスで別々に宣言されている状態です。

全文を見る
すると、全ての回答が全文表示されます。
  1. カテゴリ
  2. [技術者向] コンピューター
  3. プログラミング・開発
  4. C・C++・C#

関連するQ&A

  • C++　protectedにアクセス不可

    C++でお聞きしたいことがあります。 初心者ですのでご容赦ください。 //基本クラス class a1 { protected: a1(); }; //派生クラス class a2 :public a1 { protected: a2():a1(){}; //継承先以外から作成されたくない int t; }; class a3 : public a2 //a2を継承 { public: a3(); ~a3(); private: a2* a; //a2のポインタを持つ }; a3::a3() { t = 3; a = new a2; //コンストラクタの中でnew } a3::~a3() { delete a; //終わったらdelete } int main(void) { a3 a; return 0; } というソースでa3はa2を継承しています。 ですのでa2のprotectedメンバtにアクセスできるはず。 ここまではいいのですが、 a2のコンストラクタもa3でアクセスでき無いのでしょうか？ a3のコンストラクタでnewしようとすると error C2248: 'a2::a2' : protected メンバ (クラス 'a2' で宣言されている）にアクセスできません。 a2::a2' の宣言を確認してください。 というエラーが発生します。 やりたいのはa2のような基本クラスが6つほどあり、 a2のメンバ変数が多いので コンストラクタの引数で指定されたものだけ コンストラクタないでnewしたいのです。 なぜ、できないのか？ どうすればnewできるようになるのか…。 非常にお手数ですが、 どなたかご回答いただけませんでしょうか？ 長文になりましたが、宜しくお願いいたします。

  • C++ クラス概念

    以下の違いがよく理解できていません。 どなたかご教授お願い致します。 (1)クラスのメンバー変数に static を付けた場合と、 メンバーではない変数に static を付けた場合の違い (2)クラスのメンバー関数に static を付けた場合と、 メンバーではない関数に static を付けた場合の違い [プログラム例] class MyClass { public: static int internalCount; static void func(MyClass& a) {/* ... */} }; static int i; static void f(MyClass& a) { /* ... */ }

  • C++ クラスについて

    クラスについて今勉強しています。 そこで質問なんですが クラスの中にクラスというのは実現可能なのでしょうか？ クラスＡの中にクラスＢとクラスＣが入っている状態で、 クラスＡの中にある関数ＸでクラスＢやＣの関数を呼ぶような処理を目指しています。 またクラスＡに宣言されている変数をクラスＢ,Ｃの中で変更したりできないでしょうか？

  • c++でのクラス設計について

    　今，c++で色を表すクラスを作っています． 　色とはいっても，RGBやXYZなどいろいろな色がありますよね？もし，「各成分の合計を求める」なんて言う関数を作ろうと考えるとすると（必要ないですが^^…）RGBに対しても，XYZに対しても，というように２つ以上作るのは面倒です．そこで，n成分を持つ色のクラスというものを作って，そのクラスを継承して，RGB, XYZなどを表せないかと考えました． 　つまり… class color3{ double data[3]; ~~~~~~ }; class rgb : public color3{ ~~~~~~ }; class xyz : public color3{ ~~~~~~ }; （実際color3はtemplateクラスで，double data[3]もポインタを使って要素数を変更できるようにしています．） もちろんこの方法で正常に動いていますが…．これではrgbクラスの変数cを宣言して，成分を参照するときに，赤色ならc.data[0]，緑色ならc.data[1]というように，data[~]で参照しなければなりません．しかし，これでは見にくいと思っています><．できるなら赤色ならc.r, 緑色ならc.gというように参照したいです．もちろん派生クラスの中にdouble r, g, b;　と宣言すればいいのですが… ．そうしてしまうと，一番最初の目的が達成できません．ポインタを使ったり，メンバの値を取得する関数（getR(), getX()みたいな…）を作れば解決できますが，この方法を使うと*(c.r), c.r()のような参照をしなければならず，「c.r」というようなきれいな参照はできません．贅沢を言っているようではありますが．c.rと表せるようなクラスの設計の仕方はあるでしょうか？ 　現在ぼんやりと考えているのは class color3{ double data[3]; ~~~~~~~ }: class rgb : public color3{ double &r = data[0]; double &g = data[1]; double &b = data[2]; ~~~~~~ }; class xyz : public class3{ ~~~~~~ }; のように，参照変数を使えないか，と思っています．もちろん上の方法ではコンパイルエラー となりましたが>< 　質問が長くなってしまいすいません>< 　よろしくお願いします． 必要ないかもしれませんが参考までに… 【環　　境】MaxOS 10.6 【言　　語】c++ 【開発環境】Xcode 【 レベル 】プログラミング歴6年　c歴4年 c++歴1年の大学１年です

  • C++ のクラスの定義

    C++の初心者です。 C++のことでお聞きしたいことがあります。 namespace test1{ class A { friend class B; protected: int a; }; } とtest1の下にAクラスを作り namespace test2{ class B { public: test1::A A1; }; } と別のnamespaceにBというクラスを作り、 A1というAのインスタンスを持ちたいのです。 しかし、Aというクラスを先に宣言しているので Bというクラスが解らないらしく friendがうまく働きません。 friend class test2::B としても test2がまだ宣言されていないので、 ？？？？とコンパイルエラーが出ます。 先にtest2::B　を宣言してしまうと、 今度はtest1::Aが解らずに エラーが出てしまいます。 こんな場合はどのように書けばよろしいのでしょうか？ 環境はwindows2000でVC++ 6.0です。 どうか宜しくお願いいたします。

  • C++でfriendクラスにしているのにprivateメンバにアクセスできない

    C++でメンバ変数をprivateにして、特定のクラスにだけ公開するようにクラスをfriend指定したのですがprivateメンバにアクセスできませんとエラーが吐かれてしまいます。 先行宣言したりもしてみたのですがどうしても使用できません。 何か心当たりのあるかた教えてください。 class A { friend class B; private: int a; }; class B { public: void test( A *a ) { a->a = 0; } }; コードは違いますがこんな感じのことをしたいのです。 /* コンソールで小さなプログラムでテストしてみると動くのにいざ実際のソースに組み込むと動かないという奇妙な状態です。よろしくお願いします。 */

  • 継承・実装の関係で悩んでいます。

    継承・実装の関係について悩んでいます。 ここでは、アクセス制御を考えずに、インスタンスかstaticかabstract(ここではabstract final staticやabstract classのこと)の違いで、どう継承するのか考えています。 // 継承　はメソッドのオーバーライドのことを考えます。(オーバーロードは考えない) クラスAからクラスBでオーバーライドしたメソッドは、 クラスCでさらにオーバーライドできるのでしょうか？ クラスCが　クラスBのクラスAからオーバーライドしたメソッド をクラスBのメソッドとして見たときに、オーバーライドすることは可能なのでしょうか？ クラスA │ インスタンスフィールドA │ staticフィールドA │ │ クラスA() { } │ │ インスタンスメソッドA () { } │ staticメソッドB() { } ↓ クラスB extends クラスA │ インスタンスフィールドA // 継承 │ インスタンスフィールドB │ staticフィールドB │ │ サブクラス1() { } // コンストラクタは継承しない、super()で呼び出す │ │ インスタンスメソッドA () { } // 継承 │ │ インスタンスメソッドB () { } │ staticメソッドB () { } ↓ クラスC extends クラスB implements インタフェースD, ... ↑ インスタンスフィールドA // クラスBのフィールドを継承 │ インスタンスフィールドB // クラスBのフィールドを継承 │ インスタンスフィールドC │ staticフィールドC │ │ サブクラス2() { } │ │ インスタンスメソッドA () ｛ ｝ // クラスBのメソッドを継承 │ インスタンスメソッドB () { } // クラスBのメソッドを継承 │ インスタンスメソッドD () { } // インタフェースDのメソッドを実装 │ インスタンスメソッドE () { } // インタフェースDのメソッドを実装 │ │ │ インスタンスメソッドC () { } │ staticメソッドC() { } │ interface インタフェースD extends インタフェースE ↑ │ staticフィールドD // public static final │ │ インスタンスメソッドD() { } // public abstract │ インスタンスメソッドE() { } // 継承 │ interface インタフェースE staticフィールドE // public static final インスタンスメソッドE() { } // public abstract

    • ベストアンサー
    • Java
  • C++ 静的クラスの役割が分からない

    こんばんは。 C++で静的オブジェクトがなんなのか試しにプログラミングしているんですが どんな役割があるのか試してもよく分かりません。 static class HOGE { public: int *hoge; private: int value[2]; char moji[6]; public: HOGE() { cout << "HOGE\n"; } void MOJI() { strcpy(moji, "MOJI\n"); } void TEST() { cout << "moji = " << moji << endl; } static void staticMOJI() { } }sObj; sObjからは静的メンバ関数と静的メンバ変数は呼び出せるようです。他は呼び出せません。 そのようにするためにstatic宣言をするものなのでしょうか？ このsObjからメンバ変数(*hoge)を呼び出すことは無理なんでしょうか？

  • クラスの継承でメンバ変数が・・

    初めてクラスの継承をしている者です。 親クラスＡ　のメンバ変数らを 親クラスＡからの処理で呼び出した 子クラスＢ　で変更したいのですが、 子クラスＢ内ではちゃんと処理で数値が変化するのですが、 親クラスＡに戻ると数値が全て初期値に戻ってしまいます。 継承方法は単に　public 親クラスA　と記述しているだけで、 親クラスＡと子クラスＢは別々のファイルに１クラスずつで記述しています。 コンパイルは警告もエラーも出ず動きます。 子クラスから親クラスのメンバ変数を変更するにはどうすれば良いのでしょうか？

  • static変数を持ったクラスを継承した複数のクラスで、独立してその変

    static変数を持ったクラスを継承した複数のクラスで、独立してその変数を使いたい。 例えば、 class AbstractStatic{ public static String str; } class Ex1 extends AbstractStatic{} class Ex2 extends AbstractStatic{} というクラスがあったとして、 Ex1 e1 = new Ex1(); ex1.str = "hoge"; といれると、 ex2.str で、hogeが入ってしまいます。同じようなstatic変数を持つクラスを複数作りたいのですが、継承させると、独立して変数を持たせられません。 同じようなクラスに直接static変数を、記述するしかないのでしょうか？

    • ベストアンサー
    • Java
ポータブルブルーレイディスクの再生に問題が発生しています
このQ&Aのポイント
  • ポータブルブルーレイディスクの再生ができず困っています。購入後すぐに再生ソフトをインストールしましたが、4K ULTRAHDの再生ができません。
  • ノートパソコンでポータブルブルーレイディスクの再生を試みていますが、4K ULTRAHDの再生ができません。再生ソフトは正しくインストールしています。
  • 購入したポータブルブルーレイディスクが正常に動作せず、4K ULTRAHDの再生ができません。再生ソフトの設定を確認しましたが問題は解決しません。
回答を見る

注目のQ&A

カテゴリ

専門家に質問してみよう

職業から探して質問する

あなたにピッタリな商品が見つかる！ OKWAVEセレクト

質問する