Why am I getting an SPADE cannot be resolve to a type error?

Question

I recently decided to start using enum for my card Suit however it seem that I can't resolve this error no matter how I type up the code.

Edited:

/**
 * 
 * A basic constructor class that build the card object. 
 * The card object will contain value for both the Suit and 
 * card's Value.
 * 
 * 
 *
 */
public class Card {

    public Enum SUIT{SPADE, CLUB, DIAMOND, HEART};

    // This will identify the card suit.
    public Enum suit; 
    //This will hold the card value.
    //Jack = 11 ... Ace = 14
    public int value;


    public Card(Enum suit, int value){

        this.suit = suit;
        this.value = value;
    }
}

I have look up various answer on the site and look through the official Java Documentation on Orcacle. Can someone help me figure this out?

Update:

/**

* * A basic constructor class that build the card object. * The card object will contain value for both the Suit and * card's Value. * * * */ public class Card {

public enum Suit{SPADE, CLUB, DIAMOND, HEART};

// This will identify the card suit.
private final Suit suit; 
//This will hold the card value.
//Jack = 11 ... Ace = 14
private final int value;


public Card(Suit suit, int value){

    this.suit = suit;
    this.value = value;
}

}

Here a screenshot of the errors: A screenshot of the error

Answer 1

Enum is a class name (it is the base class of all enums). You need to declare the Suit class as:

public enum Suit{SPADE, CLUB, /* etc */}

Then, in order to accept an instance of Suit as a parameter to your constructor, make the parameter type Suit, as well as the corresponding field:

public Suit suit;
public int value;

public Card(Suit suit, int value) {
  this.suit = suit;
  this.value = value;
}

---

A couple of other observations:

• Make your suit and value fields private, and add accessor methods (getSuit(), getValue()). Currently, nothing stops somebody outside the class from changing them.
• Also, consider making them final, unless there is a good reason to want to be able to change them later inside this class, or you want to expose setters for them (I can't imagine what that reason would be)
• Check the range of value is between 2 and 14; alternatively, create an enum for the card values. It's a little bit of one-off typing, but it will lead to much more descriptive code (e.g. you can use Value.ACE instead of wondering if you should use 1 or 14 to represent that card)

Answer 2

If you want to define an enum type

// lowercase
public enum SUIT{SPADE, CLUB, DIAMOND, HEART};

If you want to have a variable SUIT

SUIT suit;

Answer 3

enum is the right keyword you are looking for.

UPDATE

To make your code work,change Suit(mostly a typo) to SUIT(enum declared by you)

public class Card {

    public enum SUIT{SPADE, CLUB, DIAMOND, HEART};

    // This will identify the card suit.
    private final SUIT suit; 
    //This will hold the card value.
    //Jack = 11 ... Ace = 14
    private final int value;


    public Card(SUIT suit, int value){

        this.suit = suit;
        this.value = value;
    }
}

Answer 4

Your latest edit shows a screenshot containing this code:

public class Card {
    public enum Suit {SPADE, CLUB, DIAMOND, HEART};

    public final SUIT suit;

Your enum declaration is fine. The fact that you are getting the error message "enum cannot be resolved to a type" suggests that your compilation settings are targetting an obsolete version of Java. Enums did not exist before Java 1.5.

Make sure you are using the latest Java compiler (1.8) and then go into Eclipse properties for your project. Click on Java Compiler. Under JDK Compliance, make sure it is set to 1.5 or later (preferably 1.8).

Your next error is the fact that SUIT != Suit. Be careful with the case of your letters.