最初に、本質問は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)
もちろん、このような事例がいくつかあったからと言って、先の私の回答へのご指摘を否定するものではありません。ただ、小さな事例に対してまでそこまで疎結合にこだわる必要はあるのだろうか?と迷いを持っていることは事実ではあります。
こうした入門レベルの小さなコードで、(恐らく)その後の拡張性はあまり意識されていないようなコードに対しても、疎結合の意識は決して欠かすことはあってはならないのか、それともそれが質問者にとって簡易・簡潔であれば例え密結合であろうとも許容れることがあってもいいものなのか、(中間的な思想もあるかもしれませんが)皆様がどのようにお考えになるかお聞かせ願えないでしょうか。
回答1件
あなたの回答
tips
プレビュー
バッドをするには、ログインかつ
こちらの条件を満たす必要があります。
2018/05/05 15:14