Android用サイコロアプリの無駄な動作を教えてほしい

Question

本文
うまく動くようになったので改善できるところを教えてください。
(例外は発生しているか試しました)

package com.mycompany.myappdiceplus;
import android.app.*;
import android.os.*;
import android.widget.*;
import android.view.View.*;
import android.view.*;
import java.util.*;
import android.media.*;
import android.content.*;
import android.widget.AutoCompleteTextView.*;

public class MainActivity extends Activity{
	MediaPlayer SE;

	
	protected void onCreate(Bundle savedInstanceState){
		super.onCreate(savedInstanceState);
		setContentView(R.layout.main);
		Button button=(Button)findViewById(R.id.bn);
		button.setOnClickListener(new OnClickListener(){
				public void onClick(View Vector){
					
					final EditText DC=(EditText)findViewById(R.id.Dc);
					TextView math=(TextView)findViewById(R.id.text);
					math.setText("???");
					final EditText MAX=(EditText)findViewById(R.id.MAX);
					final EditText kai=(EditText)findViewById(R.id.math);
					SE.start();
					int max=0;
					int dice = 0;
					
					int kaisu=0;
					
					
					max=Integer.parseInt(MAX.getText().toString());
					dice=Integer.parseInt(DC.getText().toString());
					kaisu=Integer.parseInt(kai.getText().toString());
					try
					{
						Thread.sleep(1500);
					}
					catch (InterruptedException e)
					{
						e.printStackTrace();
						System.exit(1);
					}
					int[] sum = new int[kaisu];
					StringBuilder strings=new StringBuilder();
					int Random = 0;
				
					int dex=sum.length;
					dex--;
					for(int i=0;i<=dex;i++){
						sum[i]=new Random().nextInt(max);
						Random+=sum[i];
					}
					strings.append(Random <= dice ? "成功(" : "失敗(");
					strings.append(Random).append("=");
					
					for(int index:sum){
						strings.append(index);
						strings.append(",");
					}
					strings.append(")");
					
					String ans=strings.toString();
					math.setText(ans);
				}
		});
	}

	@Override
	protected void onResume()
	{
		super.onResume();
		SE = MediaPlayer.create(this, R.raw.niconico);
		
	}

	@Override
	protected void onDestroy()
	{
		onStop();
		super.onDestroy();
	}
}

直る前の文
結果:成功か失敗(右の合計=乱数,乱数…)
と表示したいのですが、
結果:成功(0=)
どうしてでしょうか。直し方を教えてください。
一部省く…

try{
max=Integer.parseInt(MAX.getText().toString());
dice=Integer.parseInt(DC.getText().toString());
kaisu=Integer.parseInt(kai.getText().toString());
Thread.sleep(1500);
}
catch (Exception e){
}
int kaisu-1;
…

Comments

[swordone]

catch節でe.printStackTrace();を記述してどうなるか確認してみてくれませんか？(例外ログが出るかどうか)

[sun-solar-arrow]

例外は起きていないので意味がないと思います。

[yona]

なぜ、例外が発生していないと思うんですか？

[sun-solar-arrow]

いい忘れていましたが試していたからです。

[sun-solar-arrow]

もう一回試します

[swordone]

ついでに実行時の各パラメータ(max, kaisuなど)の中身も確認しておいたほうがいいですよ

[sun-solar-arrow]

分かりました。

[sun-solar-arrow]

皆さん、ありがとうございました。最初の質問が解決したためベストアンサーを決めることにしました。
この後も回答しようとしていたかたもいたかもしれませんが申し訳ございません。
皆さんありがとうございました！

Answer 1

正直無駄が多く感じ、意味のない操作があるようだったので、肝要な部分だけ抜き出して書きなおしてみました。(Java環境で動くよう乱数発生して合計し判定という部分だけで実行する)

import java.util.Random;

public class Q31458 {

	public static void main(String[] args) {
		int max = 25;
		int dice = 100;
		int kaisu = 5;

		try {
			Thread.sleep(1500);
		} catch (Exception e) {
			e.printStackTrace();
		}

		kaisu -= 1;
		int[] sum = new int[kaisu];
		StringBuilder strings = new StringBuilder();
		int random = 0;

		Random random2 = new Random();
		for (int i = 0; i < kaisu; i++) {
			sum[i] = random2.nextInt(max);
			random += sum[i];
		}

		strings.append("結果:")
			.append(random <= dice ? "成功(" : "失敗(")
			.append(random).append("=");
		for (int i : sum) {
			strings.append(i);
			strings.append(",");
		}
		strings.append(")");

		String ans = strings.toString();
		System.out.println(ans);
	}

}

これを実行した結果、

---

結果:成功(74=21,10,23,20,)

---

などのようになり、問題の現象が発生しません。
kaisu -= 1;などの意味不明な操作も気になるのですが、一番問題がありそうなのは、EditTextから取得した文字列を数値に変換する段階で例外が発生している可能性があり、しかもそれがcatch節で握りつぶされてしまっていることです。
その部分のcatchでログを出してチェックするか、catchする例外をInterruptedExceptionに限定するべきです。

Comments

[sun-solar-arrow]

直してみました。今はスマホのAIDEで開発しているのでRoot化していないためlogが分からないから
IntnrruptedExceptionをcatchするようにし、
catchしたらSystem.exit(1);
のように強制終了させるようにしましたが、強制終了はしていませんでした。
無駄な動作を省きやってみます。
しかし、初心者なので無駄な動作というのがあまり分かってません。ですから思い当たるものだけ消してみます。

[sun-solar-arrow]

kaisu-1を消したら動くようになりました。

[sun-solar-arrow]

しかし、
append(random <= dice ? "成功(" : "失敗(")
は便利です。一気にコードが短くなりました。
教えて下さりありがとうございました。

Answer 2

kaisu-=1; 
int[] sum = new int[kaisu]; 
StringBuilder strings=new StringBuilder(); 
int Random = 0;

int dex=sum.length-1; 
for(int i=0;i<=dex;i++){ 
sum[i]=new Random().nextInt(max); 
Random+=sum[i]; 
}

ここで常にdexが-1になるので
Randomが常に0になります。
そのためRandom <= diceが常に成り立ち成功になります。

Comments

[sun-solar-arrow]

次のようになりました。
成功か失敗(合計=)
になりました。(失敗にもなるようになり合計もでる)
しかし、まだ、=の右に足す前の数が出ません。
(つまり、成功か失敗(合計=足す前)になる)

[yona]

どのように修正して、どのようになったのかさっぱりわかりません。そんな状態で回答できると思いますか？
変更した内容を追記してください。

また、プログラムは専用の記述方法があるので修正してください。

[sun-solar-arrow]

dex=sum.length-1の所を
dex=sum.length;
dex--;に変えました。
しかし、
専用の記述方法というのはどんなのですか？初心者なので分かりません。そこも教えてください。

[yona]

PCなら質問入力欄の上部にボタンがあるはずです。

```言語を書く
ここにコードを書く
```

[sun-solar-arrow]

スマホなんで気が付きませんでした。

[yona]

確かにスマホだと気づかないですよね。

[yona]

Thread.sleep(1500);している意図は何でしょうか。
意味がないように思います。

[sun-solar-arrow]

サイコロの音に合わせるためです。
http://commons.nicovideo.jp/material/nc44544