crille/SoftwareCraftmanship
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fixup

Browse Source
This commit is contained in:
Christian Ohlsson
2015-04-29 09:24:59 +02:00
parent d089eedb19
commit 43e313beb4
3 changed files with 4 additions and 239 deletions
Download Patch File Download Diff File Expand all files Collapse all files

206
PokerKata/src/session3/PokerGame.java
View File

@@ -1,7 +1,6 @@
package session3; package session3;
public class PokerGame { public class PokerGame {
<<<<<<< HEAD
public Player calculateWinningPlayer(Player left, Player right) { public Player calculateWinningPlayer(Player left, Player right) {
@@ -13,214 +12,9 @@ public class PokerGame {
Player pairResult = PairRule.compare(left, right); Player pairResult = PairRule.compare(left, right);
if(pairResult != null){ if(pairResult != null){
return pairResult; return pairResult;
=======
public int getScore(String hand) {
int score = 0;
char[] values = getValues(hand);
char[] colors = getColors(hand);
if (findStraightFlush(values, colors) == true)
score = 14;
else if (findStraight(values) == true)
score = 12;
else if (findFlush(colors) == true)
score = 10;
else if (findFullHouse(values) == true)
score = 9;
else if (findFourOfAKind(values) == true)
score = 7;
else if (findTwoPairs(values) == true)
score = 6;
else if (findThreeOfAKind(values) == true)
score = 3;
else if (findPair(values) == true)
score = 2;
else
score = 0;
return score;
}
/**
* Finds a pair by looking if two cards next to each other has the same
* value
*
* @param hand
* @return true if there is a pair, otherwise false
*/
private boolean findPair(char[] values) {
// Loop again, find pair
for (int i = 0; i < values.length - 1; i++) {
if (values[i] == values[i + 1])
return true;
}
return false;
}
/**
* Finds a pair by looking if three cards next to each other has the same
* value
*
* @param hand
* @return true if there is a Three of a kind, otherwise false
*/
private boolean findThreeOfAKind(char[] values) {
// Loop again, find three of a kind
for (int i = 0; i < values.length - 2; i++) {
if (values[i] == values[i + 1] && values[i + 1] == values[i + 2])
return true;
>>>>>>> origin/master
} }
return HighCardRule.compare(left, right); return HighCardRule.compare(left, right);
} }
<<<<<<< HEAD
=======
/**
* Finds two pairs by looking for a pair, and them look if the remaining
* cards is a pair
*
* @param hand
* @return true if there is two pairs, otherwise false
*/
private boolean findTwoPairs(char[] values) {
try {
// Loop again, find pair
for (int i = 0; i < values.length - 1; i++) {
if (values[i] == values[i + 1]) {
if (values[i + 2] == values[i + 3]
|| values[i + 3] == values[i + 4]) {
return true;
}
}
}
}
catch (ArrayIndexOutOfBoundsException e) {
return false;
}
return false;
}
/**
* Finds Four of a kind by first finding Three of a kind and then looks if
* one of the remaining cards matches
*
* @param hand
* @return true if there is a Four of a kind, otherwise false
*/
private boolean findFourOfAKind(char[] values) {
// Loop again, find three of a kind
for (int i = 0; i < values.length - 2; i++) {
if (values[i] == values[i + 1] && values[i + 1] == values[i + 2]
&& values[i + 2] == values[i + 3])
return true;
}
return false;
}
/**
* Finds a Full house. We have a Full House if the first and last two cards
* are a pair, and the middle card has same value as either of them (is
* three of a kind)
*
* @param hand
* @return true if there is a Full House, otherwise false
*/
private boolean findFullHouse(char[] values) {
if (values[0] == values[1] && values[3] == values[4]
&& (values[0] == values[2] || values[2] == values[3]))
return true;
return false;
}
/**
* Finds a Flush by looking if all cards has the same colors
*
* @param hand
* @return true if there is a flush, otherwise false
*/
private boolean findFlush(char[] colors) {
// Loop again, find pair
if (colors[0] == colors[1] && colors[1] == colors[2]
&& colors[2] == colors[3] && colors[3] == colors[4])
return true;
return false;
}
/**
* Finds a Straight by looking if the value of the next card is 1 higher
* throughout all the cards
*
* @param values
* @return true if there is a straight, otherwise false
*/
private boolean findStraight(char[] values) {
for (int i = 0; i < values.length - 1; i++)
if (values[i + 1] != values[i] + 1)
return false;
return true;
}
/**
* Finds a Straight Flush by first looking for a Straight and then the Flush
*
* @param values
* @param colors
* @return
*/
private boolean findStraightFlush(char[] values, char[] colors) {
for (int i = 0; i < values.length - 1; i++)
if (values[i + 1] != values[i] + 1)
return false;
if (colors[0] == colors[4])
return true;
else
return false;
}
/**
* Extracts all the values from the given String (hand)
*
* @param hand
* @return The values of all the cards
*/
private char[] getValues(String hand) {
char[] values = "".toCharArray();
String temp = "";
// Find all values
for (int i = 1; i < hand.length(); i += 2) {
temp += hand.charAt(i);
}
values = temp.toCharArray();
Arrays.sort(values);
return values;
}
/**
* Extracts all the colors from the given String (hand)
*
* @param hand
* @return The colors of all the cards
*/
private char[] getColors(String hand) {
char[] colors = "".toCharArray();
String temp = "";
// Find all values
for (int i = 0; i < hand.length() - 1; i += 2) {
temp += hand.charAt(i);
}
colors = temp.toCharArray();
Arrays.sort(colors);
return colors;
}
>>>>>>> origin/master
} }

28
PokerKata/test/session3/PokerGameTest.java
View File

@@ -39,32 +39,4 @@ public class PokerGameTest {
assertEquals("White", winningPlayer.getPlayerName()); assertEquals("White", winningPlayer.getPlayerName());
} }
@Test
public void straight_gives_12() throws Exception {
String hand = "K2S3H4R5R6";
int result = myGame.getScore(hand);
assertEquals(12, result);
}
@Test
public void straight_flush_gives_14() throws Exception {
String hand = "K2K3K4K5K6";
int result = myGame.getScore(hand);
assertEquals(14, result);
}
@Test
@Ignore
public void royal_straight_flush_gives_16() throws Exception {
String hand = "R10RKRDRKRE";
int result = myGame.getScore(hand);
assertEquals(16, result);
}
} }

9
PokerKata/test/session3/TwoPairRuleTest.java
View File

@@ -1,8 +1,7 @@
package session3; package session3;
import static org.junit.Assert.*; import src.session3.Player;
import src.session3.TwoPairRule;
import org.junit.Test;
public class TwoPairRuleTest { public class TwoPairRuleTest {
@@ -27,7 +26,7 @@ public class TwoPairRuleTest {
assertEquals("Green", winningPlayer.getPlayerName()); assertEquals("Green", winningPlayer.getPlayerName());
} }
@Test @Test
public void first_pair_tie_second_pair_highest_wins() throws Exception { public void first_pair_tie_second_pair_highest_wins() throws Exception {
Player green = new Player("Green", "9S 9C 6D 6H AH"); Player green = new Player("Green", "9S 9C 6D 6H AH");
@@ -35,6 +34,6 @@ public class TwoPairRuleTest {
Player winningPlayer = TwoPairRule.compare(white, green); Player winningPlayer = TwoPairRule.compare(white, green);
assertEquals("Green", winningPlayer.getPlayerName()); assertEquals("Green", winningPlayer.getPlayerName());
} }
} }