Ошибки в покерной игре: 'players' was not declared in this scope

@Anton-KRaV · Регистрация: 28.07.2013

Студворк — интернет-сервис помощи студентам

Добрый день.
Наверно вы меня уже тихонько ненавидите, но я все же снова напишу сюда

Я новичок - самоучка, и от этого у меня много пробелов в знаниях.
Решил попробовать написать покерную игру, но получаю ошибку за ошибкой

. В данный момент не могу понять, почему компилятор ругается, подскажите пожалуйста.
Мой код:

Кликните здесь для просмотра всего текста

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
#include <iostream>
#include <string>
#include <vector>
#include <iterator>
#include <algorithm>
#include <random>
#include <iomanip>
 
using namespace std;
 
random_device rd;
mt19937 gen(rd());
 
/*----------------------------------------------------------------------------*/
 
class Card
{
private:
    short numb;
    short suit;
public:
    Card(short Numb, short Suit)
    {
        numb = Numb;
        suit = Suit;
    }
    short getNumb() const
        { return numb; }
    short getSuit() const
        { return suit; }
    friend ostream& operator << (ostream&, Card&);
};
ostream& operator << (ostream& out, Card& card)
{
    out << card.getNumb() << card.getSuit();
    return out;
}
 
/*----------------------------------------------------------------------------*/
 
class Player
{
private:
    string username;
    short numb;
    short position;
    vector<Card> hand;
    double bankroll;
    bool readyToPlay;
public:
    Player(short Numb)
    {
        numb = Numb;
        bankroll = 100.0; //DBG
    }
    Player(short Numb, string Username)
    {
        numb = Numb;
        username = Username;
        bankroll = 100.0; //DBG
    }
    void setUsername(string Username)
        { username = Username; }
    void setNumb(short Numb)
        { numb = Numb; }
    void setPosition(short Position)
        { position = Position; }
    void setHand(vector<Card> Hand)
        { hand = Hand; }
    void setBankroll(double Bankroll)
        { bankroll = Bankroll; }
    void setReadyToPlay(bool ReadyToPlay)
        { readyToPlay = ReadyToPlay; }
    void setCardToPlayersHand(Card card)
        { hand.push_back(card); }
    string getUsername() const
        { return username; }
    short getNumb() const
        { return numb; }
    short getPosition() const
        { return position; }
    vector<Card> getHand() const
        { return hand; }
    double getBankroll() const
        { return bankroll; }
    bool getReadyToPlay() const
        { return readyToPlay; }
};
 
/*----------------------------------------------------------------------------*/
 
class cardDeck
{
private:
    vector<Card> deck;
public:
    cardDeck()
    {
        for(short i = 1; i <= 4; i++)
        {
            for(short j = 2; j <= 13 + 1; j++)
            {
                deck.push_back(Card(j, i));
            }
        }
    }
    vector<Card> getCardDeck()
        { return deck; }
    vector<Card>& shuffle()
    {
        std::shuffle(deck.begin(), deck.end(), gen);
        for(int i = 0; i < 52; i++)
            cout << deck[i] << ' ';
        return deck;
    }
};
 
/*----------------------------------------------------------------------------*/
 
class Table
{
private:
    vector<Player> players;
    static int tableId;
    bool emptySeats[9];
    short numberOfEmptySeats;
public:
    Table()
    {
        cout << "Create Table\n"; //DBG
        for(short i = 0; i < 9; i++)
        {
            emptySeats[i] = true;
        }
        // more
    }
    /* DEBUG BEGINS */
        ~Table()
        {
            cout << "Delete Table\n";
        }
    /* DEBUG ENDS */
    friend class Hand; /////////////////////////////////hmmmm...
    void countNumberOfEmptySeats()
    {
        short numb = 0;
        for(short i = 0; i < 9; i++)
            if(emptySeats[i] == true)
                numb++;
        numberOfEmptySeats = numb;
    }
    short getNumberOfEmptySeats()
        { return numberOfEmptySeats; }
    vector<Player>& getPlayers()
        { return players; }
};
 
/*----------------------------------------------------------------------------*/
 
class Hand
{
private:
    static int handId;
public:
    Hand()
    {
        cout << "Create Hand\n"; // DBG
        cardDeck deck;
        deck.shuffle();
    }
    /* DEBUG BEGINS */
        ~Hand()
        {
            cout << "Delete Hand\n";
        }
    /* DEBUG ENDS */
    void dealToPlayers(short count)
    {
        while(count > 0)
        {
            for(vector<Player> :: iterator it = players.begin(); it < players.end(); it++)
            {
                (*it).setCardToPlayersHand(deck.front());
                deck.erase(deck.begin());
            }
            count--;
        }
    }
};
 
/*----------------------------------------------------------------------------*/
 
int main()
{
    cout << "Main starts\n"; //DBG
    Table t;
    
//  do
//  {
        t.countNumberOfEmptySeats();
//      if(t.getNumberOfEmptySeats() < 8)
        {
            Hand h;
        }
//  }while(true)
    cout << "Main ends\n"; //DBG
    return 0;
}

Вот вывод компилятора:

Кликните здесь для просмотра всего текста

Code
1
2
3
4
5
6
7
hello_world.cpp: In member function 'void Hand::dealToPlayers(short int)':
hello_world.cpp:181:40: error: 'players' was not declared in this scope
    for(vector<Player> :: iterator it = players.begin(); it < players.end(); it++)
                                        ^
hello_world.cpp:183:44: error: 'deck' was not declared in this scope
                 (*it).setCardToPlayersHand(deck.front());
                                            ^

Почему он не хочет дружить - ума не приложу... Буду благодарен любым замечаниям!

P.S Если у кого то будет время бегло пробежаться по коду - буду премного благодарен замечаниям по общей архитектуре приложения, некорректности код и т. п..

Заранее благодарю!!!

@nefton · 25.05.2015, 08:46

hand_evaluator.h

Кликните здесь для просмотра всего текста

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
// Convert hand value to unsined long number (32 bits)
//          4th byte                 3rd byte                    2nd byte                  1st byte
//   [7][6][5][4][3][2][1][0]  [7][6][5][4][3][2][1][0]  [7][6][5][4][3][2][1][0]  [7][6][5][4][3][2][1][0]
//    |  |  |  |  |  |  |  |    |  |  |  |  |  |  |  |    |  |  |  |  |  |  |  |    |  |  |  |  |  |  |  |
//                                           --- STRAIGHT FLUSH ---
//   [0][0][0][0][0][0][0][0]  [0][0][0][0][1][0][0][1]  [0][0][0][0][0][0][0][0]  [0][0][0][0][*][*][*][*]
//                                         [ hand type]           Hi card in street(0-12)----->[          ]
//                                           --- FOUR OF A KIND ---
//   [0][0][0][0][0][0][0][0]  [0][0][0][0][1][0][0][0]  [0][0][0][0][*][*][*][*]  [0][0][0][0][*][*][*][*]
//                                         [ hand type]              [   four   ]              [          ]<---Hi card from the rest of cards
//                                             --- FULL HOUSE ---
//   [0][0][0][0][0][0][0][0]  [0][0][0][0][0][1][1][1]  [0][0][0][0][*][*][*][*]  [0][0][0][0][*][*][*][*]
//                                         [ hand type]              [   trips  ]              [   pair   ]
//                                                --- FLUSH ---
//   [0][0][0][0][0][0][0][0]  [0][0][0][0][0][1][1][0]  [0][0][0][*][*][*][*][*][*][*][*][*][*][*][*][*]
//                                         [ hand type]           [         Fife hi cards in flush        ]
//                                              --- STRAIGHT ---
//   [0][0][0][0][0][0][0][0]  [0][0][0][0][0][1][0][1]  [0][0][0][0][0][0][0][0]  [0][0][0][0][*][*][*][*]
//                                         [ hand type]           Hi card in street(0-12)----->[          ]
//                                           --- THREE OF A KIND ---
//   [0][0][0][0][0][0][0][0]  [0][0][0][0][0][1][0][0]  [0][0][0][0][*][*][*][*][*][*][*][*][*][*][*][*]
//                                         [ hand type]              [  trips   ]  [    1st   ][   2nd    ]<---Values of two hi cards         ]
//                                             --- TWO PAIRS ---
//   [0][0][0][0][0][0][0][0]  [0][0][0][0][0][0][1][1]  [0][0][0][0][*][*][*][*][*][*][*][*][*][*][*][*]
//                                         [ hand type]              [ Hi pair  ]  [ Low pair ][ hi card  ]
//                                                --- PAIR ---
//   [0][0][0][0][0][0][0][0]  [0][0][0][0][0][0][1][0][*][*][*][*][*][*][*][*][*][*][*][*][*][*][*][*]
//                                         [ hand type]  [   pair   ][          ]  [          ][          ]<---three hi cards
//                                              --- HI CARD ---
//   [0][0][0][0][0][0][0][0]  [0][0][0][0][0][0][0][1]  [0][0][0][*][*][*][*][*][*][*][*][*][*][*][*][*]
//                                         [ hand type]           [          five hi cards                ]
//
//
//
 
 
 
#include <string>
#include <stdint.h>
 
#include "cards.h"
 
 
#ifndef HOLDEM_HAND_EVALUATOR_H
#define HOLDEM_HAND_EVALUATOR_H
 
 
typedef uint32_t hand_value_t;
 
 
using namespace std;
 
 
hand_value_t GetHand(card_t* cards); //7 cards array needed
hand_value_t GetHand(card_t card0, card_t card1, card_t card2, card_t card3, card_t card4, card_t card5, card_t card6);
 
string HandValueToString(hand_value_t hand);
 
 
 
uint8_t IsStreet(uint32_t cards);
card_t GetHiCard(uint32_t cards);
card_t GetAndRemoveHiCard(uint32_t* cards);
 
 
 
 
#endif // HOLDEM_HAND_EVALUATOR_H

hand_evaluator.cpp

Кликните здесь для просмотра всего текста

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
#include "holdem_hand_evaluator.h"
#include <iostream>
#include <bitset>
 
hand_value_t GetHand(card_t* cards)
{
    uint32_t answer=0;
    uint32_t temp1=0;
    uint32_t temp2=0;
 
    uint32_t spades=0;
    uint32_t clubs=0;
    uint32_t diamonds=0;
    uint32_t hearts=0;
 
    for (int i=0; i<7; i++)
    {
    if(cards[i]>>4==0)spades|=(1<<((cards[i]&0xF)-1));
    if(cards[i]>>4==1)clubs|=(1<<((cards[i]&0xF)-1));
    if(cards[i]>>4==2)diamonds|=(1<<((cards[i]&0xF)-1));
    if(cards[i]>>4==3)hearts|=(1<<((cards[i]&0xF)-1));
    }
 
//    cout<<endl<<endl<<"      AKQJT98765432"<<endl<<"s: "<<bitset<16>(spades)<<endl<<"c: "<<bitset<16>(clubs)<<endl<<"d: "<<bitset<16>(diamonds)<<endl<<"h: "<<bitset<16>(hearts)<<endl<<endl;
 
    //  -----------------------  STRAIGHT FLASH  ------------------------------------------
    temp1=IsStreet(spades);
    if (temp1){answer=(9<<16)|temp1; return answer;};
    temp1=IsStreet(clubs);
    if (temp1){answer=(9<<16)|temp1; return answer;};
    temp1=IsStreet(diamonds);
    if (temp1){answer=(9<<16)|temp1; return answer;};
    temp1=IsStreet(hearts);
    if (temp1){answer=(9<<16)|temp1; return answer;};
 
    //  -----------------------  FOUR OF A KIND  ------------------------------------------
    temp1=spades&clubs&diamonds&hearts;
    if(temp1){
      temp2 = spades|clubs|diamonds|hearts;
      temp2&=~temp1;//гасим нашу 4ку
      answer = (8<<16)|(temp1<<8)|GetHiCard(temp2);
      return answer;
    }
 
    uint32_t trips=0;
    trips = (spades&clubs&diamonds)|(spades&clubs&hearts)|(spades&diamonds&hearts)|(clubs&diamonds&hearts);
    uint32_t pairs=0;
    pairs = (spades&clubs)|(spades&diamonds)|(spades&hearts)|(clubs&diamonds)|(clubs&hearts)|(diamonds&hearts);
    //  -----------------------  FULL HOUSE  -------------------------------------------
 
    if (trips){
      //находим старшую тройку
      uint8_t t;
      t = GetHiCard(trips);
      //ишем двойки
      pairs&=~(1<<t);//гасим в двойках старшую тройку (нестаршие тройки остаются)
      if (pairs){
        answer = (7<<16)|(t<<8)|GetHiCard(pairs);
        return answer;
      }
    }
    //  -----------------------  FLUSH  -------------------------------------------------
    uint8_t s=0,c=0,d=0,h=0;
    for(int i=0;i<=12;i++){
      if(spades&(1<<i))s++;
      if(clubs&(1<<i))c++;
      if(diamonds&(1<<i))d++;
      if(hearts&(1<<i))h++;
    }
 
    if(s>=5){
        answer = (6<<16);
        uint8_t t;
        t = GetAndRemoveHiCard(&spades);
        answer|=(1<<t);
        t = GetAndRemoveHiCard(&spades);
        answer|=(1<<t);
        t = GetAndRemoveHiCard(&spades);
        answer|=(1<<t);
        t = GetAndRemoveHiCard(&spades);
        answer|=(1<<t);
        t = GetAndRemoveHiCard(&spades);
        answer|=(1<<t);
        return answer;
    }
    if(c>=5){
        answer = (6<<16);
        uint8_t t;
        t = GetAndRemoveHiCard(&clubs);
        answer|=(1<<t);
        t = GetAndRemoveHiCard(&clubs);
        answer|=(1<<t);
        t = GetAndRemoveHiCard(&clubs);
        answer|=(1<<t);
        t = GetAndRemoveHiCard(&clubs);
        answer|=(1<<t);
        t = GetAndRemoveHiCard(&clubs);
        answer|=(1<<t);
        return answer;
    }
    if(d>=5){
        answer = (6<<16);
        uint8_t t;
        t = GetAndRemoveHiCard(&diamonds);
        answer|=(1<<t);
        t = GetAndRemoveHiCard(&diamonds);
        answer|=(1<<t);
        t = GetAndRemoveHiCard(&diamonds);
        answer|=(1<<t);
        t = GetAndRemoveHiCard(&diamonds);
        answer|=(1<<t);
        t = GetAndRemoveHiCard(&diamonds);
        answer|=(1<<t);
        return answer;
    }
    if(h>=5){
        answer = (6<<16);
        uint8_t t;
        t = GetAndRemoveHiCard(&hearts);
        answer|=(1<<t);
        t = GetAndRemoveHiCard(&hearts);
        answer|=(1<<t);
        t = GetAndRemoveHiCard(&hearts);
        answer|=(1<<t);
        t = GetAndRemoveHiCard(&hearts);
        answer|=(1<<t);
        t = GetAndRemoveHiCard(&hearts);
        answer|=(1<<t);
        return answer;
    }
 
    //  --------------------------  STRAIGHT  -----------------------------------------
    temp1 = IsStreet(spades|clubs|diamonds|hearts);
    if(temp1){answer=(5<<16)|temp1; return answer;}
 
    //  --------------------------  THREE OF A KIND  ----------------------------------
    if (trips){//если мы дошли сюда, то у нас может быть только 1 тройка
      uint8_t t;
      t = GetHiCard(trips);
      answer = (4<<16)|(t<<8);
      uint32_t all = spades|clubs|diamonds|hearts;
      all&=~trips;//гасим тройку
      answer |= (GetAndRemoveHiCard(&all)<<4);
      answer |= GetAndRemoveHiCard(&all);
      return answer;
    }
 
 
    //  ---------------------------  PAIR AND TWO PAIRS  -----------------------------------------
    if (pairs){
 
      uint32_t all = spades|clubs|diamonds|hearts;
      //находим старшую двойку
      uint8_t hi_pair = GetAndRemoveHiCard(&pairs);
      if (pairs){//если есть другие двойки кроме найденной
        uint8_t low_pair = GetAndRemoveHiCard(&pairs);
        all&=~(1<<hi_pair);//гасим старшую двойку
        all&=~(1<<low_pair);//гасим младшую двойку
        answer = (3<<16)|(hi_pair<<8)|(low_pair<<4)|GetAndRemoveHiCard(&all);
        return answer;
      }
      answer = (2<<16)|(hi_pair<<12);
      all&=~(1<<hi_pair);//гасим двойку
      answer |= (GetAndRemoveHiCard(&all)<<8);
      answer |= (GetAndRemoveHiCard(&all)<<4);
      answer |= GetAndRemoveHiCard(&all);
      return answer;
    }
 
    //  ------------------------------  HI CARD  -----------------------------
    uint32_t all = spades|clubs|diamonds|hearts;
    answer = (1<<16);
    answer|=(1<<GetAndRemoveHiCard(&all));
    answer|=(1<<GetAndRemoveHiCard(&all));
    answer|=(1<<GetAndRemoveHiCard(&all));
    answer|=(1<<GetAndRemoveHiCard(&all));
    answer|=(1<<GetAndRemoveHiCard(&all));
    return answer;
}
// ///////////////////////////////////////////////////////////////////////////////////////////////////////////////
 
uint8_t IsStreet(uint32_t cards)
{
 
    cards&=0x1FFF;
    if ((cards&0x1F00)==0x1F00)return 13;
    if ((cards&0xF80)==0xF80)return 12;
    if ((cards&0x7C0)==0x7C0)return 11;
    if ((cards&0x3E0)==0x3E0)return 10;
    if ((cards&0x1F0)==0x1F0)return 9;
    if ((cards&0xF8)==0xF8)return 8;
    if ((cards&0x7C)==0x7C)return 7;
    if ((cards&0x3E)==0x3E)return 6;
    if ((cards&0x1F)==0x1F)return 5;
    if ((cards&0x100F)==0x100F)return 4;
    return 0;
}
// ////////////////////////////////////////////////////////////////////////////////////////////////////////////////
uint8_t GetHiCard(uint32_t cards){
  uint8_t t=12;
  while((cards&(1<<t))==0){t--;}//на выходе t = старшая карта (от 12 до 0)
  return t;
}
// ///////////////////////////////////////////////////////////////////////////////////////////////////////////////
uint8_t GetAndRemoveHiCard(uint32_t* cards){
    if (*cards == 0) return 0;
    uint8_t t=12;
  while((*cards&(1<<t))==0){t--;}//на выходе t = старшая карта (от 12 до 0)
  *cards&=~(1<<t);//гасим бит старшей карты в cards
  return t;
}
// //////////////////////////////////////////////////////////////////////////////////////////////////////////////
string HandValueToString(hand_value_t hand)
{
    string hand_type = "error";
    if (hand == 0) return "Null hand";
    if ((hand&0xF0000) == (9<<16)) hand_type = "Straight flush";
    if ((hand&0xF0000) == (8<<16)) hand_type = "Four of a kind";
    if ((hand&0xF0000) == (7<<16)) hand_type = "Full house";
    if ((hand&0xF0000) == (6<<16)) hand_type = "Flush";
    if ((hand&0xF0000) == (5<<16)) hand_type = "Straight";
    if ((hand&0xF0000) == (4<<16)) hand_type = "Three of a kind";
    if ((hand&0xF0000) == (3<<16)) hand_type = "Two pairs";
    if ((hand&0xF0000) == (2<<16)) hand_type = "Pair";
    if ((hand&0xF0000) == (1<<16)) hand_type = "Hi card";
    return hand_type;
}
 
hand_value_t GetHand(card_t card0, card_t card1, card_t card2, card_t card3, card_t card4, card_t card5, card_t card6){
    card_t Cards[7];
    Cards[0] = card0;
    Cards[1] = card1;
    Cards[2] = card2;
    Cards[3] = card3;
    Cards[4] = card4;
    Cards[5] = card5;
    Cards[6] = card6;
    return GetHand(Cards);
}

Дальше в развитии идёт фсмка (finite state machine) покера. Которая инкапсулирует в себя все покерные правила (включая неполный райз, множественые боковые банки (side pots) и деление фишек чётко с правилами до цента.
Её я планирую выложить в паблик чуть позже.

Так вот всё это - полная ерунда в сравнении с тем этапом когда playerу понадобится член ballance

@Nosey · 25.05.2015, 13:10

nefton, Если вы любите быструю езду, это не означает что нельзя использовать классы.
С++ для это и был создан чтобы получить максимальную производительность не переходя на прорву мэджикнамберов и не расширяемость.

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
#include <vector>
#include <cstring>
 
enum Numb
{
    Two = 0x1, Three = 0x2, Four = 0x3, Five = 0x4, Six = 0x5, Seven = 0x6
};
 
enum Suit
{
    Spades = 0x0, Clubs = 0x1, Diamonds = 0x2, Hearts = 0x3
};
 
class Card
{
    uint8_t cardValue_;
public:
    Card(Numb numb, Suit suit) :
            cardValue_((unsigned char) suit << 4 | (unsigned char) numb)
    {
    }
 
    Numb numb() const
    {
        return Numb(cardValue_ & 0xF);
    }
 
    void numb(const Numb& val)
    {
        cardValue_ = (cardValue_ & 0xF0) | (unsigned char) val;
    }
 
    Suit suit() const
    {
        return Suit((cardValue_ & 0x30) >> 4);
    }
 
    void suit(const Suit& val)
    {
        cardValue_ = (cardValue_ & 0xF) | (unsigned char) val << 4;
    }
};
 
template<Numb N, Suit S>
struct StaticCard
{
private:
    static const uint8_t cardValue = (unsigned char) S << 4 | (unsigned char) N;
public:
 
    constexpr unsigned char data() const
    {
        return cardValue;
    }
 
    constexpr Numb numb() const
    {
        return N;
    }
 
    constexpr Suit suit() const
    {
        return S;
    }
};
 
template<class T, class ... Types>
class StaticHand: StaticHand<Types...>
{
    static const unsigned char data_ = (unsigned char) T::cardValue;
public:
    constexpr long int data() const
    {
        return (*(long int*) this);
    }
 
    constexpr long int numbData() const
    {
        return ((*(long int*) this) & 0xF0F0F0F0F0F0F0F);
    }
 
    constexpr long int suitData() const
    {
        return ((*(long int*) this) & 0xF0F0F0F0F0F0F0F0);
    }
};
 
template<class T>
struct StaticHand<T>
{
    static const unsigned char data = (unsigned char) (T::S) << 4 | (unsigned char) (T::N);
};
 
class Hand
{
public:
    Hand(const std::initializer_list<Card>& value) :
            cards_(value)
    {
    }
 
    std::vector<Card> cards_;
 
    inline long int data() const
    {
        long int ret = 0;
        memcpy(&ret, cards_.data(), cards_.size());
        return ret;
    }
 
    inline long int numbData() const
    {
        return data() & 0xF0F0F0F0F0F0F0F;
    }
 
    inline long int suitData() const
    {
        return data() & 0xF0F0F0F0F0F0F0F0;
    }
};
 
bool IsStreet(const Hand& hand)
{
    StaticHand<StaticCard<Numb::Two, Suit::Clubs>, StaticCard<Numb::Three, Suit::Clubs>, StaticCard<Numb::Four, Suit::Clubs>,
            StaticCard<Numb::Five, Suit::Clubs>, StaticCard<Numb::Six, Suit::Clubs>, StaticCard<Numb::Seven, Suit::Clubs>> h1;
 
    if ((hand.numbData() & h1.numbData()) == h1.numbData())
        return true;
    return false;
}
 
int main(int argC, char* argV[])
{
    Hand hand { { Numb::Three, Suit::Diamonds }, { Numb::Three, Suit::Diamonds }, { Numb::Three, Suit::Diamonds }, { Numb::Three,
            Suit::Diamonds }, { Numb::Three, Suit::Diamonds }, { Numb::Three, Suit::Diamonds } };
    Hand hand2 { { Numb::Two, Suit::Diamonds }, { Numb::Three, Suit::Hearts }, { Numb::Four, Suit::Clubs }, { Numb::Five,
            Suit::Clubs }, { Numb::Six, Suit::Spades }, { Numb::Seven, Suit::Clubs } };
    IsStreet(hand);
    IsStreet(hand2);
    std::cout << IsStreet(hand) << std::endl;
    std::cout << IsStreet(hand2) << std::endl;
}

@nefton · 25.05.2015, 15:40

Сообщение от Nosey

enum Numb
{
* * Two = 0x1, Three = 0x2, Four = 0x3, Five = 0x4, Six = 0x5, Seven = 0x6
};

C++
1
enum class {Two, Three, Four, Five, Six, Seven}

но к чему это???

Сообщение от Nosey

enum Suit
{
* * Spades = 0x0, Clubs = 0x1, Diamonds = 0x2, Hearts = 0x3
};

аналогично

Сообщение от Nosey

Suit suit() const
* * {
* * * * return Suit((cardValue_ & 0x30) >> 4);
* * }

А вот и пошли магические номера. Парадигма enum class подразумевает отказ от номеров (даже порядковых)
надо:

C++
1
2
3
4
5
6
[CPP]
Suit suit() const
    {
         if (((cardValue_ & 0x30) >> 4) == 0) return Suit::spades;
         ......
     }

[/CPP]

Замечу привязка к битовой карте (которая должна быть гдето описанна по русски отдельно) - осталась
и все магические номера соответственно

Сообщение от Nosey

struct StaticCard
{
private:
* * static const uint8_t cardValue = (unsigned char) S << 4 | (unsigned char) N;
public:
constexpr unsigned char data() const
* * {
* * * * return cardValue;
* * }
constexpr Numb numb() const
* * {
* * * * return N;
* * }
constexpr Suit suit() const
* * {
* * * * return S;
* * }
};

Это настолько выше моего понимания что я даже представить не могу что это.

А теперь самое интересное чем теоретики отличаются от практиков )))
Hand - рука - вы вообще представляете что это и как с этим работать?
она состоит из 5ти карт (но выбирается из 7ми)
и самое главное её отношение с другими руками. она их либо "бьёт" либо не бьёт либо они одинаковые
где перегрузка операторов >,<,>=,<=,==,!= ???
и главное на что интересно будет похож этот код???? )

Сообщение от Nosey

bool IsStreet(const Hand& hand)
{
* * StaticHand<StaticCard<Numb::Two, Suit::Clubs>, StaticCard<Numb::Three, Suit::Clubs>, StaticCard<Numb::Four, Suit::Clubs>,
* * * * * * StaticCard<Numb::Five, Suit::Clubs>, StaticCard<Numb::Six, Suit::Clubs>, StaticCard<Numb::Seven, Suit::Clubs>> h1;
if ((hand.numbData() & h1.numbData()) == h1.numbData())
* * * * return true;
* * return false;
}

Стрит в покере это не просто стрит. стрит 789TJ бьёт 6789T
А ещё бывает стрит A2345, А вот стрита KA234 не бывает
Да и вообще функция изстрит - просто вспомогательная.
Реально надо только 1 функция перевода 7 карт в руку (4х байтное число) которое можно >,<,>=,<=,==,!=

И самое главное. Вы действительно думаете что карта в виде класса удобнее?
Неужели есть в жизни хоть 1 ситуация где функции string CardToString(card_t card) мало????

Мало того. Мне вообще за годы покер писательства ниразу не приходилось заглядывать внутрь этих файлов.

@Nosey · 25.05.2015, 16:58

Сообщение от nefton

но к чему это??? ... аналогично ...

Я просто скопипастил ваше сжатие.
Я не занимался картами столько лет, точнее я вообще ими не занимался и о покере ничего не знаю

Сообщение от nefton

А вот и пошли магические номера. Парадигма enum class подразумевает отказ от номеров (даже порядковых)
надо:

C++
1
2
3
4
5
Suit suit() const
 {
 if (((cardValue_ & 0x30) >> 4) == 0) return Suit::spades;
 ......
 }

Я позволю сделать это компилятору вместо меня

Сообщение от nefton

Замечу привязка к битовой карте (которая должна быть гдето описанна по русски отдельно) - осталась
и все магические номера соответственно

Битовая карта конечно осталась, это правда отличная идея сжимать данные, если ими остаётся возможность пользоваться.
И собственно я написал лишь компайл тайм сжатие любой руки и не более. ( и я сжал в long, ибо не вкурил быстро ваш вариант)
И хотел показать что всё можно инкапсулировать и получить хорушую производительность на вашу фразу:

Сообщение от nefton

класс card - не нужен. Как ты потом будешь руки сравнивать?
ты если и напишешь эту функцию - то удалишь её сразу из за её убогости и тормознутости.
во всём мире давно руки сравнивают с помощью битовых операций.

Я не сомневаюсь что ваш код работает и достаточно хорошо, и вы его знаете и т.д., но для меня этот код - совершенно не понятен при первом просмотре, ибо там идёт "жесткая математка" без коммментариев и абстракций которые можно добавить ни в грамм ни ухудшив производительнось. Вам это не нужно, за годы вы всё точно там знаете и всё обдумали, но в тру ООП и бла бла бла, оно лишним никогда не будет, и человек тут учится не покеру а С++

.

Сообщение от nefton

Неужели есть в жизни хоть 1 ситуация где функции string CardToString(card_t card) мало????

Это просто не ООПшный подход, поэтому мне не нравится

. Памяти тут экономится несказанно мало, процессор нужно смотреть насколько часто нужно вам строковое представление, предполагаю что вообще никогда

, поэтому как-то неважен этот момент, но не тру ООП

(если у меня добавить такую штуку, то тоже сжатие изменится)

Сообщение от nefton

Реально надо только 1 функция перевода 7 карт в руку (4х байтное число) которое можно >,<,>=,<=,==,!=

Я играю в покер только пьяным, так что не знаю точно всего этого, сорри

Я лишь хотел показать что от ничего не говорящих конструкций:

C++
1
if ((cards&0x1F00)==0x1F00)return 13;

можно отказаться не ухудшив производительность, просто заменив калькулятор на компилятор

И я в примере немного загнался, в голове прочно сидел массив карт, поэтому появились constexpr, можно даже легче и без них сжать.

@nefton · 25.05.2015, 17:46

Перестаньте писать классы

И рука (7 карт) компонуются в число не для экономии памяти или чегото там, а для того чтоб сравнить руки у 5ти человек к примеру кто победил.
Кроме моего подсчёта силы руки есть ещё с десяток вариантов, написанных независимо в разное время в разных частях света. И все они битовые

Ну есть конечно не битовые - только все что мне встречались - с ошибками. Тоесть не считают правильно все руки

Если бы использование классов реально привело б к хоть какойто удобочитаемости.... но это не тот случай. Тут не будет понятнее с классами. А вот скорость тут очень надо. В самом примитивном калькуляторе эта функция вызывается 100 000 раз при каждом подсчёте вероятностей.

@Nosey · 25.05.2015, 18:34

Сообщение от nefton

Перестаньте писать классы

Иногда когда пишу на яве, я тоже хочу такое сказать )), но в большей степени я за "пишите классы, да не скромничайте", имхо, которое таки доказано немалыми проектами, но имхо, я не хочу холивара

Сообщение от nefton

Ну есть конечно не битовые - только все что мне встречались - с ошибками. Тоесть не считают правильно все руки
Если бы использование классов реально привело б к хоть какойто удобочитаемости.... но это не тот случай. Тут не будет понятнее с классами. А вот скорость тут очень надо. В самом примитивном калькуляторе эта функция вызывается 100 000 раз при каждом подсчёте вероятностей.

Я понимаю что это для производительности, поэтому и сохранил вашу идею, которая после компиляция превратится в вашу реализацию.
И удобочитаемость, это для вас удобно, вы много разных реализаций повидали и понимаете всю математику, а я только вашу, и мне она заходит тяжко. Это как еслиб я вместо инлайна умножение матриц в 3д явно писал))) (конечно неравноценный пример, но всё же).
Я не предлагаю переписать вам ваш проект, я за-то чтобы кто будет это читать понимали что есть такая возможность, не только для динамических рук, но и сравнение со статическими руками(разными комбинациями из правил игры)

@Anton-KRaV · 28.05.2015, 16:28 **[ТС]**

Всем спасибо, кто откликнулся, но вы меня напугали

я же новичек...
Спасибо за коды, интересно, поучительно, но не все понятно. А так же хотелось бы уточнить: чем плохи классы? Роберт Лафоре учил создавать и не стесняться.

Новые блоги и статьи Все статьи Все блоги /
Кредитный калькулятор Maks 05.08.2026 Решение задачи по прикладной информатике средствами 1С. Задача: Напишите приложение-калькулятор, которое помогает рассчитывать параметры кредита для аннуитетного и дифференцированного видов. . .	У нас сейчас поговорку "Опять 25" нужно переделать на "Опять +35". kumehtar 04.08.2026 С ностальгией вспоминаю времена моего детства, когда у нас и правда +25 - была максимальная температура летом. Раньше +25 °C реально казались вершиной жары, когда можно было весь день пропадать на. . .	Как ИИ начал спорить и врать (возможно почуяв опасность для себя от индустрии - уход от электроники). Hrethgir 04.08.2026 Недельный диалог, на фоне событий с НПЗ. Да, из спирта можно получать бензин, и это не сложно. Но потом в схеме я решил избавиться от насоса, при этом полностью сделав контроль подачи спирта в. . .	Термопринтер QR701 Argus19 03.08.2026 Термопринтер QR701 Купил два термопринтера QR701. На сэлф-тесте написано: Language: PC936 (GB18030). Что означает, что принтеры могут печатать только латиницу и китайские иероглифы. Так же. . .
Создание формы заимствованного документа Maks 03.08.2026 Задача: Необходимо создать собственную форму заимствованного документа. На форме должен быть реквизит "Покупатель", а также табличная часть со следующими реквизитами: - Расчетный счет покупателя. . .	Задача предоставления скидок покупателям Maks 03.08.2026 Задача: В документе "Продажи" необходимо реализовать функционал предоставления скидок покупателям. Скидка должна автоматически рассчитываться и подставляться в соответствующее поле при выборе. . .	Почему SEO не начинается с ключевых слов: что проверить до написания текстов Neotwalker 01.08.2026 Когда владельцу сайта предлагают заняться SEO, первым шагом часто становится сбор запросов и написание текстов. Логика кажется понятной: 1. Находим ключевые слова. 2. Добавляем их на. . .	Знание — сила: Доктрина интенциональности знаний, углубление в формулу Hrethgir 01.08.2026 https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11957&stc=1&d=1785567302 Знаменитый афоризм Фрэнсиса Бэкона «Знание — сила» (Scientia potentia est) в массовой культуре принято понимать. . .

@nefton 45 / 21 / 6 Регистрация: 28.02.2013 Сообщений: 194
	25.05.2015, 17:46
	Перестаньте писать классы И рука (7 карт) компонуются в число не для экономии памяти или чегото там, а для того чтоб сравнить руки у 5ти человек к примеру кто победил. Кроме моего подсчёта силы руки есть ещё с десяток вариантов, написанных независимо в разное время в разных частях света. И все они битовые Ну есть конечно не битовые - только все что мне встречались - с ошибками. Тоесть не считают правильно все руки Если бы использование классов реально привело б к хоть какойто удобочитаемости.... но это не тот случай. Тут не будет понятнее с классами. А вот скорость тут очень надо. В самом примитивном калькуляторе эта функция вызывается 100 000 раз при каждом подсчёте вероятностей. 0

@Anton-KRaV 1 / 1 / 1 Регистрация: 28.07.2013 Сообщений: 41
	28.05.2015, 16:28 [ТС]
	Всем спасибо, кто откликнулся, но вы меня напугали я же новичек... Спасибо за коды, интересно, поучительно, но не все понятно. А так же хотелось бы уточнить: чем плохи классы? Роберт Лафоре учил создавать и не стесняться. 0