Thread: Conditionals

I am sorry, I tried to read your code, but it's been very difficult. so I remade a cleaner working version of your program

Code:

#include "stdio.h"
#include "stdlib.h"
#include "time.h"

enum TYPE {
	DIAMOND = 0,
	HEARTS,
	SPADES,
	CLUBS
};

const int MAX_PICKS = 100;

typedef struct {
	int type;
	int number;
} Card;

int main(int argc, char* argv[])
{
	/* first, allocate a 52 set of cards
	 * and allocate the whole deck
	 */

	int pick[MAX_PICKS];	// will contain the index of the card in the deck
	Card deck[52];
	int index = 0;

	int red_cards = 0;
	int black_cards = 0;

	for ( int type = 0; type < 4; type++ ) {
		for ( int n = 0; n < 14; n++ ) {
			deck[index].number = n;
			deck[index].type = type;
			index++;
		}
	}

	/*
	 * now, pick randomly hundred cards
	 */

	/* in order to get different pick everytime we run the program */
	srand( (unsigned)time( NULL ) );

	for ( int i = 0; i < MAX_PICKS; i++ ) {
		// get random number between 0 and 51
		pick[i] = rand() * 52 / RAND_MAX;
		if ( pick[i] >= 52 ) pick[i] = 51;	// might happen

		// at the same time, count red/black cards
		if ( deck[pick[i]].type > HEARTS ) {
			black_cards++;
		} else {
			red_cards++;
		}
	}

	/* 
	 * ok, we're done picking the 100 cards
	 * now, lets print all the results
	 */

	printf("You've been delt %d red cards amd %d black cards\n", red_cards, black_cards);
	
	return 0;
}

Hope that's what you're looking for

because everyone knows there are only 52 playin card, D'UH lol just kiddin

ha ha very funny, but really, it is 52 cards, but you are picking these cards 100 times at random. And the code that someone printed out before did not work on my compiler (i use microsoft visual c++ 6.0) and it was very complicated (i'm not that much of a c expert). But thank you for taking the time to rewrite it. Its the for loop at the end, which prints out the total black and red cards, is what i am REALLY having a problem with - everything else works.

actually the code I posted before does work under VC++ 6.0 because that's what I used.

under Visual C++ 6.0, go to File->New ->Project ->Win 32 Console Application -> Empty project

create a file, cut-paste my code, compile and say yes to include the file in your project and execute it.

It works.

BUT I ran your code and looked at it: you make a mistake in your last loop:

Code:

for (kindOfCard=1; kindOfCard<=4; kindOfCard++) {
	if (kindOfCard == 1 || kindOfCard == 2) {
		totalBlack = cardSuit[kindOfCard] + cardSuit[kindOfCard];
		printf("You were dealt %d black cards ", totalBlack);
	}
	else if (kindOfCard == 3 || kindOfCard == 4) {
		totalRed = cardSuit[kindOfCard] + cardSuit[kindOfCard];
		printf("and %d red cards.\n ", totalRed);
	}
}

because kindOfCard is going from 1 to 4. So it's going to enter TWO times in the first 'if' statement and TWO times in the second.

this is what you should have instead of the loop:

Code:

printf("You were dealt %d black cards\n", cardSuit[1] + cardSuit[2]);
printf("You were dealt %d red cards\n", cardSuit[3] + cardSuit[4]);

why do complicated when you can do simple ?

xmdvp

OH MY GOD!!!!! Thank you soooo much - that was so simple, and it worked!!!!!!! I have one more question - how would i be able to find and print the card that was picked the most times?