仕事がはじまって今週で4週間目、チケットのコードレビューをお願いした時に、こんなレビューが返って来た

「Contextは引数で渡すんじゃなくて、Applicationクラスのcontextを直接参照したらダメなんですか？」

まず、自分の担当しているプロジェクトでは、Applicationクラスを継承したクラスに、StaticでContext自身のクラスを保持していて、
そのContextを参照せよと言っているのだ。

前提として、Applicationクラスにstaticを持つこと自体がダメであろうことだし。
仮に使えるとしても、結合という観点からContextを直接参照すべきでないと思う。

これには、なんと返信したらいいのか、考えてしまった。

結局、結合の観点と、staticグローバルインスタンスがダメであることを書き、
なんとなくわかってもらえたようだった。

こんなコードがたくさんあるから、バグチケットを処理するのが大変なのです。
結合度が高くて、一箇所修正しようとしても他の箇所も修正する必要が出てくる。
そして中身を追っていくと、結局はグローバル変数にぶつかり、修正とテストが大変なことになる。

だからこのプロジェクトのチケットを片付けるのは、非常に時間がかかるので、
ちゃんと考慮にいれて、作業量を振っていただきたいです。

こういうプロジェクトに新規で一人で参入している時、自分が出会った優秀な人たちはいったいどうしているんだろうか？
前のプロジェクトで一緒だった同じ年の優秀なリーダーに聞いてみたいです。