Game of Set program, need help with for loop nesting and more

[falconsx23]

I am making a game called Set, it is a card game: here is a brief description of the rules:

The game of Set is played with 81 cards, each of which has 4 properties (number, pattern,
color and shape). The number of objects is 1, 2, or 3. The 3 possible patterns are solid,
striped, and open. The 3 possible colors are red, green, and purple. The 3 possible
shapes are diamond, squiggle, and oval. The 81 cards comprise all possible combinations
of patterns, colors and shapes. Some example cards are:
• 1 Striped red oval
• 2 Solid green squiggle
• 3 Open purple diamond

Ok now what I am having trouble with is creating my deck of 81 cards. The way I am trying to do it is by creating a buch of nested for loops (should only be 4) that goes through each array and gets a number, color, shape and pattern. If nested right I should be able to create a deck of cards that has 81 possible combination's of numbers, colors, shapes and patterns.

import java.util.Scanner;
import java.util.*;
/**
 * Write a description of class GameOfSet here.
 * @author (Michael Brown) 
 * @version (a version number or a date)
 */
public class GameOfSet
{
 
    ArrayList<Card> cards;   //An array list of card objects
 
 
    /**
     * Constructor for objects of class GameOfSet
     * @param String to determine Shape and String to determine color
     * @param number can be 1-3
     * @param color can be red, green, purple
     * @param pattern can be solid, stripped, open
     * @param shape can be diamond, squiggle, oval
     * 
     */
    public GameOfSet()
    {
 
         cards = new ArrayList<Card>();
 
        Card cards = new Card ("1", "solid", "purple", "oval" );              //One card from the Cards ArraysList that has 4 perameters 
 
 
 
 
    }
 
    /**
     * Deck of 81 cards
     * nested for loop will create 81 possible 
     * combinations of number, color, pattern, and shape
     * each card will have one number, pattern, color and shape
     */
 
    public void getDeckOfCards()
    {      
      ArrayList<String> number = new ArrayList<String>(3);                     //ArrayList of numbers
      number.add("1");
      number.add("2");
      number.add("3");
 
      ArrayList<String>  color = new ArrayList<String>(3);                     //ArrayList of colors
      color.add("red");
      color.add("green");
      color.add("purple ");
 
      ArrayList<String> pattern = new ArrayList<String>(3);                 //ArrayList of patterns
      pattern.add("solid");
      pattern.add("striped");
      pattern.add("open");
 
      ArrayList<String> shape = new ArrayList<String>(3);                   //ArrayList of Shapes
       shape.add("diamond");
       shape.add("squiggle");
       shape.add("oval");
 
 
         for(int i = 0; i < number.size(); i++)
         {
 
             for(int j = 0; j < color.size(); j++)
             {
                  Card deck = cards.get(i);
 
                  for(int k = 0; k < pattern.size(); k++)
                  {
 
 
                        for(int m = 0; m < shape.size(); m++)
                        {
 
                        }
                    }
                }
            }                     
 
    }
 
 
}
 

As you can see I have Card deck = cards.get(i);, well I should also be able to get j, k, and m. But I have no idea where I should put that. And after I get that settled, I should be able to put what ever is at i, k, k, and m, and add that to the deck.

[r035198x]

A method called getDeckOfCards should not be void. It must be returning a List<Card> instead.
Enums will help to clean up the code and logic

[falconsx23]

Right now I want to just create a deck of 81 cards for that method. Im not sure If i should be returning anything yet. Also I am confused on the Enums link to help clean up the code. I will continue looking at it though.

[r035198x]

Suppose you have an enum called Color.
The Color.values() method gives you an array of all the colors.

[Frinavale]

@falconsx23
Who told you that there should only be 4 nested loops...or even that you need use for-loops to solve this problem? Seems a bit weird...you could use something like recursion to solve the problem and then there wouldn't be 4 nested loops...

I think that the point that r035198x was getting at is that it would probably be a lot easier if you were to use enums because you have a constant list of values for your numbers, patterns, colours, and shapes.

Anyways, I'm not sure why you're having a problem with solving the problem.

If you write it out as psudo code it's pretty obvious how the loop structure should look and what you should be doing:

For each number from 1 - 3 
  For each colour in my list/enum of shapes
    For each shape in my list/enum of shapes
      For each pattern in my list/enum of patterns
        Add a new card(number, colour, shape, pattern) to my deck of cards
      Get the next pattern
    Get the next shape
  Get the next colour
Get the next number

How much more simple can it get?
It really helps if you give your variables meaningful names...instead of using a,b, c or i,j,k,l,m use things like "number", "colour", "shape" and "pattern".

@falconsx23
What do you mean you're not sure if you should be returning anything....
The name of the method is getDeckOfCards...not "setDeckOfCards"...not "populateDeckOfCards"....it's getDeckOfCards.

It seems very obvious that when someone calls the getDeckOfCards method the are expecting to get a deck of cards....how are they supposed to get anything if you don't give them something (return something to them) when they call the method? Again, pay attention to names :)

-Frinny

[falconsx23]

ok you are right that is bad programming on my part... sorry

[JosAH]

For any number n in the range [0,81) you can do the following:

int number= n/27;
int pattern= n/9%3;
int color= n/3%3;
int shape= n%3;
 

So a single loop will generate all your different cards; no recursion nor nested loops are needed.

kind regards,

Jos

[Frinavale]

@JosAH
Show off!

: )

[JosAH]

@Frinavale
Not at all; write down all numbers in the range 0 ... 80 in ternary (base 3) notation and you'll see: 0000 ... 2222. There are much trickier things one can do with this notion in any base.

kind regards,

Jos (ex-moderator)

[falconsx23]

Ok Frinavale, while I was working on this program my Lab Teacher Assistant suggested that using nested for loops will help me out, and what I have up there he said I was on the right track. Also keep in mind that I am still a beginner at Java and no expert. So what may be obvious to you is not to me and I apologize for that.

JosAH thanks alot for the response and I will try out the way you did as soon as I can get my program up.

[Frinavale]

Well the first thing you need to do is create/declare something that will represent the Deck Of Cards. An ArrayList<Cards> will work fine.

Have you learned about variable scope yet?

You could declare the deck of cards at the Class Level or at Method Level (let's just stick with these 2 scopes for now)...

If you declare the card deck in the getDeckOfCards method then the deck of cards is destroyed once the method is finished executing. If you need to use this deck of cards anywhere else then you either have to return the deck of cards to the calling code, or you need to declare the deck of cards at the class level.

If you declare the deck of cards as a member of the GameOfSet class then it will exist for as long as the GameOfSet exists. In this case I would rename the method from getDeckOfCards to "populateDeckOfCards" because "get" implies that whatever is calling the method will be getting something back from the method. If another class needs to use the same deck of cards then I would implement the getDeckOfCards() to return the deck of cards you declared as a member of the GameOfSet class.

Anyways, the important point to take away from this is that you need to declare some sort of container that will represent the deck of cards....otherwise you won't have anywhere to put the cards.

Once you've declared the DeckOfCards container you need to loop through all of the card possibilities in order to create a card for each possibility.

You already have a good idea of how to do this. How you do the loop doesn't really matter. The easiest way to for a beginner to understand would be to use For Loops....so keep on track with what you were doing.

So, for each possibility create 1 card and add it to the ArrayList<Card> that represents the deck of cards.

Once you are finished adding all of the cards to the deck return the deck (or not if it's a class member)

If you are going to return the deck then you need to change your getDeckOfCards method so that it returns the deck of cards. So you'll need to change the method's signature to:

public ArrayList<Card> getDeckOfCards()

Give this a try.
If you're still stuck, post your updated code and we can discuss it :)

-Frinny

[falconsx23]

No we have not learned about scopes yet nor recursion. And yes what you are saying about for each possibility create 1 card and add it to the ArrayList<Card> that represents the deck of cards is what I was trying to do.

That was what I am stuck on. I wasn't sure how to set up the for loops and also where to place the add method to add the card that is created to the deck.

I am thinking that I should only have to call the add method once, but I am not sure where to place that add method. What I mean by add method is something like this

cards.add(Card).
"Card" is the card being created from the nested for loops. So card should be like Card ("1", "solid", "purple", "oval" ).

What you are saying here "So, for each possibility create 1 card and add it to the ArrayList<Card> that represents the deck of cards." Would I place the add method after all of the for loops? Anyways here is what I have so far:

public class GameOfSet
{
 
    ArrayList<Card> cards;   //An array list of card objects
 
 
 
    public GameOfSet()
    {
 
         cards = new ArrayList<Card>(); 
        Card cards = new Card ("1", "solid", "purple", "oval" );              //One card from the Cards ArraysList that has 4 perameters 
 
    }
 
    public ArrayList<Card> populateDeckOfCards()
    {      
     ArrayList<Cards>();                                                                         //An ArrayList that will represent DeckOfCards like a container  
      ArrayList<String> number = new ArrayList<String>(3);                     //ArrayList of numbers
      number.add("1");
      number.add("2");
      number.add("3");
 
      ArrayList<String>  color = new ArrayList<String>(3);                     //ArrayList of colors
      color.add("red");
      color.add("green");
      color.add("purple ");
 
      ArrayList<String> pattern = new ArrayList<String>(3);                 //ArrayList of patterns
      pattern.add("solid");
      pattern.add("striped");
      pattern.add("open");
 
      ArrayList<String> shape = new ArrayList<String>(3);                   //ArrayList of Shapes
       shape.add("diamond");
       shape.add("squiggle");
       shape.add("oval");
 
 
         for(int i = 0; i < number.size(); i++)
         {
 
             for(int j = 0; j < color.size(); j++)
             {
                  Card deck = cards.get(i);
 
                  for(int k = 0; k < pattern.size(); k++)
                  {
 
 
                        for(int m = 0; m < shape.size(); m++)
                        {
 
                        }
                    }
                }
            }                     
 
    }   
}

As you can see I changed the signature of the method so that it could return something, but at the same time I am having trouble creating the container for the populateDeckOfCards. (Line 18)

[Frinavale]

The add method that I'm talking about is part of the ArrayList class.
Sun (the makers of Java) has a website that lists all of the classes along with the class's methods etc (the API). This documentation can be found here. Bookmark this resource because every time you need help using a Java class you can refer to it for help before asking us.

Anyways, like I was saying, the add method is a member of the ArrayList class.

First you need to instantiate the ArrayList class before you can use it...I'm not sure if you know what instantiate means...it means to "create an instance of".

There are classes (they are like blue prints...for a house) and then there are instances of classes (the actual house).

Before you can use any variable you have to instantiate it (create an instance of it) or else it doesn't exist and you will get Null Reference errors if you try to use it.

Ok so create a new instance of the ArrayList class to use as your deckOfCards:

 ArrayList<Cards> deckOfCards = new ArrayList<Cards>(81);     
 

Now if you want to add a card to the deck you call the ArrayList's add method, providing it a card:

Card theCard = new Card(.......);
deckOfCards.add(theCard);

Make sense?

[Frinavale]

I was just reviewing your code and I don't have a clue what you're trying to do at line 18.

What you did is create a class member variable "cards"...I've been referring to this as the deckOfCards.

So you already have a container (an ArrayList) to put the cards into when you create them.

You have already correctly instantiated this container in the constructor. (It might be a good idea to rename your "cards" variable to deckOfCards since it is representing a "deck of cards")


Great!

Don't know what you're doing at line 18.....just delete that.

Also remove line 12....there's no need to have that there. You should be creating new cards (not at line 12) in the inner most for loop instead (line 53)...and then adding it to the deck of cards (blah that name...adding it to your "cards" variable).

You need to call the "populateDeckOfCards" method in the constructor.

Since you are going to be populating a class variable, you don't need to return anything. So, (sorry) change your populateDeckOfCards method signature back to returning void:

public void populateDeckOfCards()

Now in the populateDeckOfCards you want to add to the card to the deck of cards (the "cards" variable) by using the ArrayList.add() method (**hint: line 53**)......

[falconsx23]

Yes I am following you so far. I did create a seperate class called Card like you said here is the code for it just in case you need to see it:

public class Card
{
    public String number;        
    public String color;   
    public String pattern;
    public String shape;
 
    public Card(String number, String color, String pattern, String shape)
    {
        this.number = number;
        this.color = color;
        this. pattern = pattern;
        this.shape = shape;   
    }
 
    public String getNumber()
    {
 
        return number;
    }
 
    /**
     * returns color
     */
 
    public String getColor()
   {
        return color;
    }
 
    /**
     * returns pattern
     */
    public String pattern()
    {
 
        return pattern;
    }
 
    /**
     * returns shape
     */
    public String shape()
    {  
        return shape;
    }   
}

Yea I tried using the add method from the JAVA API earlier, I was typing it wrong. OK So here is my code again:

public class GameOfSet
{
 
    ArrayList<Card> cards;   //An array list of card objects
 
    public GameOfSet()
    {
 
         cards = new ArrayList<Card>();  
          ArrayList<Card> populateDeckOfCards = new ArrayList<Card>(81); 
    }
 
 
    public  void populateDeckOfCards()
    {      
     ArrayList<String>deckOfCards = new ArrayList<String>(81);   
 
      ArrayList<String> number = new ArrayList<String>(3);                     //ArrayList of numbers
      number.add("1");
      number.add("2");
      number.add("3");
 
      ArrayList<String>  color = new ArrayList<String>(3);                     //ArrayList of colors
      color.add("red");
      color.add("green");
      color.add("purple ");
 
      ArrayList<String> pattern = new ArrayList<String>(3);                 //ArrayList of patterns
      pattern.add("solid");
      pattern.add("striped");
      pattern.add("open");
 
      ArrayList<String> shape = new ArrayList<String>(3);                   //ArrayList of Shapes
       shape.add("diamond");
       shape.add("squiggle");
       shape.add("oval");
 
 
         for(int i = 0; i < number.size(); i++)
         {
 
             for(int j = 0; j < color.size(); j++)
             {
                  Card deck = cards.get(i);
 
                  for(int k = 0; k < pattern.size(); k++)
                  {
 
                       Card theCard = new Card("","","","");
                       populateDeckOfCards().add(theCard);
 
 
                        for(int m = 0; m < shape.size(); m++)
                        {
 
                        }
 
                }
            }                     
 
    }   
}
}

Im getting an error message at line 50 stating void cannot be dereferenced. I tried calling the populateDeckOfCards method in the constructor and one thing I changed from what you originally stated was

 ArrayList<Cards> populateDeckOfCards = new ArrayList<Cards>(81); 

to

ArrayList<Card> populateDeckOfCards = new ArrayList<Card>(81); 

I was getting an error stating cannot find symbol Cards.

[Frinavale]

We need to stop for a second.
You have to understand scope before we can fix this problem.

Actually, firstly you should know that you should not have variables with the same name as a function/method.

Right now you have a method named "populateDeckOfCards()".
You also have declared an ArrayList named "populateDeckOfCards" in your constructor method....and for some reason you're trying to access it in your populateDeckOfCards() method but.......

This is a very confusing thing to do and is probably the source of your problems.

With that aside we have to go over the concept of scope and classes.

As I stated before, a class is like a blue print for creating Objects. An Object is something that exists in memory.

Objects/Classes have members. Members can be simple variables (like Integers, and Strings and such), but they can also be methods that will modify Object's state or do things.

In your case you have a Card class.
A card has a few different members:

• public String number;
• public String color;
• public String pattern;
• public String shape;

You have declared them as part of the class. This means that these variables will exist as long as the Card Object exists in memory. Once you're finished with the Card Object, they are destroyed and are no longer accessible. The scope of these methods are at the class level.

The card class also has a bunch of methods:

• public Card(String number, String color, String pattern, String shape) <-- the constructor
• public String getNumber()
• public String getColor()
• public String pattern()<--you should to rename this to getPattern because you already have a String variable named pattern.
• public String shape()<-- you should to rename this to getShape because you already have a String variable named shape.

These methods in the Card class have access to class members (the variables above)....and if you had declared the Strings as "private" the these strings would Only be accessible to the methods that are members of the Card class. (But you declared them public...which is weird...especially since you've provided get methods to retrieve them....)


Let's add another method to the Card class...let's create a method named PrintCard. Since this PrintCard method is part of the Card class it has access to all of the member variables of the Card class and so it is able to print the card details to the screen:

public void PrintCard(){
  StringBuilder theOutput=new StringBuilder();
  theOutput.append(number);
  theOutput.append(color);
  theOutput.append(pattern);
  theOutput.append(shape);
  System.out.println(theOutput.toString() );
}

The variable "theOutput" in the PrintCard method has a method level scope. This means that theOutput cannot be accessed out side of the PrintCard method. It is only available within the PrintCard method. It also means that theOuput is destroyed as soon as the method is finished executing.

Hope you're with me so far.

Say we have a loop and we declare a variable inside the loop. That variable only exists until the loop is finished executing. It cannot be access outside of the loop.

As you can see, variables have life-spans. It is important to take care in where you declare them because this will have a big impact on how things work.

Now let's look at the GameOfSet class.
It has one class member variable:

• ArrayList<Card> cards;

This means that this variable is accessible to all of the methods in the GameOfSet class.

The important thing is that this variable has to exist before you can use it....or else you're going to get Null Reference errors (like I explained before).

That is the whole purpose to the Constructor method. The constructor method is used to instantiate all of the class member variables. It is called when you use the "new" keyword to instantiate an Object of the class.

For example, when you create a new instance of the GameOfSet like:

GameOfSet theGame = new GameOfSet();

The constructor method is called. It is the constructor's responsibility to make sure that all of the class member variables are instantiated.

Therefore, your constructor needs to at least have the following:

public GameOfSet() {
         cards = new ArrayList<Card>();  
}

Now your "cards" ArrayList variable is instantiated and can be used by all of the methods in the GameOfSet class.

Your GameOfSet has one method:

• public void populateDeckOfCards()

This method doesn't return anything...
What is the purpose of having this method then?

It is supposed to populate the class member variable "cards" with Card Objects.

You should call this method after you have instantiated the "cards" ArrayList in the constructor. That way the "cards" variable will not only be instantiated, but it will also have data in it!

So your constructor should look like:

public GameOfSet() {
         cards = new ArrayList<Card>();  
          populateDeckOfCards();
}

Now to address your problem on line 50.

You have:
populateDeckOfCards().add(theCard);

First of all populateDeckOfCards is the name of the method. In Java you execute a method by calling it's name and providing it with parameters (eg: populateDeckOfCards() will call the populateDeckOfCards method). Then you have .add(theCard) but how is that supposed to work (especially since the method doesn't return anything)? .... In otherwords this really doesn't make any sense and that is why the compiler is telling you that there is a problem.

Secondly, you cannot access any variables that were declared in the constructor method in the populateDeckOfCards() method. So even if you changed the name of the variable that was declared in the constructor method, it is not accessible in the populateDeckOfCards() because this is how scope works.

But the "cards" variable is accessible in the populateDeckOfCards() method.
Because it is has been declared at the class level...therefore has scope for every method in the class.


Later I'll tell you about "public" and "private" scope.
For now I'll let you figure out how to populate the cards variable :)

-Frinny

[falconsx23]

Ok Thanks for the response I can probably figure it, Ill get back to you if i am still stuck

[JosAH]

@Frinavale
Although it can be confusing there's technically nothing wrong with it; member and method names live in separate symbol tables and the compiler can unambiguously recoginise where to look: if a name is followed by an open parenthesis it must be a method, otherwise it's a member. Have a look:

 
public class Method  {
 
    static final int method= 0;
    static final int METHOD= 1;
 
    static int method(int method) {
        return (method == Method.method)?
            Method.METHOD:
            method(method-Method.METHOD)*method;
    }
    public static void main(String[] args) {
 
        System.out.println(method(5));
    }
}
 

kind regards,

Jos

[Frinavale]

This is true for Java but please remember that I haven't used Java for quite a few years....and in VB.NET this sort of thing could be pretty bad. Especially if you declared a public class member array with the same name as a public method. In VB.NET arrays use () instead of [] and so there would be ambiguity (I don't even think it's possible to do this actually...I think the compiler would complain...mind you I would start complaining way before the compiler!)

I would try to keep things as clear as possible, and even though it is possible to name a variable with the same name as a method (or even have mulitple variables with the same name in different cases) it is going to be confusing to anyone looking at the code and I would strongly be apposed to suggesting the idea.

The problem in this case is that the OP was clearly confused by the name of the variable in the constructor, the method name, how to use/call methods and how to use variables.

Besides that it just feels wrong to have a public string "pattern" and a public method "pattern()".....they both do return the pattern string...what's the point really? The pattern string should be private in my mind and interaction should take place through the get/set methods associated with this member. This will allow for possible future design changes to the private pattern string member without effecting the public API.

I don't think that the OP understands the concept of public and private scope modifiers yet....so I was going to just leave that alone :)

Besides that your Method class with the class members: int method and int METHOD, that has an int method() method that takes an int method as a parameter and then makes a recursive call to itself is pretty gross.

Yuck...
It's like some kid just showed me a deformed bug with multiple body parts that looked like heads.
Thanks for that Jos....bleh....

It would make a lot more sense if you had named things appropriately:

public class FactorialCalculator{
    static final int ZERO = 0;
    static final int ONE = 1;
    static int factorial(int number){
        return (number == ZERO) ?  
          ONE :  
          factorial(number - ONE) * number;
    }
 
    public static void main(String[] args) {
          System.out.println(factorial(5));
    }
}

-Frinny

[JosAH]

@Frinavale
True, all true but your previous post insinuated (to me at least) that this practice would be incorrect which it isn't. If someone writes code like I did in my previous example he deserves to be defenestrated head first. Sun's naming convention partially solves the problem because methods shoud be (or start with) a verb while members are supposed to be nouns.

kind regards,

Jos

[falconsx23]

Ok its been about three weeks since we talked about this program. I made a couple of adjustments like creating the deck of cards inside the constructor and creating a method that creates 12 random cards. Each card that is created randomly will be different meaning each card will have a different number, color, pattern and shape.

Now I am trying to create the actual play of the game. I am going to have the game played by using the console (kinda confused on what that actually is) instead of GUI such as JFrame or an applet. The way I understand the console is that it would be the printed out (as output) on to the terminal window.

The way I started doing this is by using System.out.println. So right now the part that I am stuck on is actually displaying or printing the 12 random cards onto the screen. The way I tried it is shown in comments at the bottom of the program. I know what I am trying to do doesnt make sense, but I am still working on it.

import java.util.Scanner;
import java.util.*;
import javax.swing.*;
import java.awt.*;
 
/**
 * Write a description of class GameOfSet here.
 * @author (Michael Brown) 
 * @version (a version number or a date)
 */
public class GameOfSet
{
 
    ArrayList<Card> deckOfCards;   //An array list of card objects
 
 
    /**
     * Constructor for objects of class GameOfSet
     * @param String to determine Shape and String to determine color
     * @param number can be 1-3
     * @param color can be red, green, purple
     * @param pattern can be solid, stripped, open
     * @param shape can be diamond, squiggle, oval
     * 
     */
    public GameOfSet()
    {
        deckOfCards = new ArrayList<Card>(81);          //"Card" is what the arraylist is made up of
 
      ArrayList<String> number = new ArrayList<String>(3);                     //ArrayList of numbers
      number.add("1");
      number.add("2");
      number.add("3");
 
      ArrayList<String>  color = new ArrayList<String>(3);                     //ArrayList of colors
      color.add("red");
      color.add("green");
      color.add("purple ");
 
      ArrayList<String> pattern = new ArrayList<String>(3);                 //ArrayList of patterns
      pattern.add("solid");
      pattern.add("striped");
      pattern.add("open");
 
      ArrayList<String> shape = new ArrayList<String>(3);                   //ArrayList of Shapes
       shape.add("diamond");
       shape.add("squiggle");
       shape.add("oval");
 
      Card theCard = null;
         for(int i = 0; i < number.size(); i++)
         {
 
             for(int j = 0; j < color.size(); j++)
             {
 
                  for(int k = 0; k < pattern.size(); k++)
                  {
 
                        for(int m = 0; m < shape.size(); m++)
                        {
                             theCard = new Card(number.get(i),color.get(j),pattern.get(k),shape.get(m));
                             deckOfCards.add(theCard);    // adds the new card that is being created to the deck of cards array list
                      }
 
                }
            }                     
 
          }   
    }
 
     /**
      * To allow 12 cards from the deckOfCards ArrayList  to be choosen at random 
      * Then laid out in a grid of 3 rows with 4 cards each
      */
     public ArrayList<Card> createRandomNumberOfCards()
     {
         Random generator = new Random();                       //Creating the random class
         ArrayList <Integer> list = new ArrayList <Integer>();
         int randomIndex = 0;                //A random interger (range 1 to 12) so random class creates 12 random cards
         int x = 12;
         ArrayList <Card> randomCards = new ArrayList <Card>();         //Another ArrayList, when a card is pulled from the deckOfCard ArrayList, it is stored into the randomCards arrayList
         while(x>0) 
         {
             randomIndex = generator.nextInt(81);               //randomIndex is a random number ranging from 0 to 81 cards 
 
             while(list.contains(new Integer(randomIndex)))  //this is a nested list that make sure that there is more then one of the same random card being returned. 
             {
                randomIndex = generator.nextInt(81);               //If there is one of the same card being returned then it will simply make a new random card 
            }
            list.add(randomIndex);                                         // value at randomIndex is being added to the list of integers
            x--;
 
 
         }
              for(int y= 0; y < 12; y++)
             {
                                                                                                           //gets a number from the "list" ArrayList. Then deckOfCards
                 randomCards.add(deckOfCards.get(list.get(y)));  //uses that number to get the value from the deckOfcards ArraysList and places it into the random card class
 
             }
 
             return  randomCards;                                                       //This is the random card being returned 
      }
 
 
      /**
       * To create a grid that contains 12 cards
       * 3 rows and 4 columns
       */
 
      public static void main (String[] args)
      {
        System.out.println("Welcome to the Game Of Set");
        System.out.println("12 random cards out of 81 are choosen randomly");
        //System.out.println(Card(String number(),color(), pattern(), shape()));  //Prints out the 12 random cards 
        //System.out.println( ArrayList <Card> randomCards ) ;
       // System.out.println(Card randomCards());
        }
 
 
}
 

[Frinavale]

In post 16 I gave an example of a method that prints a card to a screen.

You will have to add a public method to the Card class ("printCard" or "displayCard" or something) that will output the card details to the screen.

Then you will have to add a public method to the GameOfSet class ("printDeck" or "displayDeck"...) that will loop through all of the cards in the deck of cards and call the method that outputs the card details.

Or you could just write a toString method for the Card class that returns a String representation of the card....then loop through the deck of cards and output the card details to the screen by calling the toString method for each card in the deck.

The important thing here is that in order to achieve what you want to achieve you need to loop through all of the cards in the deck of cards and call a method (or methods) that will result in the card details being printed to the screen.

-Frinny

[JosAH]

@Frinavale
Don't do that; implement a public toString() method that returns a String representation of the object; PrintStreams and Writers call that method when they have to print a non-primitive value; it's much more flexible like that.

kind regards,

Jos

[falconsx23]

Before I explain what I am stuck on, I will try to explain what I do. I agree with you Jos that going with the ToString method. Just in case you didn't know I have another class named Card, which is for creating a Card. I think it will make sense if I place the toString method inside the card class. I will use the toString method to create 12 random cards.

Then, inside my GameOfSet class I will create a main method to call the toString method, which creates 12 random cards and prints out the 12 random cards using System.out.println.

Now the problem that I am having is that I am not too familiar with using the toString so I am going to need your help on how to create the 12 random cards inside the toString method of my Card class.

public class Card
{
    public String number;        
    public String color;   
    public String pattern;
    public String shape;
 
     /**
      * a deck of cards 
      * @param number of objects on cards
      * @param color of shape and pattern
      * @param what type of pattern
      * @param what type of shape
      */
    public Card(String number, String color, String pattern, String shape)
    {
        this.number = number;
        this.color = color;
        this. pattern = pattern;
        this.shape = shape;   
    }
 
    /**
     * number of objects
     * only 1-3
     */
    public String getNumber()
    {
 
        return number;
    }
 
    /**
     * returns color
     */
 
    public String getColor()
   {
        return color;
    }
 
    /**
     * returns pattern
     */
    public String pattern()
    {
 
        return pattern;
    }
 
    /**
     * returns shape
     */
    public String shape()
    {  
        return shape;
    }
 
    public String toString ()
      {
          randomCards (String number, color, pattern, shape);
        return  "Here are the 12 random cards "  + new ArrayList<Card> randomCards(12);   //12 random cards are being shown to the user
 
 
        }
 
}
 
 

[Frinavale]

I think it would be best to create 12 random Cards Objects in the GameOfSet class...

Loop through the 12 random cards and call their toString() methods. The toString() method should return a string that contains the Card object's details. You can print this string to the screen by passing it to the the System.out.println() method.

The reason i think it would be best to create the 12 random Card Objects in the GameOfSet class is because a Card Object represents a single Card....it does not represent 12 cards, just one card; therefore, it doesn't make sense to have the Card's toString() method print 12 cards (how can it really? it only has access to that single card's details)

[falconsx23]

Yes, but in my public ArrayList<Card> createRandomNumberOfCards()method I have 12 random cards being created. Which means my randomCards arrayList holds 12 random cards from the deck of cards arrayList.

So what I am trying to say is I am not sure how to print those 12 random cards from the randomCards ArrayList using the toString method.

Here is what I doing now

   public String toString ()
      {
          for(int i = 0, i < randomCards.size, i++)
         {
            System.out.println(randomCards())
 
           return randomCards();
 
 
        }

This is in the game of Set class

[Frinavale]

@falconsx23
Ok, declare a variable to hold the ArrayList (the 12 random cards), then loop through each Card in the ArrayList and call it's toString method to retrieve the string containing the card details.

For example:

ArrayList<Card> randomCards = createRandomNumberOfCards();
for(int i= 0; i < randomCards.size(); i++){
   String cardDetails = randomCards[i].toString();
   System.System.out.println(cardDetails);
}

Please be aware that the Card class's toString() method has to return a String containing the card's details.

For example:

public String toString(){
  StringBuilder theOutput=new StringBuilder();
  theOutput.append(" number:");
  theOutput.append(number);
  theOutput.append(" colour:");
  theOutput.append(color);
  theOutput.append(" pattern:");
  theOutput.append(pattern);
  theOutput.append(" shape:");
  theOutput.append(shape);
  theOutput.append(" ");
  return theOutput.toString();
}

[falconsx23]

Ok what you are saying is making a little more sense. But before I am able to fully understand what you are saying,

    ArrayList<Card> randomCards = createRandomNumberOfCards();
    for(int i= 0; i < randomCards.size(); i++){
       String cardDetails = randomCards[i].toString();
       System.System.out.println(cardDetails);
    }

where should this be placed. To me it looks like it should be placed inside the toString method, but I am not sure. When I tried to, I received many errors, unless I was doing it wrong. And do you still feel that my toString method should be placed in the gameOfSetClass or Card class?

[Frinavale]

It cannot go in the Card's toString() method.

Because the Card's toString method returns a string representation of that instance of the Card class (that Card Object).

It needs to go in the function that is using the ArrayList of 12 card objects.
The function where that is happening is in the GameOfSet class....

-Frinny

[falconsx23]

Thanks I got that part done. Now Im trying to have the program ask the user to enter an actually set. For 3 cards to form a set they must be all different or all alike within each category. For example.

0  number:3 color:green pattern:open shape:squiggle 
1  number:1 color:red pattern:open shape:diamond 
2  number:1 color:purple  pattern:solid shape:squiggle 
3  number:1 color:green pattern:striped shape:diamond 
4  number:1 color:red pattern:solid shape:diamond
5  number:1 color:purple  pattern:solid shape:diamond 
6  number:1 color:green pattern:open shape:squiggle 
7  number:2 color:green pattern:open shape:diamond 
8  number:3 color:purple  pattern:solid shape:squiggle 
9  number:1 color:purple  pattern:striped shape:squiggle 
10  number:1 color:red pattern:striped shape:diamond 
11  number:3 color:red pattern:striped shape:diamond

Cards 1, 4, and 10 is an example of a set since they are all alike in every category except pattern.

The problem that I am having for now is how to code this. The only way I can think of is using if statements. Any suggestions? Im working on it now.

Here is some of the code that I fixed up from last time.

 public void playGame()
      {
         ArrayList<Card> randomCards = createRandomNumberOfCards();
          for(int i= 0; i < randomCards.size(); i++)   //checking for every  card in the randomCard ArrayList
          {
 
 
              String cardDetails = randomCards.get(i).toString();    //using the toString of the Card method (which converts the details of the cards into a String 
              System.out.println(i + " " + cardDetails);   //prints out the cards (details of the cards) into the terminal window 

}


I wrote the toString() method the same way you had it.

The code below is what I am working on in the GameOfSet class.

 Card one = randomCards.get(1);
          Card two = randomCards.get(2);
          Card three = randomCards.get(3);
          Card four = randomCards.get(4);
          Card five = randomCards.get(5);
          Card six = randomCards.get(6);
          Card seven = randomCards.get(7);
          Card eight = randomCards.get(8);
          Card nine = randomCards.get(9);
          Card ten = randomCards.get(10);
          Card eleven = randomCards.get(11);
 
          System.out.println("Pick your 1st card for a set  " +  one);
          Scanner in = new Scanner(System.in);
          String setH = in.nextLine(); 
 
          System.out.println("Pick your 2nd card for a set");
          System.out.println("Pick your 3rd card for a set");

I made it where each card has a variable. Now Im trying to think of a way where I can have it where the user enters one of those card variables, then another and another and then the program will compare each of those cards to each other to determine if they are a set.

[falconsx23]

So far I got my game to ask a user to input a card to determine a set. It first asks the user to enter one card (0 - eleven making 12). It asks the user to enter one card and then another and then a third card.

What I am stuck on is how do I get my program to ask the user to enter again if the user enters the wrong data. For instance the only way you can enter card one is by typing one. If the user enters 1 or fg instead of one, how do I get my program to ask the user to enter another card.

As you can see my program has tons of else if statements. I could not find an easier way to create that data. If you guys see a better and easier way can you let me know.

 public void playGame()
      {
         ArrayList<Card> randomCards = createRandomNumberOfCards();
          for(int i= 0; i < randomCards.size(); i++)   //checking for every  card in the randomCard ArrayList
          {
 
 
              String cardDetails = randomCards.get(i).toString();    //using the toString of the Card method (which converts the details of the cards into a String 
              System.out.println(i + " " + cardDetails);   //prints out the cards (details of the cards) into the terminal window 
 
 
 
        }
 
          //These cards are set to a varabiable to 
          //make it easier for the user to input
          Card zero = randomCards.get(0);
          Card one = randomCards.get(1);
          Card two = randomCards.get(2);
          Card three = randomCards.get(3);
          Card four = randomCards.get(4);
          Card five = randomCards.get(5);
          Card six = randomCards.get(6);
          Card seven = randomCards.get(7);
          Card eight = randomCards.get(8);
          Card nine = randomCards.get(9);
          Card ten = randomCards.get(10);
          Card eleven = randomCards.get(11);
 
 
          System.out.println("Pick your 1st card for the set  ");
          Scanner in = new Scanner(System.in);
          String setH = in.nextLine();
 
          if(setH.equals("one" ))
          {
             System.out.println(one);
          //    System.out.println("Pick your second card for the set ");
            //  String setJ = in.nextLine();
             // if(setJ.equals("two"))
              //{
                //  System.out.println(two);
               //}
 
          }
            else if(setH.equals("three"))
            {
                System.out.println(three);
            }
             else if(setH.equals("four"))
            {
                System.out.println(four);
            }
             else if(setH.equals("four"))
            {
                System.out.println(four);
            }
             else if(setH.equals("five"))
            {
                System.out.println(five);
            }
             else if(setH.equals("six"))
            {
                System.out.println(six);
            }
             else if(setH.equals("seven"))
            {
                System.out.println(seven);
            }
             else if(setH.equals("eight"))
            {
                System.out.println(eight);
            }
             else if(setH.equals("nine"))
            {
                System.out.println(nine);
            }
             else if(setH.equals("ten"))
            {
                System.out.println(ten);
            }
             else if(setH.equals("eleven"))
            {
                System.out.println(eleven);
            }
 
 
 
              System.out.println("Pick your second card for the set ");
              String setJ = in.nextLine();
               if(setJ.equals("zero"))
               {
                   System.out.println(zero);
                }
                  else if(setJ.equals("one"))
                  {
                      System.out.println(one);
                    }
              else if(setJ.equals("two"))
              {
                  System.out.println(two);
                }
              else if(setJ.equals("three"))
              {
                  System.out.println(three);
                }
                else if(setJ.equals("four"))
                {
                    System.out.println(five);
                }
                  else if(setJ.equals("six"))
                  {
                      System.out.println(six);
                    }
              else if(setJ.equals("seven"))
                {
                    System.out.println(seven);
                }
              else if(setJ.equals("eight"))
              {
                  System.out.println(eight);
                }
              else if(setJ.equals("eight"))
              {
                  System.out.println(eight);
                }
              else if(setJ.equals("eight"))
              {
                 System.out.println(eight);
                }
              else if(setJ.equals("nine"))
              {
                    System.out.println(nine);
                }
              else if(setJ.equals("ten"))
              {
                  System.out.println(ten);
                }
                  else if(setJ.equals("eleven"))
            {
                System.out.println(eleven);
            }
 
             System.out.println("Pick your third card for a set ");
              String setF = in.nextLine();
               if(setF.equals("zero"))
               {
                   System.out.println(zero);
                }
                else if(setF.equals("one"))
                {
                    System.out.println(one);
                }
                else if(setF.equals("two"))
                {
                    System.out.println(two);
                }
                else if(setF.equals("three"))
                {
                    System.out.println(three);
                }
                else if(setF.equals("four"))
                {
                    System.out.println(four);
                }
                else if(setF.equals("five"))
                {
                    System.out.println(five);
                }
                else if(setF.equals("six"))
                {
                    System.out.println(six);
                }
                else if(setF.equals("seven"))
                {
                    System.out.println(seven);
                }
                else if(setF.equals("eight"))
                {
                    System.out.println(eight);
                }
                else if(setF.equals("nine"))
                {
                    System.out.println(nine);
                }
                else if(setF.equals("ten"))
                {
                    System.out.println(ten);
                }
                else if(setF.equals("eleven"))
                {
                    System.out.println(eleven);
                }

[falconsx23]

ok I figured that part. I just placed my program in a while loop using a Boolean.
Now I need help with how can I actually determine a set. I just dont see how can I say if card one's color, number, and pattern equals card two's color , number, and pattern Then it is a set.

By the look of this I know I have to use an if statement. I'm just dont know how to code it.

  public void playGame()
      {
         ArrayList<Card> randomCards = createRandomNumberOfCards();
          for(int i= 0; i < randomCards.size(); i++)   //checking for every  card in the randomCard ArrayList
          {
 
 
              String cardDetails = randomCards.get(i).toString();    //using the toString of the Card method (which converts the details of the cards into a String 
              System.out.println(i + " " + cardDetails);   //prints out the cards (details of the cards) into the terminal window 
 
 
 
        }
 
          //These cards are set to a varabiable to 
          //make it easier for the user to input
          Card zero = randomCards.get(0);
          Card one = randomCards.get(1);
          Card two = randomCards.get(2);
          Card three = randomCards.get(3);
          Card four = randomCards.get(4);
          Card five = randomCards.get(5);
          Card six = randomCards.get(6);
          Card seven = randomCards.get(7);
          Card eight = randomCards.get(8);
          Card nine = randomCards.get(9);
          Card ten = randomCards.get(10);
          Card eleven = randomCards.get(11);
 
          Boolean correct = false;
          while (!correct){
              System.out.println("Pick your 1st card for the set  ");
                Scanner in = new Scanner(System.in);
                String setH = in.nextLine();
                if (setH.equals( "zero")){
 
                    correct = true;
                    System.out.println(zero);
                }
                else if(setH.equals( "one"))
                {
                    System.out.println(one);
                    correct = true;
                }
 
                  else if(setH.equals( "two"))
                {
                    System.out.println(two);
                    correct = true;
                }
                  else if(setH.equals( "three"))
                {
                    System.out.println(three);
                    correct = true;
                }
                  else if(setH.equals( "four"))
                {
                    System.out.println(four);
                    correct = true;
                }
                  else if(setH.equals( "five"))
                {
                    System.out.println(five);
                    correct = true;
                }
                  else if(setH.equals( "six"))
                {
                    System.out.println(six);
                    correct = true;
                }
                  else if(setH.equals( "seven"))
                {
                    System.out.println(seven);
                    correct = true;
                }
                  else if(setH.equals( "eight"))
                {
                    System.out.println(eight);
                    correct = true;
                }
                  else if(setH.equals( "nine"))
                {
                    System.out.println(nine);
                    correct = true;
                }
                  else if(setH.equals( "ten"))
                {
                    System.out.println(ten);
                    correct = true;
                }
                  else if(setH.equals( "eleven"))
                {
                    System.out.println(eleven);
                    correct = true;
                }
 
                else {
                    correct = false;
                    System.out.println("That is not a correct card please enter again");
                }

I followed this procedure two more times to allow the user to select up to three cards.

[falconsx23]

Ok I finished my game of set. If anyone is still looking at this thread I just wanna say thanks for the help.

GameOfSet Class

import java.util.Scanner;
import java.util.*;
import javax.swing.*;
import java.awt.*;
 
/**
 * Write a description of class GameOfSet here.
 * @author (Michael Brown) 
 * @version (a version number or a date)
 */
public class GameOfSet
{
 
    ArrayList<Card> deckOfCards;   //An array list of card objects
 
 
    /**
     * Constructor for objects of class GameOfSet
     * @param String to determine Shape and String to determine color
     * @param number can be 1-3
     * @param color can be red, green, purple
     * @param pattern can be solid, stripped, open
     * @param shape can be diamond, squiggle, oval
     * 
     */
    public GameOfSet()
    {
        deckOfCards = new ArrayList<Card>(81);          //"Card" is what the arraylist is made up of
 
      ArrayList<String> number = new ArrayList<String>(3);                     //ArrayList of numbers
      number.add("1");
      number.add("2");
      number.add("3");
 
      ArrayList<String>  color = new ArrayList<String>(3);                     //ArrayList of colors
      color.add("red");
      color.add("green");
      color.add("purple ");
 
      ArrayList<String> pattern = new ArrayList<String>(3);                 //ArrayList of patterns
      pattern.add("solid");
      pattern.add("striped");
      pattern.add("open");
 
      ArrayList<String> shape = new ArrayList<String>(3);                   //ArrayList of Shapes
       shape.add("diamond");
       shape.add("squiggle");
       shape.add("oval");
 
      Card theCard = null;
         for(int i = 0; i < number.size(); i++)
         {
 
             for(int j = 0; j < color.size(); j++)
             {
 
                  for(int k = 0; k < pattern.size(); k++)
                  {
 
                        for(int m = 0; m < shape.size(); m++)
                        {
                             theCard = new Card(number.get(i),color.get(j),pattern.get(k),shape.get(m));
                             deckOfCards.add(theCard);    // adds the new card that is being created to the deck of cards array list
                      }
 
                }
            }                     
 
          }   
    }
 
     /**
      * To allow 12 cards from the deckOfCards ArrayList  to be choosen at random 
      * Then laid out in a grid of 3 rows with 4 cards each
      */
     public ArrayList<Card> createRandomNumberOfCards()
     {
         Random generator = new Random();                       //Creating the random class
         ArrayList <Integer> list = new ArrayList <Integer>();
         int randomIndex = 0;                //A random interger (range 1 to 12) so random class creates 12 random cards
         int x = 12;
         ArrayList <Card> randomCards = new ArrayList <Card>();         //Another ArrayList, when a card is pulled from the deckOfCard ArrayList, it is stored into the randomCards arrayList
         while(x>0) 
         {
             randomIndex = generator.nextInt(81);               //randomIndex is a random number ranging from 0 to 81 cards 
 
             while(list.contains(new Integer(randomIndex)))  //this is a nested list that make sure that there is more then one of the same random card being returned. 
             {
                randomIndex = generator.nextInt(81);               //If there is one of the same card being returned then it will simply make a new random card 
            }
            list.add(randomIndex);                                         // value at randomIndex is being added to the list of integers
            x--;
 
 
         }
              for(int y= 0; y < 12; y++)
             {
                                                                                                           //gets a number from the "list" ArrayList. Then deckOfCards
                 randomCards.add(deckOfCards.get(list.get(y)));  //uses that number to get the value from the deckOfcards ArraysList and places it into the random card 
 
             }
 
             return  randomCards;                                                       //This is the random card being returned 
      }
 
 
      /**
       * To create a grid that contains 12 cards
       * 3 rows and 4 columns
       */
 
      public void playGame()
      {
         ArrayList<Card> randomCards = createRandomNumberOfCards();
          for(int i= 0; i < randomCards.size(); i++)   //checking for every  card in the randomCard ArrayList
          {
 
 
              String cardDetails = randomCards.get(i).toString();    //using the toString of the Card method (which converts the details of the cards into a String 
              System.out.println(i + " " + cardDetails);   //prints out the cards (details of the cards) into the terminal window 
 
 
 
        }
 
          //These cards are set to a varabiable to 
          //make it easier for the user to input
          Card zero = null;
          Card one = null;
          Card two = null;
 
          Boolean correct = false;
          while (!correct){
              System.out.println("Pick your 1st card for the set  ");
                Scanner in = new Scanner(System.in);
                String setH = in.nextLine();
                int number = Integer.parseInt(setH);
                if (number >= 0 && number <= 11){
 
                    correct = true;
                    System.out.println(randomCards.get(number));
                    zero = randomCards.get(number);
 
                }
 
 
                else {
                    correct = false;
                    System.out.println("That is not a correct card please enter again");
                }
 
            }
 
 
 
           Boolean correcta = false;
          while (!correcta){
              System.out.println("Pick your second card for the set  ");
                Scanner in = new Scanner(System.in);
                String setJ = in.nextLine();
                int number = Integer.parseInt(setJ);
                if ( number >= 0 && number <= 11){
 
                    correcta = true;
                    System.out.println(randomCards.get(number));
                    one = randomCards.get(number);
                }
 
 
                else
                {
                    correct = false;
                    System.out.println("That is not a correct card please enter again");
                }
            }
 
             Boolean correctb = false;
          while (!correctb){
              System.out.println("Pick your third card for the set  ");
                Scanner in = new Scanner(System.in);
                String setV = in.nextLine();
                 int number = Integer.parseInt(setV);
                if ( number >= 0 && number <= 11){
 
                    correctb = true;
                    System.out.println(randomCards.get(number));
                    two = randomCards.get(number);
                }
 
                else
                {
                       correct = false;
                    System.out.println("That is not a correct card please enter again");
                }
 
 
 
            }
 
           //setV
           //setH
           //setJ
           int same = 0;
 
           if(zero.getNumber().equals(one.getNumber()) && one.getNumber().equals(two.getNumber()))
           {
               same ++;
            } 
 
           if(zero.getColor().equals(one.getColor()) && one.getColor().equals(two.getNumber()))
           {
               same ++;
            }
             if(zero.pattern().equals(one.pattern()) && one.pattern().equals(two.pattern()))
              {
                  same ++;
                }
             if(zero.shape().equals(one.shape()) && one.shape().equals(two.shape()))
               {
                   same ++;
                }
 
                if( same >= 3)
                {
                    System.out.println("This is a set you win");
                } 
 
            if(! zero.getNumber().equals(one.getNumber()) && ! one.getNumber().equals(two.getNumber()))
            {
                same ++;
            }
             if(! zero.getColor().equals(one.getColor()) && ! one.getColor().equals(two.getNumber()))
             {
                 same ++;
              }
 
              if(! zero.pattern().equals(one.pattern()) && ! one.pattern().equals(two.pattern()))
               {
                   same ++;
                }
               if(zero.shape().equals(one.shape()) && one.shape().equals(two.shape()))
                {
                    same ++;
                }
                if(same >= 3)
                {
                  System.out.println("This is a set you win");
              }
 
 
        }
 
 
 
}

Card Class

public class Card
{
    public String number;        
    public String color;   
    public String pattern;
    public String shape;
 
     /**
      * a deck of cards 
      * @param number of objects on cards
      * @param color of shape and pattern
      * @param what type of pattern
      * @param what type of shape
      */
    public Card(String number, String color, String pattern, String shape)
    {
        this.number = number;
        this.color = color;
        this. pattern = pattern;
        this.shape = shape;   
    }
 
    /**
     * number of objects
     * only 1-3
     */
    public String getNumber()
    {
 
        return number;
    }
 
    /**
     * returns color
     */
 
    public String getColor()
   {
        return color;
    }
 
    /**
     * returns pattern
     */
    public String pattern()
    {
 
        return pattern;
    }
 
    /**
     * returns shape
     */
    public String shape()
    {  
        return shape;
    }
 
    public String toString()
    {
 
      // return Card(String number, String color, String pattern, String shape);
         StringBuilder theOutput=new StringBuilder();
         theOutput.append(" number:");
         theOutput.append(number);
         theOutput.append(" color:");
         theOutput.append(color);
         theOutput.append(" pattern:");
         theOutput.append(pattern);
         theOutput.append(" shape:");
         theOutput.append(shape);
         theOutput.append(" "); 
       //  System.out.println(theOutput);
         return theOutput.toString();
 
    }
 
}