【Java】あるアルゴリズムのクラス分け

質問

あるアルゴリズムをJavaで実装しました。
大きなファイルになったので、クラス分けしようと思います。

ソースはおおよそ以下のような構成になっています。
1.　ファイル読み出し
2.　アルゴリズム本体
3.　ファイル書き込み
これが1ファイルになっているため、各々をクラス分けします。

ここで問題になるのは、1.～3.の各項目から
アクセスする必要がある変数が存在することです。
通常ですと、解決策は

1.　変数はprivateにして、
　　アクセス専用メソッドを通じて変数にアクセスする。
　　問題点：　変数は、HashMapの巨大な変数であるため、
　　　　　　　やりとりにコストがかかる。
2.　別パッケージを使う。
　　http://d.hatena.ne.jp/kenpoco/20080626/1214440164
　　問題点：　1つのアルゴリズム中なので、別パッケージにする必要がない。
3.　あきらめて変数をpublicで公開
　　問題点：　他人から変数が見えてしまう。

というわけで、自分が知っている方法ではどれも一長一短なのですが、
一般にはこうした状況はどのように解決されているか教えてください。

回答No.2

root139 · ベストアンサー

#1の方が書かれている様にクラス型のフィールドは参照なので、そのまま返すだけなら、性能的なハンデは有りません。

また、一部のクラスにだけ公開したいメンバーが欲しいなら、パッケージプライベートを使うのが一般的でしょう。Javaではアクセス修飾子を付けなければ、パッケージプライベートとなります。
パッケージプライベートなフィールドやメソッドは同じパッケージのクラスからだけアクセスできます。

例) ------------------------------------

package aaa;
public class Class1 {
　private Map<Object, Object> map;
　Map<Object, Object> getMap() { return map; }
}

package aaa;
public class Class2 {
　public static void main(String[] args) {
　　Map<Object, Object> map = new Class1().getMap(); // OK
　}
}

package bbb;
public class Class3 {
　public static void main(String[] args) {
　　Map<Object, Object> map = new Class1().getMap(); // コンパイルエラー
　}
}

----------------------------------------

また、呼出し元によるMapに対する意図しない変更を防ぎたい場合は、java.util.Collections の unmodifiableMap を使う手も有ります。これも元のMapの要素数を気にする必要は有りません。
http://java.sun.com/javase/ja/6/docs/ja/api/java/util/Collections.html#unmodifiableMap%28java.util.Map%29

例) ------------------------------------
return java.util.Collections.unmodifiableMap(map);
----------------------------------------

もっともMapのキーや値が可変クラスであれば、それに対しての変更は防げませんが。

お礼

ありがとうございます。
実はこのソースは最終的にはjarになるはずなのですが、

>パッケージプライベートなフィールドやメソッドは
>同じパッケージのクラスからだけアクセスできます。

ひょっとすると、jarの中のアクセス修飾子のついていないメンバについても
jarの中でだけアクセスできたりするのでしょうか？

回答No.3

root139

> jarの中のアクセス修飾子のついていないメンバについても
> jarの中でだけアクセスできたりするのでしょうか？

アクセス修飾子のついていないメンバはパッケージ内でプライベートという意味になりますので、同じjarの中であっても別のパッケージに属するクラスからはアクセスできませんし、別のjarであっても同じパッケージのクラスからはアクセスできます。

ただし、クラスローダーが違う場合などは、同じパッケージでもアクセスできないケースが出てくるみたいですが、例外的な事象ですので普通は気にする事は無いかと。

回答No.1

hitomura

1.でしょうね。
というか、JavaでネットワークやDBが絡まない話で
> 変数は、HashMapの巨大な変数であるため、
> やりとりにコストがかかる。
というのはちょっと理解できないのですが……。

HashMapは参照型になるからメソッドでやりとりされるのは参照のはずで、そのやり取りでのコストは普通のクラスのやりとりと同じはずです。
ゲッターやセッターで（引数の渡し手や戻り値の受け取り手に内容を勝手に修正されることを嫌って）
引数や戻り値をcloneしているというのなら話は別ですが、あなたの質問されている状況では特にcloneする必要はなさそうに思います。

お礼

ありがとうございます。そのとおりですね。
何も考えずに、return HashMapみたいにやりとりすれば
参照なので、値は共有できますね。

補足

と思ったのですが、
そうすると、ゲッターやセッターをpublicにする必要があると思います。
そのアルゴリズムの内部では変数に自由にアクセスできるようにし、
かつ、外部からは、ゲッターやセッターも隠蔽したいのですが、
そのようなことは可能でしょうか？