getDrawable()の扱い

Question

###getDrawable()の扱い
私は画像表示の方法として、String.xmlにR.draｗableの名前を指定し、それをactivityで動的に取得するという方式をとっています。そしてそれは実際に上手くいっています。
以下はその簡略したコードです

ImageView base= (ImageView) findViewById(R.id.base);
//numberが変化する。Strings.xmlから動的にIdを取得。
int baseId = getResources().getIdentifier("base" + number, "string", getPackageName());
base.setImageResource(baseId);

<string name="base1">@drawable/taro_base</string>
<string name="base2">@drawable/hanako_base</string>
<string name="base3">@drawable/taro_base</string>
・・・・

ここからが本題で、実はある事情により、javaコードを使って画像を重ねる必要が出てきました。
さてどうしよう思い、LayerDrawableを使うことにしました。
そのさいに参考にしたのがこのサイトです。
具体的に言うと以下の部分です。

// LayerDrawable の素材となる Drawable を取得
Resources r = getResources();
Drawable shape = r.getDrawable(R.drawable.shape_sample);
Drawable droid = r.getDrawable(R.drawable.droid);
 
// LayerDrawable を生成
Drawable[] layers = { shape, droid };
LayerDrawable layerDrawable = new LayerDrawable(layers);
 
// padding を設定する
int padding = r.getDimensionPixelSize(R.dimen.padding);
layerDrawable.setLayerInset(1, padding, padding, padding, padding);
 
// View にセットする
ImageView view = (ImageView) findViewById(R.id.image_view);
view.setImageDrawable(layerDrawable);

これにならってコードを書きました。しかしアプリは止ってしまいました。
デバックしてみるとどうやらgetDrawable(baseId)のところでエラーが起きているようです。
以下にそのコードを記します。
（重ねる画像について、全てのコードを書くと冗長になるので、適宜略しています）

ImageView base= (ImageView) findViewById(R.id.base);
略ーー
int baseId = getResources().getIdentifier("base" + number, "string", getPackageName());
略ーー
Drawable base=getResources().getDrawable(baseId);//ここで止る

Drawable[] layers = { base, 略};
        LayerDrawable layerDrawable = new LayerDrawable(layers);
        ImageView view = (ImageView) findViewById(R.id.略);
        view.setImageDrawable(layerDrawable);

これはgetDrawable()がsetImageDrawable()のように、Strings.xmlで取得した文字列のIDを持ってくることができないということなのでしょうか？
参考にしたサイトにはgetDrawable(R.drawable.shape_sample)とあるのでこの引数にStrings.xmlで取得した文字列のIDを入れてあげればいいと思ったのですが・・・・

だとすればどのように解決できるでしょうか？
どなたかご教授いただけると助かります・・・・！

追記ーーー
調べたところ今はgetDrawable()が非推奨になっているようですね・・・
ですが、Drawable drawable = ResourcesCompat.getDrawable(getResources(), baseId, null);としてもだめでした・・・

Answer 1

すでにベストアンサーがついちゃったようですが、もっとシンプルに解決するためのアドバイスです。

ご質問の今の方法だと、

• strings.xml内にdrawableのリソース名を定義している

→ 文字列を取得してそこからIDを取得して、と処理が多くパフォーマンスがよくない
→ そのぶん無駄なリソースが確保されアプリの容量が増える
→ タイポした際に表示できないし気付きづらい

• そもそもgetIdentifierはあまり効率の良くないメソッドなので使うのは推奨されていない

→ 直接IDで指定できるならなるべくそうすべき
→ https://developer.android.com/reference/android/content/res/Resources?hl=ja#getIdentifier(java.lang.String,%20java.lang.String,%20java.lang.String)

といくつか無駄な部分があります。

必要なのは番号に対応するdrawableを取得するための方法だと思いますので、以下のように配列で定義してしまってはどうでしょうか？

final int[] baseIds = new int[] {
    R.drawable.taro_base,
    R.drawable.hanako_base,
    R.drawable.taro_base,
    ...
};

// 使うとき
Drawable base = getResources().getDrawable(baseIds[number]);

Comments

[hood]

kakajikaさんですね、ご無沙汰しております・・

なるほど・・・そういうやり方があるのですね。
確かに今のやり方だとコードがすごく複雑になって無駄が多いと、気づかされました。
ご指摘、ありがとうございます！
もう一度コードをスマートにするため見直してみようと思います！

Answer 2

Resources#getDrawableメソッドに、文字列リソースIDを渡しているからではないですか。

文字列リソースにドローワブルリソースが書かれているなんて事はgetDrawableメソッドの知る由もない事ですから、自分でResources#getStringメソッドで文字列を読み込んで、Resources#getIdentifierメソッドで更にドローワブルリソースIDに変換する必要があります。

あと、

<string name="base1">@drawable/taro_base</string>

「@drawable/」は要らないのでは。

---

Resources#getDrawableメソッドが非推奨なのは、テーマを指定しないメソッドですね。

Resources#getDrawable(int, Theme)や、Context#getDrawableは非推奨ではありません。
要は、~~テーマを指定しろという事です。 ~~nullでもいいからテーマを指定しろという事です。

public class MainActivity extends Acitvity {
  @Override
  protected void onCreate(Bundle savedInstanceState)
  {
    super.onCreate(savedInstanceState);

    // 非推奨
    getResources().getDrawable(R.drawable.ic_launcher);
    // 推奨
    getResources().getDrawable(R.drawable.ic_launcher, getTheme());
    // 推奨
    getDrawable(R.drawable.ic_launcher);
}

Comments

[hood]

katsukoさんですね・・・fragmentに関する質問をしたときもお世話になりました。

今回も回答ありがとうございます。
ご指摘の通り手直ししたら、無事解決しました！
Resources#getDrawableメソッドの非推奨される理由なども、教えていただき、勉強になりました。
本当になんどもありがとうございます！