【Android】カスタムビューにスクロール機能をつけたい

Question

画像を端末内から取得し、写真一覧を表示する画面を作成しているのですが、スクロールを行うことが出来ません。

下記のl.addView(new MyView(this, test));を実行してもMyViewのonDrawが実行されないので何も表示されません。
ScrollViewを使用せずに表示を試みると正常に動作します。
原因が全く分からず、二日間ずっと悩んでます...。
皆さんの回答をお待ちしています。
どうかよろしくお願い致します。

public class Tab3Activity extends Activity{
	@Override
	public void onCreate(Bundle savedInstanceState){
		super.onCreate(savedInstanceState);
		
		LinearLayout l = new LinearLayout(this);
		l.setOrientation(LinearLayout.VERTICAL);
		
		ScrollView sv = new ScrollView(this);
		setContentView(sv);
		sv.addView(l);
		
				
        Cursor cursor = getContentResolver().query(
        		MediaStore.Images.Media.EXTERNAL_CONTENT_URI,
        		null,
        		null,
        		null,
        		null
        );
        cursor.moveToFirst();
        
        int cnt = 0;
        ArrayList<Bitmap> test = new ArrayList<Bitmap>();
        BitmapFactory.Options options = new BitmapFactory.Options();
        options.inSampleSize = 35;
        do {
            test.add(BitmapFactory.decodeFile(cursor.getString(cursor.getColumnIndex(MediaStore.Images.Media.DATA)), options));
            cnt++;
            cursor.moveToNext();
        }while(cnt < 20);
        cnt = 0;
        
        l.addView(new MyView(this, test));
	}
}

//Custom View
public class MyView extends View {
	Paint paint = new Paint();
	ArrayList<Bitmap> mBmps;

	public MyView(Context context, ArrayList<Bitmap> bmps) {
		super(context);
		mBmps = new ArrayList<Bitmap>(bmps);
		setWillNotDraw(false);
	}
	
	@Override
	public void onDraw(Canvas canvas){
		int height = 10;
		
		for(int i = 0; i < mBmps.size(); i++){
			if(i % 2 == 0){
				canvas.drawBitmap(mBmps.get(i), 10, height, paint);
				height += 100;
			}else{
				canvas.drawBitmap(mBmps.get(i), 200, height, paint);
			}
		}
	}
}

Answer 1

MyViewのサイズはいくつですか?どこで設定されていますか?されていませんよね?
となると、サイズは0になっていると考えるのが妥当だと思います。

Viewを派生したカスタムビューのサイズを決めるには、MyViewの中でonMeasureメソッドをオーバーライドして適切な処理を行ってください。

• カスタムビューを作る際に重要なonMeasure()とは？
• onMeasureとonLayoutについて理解する

もう一つ、LayoutParamsで指定する方法もあります。

@Override
public void onCreate(Bundle savedInstanceState){
    LinearLayout l = new LinearLayout(this);
    l.setOrientation(LinearLayout.VERTICAL);

    // とりあえず高さを256pxに設定
    l.addView(new MyView(this, test),
          new LinearLayout.LayoutParams(ViewGroup.LayoutParams.MATCH_PARENT, 256));

前者の方が行儀のイイやり方だと思いますが、どちらの手段を取るかはご自身でお決めください。

Comments

[panasonicsite]

ありがとうございます！
「katsuko」さんのアドバイス通りに改良するとスクロールすることができました！！
独学でやっているので質問する人がおらず大変困っていたのですが、無事解決することができてよかったです。
本当にありがとうございました！！！

Answer 2

正直に言えば、どこで何をしているかコードからわかりません。特に怪しいコードも見当たりません。しかし、１つのクラスで
複数のBitmapクラスを過度に使うことはあまりいいこととは言えないです。
ListViewを使うことはできないでしょうか？

Comments

[panasonicsite]

回答ありがとうございます！！

Bitmapクラスを使用すると速度が格段に落ちるので、本当は使用したくないです。
しかし、今私が作っているアプリは画像をグリッド表示したいので、ListViewは使用出来ない状態です。
それならGridViewを使用するのが定石だと思うのですが、Viewを大量に作ることで動作速度が極めて遅くなった為、上記のような画像描画のみを行うカスタムViewを作っています。

そこで気になるのですが、Bitmapクラスを使わずに端末内にある数千枚の画像を読み込み、操作性に不自由ない動作速度で画像をグリッド表示する方法はあるのでしょうか？

Androidにプリインストールされている「アルバム」アプリは動作がめちゃくちゃ早いのでどうやって作っているのか非常に気になります。
Android NDKなどを使って、CやＣ＋＋で画像取得、表示部分だけを作らないといけないのでしょうか．．．。

Androidアプリ作成初心者なので間違ったことを言っているかもしれないです。
その時は、ご指摘願います。