SlideShare a Scribd company logo

Bowling Game Kata in C# Adapted

Download as PPTX, PDF
Mike Clement, profile picture
Mike Clement

The document describes the scoring rules for a bowling game. It consists of 10 frames where a player has two rolls per frame, except for the 10th frame which allows a third roll for a spare or strike. Scoring is the total pins knocked down plus bonuses for spares and strikes. A spare gives a bonus of the next roll, while a strike gives a bonus of the next two rolls. The document then discusses using test-driven development to design the game classes and methods to track scoring according to the rules.

TechnologyBusinessEntertainment & Humor
1 of 49
Downloaded 82 times
1
2
3
4
5
6
7
8
9
Most read
10
Most read
11
Most read
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
Bowling Game KataAdapted for C# and nUnitBy Dan Stewart (with modifications by Mike Clement)dan@stewshack.commike@softwareontheside.com
Scoring Bowling.The game consists of 10 frames as shown above. In each frame the player has two opportunities to knock down 10 pins. The score for the frame is the total number of pins knocked down, plus bonuses for strikes and spares.A spare is when the player knocks down all 10 pins in two tries. The bonus for that frame is the number of pins knocked down by the next roll. So in frame 3 above, the score is 10 (the total number knocked down) plus a bonus of 5 (the number of pins knocked down on the next roll.)A strike is when the player knocks down all 10 pins on his first try. The bonus for that frame is the value of the next two balls rolled.In the tenth frame a player who rolls a spare or strike is allowed to roll the extra balls to complete the frame. However no more than three balls can be rolled in tenth frame.
A quick design sessionClearly we need the Game class.
A quick design sessionA game has 10 frames.
A quick design sessionA frame has 1 or two rolls.
A quick design sessionThe tenth frame has two or three rolls.It is different from all the other frames.
A quick design sessionThe score function mustinclude all the frames, and calculate all their scores.
A quick design sessionThe score for a spare or a strike depends on the frame’s successor
Begin.Create a class library named BowlingGameKata
Begin.Add a test fixture named BowlingGameTests to the projectusingNUnit.Framework;namespaceBowlingGameKata{[TestFixture]publicclassBowlingGameTests{ [Test]publicvoidGutterGameTest() { } }}
The first test.[Test]publicvoidGutterGameTest() { Game g = new Game();}
The first test.namespaceBowlingGameKata{publicclassGame { }}[Test]publicvoidGutterGameTest(){ Game g = new Game();}
The first test.[Test]publicvoidGutterGameTest(){ Game g = new Game();}publicclassGame{}
The first test.[Test]publicvoidGutterGameTest(){ Gameg = newGame();for(int i = 0; i < 20; i++) {g.Roll(0); }}publicclassGame{}
The first test.public class Game{ publicvoid Roll(int p){ thrownewSystem.NotImplementedException();}}[Test]publicvoidGutterGameTest(){ Gameg = newGame();for(int i = 0; i < 20; i++) {g.Roll(0); }}
The first test.[Test]publicvoidGutterGameTest(){ Gameg = newGame();for(int i = 0; i < 20; i++){g.Roll(0);}Assert.That(g.Score(), Is.EqualTo(0));}public class Game{ publicvoid Roll(int p){ thrownewSystem.NotImplementedException(); }}
The first test.[Test]publicvoidGutterGameTest(){ Gameg = newGame();for(int i = 0; i < 20; i++){g.Roll(0);}Assert.That(g.Score(), Is.EqualTo(0));}publicclassGame{ publicvoid Roll(int p) {thrownew System.NotImplementedException(); }publicobject Score(){thrownew System.NotImplementedException(); }}Failed GutterGameTestthrew exception
The first test.[Test]publicvoidGutterGameTest(){ Gameg = newGame();for(int i = 0; i < 20; i++) {g.Roll(0); }Assert.That(g.Score(), Is.EqualTo(0));}publicclassGame{privateint score;publicvoid Roll(int pins) { }publicintScore() {returnscore; }}
The Second test.[Test]publicvoidAllOnesTest(){ Gameg = newGame(); for(int i = 0; i < 20; i++) {g.Roll(1); }Assert.That(g.Score(), Is.EqualTo(20));}publicclassGame{privateint score;publicvoid Roll(int pins) { }publicintScore() {returnscore; }}
The Second test.- Game creation is duplicated- roll loop is duplicated[Test]publicvoidAllOnesTest(){ Gameg = newGame(); for(int i = 0; i < 20; i++) {g.Roll(1); }Assert.That(g.Score(), Is.EqualTo(20));}publicclassGame{privateint score;publicvoid Roll(int pins) { }publicintScore() {returnscore; }}
The Second test.- Game creation is duplicated- roll loop is duplicated[Test]publicvoidAllOnesTest(){ Gameg = newGame(); for(int i = 0; i < 20; i++) {g.Roll(1); }Assert.That(g.Score(), Is.EqualTo(20));}publicclassGame{privateint score;publicvoid Roll(int pins){}publicintScore(){return score; }}Assert.AreEqual failed. Expected:<20>. Actual:<0>.
The Second test.- roll loop is duplicatedprivateGame g;[SetUp]publicvoidSetup() { g = newGame();}[Test]publicvoidGutterGameTest(){for(int i = 0; i < 20; i++) {g.Roll(0); }Assert.That(g.Score(), Is.EqualTo(0));}[Test]publicvoidAllOnesTest(){for(int i = 0; i < 20; i++) {g.Roll(1); }Assert.That(g.Score(), Is.EqualTo(20));}publicclassGame{privateint score;publicvoid Roll(int pins){score += pins;}publicint Score(){return score; }}
The Second test.- roll loop is duplicated[Test]publicvoidGutterGameTest(){int rolls = 20;intpins = 0;for(int i = 0; i < rolls; i++){g.Roll(pins);}Assert.That(g.Score(), Is.EqualTo(0));}publicclassGame{privateint score;publicvoid Roll(int pins){score += pins;}publicint Score(){return score; }}
The Second test.- roll loop is duplicated[Test]publicvoidGutterGameTest(){int rolls = 20;intpins = 0;RollMany(rolls, pins);for(int i = 0; i < rolls; i++) {g.Roll(pins); }Assert.That(g.Score(), Is.EqualTo(0));}publicclassGame{privateint score;publicvoid Roll(int pins){score += pins;}publicint Score(){return score; }}
The Second test.publicclassGame{privateint score;publicvoid Roll(int pins){score += pins;}publicint Score(){return score; }}[Test]publicvoidGutterGameTest(){RollMany(20, 0);Assert.That(g.Score(), Is.EqualTo(0));}privatevoidRollMany(introlls, int pins){ for(int i = 0; i < rolls; i++) {g.Roll(pins); }}
The Second test.[Test]publicvoidGutterGameTest(){RollMany(20, 0);Assert.That(g.Score(), Is.EqualTo(0));}[Test]publicvoidAllOnesTest(){RollMany(20, 1);Assert.That(g.Score(), Is.EqualTo(20));}privatevoidRollMany(int rolls, int pins){ for(int i = 0; i < rolls; i++) {g.Roll(pins); }}publicclassGame{privateint score;publicvoid Roll(int pins){score += pins;}publicint Score(){return score; }}
The Third test.- ugly comment in test.[Test]publicvoidOneSpareTest(){g.Roll(5);g.Roll(5); // spareg.Roll(3);RollMany(17, 0);Assert.That(g.Score(), Is.EqualTo(16));}publicclassGame{privateint score;publicvoid Roll(int pins){score += pins;}publicint Score(){return score; }}Failed Assert.AreEqual Expected:<16>. Actual:<13>
The Third test.- ugly comment in test.publicclassGame{privateint score;publicvoid Roll(int pins){score += pins;}publicint Score(){return score; }}[Test]publicvoidOneSpareTest(){g.Roll(5);g.Roll(5); // spareg.Roll(3);RollMany(17, 0);Assert.That(g.Score(), Is.EqualTo(16));}tempted to use flag to remember previous roll. So design must be wrong.Failed Assert.AreEqual Expected:<16>. Actual:<13>
The Third test.- ugly comment in test.Roll() calculates score, but name does not imply that.publicclassGame{privateint score;publicvoid Roll(int pins){score += pins;}publicint Score(){return score; }}[Test]publicvoidOneSpareTest(){g.Roll(5);g.Roll(5); // spareg.Roll(3);RollMany(17, 0);Assert.AreEqual(16, g.Score);}Score() does not calculate score, but name implies that it does.Design is wrong. Responsibilities are misplaced.Failed Assert.AreEqual Expected:<16>. Actual:<13>
The Third test.- ugly comment in test.[Test]publicvoidOneSpareTest(){g.Roll(5);g.Roll(5); // spareg.Roll(3);RollMany(17, 0);//Assert.That(g.Score(), Is.EqualTo(16));}publicclassGame{privateint score;publicvoid Roll(int pins){score += pins;}publicint Score(){return score; }}
The Third test.- ugly comment in test.publicclassGame{privateint score;privateint[] rolls = newint[21];privateint currentRoll;publicvoid Roll(int pins){rolls[currentRoll++] = pins;score += pins;}publicint Score(){return score; }}[Test]publicvoidOneSpareTest(){g.Roll(5);g.Roll(5); // spareg.Roll(3);RollMany(17, 0);//Assert.That(g.Score(), Is.EqualTo(16));}
The Third test.- ugly comment in test.publicclassGame{ privateint[] rolls = newint[21];privateint currentRoll;publicvoid Roll(int pins){rolls[currentRoll++] = pins;}publicint Score(){int score = 0;for (int i = 0; i < rolls.Length; i++){score += rolls[i];}return score; }}[Test]publicvoidOneSpareTest(){g.Roll(5);g.Roll(5); // spareg.Roll(3);RollMany(17, 0);//Assert.That(g.Score(), Is.EqualTo(16));}
The Third test.- ugly comment in test.[Test]publicvoidOneSpareTest(){g.Roll(5);g.Roll(5); // spareg.Roll(3);RollMany(17, 0);Assert.That(g.Score(), Is.EqualTo(16));}publicclassGame{ privateint[] rolls = newint[21];privateint currentRoll;publicvoid Roll(int pins){rolls[currentRoll++] = pins;}publicint Score(){int score = 0;for (int i = 0; i < rolls.Length; i++){score += rolls[i];}return score; }}Failed Assert.AreEqual Expected:<16>. Actual:<13>
The Third test.- ugly comment in test.publicint Score(){int score = 0;for (int i = 0; i < rolls.Length; i++) { // spare if(rolls[i] + rolls[i+1] == 10){score += ...}score += rolls[i];}return score;}[Test]publicvoidOneSpareTest(){g.Roll(5);g.Roll(5); // spareg.Roll(3);RollMany(17, 0);Assert.That(g.Score(), Is.EqualTo(16));}This isn’t going to work because i might not refer to the first ball of the frame.Design is still wrong.Need to walk through array two balls (one frame) at a time.Failed Assert.AreEqual Expected:<16>. Actual:<13>
The Third test.- ugly comment in test.publicclassGame{ privateint[] rolls = newint[21];privateint currentRoll;publicvoid Roll(int pins){rolls[currentRoll++] = pins;}publicint Score(){int score = 0;for (int i = 0; i < rolls.Length; i++){score += rolls[i];}return score; }}[Test]publicvoidOneSpareTest(){g.Roll(5);g.Roll(5); // spareg.Roll(3);RollMany(17, 0);//Assert.That(g.Score(), Is.EqualTo(16));Assert.Inconclusive();}
The Third test.- ugly comment in test.publicint Score(){int score = 0;inti = 0;for (int frame = 0; frame < 10; frame++){ score += rolls[i] + rolls[i + 1]; i+= 2;}returnscore;}[Test]publicvoidOneSpareTest(){g.Roll(5);g.Roll(5); // spareg.Roll(3);RollMany(17, 0);//Assert.That(g.Score(), Is.EqualTo(16));Assert.Inconclusive();}
The Third test.- ugly comment in test.[Test]publicvoidOneSpareTest(){g.Roll(5);g.Roll(5); // spareg.Roll(3);RollMany(17, 0);Assert.That(g.Score(), Is.EqualTo(16));}publicint Score(){int score = 0;int i = 0;for (int frame = 0; frame < 10; frame++) { score += rolls[i] + rolls[i + 1]; i += 2; }return score;}Failed Assert.AreEqual Expected:<16>. Actual:<13>
The Third test.- ugly comment in test.publicint Score(){int score = 0;inti = 0;for (int frame = 0; frame < 10; frame++){// spareif (rolls[i] + rolls[i + 1] == 10){score += 10 + rolls[i + 2];i += 2;}else{ score += rolls[i] + rolls[i + 1]; i += 2; } }return score;}[Test]publicvoidOneSpareTest(){g.Roll(5);g.Roll(5); // spareg.Roll(3);RollMany(17, 0);Assert.That(g.Score(), Is.EqualTo(16));}
The Third test.ugly comment in test.
ugly comment in conditional.publicint Score(){int score = 0;int roll = 0;for (int frame = 0; frame < 10; frame++){// spareif (rolls[roll] + rolls[roll + 1] == 10){score += 10 + rolls[roll + 2];roll += 2;}else{ score += rolls[roll] + rolls[roll + 1]; roll += 2; } }return score;}[Test]publicvoidOneSpareTest(){g.Roll(5);g.Roll(5); // spareg.Roll(3);RollMany(17, 0);Assert.That(g.Score(), Is.EqualTo(16));}
The Third test.- ugly comment in test.publicint Score(){int score = 0;int roll = 0;for (int frame = 0; frame < 10; frame++){ if(IsSpare(roll)) { score += 10 + rolls[roll + 2]; roll += 2; }else { score += rolls[roll] + rolls[roll + 1]; roll += 2;} }returnscore;}privateboolIsSpare(int roll){ returnrolls[roll] + rolls[roll + 1] == 10;}[Test]publicvoidOneSpareTest(){g.Roll(5);g.Roll(5); // spareg.Roll(3);RollMany(17, 0);Assert.That(g.Score(), Is.EqualTo(16));}
The Third test.[Test]publicvoidOneSpareTest(){RollSpare();g.Roll(3);RollMany(17, 0);Assert.That(g.Score(), Is.EqualTo(16));}privatevoidRollSpare(){g.Roll(5);g.Roll(5);}publicint Score(){int score = 0;int roll = 0;for (int frame = 0; frame < 10; frame++){ if(IsSpare(roll)) { score += 10 + rolls[roll + 2]; roll += 2; }else { score += rolls[roll] + rolls[roll + 1]; roll += 2; }}returnscore;}privateboolIsSpare(int roll){ returnrolls[roll] + rolls[roll + 1] == 10;}
The Fourth test.- ugly comment in OneStrikeTest.[Test]publicvoidOneStrikeTest(){g.Roll(10); // strikeg.Roll(3);g.Roll(4);RollMany(16, 0);Assert.That(g.Score(), Is.EqualTo(24));}publicint Score(){int score = 0;int roll = 0;for (int frame = 0; frame < 10; frame++){ if(IsSpare(roll)){ score += 10 + rolls[roll + 2];}else{ score += rolls[roll] + rolls[roll + 1];}roll += 2;}returnscore;}privatebool IsSpare(int roll){ returnrolls[roll] + rolls[roll + 1] == 10;}Failed Assert.AreEqual Expected:<24>. Actual:<17>
The Fourth test.ugly comment in OneStrikeTest.
ugly comment in conditional.
ugly expressions.publicint Score(){ intscore = 0;introll = 0;for(int frame = 0; frame < 10; frame++){if(rolls[roll] == 10) // strike{ score += 10 + rolls[roll + 1] + rolls[roll + 2];roll++;}elseif (IsSpare(roll)){ score += 10 + rolls[roll + 2];roll += 2;}else{ score += rolls[roll] + rolls[roll + 1];roll += 2;} } returnscore;}[Test]publicvoidOneStrikeTest(){g.Roll(10); // strikeg.Roll(3);g.Roll(4);RollMany(16, 0);Assert.That(g.Score(), Is.EqualTo(24));}
The Fourth test.ugly comment in OneStrikeTest.
ugly comment in conditional.publicint Score(){ intscore = 0;introll = 0;for(int frame = 0; frame < 10; frame++){if(rolls[roll] == 10) // strike{ score += 10 + StrikeBonus(roll);roll++;}elseif (IsSpare(roll)){ score += 10 + SpareBonus(roll);roll += 2;}else{ score += SumOfBallsInFrame(roll);roll += 2;}}returnscore;}privateint SumOfBallsInFrame(int roll){ returnrolls[roll] + rolls[roll + 1];}privateint SpareBonus(int roll){ returnrolls[roll + 2];}privateint StrikeBonus(int roll){ returnrolls[roll + 1] + rolls[roll + 2];}[Test]publicvoidOneStrikeTest(){g.Roll(10); // strikeg.Roll(3);g.Roll(4);RollMany(16, 0);Assert.That(g.Score(), Is.EqualTo(24));}
The Fourth test.- ugly comment in OneStrikeTest.publicint Score(){int score = 0;int roll = 0;for (int frame = 0; frame < 10; frame++){if(IsStrike(roll)){ score += 10 + StrikeBonus(roll);roll++;}elseif (IsSpare(roll)){score += 10+ SpareBonus(roll);roll += 2;}else{ score += SumOfBallsInFrame(roll);roll += 2;}}returnscore;}privatebool IsStrike(int roll){ returnrolls[roll] == 10;}[Test]publicvoidOneStrikeTest(){g.Roll(10); // strikeg.Roll(3);g.Roll(4);RollMany(16, 0);Assert.That(g.Score(), Is.EqualTo(24));}

More Related Content

PPT
Bowling Game Kata
guest958d7
 
PPTX
Bowling Game Kata C#
Dan Stewart
 
PPTX
Java: Regular Expression
Masudul Haque
 
PDF
Curso de Java: Threads
Helder da Rocha
 
PPTX
Java 8 Lambda and Streams
Venkata Naga Ravi
 
PPTX
Regular Expressions in Java
OblivionWalker
 
PPTX
Java Foundations: Basic Syntax, Conditions, Loops
Svetlin Nakov
 
PDF
Curso de Java: Introdução a lambda e Streams
Helder da Rocha
 
Bowling Game Kata
guest958d7
 
Bowling Game Kata C#
Dan Stewart
 
Java: Regular Expression
Masudul Haque
 
Curso de Java: Threads
Helder da Rocha
 
Java 8 Lambda and Streams
Venkata Naga Ravi
 
Regular Expressions in Java
OblivionWalker
 
Java Foundations: Basic Syntax, Conditions, Loops
Svetlin Nakov
 
Curso de Java: Introdução a lambda e Streams
Helder da Rocha
 

What's hot (20)

PPTX
Loops in C# for loops while and do while loop.
Abid Kohistani
 
PDF
Workshop unit test
Francesco Garavaglia
 
PPTX
Introduction to Regular Expressions
Matt Casto
 
PPTX
Java Lambda Expressions.pptx
SameerAhmed593310
 
PPTX
Java Unit Testing
Nayanda Haberty
 
PDF
Page Object Model and Implementation in Selenium
Zoe Gilbert
 
PPTX
Whitebox testing of Spring Boot applications
Yura Nosenko
 
PPTX
Paging Like A Pro
Gabor Varadi
 
PPT
05 junit
mha4
 
PPTX
Java Generics
Zülfikar Karakaya
 
PPT
Java Collections Framework
Sony India Software Center
 
PPT
JUnit 4
Sunil OS
 
PDF
GMock framework
corehard_by
 
PDF
JUnit 5
Scott Leberknight
 
PPTX
20.4 Java interfaces and abstraction
Intro C# Book
 
PDF
Mocking in Java with Mockito
Richard Paul
 
PPT
7.data types in c#
Zeeshan Ahmad
 
PDF
Core Java Tutorial
eMexo Technologies
 
PPTX
for loop in java
Majid Ali
 
PDF
Algo poo ts
mohamed El babili
 
Loops in C# for loops while and do while loop.
Abid Kohistani
 
Workshop unit test
Francesco Garavaglia
 
Introduction to Regular Expressions
Matt Casto
 
Java Lambda Expressions.pptx
SameerAhmed593310
 
Java Unit Testing
Nayanda Haberty
 
Page Object Model and Implementation in Selenium
Zoe Gilbert
 
Whitebox testing of Spring Boot applications
Yura Nosenko
 
Paging Like A Pro
Gabor Varadi
 
05 junit
mha4
 
Java Generics
Zülfikar Karakaya
 
Java Collections Framework
Sony India Software Center
 
JUnit 4
Sunil OS
 
GMock framework
corehard_by
 
JUnit 5
Scott Leberknight
 
20.4 Java interfaces and abstraction
Intro C# Book
 
Mocking in Java with Mockito
Richard Paul
 
7.data types in c#
Zeeshan Ahmad
 
Core Java Tutorial
eMexo Technologies
 
for loop in java
Majid Ali
 
Algo poo ts
mohamed El babili
 
Ad

Similar to Bowling Game Kata in C# Adapted (15)

PPT
Bowling Game Kata by Robert C. Martin
Lalit Kale
 
PPT
Bowling game kata
Carol Bruno
 
PPT
Bowling game kata
Joe McCall
 
PPT
Bowling game kata
Rahman Ash
 
PDF
Test-Driven Development Fundamentals on Force.com
Salesforce Developers
 
PPT
Csphtp1 06
HUST
 
PPT
Introduction to Spec#
Pieter Joost van de Sande
 
PPT
Csphtp1 07
HUST
 
PDF
C++You will design a program to play a simplified version of war, .pdf
ezzi97
 
PDF
Code retreat june24th
Jean-Laurent de Morlhon
 
PPT
Csharp In Detail Part2
Mohamed Krar
 
PPTX
Thinking in F#
Mike Clement
 
DOCX
Java Helpa4.pdfASSIGNMENT 4 A game and an email system.docx
priestmanmable
 
PPT
Classes1
phanleson
 
PDF
Core C#
Jussi Pohjolainen
 
Bowling Game Kata by Robert C. Martin
Lalit Kale
 
Bowling game kata
Carol Bruno
 
Bowling game kata
Joe McCall
 
Bowling game kata
Rahman Ash
 
Test-Driven Development Fundamentals on Force.com
Salesforce Developers
 
Csphtp1 06
HUST
 
Introduction to Spec#
Pieter Joost van de Sande
 
Csphtp1 07
HUST
 
C++You will design a program to play a simplified version of war, .pdf
ezzi97
 
Code retreat june24th
Jean-Laurent de Morlhon
 
Csharp In Detail Part2
Mohamed Krar
 
Thinking in F#
Mike Clement
 
Java Helpa4.pdfASSIGNMENT 4 A game and an email system.docx
priestmanmable
 
Classes1
phanleson
 
Core C#
Jussi Pohjolainen
 
Ad

More from Mike Clement (19)

PPTX
Collaboration Principles from Mob Programming
Mike Clement
 
PPTX
Focus on Flow: Lean Principles in Action
Mike Clement
 
PPTX
Taming scary production code that nobody wants to touch
Mike Clement
 
PPTX
Develop your sense of code smell
Mike Clement
 
PPTX
Maps over Backlogs: User Story Mapping to Share the Big Picture
Mike Clement
 
PPTX
Escaping the Pitfalls of Software Product Development
Mike Clement
 
PPTX
Put the Tests Before the Code
Mike Clement
 
PDF
Mob Programming for Continuous Learning
Mike Clement
 
PPTX
Play to Learn: Agile Games with Cards and Dice
Mike Clement
 
PPTX
The Quest for Continuous Delivery at Pluralsight
Mike Clement
 
PPTX
Software Craftsmanship and Agile Code Games
Mike Clement
 
PPTX
Transformation Priority Premise: TDD Test Order Matters
Mike Clement
 
PPTX
Power of Patterns: Refactoring to (or away from) Patterns
Mike Clement
 
PPTX
Code Katas Spring 2012
Mike Clement
 
PPTX
FizzBuzz Guided Kata
Mike Clement
 
PPTX
Linq (from the inside)
Mike Clement
 
PPTX
Code Katas: Practicing Your Craft
Mike Clement
 
PPTX
Software Craftsmanship
Mike Clement
 
PPTX
Using Rhino Mocks for Effective Unit Testing
Mike Clement
 
Collaboration Principles from Mob Programming
Mike Clement
 
Focus on Flow: Lean Principles in Action
Mike Clement
 
Taming scary production code that nobody wants to touch
Mike Clement
 
Develop your sense of code smell
Mike Clement
 
Maps over Backlogs: User Story Mapping to Share the Big Picture
Mike Clement
 
Escaping the Pitfalls of Software Product Development
Mike Clement
 
Put the Tests Before the Code
Mike Clement
 
Mob Programming for Continuous Learning
Mike Clement
 
Play to Learn: Agile Games with Cards and Dice
Mike Clement
 
The Quest for Continuous Delivery at Pluralsight
Mike Clement
 
Software Craftsmanship and Agile Code Games
Mike Clement
 
Transformation Priority Premise: TDD Test Order Matters
Mike Clement
 
Power of Patterns: Refactoring to (or away from) Patterns
Mike Clement
 
Code Katas Spring 2012
Mike Clement
 
FizzBuzz Guided Kata
Mike Clement
 
Linq (from the inside)
Mike Clement
 
Code Katas: Practicing Your Craft
Mike Clement
 
Software Craftsmanship
Mike Clement
 
Using Rhino Mocks for Effective Unit Testing
Mike Clement
 

Recently uploaded (20)

PDF
Agricultural_Statistics_at_a_Glance_2022_0.pdf
SandeepSingh286037
 
PDF
MIND Revenue Release Quarter 2 2025 Press Release
MIND CTI
 
PDF
DP Operators-handbook-extract for the Mautical Institute
LeonelMejiaLarios
 
PDF
A comparative study of natural language inference in Swahili using monolingua...
IAESIJAI
 
PPTX
Group 1 Presentation -Planning and Decision Making .pptx
MatthewLewis227954
 
PDF
From MVP to Full-Scale Product A Startup’s Software Journey.pdf
KodekX
 
PPTX
TechTalks-8-2019-Service-Management-ITIL-Refresh-ITIL-4-Framework-Supports-Ou...
bonakiduza1024
 
PDF
August Patch Tuesday
Ivanti
 
PDF
7 ChatGPT Prompts to Help You Define Your Ideal Customer Profile.pdf
SOFTTECHHUB
 
PPTX
Tartificialntelligence_presentation.pptx
ry7r52mzb4
 
PDF
A comparative analysis of optical character recognition models for extracting...
IAESIJAI
 
PPTX
cloud_computing_Infrastucture_as_cloud_p
mainakbhattacharya20
 
PDF
Transform Your ITIL® 4 & ITSM Strategy with AI in 2025.pdf
PDCA Consulting
 
PDF
Building Integrated photovoltaic BIPV_UPV.pdf
SandeepSingh286037
 
PDF
Enhancing emotion recognition model for a student engagement use case through...
IAESIJAI
 
PDF
project resource management chapter-09.pdf
ssuser00e6e21
 
PDF
Encapsulation_ Review paper, used for researhc scholars
gurumoop
 
PPTX
KOM of Painting work and Equipment Insulation REV00 update 25-dec.pptx
muhammadmuneebnaeem7
 
PPTX
Digital-Transformation-Roadmap-for-Companies.pptx
David Malick Dieng
 
PDF
Encapsulation theory and applications.pdf
gurumoop
 
Agricultural_Statistics_at_a_Glance_2022_0.pdf
SandeepSingh286037
 
MIND Revenue Release Quarter 2 2025 Press Release
MIND CTI
 
DP Operators-handbook-extract for the Mautical Institute
LeonelMejiaLarios
 
A comparative study of natural language inference in Swahili using monolingua...
IAESIJAI
 
Group 1 Presentation -Planning and Decision Making .pptx
MatthewLewis227954
 
From MVP to Full-Scale Product A Startup’s Software Journey.pdf
KodekX
 
TechTalks-8-2019-Service-Management-ITIL-Refresh-ITIL-4-Framework-Supports-Ou...
bonakiduza1024
 
August Patch Tuesday
Ivanti
 
7 ChatGPT Prompts to Help You Define Your Ideal Customer Profile.pdf
SOFTTECHHUB
 
Tartificialntelligence_presentation.pptx
ry7r52mzb4
 
A comparative analysis of optical character recognition models for extracting...
IAESIJAI
 
cloud_computing_Infrastucture_as_cloud_p
mainakbhattacharya20
 
Transform Your ITIL® 4 & ITSM Strategy with AI in 2025.pdf
PDCA Consulting
 
Building Integrated photovoltaic BIPV_UPV.pdf
SandeepSingh286037
 
Enhancing emotion recognition model for a student engagement use case through...
IAESIJAI
 
project resource management chapter-09.pdf
ssuser00e6e21
 
Encapsulation_ Review paper, used for researhc scholars
gurumoop
 
KOM of Painting work and Equipment Insulation REV00 update 25-dec.pptx
muhammadmuneebnaeem7
 
Digital-Transformation-Roadmap-for-Companies.pptx
David Malick Dieng
 
Encapsulation theory and applications.pdf
gurumoop
 

Bowling Game Kata in C# Adapted

  • 1. Bowling Game KataAdapted for C# and nUnitBy Dan Stewart (with modifications by Mike Clement)dan@stewshack.commike@softwareontheside.com
  • 2. Scoring Bowling.The game consists of 10 frames as shown above. In each frame the player has two opportunities to knock down 10 pins. The score for the frame is the total number of pins knocked down, plus bonuses for strikes and spares.A spare is when the player knocks down all 10 pins in two tries. The bonus for that frame is the number of pins knocked down by the next roll. So in frame 3 above, the score is 10 (the total number knocked down) plus a bonus of 5 (the number of pins knocked down on the next roll.)A strike is when the player knocks down all 10 pins on his first try. The bonus for that frame is the value of the next two balls rolled.In the tenth frame a player who rolls a spare or strike is allowed to roll the extra balls to complete the frame. However no more than three balls can be rolled in tenth frame.
  • 3. A quick design sessionClearly we need the Game class.
  • 4. A quick design sessionA game has 10 frames.
  • 5. A quick design sessionA frame has 1 or two rolls.
  • 6. A quick design sessionThe tenth frame has two or three rolls.It is different from all the other frames.
  • 7. A quick design sessionThe score function mustinclude all the frames, and calculate all their scores.
  • 8. A quick design sessionThe score for a spare or a strike depends on the frame’s successor
  • 9. Begin.Create a class library named BowlingGameKata
  • 10. Begin.Add a test fixture named BowlingGameTests to the projectusingNUnit.Framework;namespaceBowlingGameKata{[TestFixture]publicclassBowlingGameTests{ [Test]publicvoidGutterGameTest() { } }}
  • 12. The first test.namespaceBowlingGameKata{publicclassGame { }}[Test]publicvoidGutterGameTest(){ Game g = new Game();}
  • 13. The first test.[Test]publicvoidGutterGameTest(){ Game g = new Game();}publicclassGame{}
  • 14. The first test.[Test]publicvoidGutterGameTest(){ Gameg = newGame();for(int i = 0; i < 20; i++) {g.Roll(0); }}publicclassGame{}
  • 15. The first test.public class Game{ publicvoid Roll(int p){ thrownewSystem.NotImplementedException();}}[Test]publicvoidGutterGameTest(){ Gameg = newGame();for(int i = 0; i < 20; i++) {g.Roll(0); }}
  • 16. The first test.[Test]publicvoidGutterGameTest(){ Gameg = newGame();for(int i = 0; i < 20; i++){g.Roll(0);}Assert.That(g.Score(), Is.EqualTo(0));}public class Game{ publicvoid Roll(int p){ thrownewSystem.NotImplementedException(); }}
  • 17. The first test.[Test]publicvoidGutterGameTest(){ Gameg = newGame();for(int i = 0; i < 20; i++){g.Roll(0);}Assert.That(g.Score(), Is.EqualTo(0));}publicclassGame{ publicvoid Roll(int p) {thrownew System.NotImplementedException(); }publicobject Score(){thrownew System.NotImplementedException(); }}Failed GutterGameTestthrew exception
  • 18. The first test.[Test]publicvoidGutterGameTest(){ Gameg = newGame();for(int i = 0; i < 20; i++) {g.Roll(0); }Assert.That(g.Score(), Is.EqualTo(0));}publicclassGame{privateint score;publicvoid Roll(int pins) { }publicintScore() {returnscore; }}
  • 19. The Second test.[Test]publicvoidAllOnesTest(){ Gameg = newGame(); for(int i = 0; i < 20; i++) {g.Roll(1); }Assert.That(g.Score(), Is.EqualTo(20));}publicclassGame{privateint score;publicvoid Roll(int pins) { }publicintScore() {returnscore; }}
  • 20. The Second test.- Game creation is duplicated- roll loop is duplicated[Test]publicvoidAllOnesTest(){ Gameg = newGame(); for(int i = 0; i < 20; i++) {g.Roll(1); }Assert.That(g.Score(), Is.EqualTo(20));}publicclassGame{privateint score;publicvoid Roll(int pins) { }publicintScore() {returnscore; }}
  • 21. The Second test.- Game creation is duplicated- roll loop is duplicated[Test]publicvoidAllOnesTest(){ Gameg = newGame(); for(int i = 0; i < 20; i++) {g.Roll(1); }Assert.That(g.Score(), Is.EqualTo(20));}publicclassGame{privateint score;publicvoid Roll(int pins){}publicintScore(){return score; }}Assert.AreEqual failed. Expected:<20>. Actual:<0>.
  • 22. The Second test.- roll loop is duplicatedprivateGame g;[SetUp]publicvoidSetup() { g = newGame();}[Test]publicvoidGutterGameTest(){for(int i = 0; i < 20; i++) {g.Roll(0); }Assert.That(g.Score(), Is.EqualTo(0));}[Test]publicvoidAllOnesTest(){for(int i = 0; i < 20; i++) {g.Roll(1); }Assert.That(g.Score(), Is.EqualTo(20));}publicclassGame{privateint score;publicvoid Roll(int pins){score += pins;}publicint Score(){return score; }}
  • 23. The Second test.- roll loop is duplicated[Test]publicvoidGutterGameTest(){int rolls = 20;intpins = 0;for(int i = 0; i < rolls; i++){g.Roll(pins);}Assert.That(g.Score(), Is.EqualTo(0));}publicclassGame{privateint score;publicvoid Roll(int pins){score += pins;}publicint Score(){return score; }}
  • 24. The Second test.- roll loop is duplicated[Test]publicvoidGutterGameTest(){int rolls = 20;intpins = 0;RollMany(rolls, pins);for(int i = 0; i < rolls; i++) {g.Roll(pins); }Assert.That(g.Score(), Is.EqualTo(0));}publicclassGame{privateint score;publicvoid Roll(int pins){score += pins;}publicint Score(){return score; }}
  • 25. The Second test.publicclassGame{privateint score;publicvoid Roll(int pins){score += pins;}publicint Score(){return score; }}[Test]publicvoidGutterGameTest(){RollMany(20, 0);Assert.That(g.Score(), Is.EqualTo(0));}privatevoidRollMany(introlls, int pins){ for(int i = 0; i < rolls; i++) {g.Roll(pins); }}
  • 26. The Second test.[Test]publicvoidGutterGameTest(){RollMany(20, 0);Assert.That(g.Score(), Is.EqualTo(0));}[Test]publicvoidAllOnesTest(){RollMany(20, 1);Assert.That(g.Score(), Is.EqualTo(20));}privatevoidRollMany(int rolls, int pins){ for(int i = 0; i < rolls; i++) {g.Roll(pins); }}publicclassGame{privateint score;publicvoid Roll(int pins){score += pins;}publicint Score(){return score; }}
  • 27. The Third test.- ugly comment in test.[Test]publicvoidOneSpareTest(){g.Roll(5);g.Roll(5); // spareg.Roll(3);RollMany(17, 0);Assert.That(g.Score(), Is.EqualTo(16));}publicclassGame{privateint score;publicvoid Roll(int pins){score += pins;}publicint Score(){return score; }}Failed Assert.AreEqual Expected:<16>. Actual:<13>
  • 28. The Third test.- ugly comment in test.publicclassGame{privateint score;publicvoid Roll(int pins){score += pins;}publicint Score(){return score; }}[Test]publicvoidOneSpareTest(){g.Roll(5);g.Roll(5); // spareg.Roll(3);RollMany(17, 0);Assert.That(g.Score(), Is.EqualTo(16));}tempted to use flag to remember previous roll. So design must be wrong.Failed Assert.AreEqual Expected:<16>. Actual:<13>
  • 29. The Third test.- ugly comment in test.Roll() calculates score, but name does not imply that.publicclassGame{privateint score;publicvoid Roll(int pins){score += pins;}publicint Score(){return score; }}[Test]publicvoidOneSpareTest(){g.Roll(5);g.Roll(5); // spareg.Roll(3);RollMany(17, 0);Assert.AreEqual(16, g.Score);}Score() does not calculate score, but name implies that it does.Design is wrong. Responsibilities are misplaced.Failed Assert.AreEqual Expected:<16>. Actual:<13>
  • 30. The Third test.- ugly comment in test.[Test]publicvoidOneSpareTest(){g.Roll(5);g.Roll(5); // spareg.Roll(3);RollMany(17, 0);//Assert.That(g.Score(), Is.EqualTo(16));}publicclassGame{privateint score;publicvoid Roll(int pins){score += pins;}publicint Score(){return score; }}
  • 31. The Third test.- ugly comment in test.publicclassGame{privateint score;privateint[] rolls = newint[21];privateint currentRoll;publicvoid Roll(int pins){rolls[currentRoll++] = pins;score += pins;}publicint Score(){return score; }}[Test]publicvoidOneSpareTest(){g.Roll(5);g.Roll(5); // spareg.Roll(3);RollMany(17, 0);//Assert.That(g.Score(), Is.EqualTo(16));}
  • 32. The Third test.- ugly comment in test.publicclassGame{ privateint[] rolls = newint[21];privateint currentRoll;publicvoid Roll(int pins){rolls[currentRoll++] = pins;}publicint Score(){int score = 0;for (int i = 0; i < rolls.Length; i++){score += rolls[i];}return score; }}[Test]publicvoidOneSpareTest(){g.Roll(5);g.Roll(5); // spareg.Roll(3);RollMany(17, 0);//Assert.That(g.Score(), Is.EqualTo(16));}
  • 33. The Third test.- ugly comment in test.[Test]publicvoidOneSpareTest(){g.Roll(5);g.Roll(5); // spareg.Roll(3);RollMany(17, 0);Assert.That(g.Score(), Is.EqualTo(16));}publicclassGame{ privateint[] rolls = newint[21];privateint currentRoll;publicvoid Roll(int pins){rolls[currentRoll++] = pins;}publicint Score(){int score = 0;for (int i = 0; i < rolls.Length; i++){score += rolls[i];}return score; }}Failed Assert.AreEqual Expected:<16>. Actual:<13>
  • 34. The Third test.- ugly comment in test.publicint Score(){int score = 0;for (int i = 0; i < rolls.Length; i++) { // spare if(rolls[i] + rolls[i+1] == 10){score += ...}score += rolls[i];}return score;}[Test]publicvoidOneSpareTest(){g.Roll(5);g.Roll(5); // spareg.Roll(3);RollMany(17, 0);Assert.That(g.Score(), Is.EqualTo(16));}This isn’t going to work because i might not refer to the first ball of the frame.Design is still wrong.Need to walk through array two balls (one frame) at a time.Failed Assert.AreEqual Expected:<16>. Actual:<13>
  • 35. The Third test.- ugly comment in test.publicclassGame{ privateint[] rolls = newint[21];privateint currentRoll;publicvoid Roll(int pins){rolls[currentRoll++] = pins;}publicint Score(){int score = 0;for (int i = 0; i < rolls.Length; i++){score += rolls[i];}return score; }}[Test]publicvoidOneSpareTest(){g.Roll(5);g.Roll(5); // spareg.Roll(3);RollMany(17, 0);//Assert.That(g.Score(), Is.EqualTo(16));Assert.Inconclusive();}
  • 36. The Third test.- ugly comment in test.publicint Score(){int score = 0;inti = 0;for (int frame = 0; frame < 10; frame++){ score += rolls[i] + rolls[i + 1]; i+= 2;}returnscore;}[Test]publicvoidOneSpareTest(){g.Roll(5);g.Roll(5); // spareg.Roll(3);RollMany(17, 0);//Assert.That(g.Score(), Is.EqualTo(16));Assert.Inconclusive();}
  • 37. The Third test.- ugly comment in test.[Test]publicvoidOneSpareTest(){g.Roll(5);g.Roll(5); // spareg.Roll(3);RollMany(17, 0);Assert.That(g.Score(), Is.EqualTo(16));}publicint Score(){int score = 0;int i = 0;for (int frame = 0; frame < 10; frame++) { score += rolls[i] + rolls[i + 1]; i += 2; }return score;}Failed Assert.AreEqual Expected:<16>. Actual:<13>
  • 38. The Third test.- ugly comment in test.publicint Score(){int score = 0;inti = 0;for (int frame = 0; frame < 10; frame++){// spareif (rolls[i] + rolls[i + 1] == 10){score += 10 + rolls[i + 2];i += 2;}else{ score += rolls[i] + rolls[i + 1]; i += 2; } }return score;}[Test]publicvoidOneSpareTest(){g.Roll(5);g.Roll(5); // spareg.Roll(3);RollMany(17, 0);Assert.That(g.Score(), Is.EqualTo(16));}
  • 39. The Third test.ugly comment in test.
  • 40. ugly comment in conditional.publicint Score(){int score = 0;int roll = 0;for (int frame = 0; frame < 10; frame++){// spareif (rolls[roll] + rolls[roll + 1] == 10){score += 10 + rolls[roll + 2];roll += 2;}else{ score += rolls[roll] + rolls[roll + 1]; roll += 2; } }return score;}[Test]publicvoidOneSpareTest(){g.Roll(5);g.Roll(5); // spareg.Roll(3);RollMany(17, 0);Assert.That(g.Score(), Is.EqualTo(16));}
  • 41. The Third test.- ugly comment in test.publicint Score(){int score = 0;int roll = 0;for (int frame = 0; frame < 10; frame++){ if(IsSpare(roll)) { score += 10 + rolls[roll + 2]; roll += 2; }else { score += rolls[roll] + rolls[roll + 1]; roll += 2;} }returnscore;}privateboolIsSpare(int roll){ returnrolls[roll] + rolls[roll + 1] == 10;}[Test]publicvoidOneSpareTest(){g.Roll(5);g.Roll(5); // spareg.Roll(3);RollMany(17, 0);Assert.That(g.Score(), Is.EqualTo(16));}
  • 42. The Third test.[Test]publicvoidOneSpareTest(){RollSpare();g.Roll(3);RollMany(17, 0);Assert.That(g.Score(), Is.EqualTo(16));}privatevoidRollSpare(){g.Roll(5);g.Roll(5);}publicint Score(){int score = 0;int roll = 0;for (int frame = 0; frame < 10; frame++){ if(IsSpare(roll)) { score += 10 + rolls[roll + 2]; roll += 2; }else { score += rolls[roll] + rolls[roll + 1]; roll += 2; }}returnscore;}privateboolIsSpare(int roll){ returnrolls[roll] + rolls[roll + 1] == 10;}
  • 43. The Fourth test.- ugly comment in OneStrikeTest.[Test]publicvoidOneStrikeTest(){g.Roll(10); // strikeg.Roll(3);g.Roll(4);RollMany(16, 0);Assert.That(g.Score(), Is.EqualTo(24));}publicint Score(){int score = 0;int roll = 0;for (int frame = 0; frame < 10; frame++){ if(IsSpare(roll)){ score += 10 + rolls[roll + 2];}else{ score += rolls[roll] + rolls[roll + 1];}roll += 2;}returnscore;}privatebool IsSpare(int roll){ returnrolls[roll] + rolls[roll + 1] == 10;}Failed Assert.AreEqual Expected:<24>. Actual:<17>
  • 44. The Fourth test.ugly comment in OneStrikeTest.
  • 45. ugly comment in conditional.
  • 46. ugly expressions.publicint Score(){ intscore = 0;introll = 0;for(int frame = 0; frame < 10; frame++){if(rolls[roll] == 10) // strike{ score += 10 + rolls[roll + 1] + rolls[roll + 2];roll++;}elseif (IsSpare(roll)){ score += 10 + rolls[roll + 2];roll += 2;}else{ score += rolls[roll] + rolls[roll + 1];roll += 2;} } returnscore;}[Test]publicvoidOneStrikeTest(){g.Roll(10); // strikeg.Roll(3);g.Roll(4);RollMany(16, 0);Assert.That(g.Score(), Is.EqualTo(24));}
  • 47. The Fourth test.ugly comment in OneStrikeTest.
  • 48. ugly comment in conditional.publicint Score(){ intscore = 0;introll = 0;for(int frame = 0; frame < 10; frame++){if(rolls[roll] == 10) // strike{ score += 10 + StrikeBonus(roll);roll++;}elseif (IsSpare(roll)){ score += 10 + SpareBonus(roll);roll += 2;}else{ score += SumOfBallsInFrame(roll);roll += 2;}}returnscore;}privateint SumOfBallsInFrame(int roll){ returnrolls[roll] + rolls[roll + 1];}privateint SpareBonus(int roll){ returnrolls[roll + 2];}privateint StrikeBonus(int roll){ returnrolls[roll + 1] + rolls[roll + 2];}[Test]publicvoidOneStrikeTest(){g.Roll(10); // strikeg.Roll(3);g.Roll(4);RollMany(16, 0);Assert.That(g.Score(), Is.EqualTo(24));}
  • 49. The Fourth test.- ugly comment in OneStrikeTest.publicint Score(){int score = 0;int roll = 0;for (int frame = 0; frame < 10; frame++){if(IsStrike(roll)){ score += 10 + StrikeBonus(roll);roll++;}elseif (IsSpare(roll)){score += 10+ SpareBonus(roll);roll += 2;}else{ score += SumOfBallsInFrame(roll);roll += 2;}}returnscore;}privatebool IsStrike(int roll){ returnrolls[roll] == 10;}[Test]publicvoidOneStrikeTest(){g.Roll(10); // strikeg.Roll(3);g.Roll(4);RollMany(16, 0);Assert.That(g.Score(), Is.EqualTo(24));}
  • 50. The Fourth test.[Test]publicvoidOneStrikeTest(){RollStrike();g.Roll(3);g.Roll(4);RollMany(16, 0);Assert.That(g.Score(), Is.EqualTo(24));}privatevoidRollStrike(){g.Roll(10);}publicint Score(){int score = 0;int roll = 0;for (int frame = 0; frame < 10; frame++){if(IsStrike(roll)){ score += 10 + StrikeBonus(roll);roll++;}elseif (IsSpare(roll)){score += 10+ SpareBonus(roll);roll += 2;}else{ score += SumOfBallsInFrame(roll);roll += 2;}}returnscore;}
  • 51. The Fifth test.[Test]publicvoidPerfectGameTest(){RollMany(12, 10);Assert.That(g.Score(), Is.EqualTo(300));}publicint Score(){int score = 0;int roll = 0;for (int frame = 0; frame < 10; frame++){if(IsStrike(roll)){ score += 10 + StrikeBonus(roll);roll++;}elseif (IsSpare(roll)){score += 10+ SpareBonus(roll);roll += 2;}else{ score += SumOfBallsInFrame(roll);roll += 2;}}returnscore;}

Editor's Notes

  • #50: I would like to thank Uncle Bob Martin and Object Mentor for the bowling game kata.