Right way to create method?

Question

Task - The Game class has two elds representing Player one and Player two.
The constructor for this class has two parameters which are the names of the two players. The
constructor will initialise the two fields.
The Game class has a single method called play which has no parameters and does not return
a value. This method should print out a suitable introduction and then:
*1. Instruct player one to play first. The instruction should give the name of player one.*
*2. Invoke the play method of Player one.*
*3. Announce the score for Player one.*
*4. Instruct player two to play next. The instuction should give the name of player two.*
*5. Announce the score for Player Two*
*6. Print message naming the winning player or annoucing a draw depending on the scores of the two players.*
private void play()
printWelcome();
player1.getName();
System.out.println("You're player one, its your turn first");
player1.play();
player1.getScore();
player2.getName();
System.out.println("You're player two, its your turn next");
player2.getScore();
}am i headed in the right direction with this?

Answer 1

import java.util.ArrayList;
import java.util.Random;
public class PackOfCards
    private ArrayList<Card> cards;
    private Random rnd;
     * Constructor for objects of class PackOfCards
    public PackOfCards()
        // initialise instance variables
        cards = new ArrayList<Card>();
        rnd = new Random();
        fillPack();
    private void fillPack() {
        for (int i=0; i<52; i++) {
            cards.add(new Card(i));
     * Select a card at random from the pack
     * @return     the card that was selected, or null if the pack is empty
    public Card selectCard()
        if (cards.size() > 0) {
            return cards.remove(rnd.nextInt(cards.size()));
        } else {
            return null;
* This class models a player.
public class Player
    private String name;
    private Card[] card = new Card[5];
    private int score = 0;
     * Contructor for objects of class Player.
     * @param A String which initialises the name of the player.
    public Player(String name)
        this.name = name;
     * Gets the score of the player.
     * @return An integer value representing the score of the player.
    public int getScore()
        return score;
     * Gets the name of the player.
     * @return A String representing the name of the player.
    public String getName()
        return name;
     * Gives the player 5 cards from the pack.
    private void fill()
        PackOfCards deck = new PackOfCards();
        int i = 0;
        while(i < card.length) {
            card[i] = deck.selectCard();
            i++;
     * Calls the fill() method and creates the method to enable the player to play the game.
    public void play()
        fill();
        Reader reader = new Reader();
        boolean stillPlaying = true;
        System.out.println(card[0].toString());
        int index = 1;
        while(stillPlaying = true) {
            if(reader.equals("higher") /*&& is correct*/) {
               if (card[index].getRank() > card[index-1].getRank())
                    score++;
                    index++;
                    System.out.println(card[index].toString());
            if(reader.equals("higher") /*&& is wrong*/) {
                    score = 0;
                    index++;
                    System.out.println(card[index].toString());
                    stillPlaying = false;
            if(reader.equals("lower") /*&& is correct*/) {
                score++;
                index++;
                System.out.println(card[index].toString());
            if(reader.equals("lower") /*&& is wrong*/) {
                score = 0;
                index++;
                System.out.println(card[index].toString());
                stillPlaying = false;
            if(reader.equals("freeze")) {
                stillPlaying = false;
        Reader.close();
}There are a couple of other classes.
This is the one im having problems with -
public class Game
    // instance variables - replace the example below with your own
    private Player player1;
    private Player player2;
     * Constructor for objects of class Game
    public Game(Player player1, Player player2)
        // initialise instance variables
        player1 = player1;
        player2 = player2;
    private void play()
        printWelcome();
        player1.getName();
        System.out.println("You're player one, its your turn first");
        player1.play();
        player1.getScore();
        player2.getName();
        System.out.println("You're player two, its your turn next");
        player2.play();
        player2.getScore();
    }