Ugly Enum Code. Please help make it more proper

[Francis Zabala]

Hello again guys!

I have a question on enums. First, the code:

enum ChowderChars {
	CHOWDER("Food!"), MUNG_DAAL("Feedle Deedle!"), SHNITZEL("Radda Radda!"), TRUFFLES("What's going on!"), GIZPACHIO("Mom!") 
	{
		public String SayIt(String origCP) {
			return origCP + " What the? ";
		}
		/*public String SayIt() {
			return catchphrase + " What the? ";
		}*/
		public String SayItRather() {
			return "Will it work?";
		}
	};
	
	ChowderChars(String s) {
		this.catchphrase = s;
	}
	
	private String catchphrase;
	
	public String SayIt() {
		return catchphrase;
	}
	
	public String SayIt(String s) {
		return catchphrase;
	}
	
	public String SayItRather() {
			return "I dunno";
		}

}

public class ChowderShow {
	ChowderChars star;
	
	public static void main(String args[]) {
		ChowderShow cs = new ChowderShow();
		cs.star = ChowderChars.CHOWDER;
		System.out.println(cs.star.SayIt());
		System.out.println(cs.star.SayItRather());
		cs.star = ChowderChars.GIZPACHIO;
		/*System.out.println(cs.star.SayIt(cs.star.SayIt()));*/
	}
}

I was trying to add "What the? " and "Mom" when GIZPACHIO is used so that the output of

		cs.star = ChowderChars.CHOWDER;
		System.out.println(cs.star.SayIt())

would be "Mom! What the?"

I thought that line 7 to line 9 would work but well..

ChowderShow.java:10: non-static variable catchphrase cannot be referenced from a static context
			return catchphrase + " What the? ";
			       ^
1 error

Is there a way to achieve this without doing the code /*System.out.println(cs.star.SayIt(cs.star.SayIt()));*/ ?

[Wouter Oet]

I'm not really sure why it isn't allowed but it can be fixed by using:

//...
GIZPACHIO("Mom!")

	{
        public String SayIt(String origCP) {
            return origCP + " What the? ";
        }

		@Override
        public String SayIt() {
            return GIZPACHIO.catchphrase + " What the? ";
        }
        public String SayItRather() {
            return "Will it work?";
        }
    };
//...

and instead of

ChowderShow cs = new ChowderShow();   
cs.star = ChowderChars.CHOWDER;   
System.out.println(cs.star.SayIt());

You can use

System.out.println(ChowderChars.CHOWDER.SayIt());

And finally methods should start with a lowercase letter.

[Francis Zabala]

Sorry for the late response and thanks Wouter Oet! Hahaha! Man, that was so simple!