如何洗牌存放在阵列中的一副牌？

Question

I was developing a simple application that It is simple game of cards, and I had created an array which contains all the card of the game but the problem is that I don't know why I can't shuffle this array?

I tried to use the RANDOM but I didn't succeed.

public class Mazzo {
    private Carta[] carteNelMazzo ;

    public Mazzo(){// create the deck
        carteNelMazzo = creaMazzo();
        mescolaMazzo(carteNelMazzo);
    }

   /**methods of the deck */

    public Carta PescaCarta (){

        return (carteNelMazzo==null||(carteNelMazzo.length>0)) ? pescaCarta(carteNelMazzo) : null;
    }

    public Carta pescaBriscola(){
        return (carteNelMazzo==null||(carteNelMazzo.length>0)) ? carteNelMazzo[carteNelMazzo.length-1] : null;
    }
    /**
     * @param carte deve avere lunghezza maggiore uguale ad 1
     * @return la prima carta del mazzo
     */
    private Carta pescaCarta(Carta[] carte){
        Carta[] nuoveCarte=new Carta[carte.length-1];
        Carta pescata= carte[0];
        System.arraycopy(carte,1,nuoveCarte,0,carte.length);
        carte = nuoveCarte;
        return pescata;
    }

    private Carta[] creaMazzo(){
        ArrayList<Carta> nuovoMazzo=new ArrayList<>();
        for(int i =0; i<4; i++){
            // selezione del seme

            for(int j = 0;j<10;j++){
                // creation of the card from another calss
                Carta nuovaCarta= new Carta(Carta.SEME.values()[i],j);
                nuovoMazzo.add(nuovaCarta);
            }
        }
        return (Carta[]) nuovoMazzo.toArray();
    }

//shuffle deck

    private void mescolaMazzo(Carta[] carte){

       Random rand = new Random();
       int elements = (int) (40 * Math.random());

       int elements = carteNelMazzo.length;
    }
}

At the end I want this array with all the cards remix at random.

Answer 1

This here:

int elements = (int) (40 * Math.random());
int elements = carteNelMazzo.length;

doesn't shuffle anything. It assigns a random number to a local variable which is then thrown away. ( I actually think this shouldn't even compile, as you declare the same local variable twice in the same scope )

What you need instead: to create a function that maps your 40 indexes to new values, like:

0, 1, 3, ... 39 

becomes

14, 2, 7, ...

An easy way to get there: Collections.shuffle(someList); .

For more ideas (also using arrays directly), see here .

But as you are probably doing this to learn things, I suggest you carefully digest what I told you upfront. Start by thinking how you would could shuffle a list of cards "manually" (not by touching them, but when you are told the order they have how you could "mentally" re-order them). From there, think how you could instruct a computer to do that.

Answer 2

采用：

Collections.shuffle(<<arrayname>>);

Answer 3

You can try something like this:

private void shuffleCards (Card[] cards) {

    for (int i = 0; i < cards.length; i++) {
        Card temp = cards[i];

        //random index of the array
        int rnd = Math.floor(Math.random() * cards.length);
        cards[i] = cards[rnd];
        cards[rnd] = temp;
    }
}

PS.: if this code throws an ArrayIndexOutOfBoundsException change the line int rnd = Math.floor(Math.random() * cards.length); to int rnd = Math.floor(Math.random() * (cards.length - 1));

Answer 4

Here's one I learned many years ago.

  int[] cards = IntStream.rangeClosed(0, 51).toArray();
  Random r = new Random();

  for (int i = cards.length - 1; i >= 0; i--) {
     int idx = r.nextInt(i + 1);
     int card = cards[idx];
     cards[idx] = cards[i];
     cards[i] = card;
  }

And then there's always the Collections.shuffle() class.