MainActivityの記述を分散させる

Question

やりたいこと

MainActivityの記述を減らして見やすくするために、oncreate内でbackgroundcolorを設定する記述を、別に用意したjavaファイルに移して呼び出したいと考えています。
具体的には

public class MainActivity extends AppCompatActivity {

Button button;

@Override
protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity_main);

    button = (Button) findViewById(R.id.button);

    Set set = new Set();
    set.color();
}

}

public class Set extends MainActivity {

void color(){
   button.setBackgroundColor(Color.RED);
}

}

とするとNullPointerExceptionのエラーが出ます。

しかし、

public class MainActivity extends AppCompatActivity {

Button button;

@Override
protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity_main);

    button = (Button) findViewById(R.id.button);

    Set set = new Set();
    set.color(button);
}

}

public class Set extends MainActivity {

void color(Button bt){
   bt.setBackgroundColor(Color.RED);
}

}

とすると、希望通りに動きます。
しかし、ボタン1つの色を設定するだけならばいいのですが、例えばactivity_main.xmlにボタンが１００個あって、そのボタンの色をそれぞれoncreate内にて指定したい場合、自作のcolor()メソッドの引数が１００こ必要になってきてしまいます。

どうにかして、MainActivityのoncreate内では
set.color();
といったような引数なしの簡潔な方法で記述できる方法はないでしょうか？

Layoutinflaterを使ったらいいのかとも考えたのですが、勉強不足でこの場合どう使ったらいいのかよくわかりません。

Answer 1

可変引数にするか、リストで渡すかすると引数はいくらでもわたせますね。

コードはコード用の記述を使ってください。
また、Activityを継承しているのは何故ですか？無意味ですよ。

Comments

[hijiri]

ありがとうございます。

リストで渡すといいますのは、配列かArrayListを作っておいて、メソッドの引数にそれを渡すということでしょうか？

勉強のために、最小構成にして色々試してみているところですので、コードはこれで全部で、実際に動かして試しているコードを貼り付けております。

MainActivityを継承していますのは、MainActivityのoncreateにて

Button button;

@Override
protected void onCreate(Bundle savedInstanceState) {
super.onCreate(savedInstanceState);
setContentView(R.layout.activity_main);

button = (Button) findViewById(R.id.button);

と定義したbuttonをSetクラスにて

void color(){
button.setBackgroundColor(Color.RED);
}

とすることで、親クラスのbuttonをそのまま使えるかなと考えたんですが・・・
結局button がnullになってしまっているようで、そのまま使うことが出来ませんでした。

もしよろしければ、その理由など教えていただけましたらありがたく存じます。

[yona]

そもそも、Activityはプログラマーがインスタンス化するものではありません。
また、継承には親クラスでインスタンス化したものをそのまま利用する機能はありません。

コード用の記述に修正してください。

[hijiri]

そうなんですね、親クラスの変数を利用できるという点から勘違いしていたようです・・・
てっきりボタンなどのインスタンスも利用できるのかと思っていました。

すみません、「コード用の記述に修正」とは、コードを全て記述しろという意味ではなく、｛　｝の中の段を下げてそろえるとか、そういうコードの形式的な修正をしろ、という意味なのでしょうか？
無知で申し訳ありません・・・

[yona]

再利用できないです。

他の質問を見たことはありませんか？
コードの記述に違いはありませんか？
無知かどうかは知りませんが、違いに違和感を持ちましょう。

[hijiri]

了解いたしました！

他の方の質問見て、勉強するようにいたします！
有り難う御座います！