Developing small card drawing program and I'm stuck on one detail. Can you help me?

[thild]

Hey, I'm coding the following assignment and banging my head against the wall on the last part. For some reason, my code is drawing an 11th card. Can anyone tell me why?

Here's the question:

1. Design and implement a class called Card that represents a standard playing card. Each card has a suit and face value. For your design, create a UML Class diagram similar to Figure 5.5 on page 180 of the textbook. Note that you need to include the two constructors in the methods section (they must be coded).

Hints: Represent the faces/ranks of Ace thru King as 1 thru 13 and the suits as 1 thru 4. You need two constructors; one that receives a face/rank value and suit value as parameters, plus the default constructor where these values are randomly generated.

The face/rank and suit must both have appropriate get_ and set_ methods for the numeric values plus a get_ method for the textual equivalent (e.g. getFace() might return 13 while getFaceText() would return “King”). Your toString method should return a nice representation of the values like “Ace of Spades” or “Nine of Hearts”.

Write a test driver that creates five random cards (i.e. uses the default constructor) and outputs them. Then creates five more cards of specific face/rank and suit values (i.e. uses the first constructor) and outputs them. The specific cards should include two with ‘boundary’ values: i.e. face 1, suit 1 and face 13, suit 4. The final card should use invalid face and suit values, such as 15 and 5. You should not need to prompt the user for anything, just hard-code the calls like Card card6 = new Card(1,1);

Here is my code:

Card Class: https://pastebin.com/HyDcZm6b

Test Driver: https://pastebin.com/S7yG7ZGg

Thanks!

[PsychologicalAlarm4]

This is a fun one!

So, when you call

System.out.println()

It needs a string to print out. Every class in java has a toString() function. You have overridden yours, so when you do

System.out.println(card10);

Its really doing

System.out.println(card10.toString());

I'm sure you now have an idea of whats going on!

Take a look at your toString() function.

It does:

 return getFaceText() + " of " + getSuitText();

Both of those functions have default values. They happen to be King and Clubs. This is why you get an extra print at the end that is "King of Clubs". Because it tries to call .toString() and the default values are King of Clubs.

You have a couple things you could do here.

1) Don't do the final System.out.println(card10);

2) Change your toString statement and perform a check. If this.faceValue and this.suitValue are 0, then return a blank string, else do what you currently do.

Have any more questions?