2013-04-29 107 views
0

我已经设置好了,如果返回0,比它上面有1的骰子图像被设置为按钮图像的位置。它也是用不同的值和图像来做到这一点。 0 =骰子1,1 =骰子2,2 =骰子3,3 =骰子4,4 =骰子5.问题是所有返回的数字总是0,我知道这是因为按钮上都有一个图像。我看过无数的教程,不知道发生了什么!请帮我上网的朋友!为什么我的随机发生器总是返回0?

这里是我的代码:

package com.example.yahtzee; 

import java.util.Random; 

import android.os.Bundle; 
import android.app.Activity; 
import android.app.AlertDialog; 
import android.content.Context; 
import android.content.DialogInterface; 
import android.graphics.Bitmap; 
import android.graphics.drawable.Drawable; 
import android.view.Menu; 
import android.view.View; 
import android.widget.Button; 
import android.widget.ImageButton; 

public class MainActivity extends Activity { 

static int mButtonAmount = 7; 
static int SidesOfDice = 6; 
final static int NumberOfDice = 6; 
int Rolls = 1; 
boolean mCACV = true; 
String dialog_title; 
String dialog_message; 
String positive_button; 
String neutral_button; 
String negitive_button; 
int DiceNumber; 
int placeHolder; 
int max = NumberOfDice + 1; 

Button[]mButtons = new Button[mButtonAmount]; 
Boolean[]mButtonBools = new Boolean[mButtonAmount]; 
Drawable[]mDiceImages = new Drawable[SidesOfDice]; 
Boolean[]mIsDiceHeld = new Boolean[NumberOfDice]; 
ImageButton[]mImageButtons = new ImageButton[NumberOfDice]; 
int[]DieNumber = new int[NumberOfDice]; 

private static final Random RANDOM = new Random(); 

@Override 
protected void onCreate(Bundle savedInstanceState) { 
    super.onCreate(savedInstanceState); 
    setContentView(R.layout.activity_main); 
    mButtons[0] = (Button) findViewById(R.id.start_roll); 
    mButtons[0].setEnabled(true); 
    mButtons[1] = (Button) findViewById(R.id.stop_roll); 
    mButtons[1].setEnabled(false); 
    mButtonBools[0] = getResources().getBoolean(R.bool.is_start_roll_button_pressable); 
    mButtonBools[1] = getResources().getBoolean(R.bool.is_stop_roll_button_pressable); 

    mDiceImages[0] = getResources().getDrawable(R.drawable.diceside1); 
    mDiceImages[1] = getResources().getDrawable(R.drawable.diceside2); 
    mDiceImages[2] = getResources().getDrawable(R.drawable.diceside3); 
    mDiceImages[3] = getResources().getDrawable(R.drawable.diceside4); 
    mDiceImages[4] = getResources().getDrawable(R.drawable.diceside5); 
    mDiceImages[5] = getResources().getDrawable(R.drawable.diceside6); 

    mImageButtons[0] = (ImageButton) findViewById(R.id.die_1); 
    mImageButtons[0].setClickable(false); 
    mImageButtons[1] = (ImageButton) findViewById(R.id.die_2); 
    mImageButtons[1].setClickable(false); 
    mImageButtons[2] = (ImageButton) findViewById(R.id.die_3); 
    mImageButtons[2].setClickable(false); 
    mImageButtons[3] = (ImageButton) findViewById(R.id.die_4); 
    mImageButtons[3].setClickable(false); 
    mImageButtons[4] = (ImageButton) findViewById(R.id.die_5); 
    mImageButtons[4].setClickable(false); 

    mIsDiceHeld[0] = false; 
    mIsDiceHeld[1] = false; 
    mIsDiceHeld[2] = false; 
    mIsDiceHeld[3] = false; 
    mIsDiceHeld[4] = false; 

    controlVariables(); 
} 

public void controlVariables() 
{ 
    rollDice(); 
      numberToImage(); 
} 

public void checkAndChangeButtons() 
{ 
    checkStartButton(); 
    checkStopButton(); 
} 


public void rollDice() 
{ 
    for(int i = 0; i < DieNumber.length; i++) 
    { 
    } 

public void numberToImage() 
{ 
    for(int i = 0; i < 5; i++) 
    { 
     mImageButtons[i].setBackground(mDiceImages[DieNumber[i]]); 
    } 
} 

public int nextInt(int n) 
{ 
    n = RANDOM.nextInt(); 
    return (n < 0 ? -n : n) % max; 
} 
+0

http://stackoverflow.com/questions/363681/generating-random-number-in-a-range-with-java – Nimooli 2013-04-29 20:14:00

回答

0

首先,你应该有一个时间的种子初始化随机:

Random rand = new Random(System.currentTimeMillis()); 

然后,你可以用

得到0和最大值之间的int值
int i = rand.nextInt(max); 
+0

不应该需要对currentTimeMillis进行调用,Javadoc说'public Random(long seed) 使用单个长种子创建一个新的随机数生成器。种子是伪随机数生成器的内部状态的初始值,其由方法next(int)维持 – pjs 2013-04-29 22:36:35

0

你可以使用SimonSays的答案,或者如果你想在特定范围内生成数字,你可以E该

Random generator = new Random(); 
int i = generator.nextInt(10) + 1; 

这将产生介于1至11的随机数

0

实施例随机发生器:

long RandomGenerator(long from,long to) throws InvalidParameterSpecException{ 
    if(to<from){throw new InvalidParameterSpecException("Parametr 'from' must bigger than 'to'");} 
    return System.currentTimeMillis()%(to-from)+from; 
} 

RandomGenerator(-200,-100)可以生成-150,-175(数从-200到-100),但不能产生150.第一参数必须小于第二参数。

0

你的逻辑是正确的。您可以通过这种方式生成一个随机数。 我实现了这里就像你,看看(你可以在这里找到完整的例子:,点击Ø随便举个例子主github full example

可能是你的错误是在瞬间获得的图像,如果他们依靠的随机数

这里是我的活动代码:

public class RandomNumberActivity extends SherlockActivity{ 

    private static final Random RANDOM = new Random(); 
    private int count; 

    private TextView numbers; 

    @Override 
    protected void onCreate(Bundle savedInstanceState) { 
     super.onCreate(savedInstanceState); 

     setContentView(R.layout.random_numbers); 
     count = 0; 
     numbers = (TextView) findViewById(R.id.numbers); 

    } 


    public void generate(View button) 
    {   
     int i = RANDOM.nextInt(); 
     count++; 
     i = (i < 0 ? -i : i) % 7; 

     String current = numbers.getText().toString(); 
     numbers.setText(current.concat(count+"º: " + i+",\n"));  
    } 
} 

在这里,我的布局:

<?xml version="1.0" encoding="utf-8"?> 
<RelativeLayout xmlns:android="http://schemas.android.com/apk/res/android" 
android:layout_width="match_parent" 
android:layout_height="match_parent" 
android:orientation="vertical" > 

    <Button android:id="@+id/generate" 
     android:layout_height="wrap_content" 
     android:layout_width="wrap_content" 
     android:text="@string/generate" 
     android:layout_alignParentBottom="true" 
     android:layout_centerHorizontal="true" 
     android:onClick="generate" 
    /> 

    <ScrollView 
     android:layout_height="match_parent" 
     android:layout_width="match_parent" 
     android:layout_above="@id/generate"> 
     <TextView 
      android:id="@+id/numbers" 
      android:layout_height="wrap_content" 
      android:layout_width="match_parent" 
      android:textColor="@android:color/black"/> 
</ScrollView> 


</RelativeLayout>