複数のクラスで共通で利用する変数をどのように管理するか。

Question

複数のクラスで共通で利用する変数がありますが、どのように管理するのが良いのでしょうか。

例えば、ClassA,ClassB,ClassCで共通で利用する変数があります。この変数は、それぞれのクラスで読み込んで利用したり、値を変えたりします。

この変数を仮にClassAのメンバー変数にしてgetter、setterのメソッドを作りClassB,ClassCにClassAのsetterメソッドを作りそれでsetして上記の変数をそれぞれのクラスで利用出来るようにしたのですが、密結合だし、なんか変です。

どのようにすべきかご教示お願いします。

class A {
        int value;

        int getValue() {
            return value;
        }

        void setValue(int value) {
            this.value = value;
        }
    }
    
    class B {
        A a;

        void setA(A a) {
            this.a = a;
        }
    }

    class C {
        A a;

        void setA(A a) {
            this.a = a;
        }
    }

Answer 1

変数を保持するクラスを別に作り、必要によってはシングルトン化を行なったり、アクセス用のインターフェースを作成したりするといいと思いますよ。
他にはファイルに書き込んだり、データベースに保存したりするのも方法としてありますね。

Answer 2

interfaceを使ってgetValue,setValue と setAの実装を強制づけてみました。
変更
依存度を下げてみました。
class B、class C はvalueをどのクラスが持っているか、気にしなくてもよいようにしました。
さらに変更
valueを直接所有しているクラスはValueOwnerを実装し、valueを間接利用しているクラスはValueSharerを実装して役割を明確化しました。
valueを直接所有しているclass Aだけは、クラスの生成法もクラス間の関係も知っている、というイメージです。

interface ValueAccessor {
	int getValue();
	void setValue(int value);
}

interface ValueOwner extends ValueAccessor {
	void setSharers(ValueSharer... sharers);
}

interface ValueSharer extends ValueAccessor {
	void setOwner(ValueOwner owner);
}

class A implements ValueOwner {
	private int value;

	public static void main(String[] args) {
		A a = new A();
		B b = new B();
		C c = new C();

		a.setSharers(b, c);
	}

	@Override
	public void setSharers(ValueSharer... sharers) {
		for (ValueSharer sharer : sharers) {
			sharer.setOwner(this);
		}
	}

	@Override
	public int getValue() {
		return value;
	}

	@Override
	public void setValue(int value) {
		this.value = value;
	}

}

class B implements ValueSharer {
	private ValueOwner owner;

	@Override
	public void setOwner(ValueOwner owner) {
		this.owner = owner;
	}

	@Override
	public int getValue() {
		return owner.getValue();
	}

	@Override
	public void setValue(int value) {
		owner.setValue(value);
	}
}

class C implements ValueSharer {
	// class B と同様
}

Comments

[退会済みユーザー]

ご回答ありがとうございます。

なるほどinterfaceでやりとりですね。良いかもしれないです。

Answer 3

クラスDを新しく作ってA,B,Cで共通してその中の
private static int value;
を使うといいのではないでしょうか A,B,Cいずれかのクラスに
private static int value;
及びstaticのsetter,getterを作ってそれを使うのでも同じ意味があると思います

class D {
        private static int value;

        static int getValue() {
            return value;
        }

        static void setValue(int val) {
            value = val;
        }
    }

Answer 4

疎結合な実装方法を探しているようですので、DI（※Dependency Injection）について説明します。
※どのタイミングで共有する値が変わるのか、によって実装は変わってきます。もしも保存時だけ…というのであれば、リポジトリに共有するための機能を追加していきます。

Dependency Injectionとは、依存性を外から注入する…という意味です。masuoさんの例の場合は、「値を共有するためのオブジェクトを渡してあげる」ということです。この利点は、DのスコープによってABCクラスで共有する値のスコープも変えることができることです。
例えば、常に値が共有され続けるならばクラスDはシングルトンにすればいいでしょう。とあるシーン（画面中）でのみ同じという場合にはその都度生成し、ABCのインスタンス化時に同じDインスタンスを渡します。
疎結合を保て、利用すべきシナリオに合わせて共有方法が変えられるというのがDIのメリットです。デメリットとしては、Dの取得元（インスタンス化方法）を参照することでしか共有方法が分からないことや、コーディング量が増えてしまうというのがあり、最初の方は使いこなすまでに慣れがいると思います。

public class A
{
    private D d;
    public A(D d)
    {
        this.d = d;
    }

    public setValue(int value)
    {
        d.setValue(value);
    }
    public getValue()
    {
        return d.getValue();
    }
}

// 他のBCについて同様、Dはvalueプロパティを持つだけのbeanみたいなものです（この例では）
public class B;
public class C;

// ファクトリを作らなければならないわけではないです、説明用
// ※これは手抜き実装ですが、シナリオによっては十分な実装にもなりえます。
public class ABCFactory
{
    // このdの寿命でABCのデータの共有のされ方が決まる。
    // 例えばDがシングルトンなら、全てのABCで常に値が共有される。
    private static D d;
    private static D getD()
    {
        if (d == null)
        {
            d = new D();
        }
        return d;
    }
    
    public static A createA()
    {
        return new A(getD());
    }
    // B & C も同様
}

上記Factoryは固定で書かれており、柔軟とは言えませんが…DIコンテナという、上記のFactoryをより柔軟に実装するためのツールがあります。ただDIそのものは慣れるまで使うのが結構大変です。アイディアだけ盗むのが最初は良いんじゃないかと思います。

Comments

[退会済みユーザー]

ご回答ありがとうございます。

DIコンテナーはちょっとだけ齧ったことがあります。Daggerとか。確かにいいかもしれませんね。検討させて頂きます。