プログラマ専用SNS ミクプラ

class Character
{
public:

	Character(void);
	virtual ~Character(void);

	virtual void update(void) = 0;
	virtual void draw(void);

	//位置アクセサ
	int getX(void);
	int getY(void);
	void setX(int x);
	void setY(int y);
private:
	//左上座標
	int x, y;
};


class Player :
	public Character
{
public:
	Player(void);
	~Player(void);

	void update(void);
	void draw(void);
private:
	Weapon *weapon;
};

class Weapon
{
public:
	Weapon(void);
	~Weapon(void);

	//弾などの描画、更新関数
	virtual void draw(void) = 0;
	virtual void update(void) = 0;
};


class RapidShotWeapon :
	public Weapon
{
public:
	RapidShotWeapon(void);
	~RapidShotWeapon(void);

	void update(void);
	void draw(void);
};

• Weaponクラスのupdate()関数の引数で座標を持ってくるようにする
• RapidShotWeaponクラスのコンストラクタの引数でPlayerクラスのポインタを持ってくるようにする

h2so5 さんが書きました:updateの引数で渡すのが自然だと思います。Playerクラスの座標が必要でないならば使わなければいいだけなので。

softya（ソフト屋) さんが書きました:コントローラクラスが座標などを中継してはどうしょう。
そうすれば、Weaponクラス側がPlayerクラスの直接参照を必要としなくなります。
コントローラクラスは各クラスの依存ベタベタになりますが、その他のクラスの独立性は高まります。

usao さんが書きました:最も簡単には，
プレイヤーの座標を取得するための関数　を用意すれば良いのではないでしょうか．

softya（ソフト屋) さんが書きました:何故クラス同士を疎結合にするか、カプセル化は何のためかを常に考えて作れば自然と収まる形に近づくと思います。
疎結合やカプセル化はバグを出さない・メンテナンス性を上げる手段にすぎないのです。何時でも綺麗で完璧な答えは難問です。
試行錯誤は一生続きますよ。失敗もあるし。

usao さんが書きました:>それはグローバル関数か、softyaさんのように外部クラスを用意するということでしょうか？

どちらでも良いと思います．
ともかくその時点でのプレイヤの位置という情報をどこかからどうにかして引っ張ってきてくれさえすれば．

（>RapidShotWeaponクラスがPlayerクラスを参照
　というのを避けたいということであれば間に何かしらワンクッション挟めば良い
　という意味においては同じ）

ISLe() さんが書きました:解決しているようですが、わたしがふだん使っている手法を書いておきます。
この掲示板で過去に同じ話題で書いたこともあるのですが。

RapidShotWeaponにはあらかじめゲームコンテキストというオブジェクトへのポインタを渡しておきます。
ゲームコンテキストは、直接関連を持たないオブジェクトの情報が必要な処理へのインターフェースです。

例えばゲームコンテキストへのポインタがpContextだとして、
pContext->GetShotTarget()->GetPosition()
というふうにショットが向かう先の座標を取得します。

GetShotTargetでショットが向かう先を対象にしたインターフェースを取得します。
直接Playerクラスとはしないで抽象化し、インターフェースを実装する先で柔軟に対応できるようにします。
インターフェース(へのポインタ)を挟むことで、直接の関係を持たないソースファイルには定義を非公開とすることができます。

コンテキストからどのようなインターフェースを取得できるようにすると良いかは設計によりますが、インターフェースを取得する際に引数も使えますし、けっこう少ない数にまとめることができるので複雑にはなりません。
インターフェースからさらにインターフェースを取得するような構造にすればソースファイルの依存関係をさらに絞り込めます。
あらかじめ必要なインターフェースを全部用意できれば良いのですが、なかなか難しいことなのですよね。