mike1729/bridgetcs1

bridgetcs1's Issues

Scorer nie informuje o końcu gry.

Np. zwracając bool z metody update, lub lepiej za pomocą gettera gameOver.

Licytacja - format

Dobrze by było, żeby format odzywki dla przebicia był [1-7] [c,d,h,s,n] (n = no trump)

Unrealistic Bot bids

Licytację zawsze rozpoczyna gracz 1

No właśnie... a powinno być w kółko (rozpoczynać pierwszą może pierwszy, albo losowy).

Game and Application

Czy ktoś był przydzielony do jakiegoś grzebania w Game albo Application ? Albo z jakichś przyczyn ma zamiar mieć z tym coś wspólnego ?

nie kompiluje się

Bardzo proszę robić commit tylko jeśli się kompiluje!

Interfejs tekstowy nie do końca czytelny

Brakuje opisów, czasem brakuje odstępów, czasem jest za dużo :)

Testy do klasy Scorer

Arbiter not compiling

The Arbiter class is not compiling because it contains a vector of Cards, which have their copying constructor deleted -- vectors apparently do not like that. See: line 17 of Arbiter.hpp.

Rubber game missing

Póki co gra składa sie z jednego rozdania.
Do napisania pętla zewnętrzna.

Bidding sometimes returns Contract with wrong declarer

Just run make test in src/bidding -- it doesn't pass two tests because of that.

Set may fall apart in Observable

Since ObserverPtr is a weak_pointer the object its pointing at it might stop existing without warning. This would change the ordering of the elements of the set.

Bot can return invalid call and hang the application

In Bot.cpp, lines 293-306:

In the case when Bot is making the second call of the bidding (isOpening(bidding) returns true), it doesn't check if the bid call is greater than the first one and the arbiter will end up asking Bot for a valid call forever.

Wyświetlanie kart ręka/dziadek

To dla pana Łukasza Kapicy, na zaliczenie:
chciałbym, żeby karty w ręce i dziadka były wyświetlane zamiast:
2C 4C 6C TC JD AD 2H QH TS JS QS KS AS
tak:
S: A K Q J T
H: Q 2
D: A J T
C: T 6 4 2
czyli kolorami w osobnych linijkach i od największej.

Contract failed. -2 undertricks. LOL

czyli 2 overtricks? ;)

Bot is in inconsitent state

Bot::getPartnerCall() is code stub (always returning PASS). But Bot::makeCall supposes that it's not a stub and takes denomination from its result without any checking (well, it's not very wise anyway, is it?), and this denomination is later used in another functions like cardsInSuit and so on - overall, it looks like a spectacular failure when the program is being started for the first time (which I believe will take place today or tomorrow, so it'd be nice if Bot didn't segfaulted then).