Randomで出た画像を配列の中から表示したい

Question

gat～gat6までのImageViewに、それぞれRandomクラスで出た変数rの数に応じて、４つある配列のうち１つの配列を選び、
その配列のうち１枚の画像を表示する、という事を６回繰り返すプログラムを組もうとしています
しかし『java.lang.Iterable の配列またはインスタンスのみ繰り返すことができます』というエラーが出てしまい、
それを消すには何というコードを入れれば良いのでしょうか
また、意図した通りにコードを動かすにあたって他にコードの間違いがありましたらご指摘をお願いします

package jp.example.wgamen;

import java.util.Random;

import android.app.Activity;
import android.content.Intent;
import android.content.res.TypedArray;
import android.os.Bundle;
import android.view.View;
import android.widget.Button;
import android.widget.ImageView;

public class SubActivity extends Activity {

	ImageView gat;
	ImageView gat2;
	ImageView gat3;
	ImageView gat4;
	ImageView gat5;
	ImageView gat6;

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

	// 画面に配置したImageViewを設定
	gat = (ImageView)findViewById(R.id.v1);
	gat2 = (ImageView)findViewById(R.id.v2);
	gat3 = (ImageView)findViewById(R.id.v3);
	gat4 = (ImageView)findViewById(R.id.v4);
	gat5 = (ImageView)findViewById(R.id.v5);
	gat6 = (ImageView)findViewById(R.id.v6);

        //整数を入れる為の変数を宣言
        int gatya2;


	// 画像のリストを取得
	TypedArray images = getResources().obtainTypedArray(R.array.r);
	TypedArray images2 = getResources().obtainTypedArray(R.array.sr);
	TypedArray images3 = getResources().obtainTypedArray(R.array.ur);
	TypedArray images4 = getResources().obtainTypedArray(R.array.lr);

	// Randomクラスのインスタンスを生成
	Random r = new Random();

        //０以上100未満の整数の乱数を作成する
        gatya2 = r.nextInt(100);

        //ランダムで出た数が２以下ならLRの配列から１枚画像を表示する
        if(gatya2 <= 2){
        gat.setImageResource(R.array.lr);

}else
        //ランダムで出た数が３以上12以下ならURの配列から１枚画像を表示する
        if(gatya2 <= 12){
        gat.setImageResource(R.array.ur);
}else
        //ランダムで出た数が13以上39以下ならSRの配列から１枚画像を表示する
        if(gatya2 <= 39){
        gat.setImageResource(R.array.sr);
}else{
        //それ以外（それ以降）の数字ならRの配列から１枚画像を表示する
	    gat.setImageResource(R.array.r);
}

//2～5まで繰り返しなので省略・

        //ランダムで出た数が２以下ならLRの配列から１枚画像を表示する
        if(gatya2 <= 2){
        gat6.setImageResource(R.array.lr);
}else
        //ランダムで出た数が３以上12以下ならURの配列から１枚画像を表示する
        if(gatya2 <= 12){
        gat6.setImageResource(R.array.ur);
}else
        //ランダムで出た数が13以上39以下ならSRの配列から１枚画像を表示する
        if(gatya2 <= 39){
        gat6.setImageResource(R.array.sr);
}else{
        //それ以外（それ以降）の数字ならRの配列から１枚画像を表示する
	gat6.setImageResource(R.array.r);
}

	// 各ImageViewに画像を設定 ↓ここで？の部分に何を入れればいいか分からずエラーが出ます
    	for(ImageView tmp : ?)
    	{
    	tmp.setImageDrawable(images.getDrawable(r.nextInt(images.length())));
    	tmp.setImageDrawable(images2.getDrawable(r.nextInt(images2.length())));
    	tmp.setImageDrawable(images3.getDrawable(r.nextInt(images3.length())));
    	tmp.setImageDrawable(images4.getDrawable(r.nextInt(images4.length())));
    	}

よろしくお願いします

追記：回答下さった皆様のおかげで、今こういうコードになっています

package jp.example.wgamen;

import java.util.Arrays;
import java.util.Random;

import android.annotation.SuppressLint;
import android.app.Activity;
import android.content.Intent;
import android.content.res.TypedArray;
import android.os.Bundle;
import android.view.View;
import android.widget.Button;
import android.widget.ImageView;

public class SubActivity extends Activity {

	ImageView gat;
	ImageView gat2;
	ImageView gat3;
	ImageView gat4;
	ImageView gat5;
	ImageView gat6;

	@SuppressLint("Recycle")
	@Override
	protected void onCreate(Bundle savedInstanceState) {
	super.onCreate(savedInstanceState);
	setContentView(R.layout.sub);

	// 画面に配置したImageViewを設定
	gat = (ImageView)findViewById(R.id.v1);
	gat2 = (ImageView)findViewById(R.id.v2);
	gat3 = (ImageView)findViewById(R.id.v3);
	gat4 = (ImageView)findViewById(R.id.v4);
	gat5 = (ImageView)findViewById(R.id.v5);
	gat6 = (ImageView)findViewById(R.id.v6);

        //整数を入れる為の変数を宣言
        int gatya2;


	// 画像のリストを取得
	TypedArray images = getResources().obtainTypedArray(R.array.r);
	TypedArray images2 = getResources().obtainTypedArray(R.array.sr);
	TypedArray images3 = getResources().obtainTypedArray(R.array.ur);
	TypedArray images4 = getResources().obtainTypedArray(R.array.lr);

	// Randomクラスのインスタンスを生成
	Random r = new Random();

        for(ImageView tmp : Arrays.asList(gat, gat2, gat3, gat4, gat5, gat6)) {
            //０以上100未満の整数の乱数を作成する
            gatya2 = r.nextInt(100);
          //ランダムで出た数が２以下ならLRの配列から１枚画像を表示する
            if(gatya2 <= 2){
                gat.setImageDrawable(images4.getDrawable(r.nextInt(images4.length())));
          //ランダムで出た数が３以上12以下ならURの配列から１枚画像を表示する
            }else if(gatya2 <= 12){
                gat.setImageDrawable(images3.getDrawable(r.nextInt(images3.length())));
              //ランダムで出た数が13以上39以下ならSRの配列から１枚画像を表示する
            }else if(gatya2 <= 39){
                gat.setImageDrawable(images2.getDrawable(r.nextInt(images2.length())));
              //それ以外（それ以降）の数字ならRの配列から１枚画像を表示する
            }else{
                gat.setImageDrawable(images.getDrawable(r.nextInt(images.length())));
            }
//gat2～gat5まで同じなので省略
            if(gatya2 <= 2){
                gat6.setImageDrawable(images4.getDrawable(r.nextInt(images4.length())));
            }else if(gatya2 <= 12){
                gat6.setImageDrawable(images3.getDrawable(r.nextInt(images3.length())));
            }else if(gatya2 <= 39){
                gat6.setImageDrawable(images2.getDrawable(r.nextInt(images2.length())));
            }else{
                gat6.setImageDrawable(images.getDrawable(r.nextInt(images.length())));
            }
        }

かなり完成度が高いコードになったと思うのですが、何故か強制終了してしまいます
何度もコードを読み返しているのですが、何が原因なのか分かりません
気になる所といえば、for(ImageView tmp : のtmpに『ローカル変数tmpの値は使用されていません』という警告が出ている事くらいです

Answer 1

多分やりたいのはこういうことですかね？
Random r = new Random();の行の次からとお考え下さい．

for(ImageView tmp : Arrays.asList(gat, gat2, gat3, gat4, gat5, gat6)) {
    //各ImageViewに対して乱数を振り、その結果でDrawableを取り出す配列を決定
    gatya2 = r.nextInt(100);
    TypedArray array = null;
    if(gatya2 <= 2){
        array = images4;
    }else if(gatya2 <= 12){
        array = images3;
    }else if(gatya2 <= 39){
        array = images2;
    }else{
        array = images;
    }
    //決定した配列の長さから乱数を生成し，それをインデックスにDrawableを取り出しセット
    //(ここは個人的に括弧が重なりすぎて読みにくかったため分解)
    int i = r.nextInt(array.length());
    tmp.setImageDrawable(array.getDrawable(i));
}

Comments

[mari_tom]

ありがとうございます！なるほどそう書けばrやsrやurからもちゃんと画像が出るようになりますね！
早速コードをそのように書き換えて実行したのですが、何故か強制終了してしまいます
エラーなどは特に出ていないのですが…原因が他にある事も十分に考えられるので、もう一度書いたコードを見直してきます

[swordone]

強制終了の原因はたいてい何かしらの例外発生で，その例外のログがLogcatに出ているはずです．それを貼って下さい．

[swordone]

追記したコードですが，これでは修正前のコード(6種のgat*に配列から引き出しセット)をただ6周してるだけのコードです．しかも1周ごとにgatya2の値を共用してるので，引き出し元の配列は同じになってしまいます．

[mari_tom]

お待たせしてしみません、LogcatのERRORメッセージがあまりにも多く、どれが原因なのか分からないので、全て貼らせていただきます
06-19 23:31:26.910: E/InputDispatcher(5423): Motion event has invalid pointer count 0; value must be between 1 and 16.
06-19 23:31:26.990: E/InputDispatcher(5423): Motion event has invalid pointer count 0; value must be between 1 and 16.
06-19 23:31:27.160: E/EmojiFactory_jni(5423): Failed to load libemoji.so: dlopen failed: library "libemoji.so" not found
06-19 23:31:27.180: E/Sensor-AccelerometerUI(5600): java.io.IOException: Connection refused
06-19 23:31:28.820: E/InputDispatcher(5423): Motion event has invalid pointer count 0; value must be between 1 and 16.
06-19 23:31:28.910: E/InputDispatcher(5423): Motion event has invalid pointer count 0; value must be between 1 and 16.
06-19 23:31:28.980: E/AndroidRuntime(6967): in writeCrashedAppName, pkgName :jp.example.wgamen
06-19 23:31:28.980: E/AndroidRuntime(6967): FATAL EXCEPTION: main
06-19 23:31:28.980: E/AndroidRuntime(6967): Process: jp.example.wgamen, PID: 6967
06-19 23:31:28.980: E/AndroidRuntime(6967): java.lang.RuntimeException: Unable to start activity ComponentInfo{jp.example.wgamen/jp.example.wgamen.SubActivity}: java.lang.ClassCastException: android.widget.Button cannot be cast to android.widget.ImageView
06-19 23:31:28.980: E/AndroidRuntime(6967): at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2195)
06-19 23:31:28.980: E/AndroidRuntime(6967): at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:2245)
06-19 23:31:28.980: E/AndroidRuntime(6967): at android.app.ActivityThread.access$800(ActivityThread.java:135)
06-19 23:31:28.980: E/AndroidRuntime(6967): at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1196)
06-19 23:31:28.980: E/AndroidRuntime(6967): at android.os.Handler.dispatchMessage(Handler.java:102)
06-19 23:31:28.980: E/AndroidRuntime(6967): at android.os.Looper.loop(Looper.java:136)
06-19 23:31:28.980: E/AndroidRuntime(6967): at android.app.ActivityThread.main(ActivityThread.java:5021)
06-19 23:31:28.980: E/AndroidRuntime(6967): at java.lang.reflect.Method.invokeNative(Native Method)
06-19 23:31:28.980: E/AndroidRuntime(6967): at java.lang.reflect.Method.invoke(Method.java:515)
06-19 23:31:28.980: E/AndroidRuntime(6967): at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:827)
06-19 23:31:28.980: E/AndroidRuntime(6967): at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:643)
06-19 23:31:28.980: E/AndroidRuntime(6967): at dalvik.system.NativeStart.main(Native Method)
06-19 23:31:28.980: E/AndroidRuntime(6967): Caused by: java.lang.ClassCastException: android.widget.Button cannot be cast to android.widget.ImageView
06-19 23:31:28.980: E/AndroidRuntime(6967): at jp.example.wgamen.SubActivity.onCreate(SubActivity.java:36)
06-19 23:31:28.980: E/AndroidRuntime(6967): at android.app.Activity.performCreate(Activity.java:5231)
06-19 23:31:28.980: E/AndroidRuntime(6967): at android.app.Instrumentation.callActivityOnCreate(Instrumentation.java:1090)
06-19 23:31:28.980: E/AndroidRuntime(6967): at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2159)
06-19 23:31:28.980: E/AndroidRuntime(6967): ... 11 more
06-19 23:31:29.120: E/AndroidRuntime(6985): in writeCrashedAppName, pkgName :jp.example.wgamen
06-19 23:31:29.140: E/AndroidRuntime(6985): FATAL EXCEPTION: main
06-19 23:31:29.140: E/AndroidRuntime(6985): Process: jp.example.wgamen, PID: 6985
06-19 23:31:29.140: E/AndroidRuntime(6985): java.lang.RuntimeException: Unable to start activity ComponentInfo{jp.example.wgamen/jp.example.wgamen.SubActivity}: java.lang.ClassCastException: android.widget.Button cannot be cast to android.widget.ImageView
06-19 23:31:29.140: E/AndroidRuntime(6985): at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2195)
06-19 23:31:29.140: E/AndroidRuntime(6985): at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:2245)
06-19 23:31:29.140: E/AndroidRuntime(6985): at android.app.ActivityThread.access$800(ActivityThread.java:135)
06-19 23:31:29.140: E/AndroidRuntime(6985): at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1196)
06-19 23:31:29.140: E/AndroidRuntime(6985): at android.os.Handler.dispatchMessage(Handler.java:102)
06-19 23:31:29.140: E/AndroidRuntime(6985): at android.os.Looper.loop(Looper.java:136)
06-19 23:31:29.140: E/AndroidRuntime(6985): at android.app.ActivityThread.main(ActivityThread.java:5021)
06-19 23:31:29.140: E/AndroidRuntime(6985): at java.lang.reflect.Method.invokeNative(Native Method)
06-19 23:31:29.140: E/AndroidRuntime(6985): at java.lang.reflect.Method.invoke(Method.java:515)
06-19 23:31:29.140: E/AndroidRuntime(6985): at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:827)
06-19 23:31:29.140: E/AndroidRuntime(6985): at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:643)
06-19 23:31:29.140: E/AndroidRuntime(6985): at dalvik.system.NativeStart.main(Native Method)
06-19 23:31:29.140: E/AndroidRuntime(6985): Caused by: java.lang.ClassCastException: android.widget.Button cannot be cast to android.widget.ImageView
06-19 23:31:29.140: E/AndroidRuntime(6985): at jp.example.wgamen.SubActivity.onCreate(SubActivity.java:36)
06-19 23:31:29.140: E/AndroidRuntime(6985): at android.app.Activity.performCreate(Activity.java:5231)
06-19 23:31:29.140: E/AndroidRuntime(6985): at android.app.Instrumentation.callActivityOnCreate(Instrumentation.java:1090)
06-19 23:31:29.140: E/AndroidRuntime(6985): at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2159)
06-19 23:31:29.140: E/AndroidRuntime(6985): ... 11 more
06-19 23:31:29.210: E/AndroidRuntime(6998): in writeCrashedAppName, pkgName :jp.example.wgamen
06-19 23:31:29.210: E/AndroidRuntime(6998): FATAL EXCEPTION: main
06-19 23:31:29.210: E/AndroidRuntime(6998): Process: jp.example.wgamen, PID: 6998
06-19 23:31:29.210: E/AndroidRuntime(6998): java.lang.RuntimeException: Unable to start activity ComponentInfo{jp.example.wgamen/jp.example.wgamen.SubActivity}: java.lang.ClassCastException: android.widget.Button cannot be cast to android.widget.ImageView
06-19 23:31:29.210: E/AndroidRuntime(6998): at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2195)
06-19 23:31:29.210: E/AndroidRuntime(6998): at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:2245)
06-19 23:31:29.210: E/AndroidRuntime(6998): at android.app.ActivityThread.access$800(ActivityThread.java:135)
06-19 23:31:29.210: E/AndroidRuntime(6998): at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1196)
06-19 23:31:29.210: E/AndroidRuntime(6998): at android.os.Handler.dispatchMessage(Handler.java:102)
06-19 23:31:29.210: E/AndroidRuntime(6998): at android.os.Looper.loop(Looper.java:136)
06-19 23:31:29.210: E/AndroidRuntime(6998): at android.app.ActivityThread.main(ActivityThread.java:5021)
06-19 23:31:29.210: E/AndroidRuntime(6998): at java.lang.reflect.Method.invokeNative(Native Method)
06-19 23:31:29.210: E/AndroidRuntime(6998): at java.lang.reflect.Method.invoke(Method.java:515)
06-19 23:31:29.210: E/AndroidRuntime(6998): at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:827)
06-19 23:31:29.210: E/AndroidRuntime(6998): at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:643)
06-19 23:31:29.210: E/AndroidRuntime(6998): at dalvik.system.NativeStart.main(Native Method)
06-19 23:31:29.210: E/AndroidRuntime(6998): Caused by: java.lang.ClassCastException: android.widget.Button cannot be cast to android.widget.ImageView
06-19 23:31:29.210: E/AndroidRuntime(6998): at jp.example.wgamen.SubActivity.onCreate(SubActivity.java:36)
06-19 23:31:29.210: E/AndroidRuntime(6998): at android.app.Activity.performCreate(Activity.java:5231)
06-19 23:31:29.210: E/AndroidRuntime(6998): at android.app.Instrumentation.callActivityOnCreate(Instrumentation.java:1090)
06-19 23:31:29.210: E/AndroidRuntime(6998): at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2159)
06-19 23:31:29.210: E/AndroidRuntime(6998): ... 11 more

[mari_tom]

追記のコードについてもご指摘ありがとうございます！gatya2以外の変数を作りそれぞれに振り分けてきます！

[swordone]

java.lang.ClassCastException: android.widget.Button cannot be cast to android.widget.ImageView
findViewByIdで持ってきたビューのどれかが実はButtonで，それをImageViewにキャストしようとした事による例外です．
at jp.example.wgamen.SubActivity.onCreate(SubActivity.java:36)
SubActivity.javaの36行目と言っていますが，36行目で扱っているViewを確認して下さい．

[mari_tom]

ありがとうございます！確かに画像の下に戻るボタンのコードも書いていたので何か関係があるのでしょうか…
36行目を確認したら gat5 = (ImageView)findViewById(R.id.v5);　でした
ちなみにボタンのコードはこうなっています↓
//ボタンクリックイベントリスナー
btn = (Button) findViewById(R.id.sub_button);
btn.setOnClickListener(new View.OnClickListener() {

//ボタンが押されたら何かする
@Override
public void onClick(View v) {

//インテントに、この画面を指定する
Intent intent = new Intent();

//戻り値設定
setResult(RESULT_OK, intent);

//サブ画面終了
finish();
}
});

[swordone]

レイアウトxmlに設定しているidは正しいですか？
NO:コードのR.id.**に合うように修正して下さい
YES:一旦そのレイアウトxmlのソースをすべて「切り取り」し，「プロジェクト」>「プロジェクトのビルド」(または「すべてビルド」)
その後，切り取ったソースを「貼り付け」し，「プロジェクト」>「プロジェクトのビルド」(または「すべてビルド」)

YESのパターンは，レイアウトxmlを書いたあとレイアウトパーツをドラッグ＆ドロップすると起こりえます(実体験)．

[mari_tom]

ありがとうございます！YESだったのでその通りにしましたら、今度はat jp.example.wgamen.SubActivity.onCreate(SubActivity.java:34)というエラーが出て強制終了しました
34行目のコードは　gat3 = (ImageView)findViewById(R.id.v3);で、これまたYESなのでもう一度同じ事を繰り返しましたところ、今度はat jp.example.wgamen.SubActivity.onCreate(SubActivity.java:35)とエラーが出て、また強制終了しました
こうなったらエラーが出なくなるまで切り貼りビルドを繰り返してみようと思います！

[mari_tom]

ありがとうございます！もう一度切り取り→ビルド→貼り付け→ビルド→実行、で強制終了する事はなくなり、ちゃんとrからlrまでランダムに画像を表示させる事ができました！！
本当にありがとうございました！！

[swordone]

なんか私がキャスト例外を指摘する前のコメントと追記のコードを見てると，貴方がループの部分で勘違いしているような気がしてならないのですが・・・

[mari_tom]

すみません、数回テストしたらちゃんと思い通りに動いてくれたので浮かれてしまいました；
コードの全容はpackage jp.example.wgamen;

import java.util.Arrays;
import java.util.Random;

import android.annotation.SuppressLint;
import android.app.Activity;
import android.content.Intent;
import android.content.res.TypedArray;
import android.os.Bundle;
import android.view.View;
import android.widget.Button;
import android.widget.ImageView;

public class SubActivity extends Activity {
private Button btn;

ImageView gat;
ImageView gat2;
ImageView gat3;
ImageView gat4;
ImageView gat5;
ImageView gat6;

@SuppressLint("Recycle")
@Override
protected void onCreate(Bundle savedInstanceState) {
super.onCreate(savedInstanceState);
setContentView(R.layout.sub);

// 画面に配置したImageViewを設定
gat = (ImageView)findViewById(R.id.v1);
gat2 = (ImageView)findViewById(R.id.v2);
gat3 = (ImageView)findViewById(R.id.v3);
gat4 = (ImageView)findViewById(R.id.v4);
gat5 = (ImageView)findViewById(R.id.v5);
gat6 = (ImageView)findViewById(R.id.v6);

//整数を入れる為の変数を宣言
int gatya1;
int gatya2;
int gatya3;
int gatya4;
int gatya5;
int gatya6;

// 画像のリストを取得
TypedArray images = getResources().obtainTypedArray(R.array.r);
TypedArray images2 = getResources().obtainTypedArray(R.array.sr);
TypedArray images3 = getResources().obtainTypedArray(R.array.ur);
TypedArray images4 = getResources().obtainTypedArray(R.array.lr);

// Randomクラスのインスタンスを生成
Random r = new Random();

for(ImageView tmp : Arrays.asList(gat, gat2, gat3, gat4, gat5, gat6)) {
//０以上100未満の整数の乱数を作成する
gatya1 = r.nextInt(100);
//ランダムで出た数が２以下ならLRの配列から１枚画像を表示する
if(gatya1 <= 2){
gat.setImageDrawable(images4.getDrawable(r.nextInt(images4.length())));
//ランダムで出た数が３以上12以下ならURの配列から１枚画像を表示する
}else if(gatya1 <= 12){
gat.setImageDrawable(images3.getDrawable(r.nextInt(images3.length())));
//ランダムで出た数が13以上39以下ならSRの配列から１枚画像を表示する
}else if(gatya1 <= 39){
gat.setImageDrawable(images2.getDrawable(r.nextInt(images2.length())));
//それ以外（それ以降）の数字ならRの配列から１枚画像を表示する
}else{
gat.setImageDrawable(images.getDrawable(r.nextInt(images.length())));
}

//０以上100未満の整数の乱数を作成する
gatya2 = r.nextInt(100);
//ランダムで出た数が２以下ならLRの配列から１枚画像を表示する
if(gatya2 <= 2){
gat2.setImageDrawable(images4.getDrawable(r.nextInt(images4.length())));
//ランダムで出た数が３以上12以下ならURの配列から１枚画像を表示する
}else if(gatya2 <= 12){
gat2.setImageDrawable(images3.getDrawable(r.nextInt(images3.length())));
//ランダムで出た数が13以上39以下ならSRの配列から１枚画像を表示する
}else if(gatya2 <= 39){
gat2.setImageDrawable(images2.getDrawable(r.nextInt(images2.length())));
//それ以外（それ以降）の数字ならRの配列から１枚画像を表示する
}else{
gat2.setImageDrawable(images.getDrawable(r.nextInt(images.length())));
}

//０以上100未満の整数の乱数を作成する
gatya3 = r.nextInt(100);
//ランダムで出た数が２以下ならLRの配列から１枚画像を表示する
if(gatya3 <= 2){
gat3.setImageDrawable(images4.getDrawable(r.nextInt(images4.length())));
//ランダムで出た数が３以上12以下ならURの配列から１枚画像を表示する
}else if(gatya3 <= 12){
gat3.setImageDrawable(images3.getDrawable(r.nextInt(images3.length())));
//ランダムで出た数が13以上39以下ならSRの配列から１枚画像を表示する
}else if(gatya3 <= 39){
gat3.setImageDrawable(images2.getDrawable(r.nextInt(images2.length())));
//それ以外（それ以降）の数字ならRの配列から１枚画像を表示する
}else{
gat3.setImageDrawable(images.getDrawable(r.nextInt(images.length())));
}

//０以上100未満の整数の乱数を作成する
gatya4 = r.nextInt(100);
//ランダムで出た数が２以下ならLRの配列から１枚画像を表示する
if(gatya4 <= 2){
gat4.setImageDrawable(images4.getDrawable(r.nextInt(images4.length())));
//ランダムで出た数が３以上12以下ならURの配列から１枚画像を表示する
}else if(gatya4 <= 12){
gat4.setImageDrawable(images3.getDrawable(r.nextInt(images3.length())));
//ランダムで出た数が13以上39以下ならSRの配列から１枚画像を表示する
}else if(gatya4 <= 39){
gat4.setImageDrawable(images2.getDrawable(r.nextInt(images2.length())));
//それ以外（それ以降）の数字ならRの配列から１枚画像を表示する
}else{
gat4.setImageDrawable(images.getDrawable(r.nextInt(images.length())));
}

//０以上100未満の整数の乱数を作成する
gatya5 = r.nextInt(100);
//ランダムで出た数が２以下ならLRの配列から１枚画像を表示する
if(gatya5 <= 2){
gat5.setImageDrawable(images4.getDrawable(r.nextInt(images4.length())));
//ランダムで出た数が３以上12以下ならURの配列から１枚画像を表示する
}else if(gatya5 <= 12){
gat5.setImageDrawable(images3.getDrawable(r.nextInt(images3.length())));
//ランダムで出た数が13以上39以下ならSRの配列から１枚画像を表示する
}else if(gatya5 <= 39){
gat5.setImageDrawable(images2.getDrawable(r.nextInt(images2.length())));
//それ以外（それ以降）の数字ならRの配列から１枚画像を表示する
}else{
gat5.setImageDrawable(images.getDrawable(r.nextInt(images.length())));
}

//０以上100未満の整数の乱数を作成する
gatya6 = r.nextInt(100);
//ランダムで出た数が２以下ならLRの配列から１枚画像を表示する
if(gatya6 <= 2){
gat6.setImageDrawable(images4.getDrawable(r.nextInt(images4.length())));
//ランダムで出た数が３以上12以下ならURの配列から１枚画像を表示する
}else if(gatya6 <= 12){
gat6.setImageDrawable(images3.getDrawable(r.nextInt(images3.length())));
//ランダムで出た数が13以上39以下ならSRの配列から１枚画像を表示する
}else if(gatya6 <= 39){
gat6.setImageDrawable(images2.getDrawable(r.nextInt(images2.length())));
//それ以外（それ以降）の数字ならRの配列から１枚画像を表示する
}else{
gat6.setImageDrawable(images.getDrawable(r.nextInt(images.length())));
}
}

//ボタンクリックイベントリスナー
btn = (Button) findViewById(R.id.sub_button);
btn.setOnClickListener(new View.OnClickListener() {

//ボタンが押されたら何かする
@Override
public void onClick(View v) {

//インテントに、この画面を指定する
Intent intent = new Intent();

//戻り値設定
setResult(RESULT_OK, intent);

//サブ画面終了
finish();
}

});
}

}

です

[swordone]

コード内のコメントが誤解を与えたのなら申し訳ないのですが，
「各ImageViewに対して乱数を振る」というのは，forによって順番にImageViewが取り出されるわけですが，そのそれぞれに対して乱数を振る，という意味です．
私の回答のコードでそれは満たされているのですが，あなたの書き方だと，「それぞれに対して乱数を振り，画像をセットする」この動作が6回繰り返されます．
はっきり言って，かなり無駄が多いです．

[swordone]

端的に言うと，貴方が最初のコードで
//2～5まで繰り返しなので省略・
と言っていた部分と，1番目，6番目も含めた部分を私が書いたfor文で解決しています．

[mari_tom]

いえ、こちらの読解力や知識が欠けており、間違って解釈してコードを書いてしまったようで、すみませんでした。
もっとコードをすっきりと簡潔に書く事ができるのですね、考えもしませんでした。
それぞれに乱数を振るだけで良く、画像をセットするコードは不要なんですね。ご指摘いただくまで全く気が付きませんでした…
これで良いのでしょうか？
for(ImageView tmp : Arrays.asList(gat, gat2, gat3, gat4, gat5, gat6)) {

//０以上100未満の整数の乱数を作成する
gatya1 = r.nextInt(100);
TypedArray array = null;
if(gatya1 <= 2){
array = images4;
}else if(gatya1 <= 12){
array = images3;
}else if(gatya1 <= 39){
array = images2;
}else{
array = images;
}
//決定した配列の長さから乱数を生成し，それをインデックスにDrawableを取り出しセット
//(ここは個人的に括弧が重なりすぎて読みにくかったため分解)
int i = r.nextInt(array.length());
tmp.setImageDrawable(array.getDrawable(i));
//2～5まで省略
gatya6 = r.nextInt(100);
TypedArray array6 = null;
if(gatya6 <= 2){
array6 = images4;
}else if(gatya6 <= 12){
array6 = images3;
}else if(gatya6 <= 39){
array6 = images2;
}else{
array6 = images;
}
int i6 = r.nextInt(array6.length());
tmp.setImageDrawable(array6.getDrawable(i6));
}
これで実行しましたら、これもちゃんと動いてくれました。

[swordone]

for文の意味を理解しなおしたほうが良いかと．
for(ImageView tmp : Arrays.asList(gat, gat2, gat3, gat4, gat5, gat6))
これはgat~gat6までをList化して拡張for文で使えるようにし，
一個ずつ取り出してループする文です．
私のコードでは，まずtmp=gatとなり，forブロックの中の文が実行されます．
最後の行でtmp.setImageDrawable(array6.getDrawable(i6));がありますが，
ここでtmp=gatなので，これはgatに対して画像セットの動作をしています．
ここで1ループ終わります．
次にtmp=gat2となり，forブロックの中の文が実行されます．
最後の行はtmp=gat2なので，gat2に対する操作です．
こうしてループして，gat~gat6それぞれにランダムに画像セットしています．
あなたのコードは，
まずtmp=gatにして，「乱数を振り，リストを決め，画像をセット」というこの一連の操作を"tmp=gatに対して計6回"行っています．これが1ループです．
それをgat~gat6それぞれに行っていて，大変無駄です．

[mari_tom]

何度もお手を煩わせてしまってすみません。もう一度考え直したらかなり簡略化できました！
//整数を入れる為の変数を宣言
int gatya1;

// 画像のリストを取得
TypedArray images = getResources().obtainTypedArray(R.array.r);
TypedArray images2 = getResources().obtainTypedArray(R.array.sr);
TypedArray images3 = getResources().obtainTypedArray(R.array.ur);
TypedArray images4 = getResources().obtainTypedArray(R.array.lr);

// Randomクラスのインスタンスを生成
Random r = new Random();

for(ImageView tmp : Arrays.asList(gat, gat2, gat3, gat4, gat5, gat6)) {

//０以上100未満の整数の乱数を作成する
gatya1 = r.nextInt(100);
TypedArray array = null;
if(gatya1 <= 2){
array = images4;
}else if(gatya1 <= 12){
array = images3;
}else if(gatya1 <= 39){
array = images2;
}else{
array = images;
}
//決定した配列の長さから乱数を生成し，それをインデックスにDrawableを取り出しセット
//(ここは個人的に括弧が重なりすぎて読みにくかったため分解)
int i = r.nextInt(array.length());
tmp.setImageDrawable(array.getDrawable(i));

}
これだけでも実行したらv1~v6それぞれにランダムで画像が表示されていました！
改めて自分の至らなさを痛感したので、もう一度for文をはじめとするJavaの基礎から学び直そうと思います。
本当に失礼をいたしました。

Answer 2

拡張for文の?の部分には、java.lang.Iterableの実装クラスか、配列を指定します。
java.lang.Iterableの実装クラスとは、例えばListなどのコレクションです。

この場合は、gat～gat6の6つのImageViewをjava.lang.Iterableの実装クラスか、配列に格納して、それを?に指定します。これはArrays.asList()を使うのが簡単だと思います。

// import java.util.Arrays

for(ImageView tmp : Arrays.asList(gat, gat2, gat3, gat4, gat5, gat6)) {
}

Comments

[mari_tom]

ありがとうございます！教えていただいた通りにしましたらエラーが消えました！
ただ実行してみたら、何故か一番低確率のはずの配列lrからしか画像が出ません…
rやsr、urも交えたRandomにするには何が間違っているのでしょうか？
// 画像のリストを取得の所のobtainTypedArrayに『This TypedArray should be recycled after use with #recycle()』という警告が出ています

[argius]

gatya2 = r.nextInt(100);したあとのgatya2の値がどうなっているか分かりますか？
その値が毎回0,1,2になっているのならRandomの問題かも知れません。そうでなければ、それ以外の箇所に問題がある可能性があります。

obtainTypedArrayの警告は、TypedArrayは使い終わったらrecycle()メソッドで破棄すべき、という意味です。

[mari_tom]

警告の意味を教えていただいてありがとうございます！
gatya2の値を表示して確認しようとソースを下記のように書いたら、強制終了してしまい、結局gatya2の値がどうなっているのか確認できませんでした、すみません
// Randomクラスのインスタンスを生成
Random r = new Random();
//０以上100未満の整数の乱数を作成する
gatya2 = r.nextInt(100);
System.out.println(gatya2);
もう一度、書いたソースを見直してきます

[swordone]

setImageResource(R.array.***)で何をやっているか不明ですが，
for(ImageView tmp : ?)
{
tmp.setImageDrawable(images.getDrawable(r.nextInt(images.length())));
tmp.setImageDrawable(images2.getDrawable(r.nextInt(images2.length())));
tmp.setImageDrawable(images3.getDrawable(r.nextInt(images3.length())));
tmp.setImageDrawable(images4.getDrawable(r.nextInt(images4.length())));
}
lrはimages4なので，このループを実行すると最後の行の結果だけが各ImageViewに残るので，lrからしか出ないのは当然です．

[argius]

swordoneさん

あれ、本当ですね。全部tmpになってるんですね。すみません、勘違いしてました。

[mari_tom]

swordoneさん、ありがとうございます！
自分の書き方だと最後の行の結果しか出なくなってしまうんですね…
lrだけでなくrやSR、urからも画像が出るようにするには、どのようにコードを書き換えたら良いのでしょうか…
もう少し自分でも考えてみます