TryCatchのコード簡略化

プログラミングを初めて１ヶ月の初心者です。
Assetsから画像をTry Catchで読み込んでいるのですが、画像が約１０枚になった為、コードが長くなり、見にくい状態です。
見やすく記述できる方法はないでしょうか。
よろしくお願いします。

Java
1 ImageView imageView1 = findViewById(R.id.image);
2
3        try (InputStream istream = getResources().getAssets().open("images_activity/rugby.jpg")) {
4            Bitmap bitmap = BitmapFactory.decodeStream(istream);
5            imageView1.setImageBitmap(bitmap);
6        } catch (IOException e) {
7            e.printStackTrace();
8        }
9
10        ImageView imageView2 = findViewById(R.id.image1);
11
12        try(InputStream istream = getResources().getAssets().open("images_activity/post_horse.jpg")){
13            Bitmap bitmap = BitmapFactory.decodeStream(istream);
14            imageView2.setImageBitmap(bitmap);
15        }catch (IOException e){
16            e.printStackTrace();
17        }
18    }

行動規範の内容に同意します

回答1件

ベストアンサー

ImageView imageView1 = findViewById(R.id.image);

try (InputStream istream = getResources().getAssets().open("images_activity/rugby.jpg")) {
    Bitmap bitmap = BitmapFactory.decodeStream(istream);
    imageView1.setImageBitmap(bitmap);
} catch (IOException e) {
    e.printStackTrace();
}

↑ここをメソッドとして切り出せばよいのでは？

// メソッド部：
ImageView createView(int viewId, String path){
	ImageView imageView1 = findViewById(viewId);

	try (InputStream istream = getResources().getAssets().open(path)) {
		Bitmap bitmap = BitmapFactory.decodeStream(istream);
		imageView1.setImageBitmap(bitmap);
	} catch (IOException e) {
		e.printStackTrace();
	}
	
	return imageView1;
}

// 使用箇所：
ImageView imageView1 = createView(R.id.image, "images_activity/rugby.jpg");
ImageView imageView2 = createView(R.id.image1, "images_activity/post_horse.jpg");

投稿2018/06/05 09:45