Thread: Random suites for black jack game..

Ok I am trying to add card suits to my game. I completed the game simulation but just trying to add some more to it. currently the game will generate random cards ie(numbers) include k for king, Q for queen, j for jack, and A for ace. Now I want to add random suits to this. I created the function below but i keep getting. the following error message when I compile.


342 conflicting types for 'dealFace'
44 previous declaration of 'dealFace' was here

currently the program will print out all four suits accross the top like a logo. Now I would like to add suits to my random card numbers.

Also how do I add color to background. Like change it to a green, or have the screen flash when player wins. or some kinds of chime.

which ever is simplist I will try to use. I'm just a beginner so if it is complicated for me to follow right now. I will not use, but appreciate any suggestions.
Thanks.. Code below.

Code:

 const char HEART = (char)259;  
  const char DIAMOND = (char)260;
  const char SPADES = (char)262;
  const char CLUBS = (char)261;


 char dealFace( char face[] );
  char face [4] = { HEART, DIAMOND, SPADES, CLUBS};

 dealFace( char face[] ) char dealFace( char face[] );
{
  
    int value = 0;
    int random = 0; 

    random = rand() % 52 + 1;    //Random number generation
    
    while( face[random] == 0 ) {
      random = rand() % 52 + 1;
                 
    }
  
    value = face[random];
    face[random] = 0;
  
    return value; 
  
}

Originally Posted by Darius Dempsey

Code:

 char dealFace( char face[] );
 
 dealFace( char face[] ) char dealFace( char face[] );
{
// ...
}

You have a double line there, is that in your actual code?

All your global variables are overflowing.

face is an array of size 4, but you address it as if it where size 52.

Well I originally had it declard as Char since it is an actually symbol/charther I am trying to display. So should it be Char or int?

Either way, declare value as char and return char, or declare value as int and return int. In the code you are showing here you aren't using characters.

ok so is this how I should declare it? because i am getting syntax error of invalid type.

Code:

char dealFace( char face[] );
  char face [52] = {1 - 13 = HEART, 14 - 26 = DIAMOND, 27 - 40 = SPADES, 41 - 52 = CLUBS};

No that wont work, you need to initialize the entire array. Although you could use that approach, for practical reasons quazah's suggestions is probably better.

My comments so far is related to the code you have posted, not the best way to solve what you are trying to achieve.

Code:

suit = card / 13;

Will give you a value in the range of 0 - 3, that you then can use to index your array. suit needs to be an integer type, then truncation will take care of the rest.

Originally Posted by quzah

The difference between this and what I've already shown is that this group all of the 1s together, and all of the 2s together, etc., instead of grouping by suit.


Quzah.

Yes, it's a different organization of the deck. But it's one I've found convenient in code...
Nobody said your was was wrong. I just thought mine would make an interesting alternative.

Thanks guys I finally got this to work.. but believe or not, I still can't get characters to work in my program...although when I did them outside of the prog they did work.. I checked declarations type, I do have variable declared as char, and the scanf set up to get %c.. but wont work in program... anyways that was a different post.. just saying you would think get Chars would be easier than setting up arrays and functions...

Originally Posted by quzah

I was mainly explaining the difference for the OP, not saying either way was better. They're both better than maintaining a struct with the suit and face value in my opinion.

Agreed. Some things just don't lend themselves well to structs... others do.