метод, использующий цикл for, не будет перемешивать элементы массива или переписывать их все - PullRequest
Новая
       12

метод, использующий цикл for, не будет перемешивать элементы массива или переписывать их все

0 голосов
/ 27 января 2019

У меня есть класс Deck_of_Cards, в котором есть конструктор по умолчанию, который назначает вновь определенный массив размером 52 в «deck», а затем активирует метод обновления класса.

В методе обновления используется вложенный цикл forназначать карты в колоду, а затем активировать метод класса shuffle, который, как вы, наверное, догадались, использует цикл for для рандомизации расположения всех элементов массива колоды.Выполнение тестового кода должно дать пять разных карт, вместо этого я получаю одну карту пять раз, пример показан ниже. 

Jack of Diamonds
Jack of Diamonds
Jack of Diamonds
Jack of Diamonds
Jack of Diamonds

Тестовый код 

public class test {

    Deck_of_Cards X = new Deck_of_Cards();

    int drawCount = 5;

    for(int i = 0; i<drawCount; i++)
    {
        printCard(X.drawCard());
    }

    //Outputs the given card's data as "<rank> of <suit>".
    public static void printCard(Card c) {
        String st;

        if (c.getRank() == '1')
            st = c.getRank() + "0 of " + c.getSuit();

        else if (c.getRank() == 'A')
            st = "Ace of " + c.getSuit();

        else if (c.getRank() == 'J')
            st = "Jack of " + c.getSuit();

        else if (c.getRank() == 'Q')
            st = "Queen of " + c.getSuit();

        else if (c.getRank() == 'K')
            st = "King of " + c.getSuit();

        else
            st = c.getRank() + " of " + c.getSuit();


        switch (c.getSuit()) {

Класс карты 

public class Card {
    private char suit;
    private char rank;

    //Constructor 
    public Card(char s, char r) 
    {
        // Suit if-statement
        if (s == 'C' || s == 'D' || s == 'H' || s == 'S' )
        {
            s = suit;
        }
        else 
        {
            throw new IllegalArgumentException("Not a valid suit!");
        }

        // Rank if-statement
        if (r == 'A' || r == '2' || r == '3' || r == '4' || r == '5' || r == '6' || r == '7' || r == '8' || r == '9' || r == 'J' || r == 'Q' || r == 'K')
        {
            r = rank;
        }
        else 
        {
            throw new IllegalArgumentException("Not a valid rank!");
        }

    }

    // Accessors
    public char getRank()
    {
        return rank;
    }

    public char getSuit()
    {
        return suit;
    }

Класс карт Deck_of_ 

import java.util.*;

public class Deck_of_Cards {

    private Card deck [];
    private int position;


    Random rng = new Random();


    public Deck_of_Cards ()
    {
        deck = new Card[51];


        refresh();
    }

    public void refresh()
    {   
        char suit[] = {'C', 'D', 'H', 'S'};
        char rank[] = {'A', '2', '3', '4', '5', '6', '7', '8', '9', 'J', 'Q', 'K'};

        int index = 0;
        for (int i = 0; i < suit.length; i++)
        {
            for (int j = 0; j < rank.length; j++) 
            {
                deck[index] = new Card(suit[i], rank[j]);
                index++;
            }
        }

        shuffle();
    }

    public void shuffle()
    {
        for (int x = deck.length; x >= 0; x--)
        {
            position = rng.nextInt(x+1);
            Card g = deck [x];

            deck[x] = deck[position];
            deck[position] = g;

            //Reseting position back to zero
            if(x == 0)
            {
                position = 0;
            }
        }



    }

    public Card drawCard()
    {
        return deck[position];
    }

}

1 Ответ

0 голосов
/ 27 января 2019

Проблема в том, что в drawCard: 

public Card drawCard()
{
    return deck[position];
}

Вы не меняете position, поэтому каждый раз, когда вы его вызываете, вы будете разыгрывать одну и ту же карту снова и снова.Измените его на: 

public Card drawCard()
{
    return deck[position++];
}

Для увеличения position (или position-- в зависимости от того, как вы визуализируете "колоду")

...