invoke void method

Question

Мне нужно вызвать метод void перед вызовом другого метода.

У меня есть этот метод

public void shuffle() { 
    various = new Random(); 

    currentCard = 0; 
    currentTotal1 = 0; 
    currentTotal2 = 0; 

    for (int first = 0; first < deckOfCards.length; first++) { 
     int second = various.nextInt(number_cards); 

     Card temp = deckOfCards[first]; 
     deckOfCards[first] = deckOfCards[second]; 
     deckOfCards[second] = temp; 
    } 
} 

И в другом классе у меня есть:

public class GameRules { 
final deck myDeckOfCards = new deck(); 
myDeckOfCards.shuffle(); //error here 

// first 
public ImageIcon GameRules1() { 
    return myDeckOfCards.giveCardPlayer1().getImage(); 
} 

Основная проблема заключается в том, что мне нужно сделать перетасовать в колоде карт, прежде чем показать карту. Без метода тасования порядок карт является последовательным

Любая идея? Если я помещаю метод внутри public ImageIcon GameRules1(), это не дает ошибки, но мне нужно перетасовать все карты перед игрой, не раньше, чем каждый метод карты.

благодарит

Answer 1

Вы должны поместить этот вызов внутри вашего конструктора для класса:

public GameRules() { 
    myDeckOfCards.shuffle(); 
} 

Конструктор является местом, где вы делаете задачи инициализации для ваших объектов.

Кроме того, у вас действительно не должно быть метода, называемого GameRules(), который возвращает ImageIcon. Вы должны переименовать этот метод getImage или что-то в этом роде. Затем, когда вы создаете GameRules с помощью контура и назовите его getImage, колода будет перетасована.

Пример:

GameRules gr = new GameRules(); // deck gets shuffled in constructor call 
JLabel test2 = new JLabel(gr.getImage()); 

Заканчивать учебники Java для хорошей tutorial on writing constructors for your classes.

Answer 2

Почему вы не звоните shuffle метод в deck конструктора напрямую?

Answer 3

Либо использовать конструктор как это было предложено, или, если в какой-то момент у вас есть несколько конструкторов, вы можете переместить код, который является общим для них инициализаторе:

public class GameRules { 
final deck myDeckOfCards = new deck(); 

// instance initalizer 
{ 
    myDeckOfCards.shuffle(); // no more error 
} 

Но, как было отмечено в комментариях лучше использовать конструктор, при необходимости создавая конструкторы цепочки. Оказывается, что инициализаторы экземпляров лучше всего использовать в анонимных классах.