Androidプログラミングにおける結合度の考え方について

Question

最初に、本質問はAndroidのプログラミングが題材ではありますが、Javaひいてはオブジェクト指向全般のプログラム設計の在り方の観点からお答えを頂戴したいものでありますので、そのようにご覧いただければと思います。ここまで多くの質問に回答する側でありながら、基礎的な概念が欠落していることを露呈するような質問をするのは大変お恥ずかしいことですし、これまで私の回答をご覧頂いた全ての質問者様に申し訳ないことではありますが。

Androidアプリケーションにおいて、画面を構成する主体となるActivityクラスが持つメソッドなどを、他のクラスから呼び出したいという類の質問をしばしば目にします。大雑把な例としてはこのようなものです。

SubClass.java

class SubClass {
    void function() {
        // ここでActivity#findViewbyId()を使いたい
    }
}

こうした質問において、かつて私は下記のような回答をすることがしばしばありました。（この例では渡すのがMainActivityである必要はありませんが、その必要がある事例でも同様のパターンで回答していたということで括ってください）

SubClass.java

class SubClass {
    private MainActivity mActivity;

    SubClass(MainActivity activity) {
        mActivity = activity;
    }

    void function() {
        // ここでActivity#findViewbyId()を使いたい
        TextView textView = mActivity.findViewById(R.id.textView);
        textView.setText("あいうえお");
    }
}

この時点でお分かりの方も多かろうと思いますが、これは結合度の観点から非常に好ましくない回答であったようです。

実際の例として、かつて下記のような質問でいずれも密結合である指摘を受けました（上の例についてはまた違った問題も孕んでいるが）。
変数が【NullPointerException】。その対処方法に関して - teratail
android studio あるタイミングでのsetText()について - teratail

いずれの指摘も大変ごもっともなものであり、私の不勉強においては汗顔の至りです。ただ、その一方で同種の問題においてよくよく調べると、単純に呼び出し元のインスタンス参照を渡すことを提案しているもの（上記に記した「かつての回答」と同じ手法）も見受けられます。そのほんの一例ではありますが。
Activityを継承していないクラスから画面遷移を行う - Stack Overflow (JP)

もちろん、このような事例がいくつかあったからと言って、先の私の回答へのご指摘を否定するものではありません。ただ、小さな事例に対してまでそこまで疎結合にこだわる必要はあるのだろうか？と迷いを持っていることは事実ではあります。

こうした入門レベルの小さなコードで、（恐らく）その後の拡張性はあまり意識されていないようなコードに対しても、疎結合の意識は決して欠かすことはあってはならないのか、それともそれが質問者にとって簡易・簡潔であれば例え密結合であろうとも許容れることがあってもいいものなのか、（中間的な思想もあるかもしれませんが）皆様がどのようにお考えになるかお聞かせ願えないでしょうか。

Answer 1

私は、回答はできるだけ一般的に正しいとされている考え方に基づいてするべきと考えています。
その場しのぎの方法ではなくて、質問者の方が今後も生かせるような考え方をしたほうが良いかなと。
その意味では、疎結合にするべきだし、Androidアプリでいうと、具体的なActivityのインスタンス(MainActivityなど)を意識するという事自体がおかしいというか、普通はやらないかなと。
入門レベルのコードだからといって、拡張性を意識しない回答はよろしくないなと思います。
逆にある程度経験のある方で、いろいろ分かっている人がわざと崩すというのはアリでしょうね。
とはいえ、teratailの質問者の方は基本的にプログラミング初心者で何が正しいのかも分からないような人が多いと思いますので、ある程度一般的にはこうするべきというものを回答してあげるのがベストと思います。

これはあくまでも私自身の考え方です。
回答者は自由に回答していいのですから、正解はないと思います。
したがって、入門レベルのコードに対して拡張性は不要だし、問題が解決するほうが重要というのも全然ありです。

Comments

[keicha_hrs]

ご回答ありがとうございます。まさにこのようなご意見をいただきたかったので、感謝します。