views:

49

answers:

1

I am trying to make a program that works with poker (texas holdem) starting hands; each hand has a value from 1 to 169, and i want to be able to input each card and whether they are suited or not, and have those values correspond to a series of structs. Here is the code so far, i cant seem to get it to work (im a beginning programmer). oh and im using visual studio 2005 by the way

#include "stdafx.h"
#include <iostream>


int main()
{
    using namespace std;


    struct FirstCard
    {
        struct SecondCard
        {

            int s;  //suited
            int n;  //non-suited

        };

        SecondCard s14;
        SecondCard s13;
        SecondCard s12;
        SecondCard s11;
        SecondCard s10;
        SecondCard s9;
        SecondCard s8;
        SecondCard s7;
        SecondCard s6;
        SecondCard s5;
        SecondCard s4;
        SecondCard s3;
        SecondCard s2;


    };

    FirstCard s14; //ace
    FirstCard s13; //king
    FirstCard s12; //queen
    FirstCard s11; //jack
    FirstCard s10;
    FirstCard s9;
    FirstCard s8;
    FirstCard s7;
    FirstCard s6;
    FirstCard s5;
    FirstCard s4;
    FirstCard s3;
    FirstCard s2;


    s14.s14.n = 169; // these are the values that each combination
    s13.s13.n = 168; // will evaluate to, would eventually have 
    s12.s12.n = 167; // hand combinations all the way down to 1
    s11.s11.n = 166;
    s14.s13.s = 165;
    s14.s13.s = 164;
    s10.s10.n = 163; //10, 10, nonsuited
    s14.s13.n = 162;
    s14.s11.s = 161;
    s13.s12.s = 160;// king, queen, suited
    s9.s9.n = 159;
    s14.s10.s = 158;
    s14.s12.n = 157;
    s13.s11.s = 156;
    s8.s8.n = 155;
    s12.s11.s = 154;
    s13.s10.s = 153;
    s14.s9.s = 152;
    s14.s11.n = 151;

    cout << "enter first card: " << endl;

    cin >> somthing?//no idea what to put here, but this would somehow
            //read out the user input (a number from 2 to 14)
            //and assign it to the corresponding struct

    cout << firstcard.secondcard.suited_or_not << endl; //this would change depending
                                                            //on what the user inputs
    system("Pause");

} 
+1  A: 

First off, instead of learning all by trial and error, consider reading a good introductory book (list here). Every book worth reading address basic input in C++ and how to handle repetitive code by e.g. using containers.

That being said: I don't know anything about poker, but you definitely need to read about containers - if you repeatedly define variables, there has to be some way to abstract it.

Going from what you have, lets say we group the values in one struct:

struct Values {
    unsigned suited;
    unsigned unsuited;
    Values() : suited(0), unsuited(0) {} // some default
};

Then we can use containers that allows us to access them by their number:

typedef std::map<unsigned, Values> SecondCardMap;
typedef std::map<unsigned, SecondCardMap> CardMap;

With that we can insert and modify the structs conveniently via their numbers:

CardMap cards;
cards[14][14].unsuited = 169;
cards[13][13].unsuited = 168;

Now you'd only have to read in indices from the user:

unsigned first, second;
std::cout << "enter first: ";
std::cin >> first;
std::cout << "enter second: ";
std::cin >> second;

And could access the values according to that:

cards[first][second].unsuited = 1234;

Note that the above doesn't address any error handling etc.

Georg Fritzsche