18/01/13 20:56:00.55 ogercqIA.net
さんすうのことはわからない
でもまず根本的にだめなところとして
変数が実質すべてグローバル変数になっている。
これならprivateなメソッドはStaticにして引数渡しにすべき。メソッドが何やってんかわからん
コンストラクタでクラスのメンバにアクセスするようなメソッドを呼び出してはいかん、
上と同じ理由で極力引数渡しにして戻り値を自分で設定すべき
ついでにクラス設計がひどい。トポロジーの配列を返すようなメソッドじゃいかんのか
自身のメンバ変数を更新しながら随時計算するようなクラスは典型的なアンチパターン