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