Download as PDF, PPTX
![public class RecentlyUsedList
{
private List<string> items;
public RecentlyUsedList()
{
items = new List<string>();
}
public void Add(string newItem)
{
if (items.Contains(newItem))
{
int position = items.IndexOf(newItem);
string existingItem = items[position];
items.RemoveAt(position);
items.Insert(0, existingItem);
}
else
{
items.Insert(0, newItem);
}
}
public int Count
{
get
{
int size = items.Count;
return size;
}
}
public string this[int index]
{
get
{
int position = 0;
foreach (string item in items)
{
if (position == index)
return item;
++position;
}
throw new ArgumentOutOfRangeException();
}
}
}](https://image.slidesharecdn.com/sevenineffectivecodinghabitsofmanyprogrammers-150302052912-conversion-gate02/75/Seven-Ineffective-Coding-Habits-of-Many-Programmers-8-2048.jpg)
![public class RecentlyUsedList
{
private List<string> items;
public RecentlyUsedList()
{
items = new List<string>();
}
public void Add(string newItem)
{
if (items.Contains(newItem))
{
int position = items.IndexOf(newItem);
string existingItem = list[position];
items.RemoveAt(position);
items.Insert(0, existingItem);
}
else
{
items.Insert(0, newItem);
}
}
public int Count
{
get
{
int size = items.Count;
return size;
}
}
public string this[int index]
{
get
{
int position = 0;
foreach (string value in items)
{
if (position == index)
return value;
++position;
}
throw new ArgumentOutOfRangeException();
}
}
}
public class RecentlyUsedList
{
private List<string> items = new List<string>();
public void Add(string newItem)
{
items.Remove(newItem);
items.Add(newItem);
}
public int Count
{
get
{
return items.Count;
}
}
public string this[int index]
{
get
{
return items[Count - index - 1];
}
}
}](https://image.slidesharecdn.com/sevenineffectivecodinghabitsofmanyprogrammers-150302052912-conversion-gate02/75/Seven-Ineffective-Coding-Habits-of-Many-Programmers-9-2048.jpg)
![public class RecentlyUsedList
{
private IList<string> items = new List<string>();
public int Count
{
get
{
return items.Count;
}
}
public string this[int index]
{
get
{
return items[index];
}
}
public void Add(string newItem)
{
if(newItem == null)
throw new ArgumentNullException();
items.Remove(newItem);
items.Insert(0, newItem);
}
...
}](https://image.slidesharecdn.com/sevenineffectivecodinghabitsofmanyprogrammers-150302052912-conversion-gate02/75/Seven-Ineffective-Coding-Habits-of-Many-Programmers-62-2048.jpg)
![public class RecentlyUsedList
{
private IList<string> items = new List<string>();
public int Count
{
get
{
return items.Count;
}
}
public string this[int index]
{
get
{
return items[Count – index - 1];
}
}
public void Add(string newItem)
{
if(newItem == null)
throw new ArgumentNullException();
items.Remove(newItem);
items.Add(newItem);
}
...
}](https://image.slidesharecdn.com/sevenineffectivecodinghabitsofmanyprogrammers-150302052912-conversion-gate02/75/Seven-Ineffective-Coding-Habits-of-Many-Programmers-63-2048.jpg)
![public class RecentlyUsedList
{
...
public RecentlyUsedList() ...
public int Count
{
get...
}
public string this[int index]
{
get...
}
public void Add(string newItem) ...
...
}](https://image.slidesharecdn.com/sevenineffectivecodinghabitsofmanyprogrammers-150302052912-conversion-gate02/75/Seven-Ineffective-Coding-Habits-of-Many-Programmers-77-2048.jpg)
![[TestFixture]
public class RecentlyUsedListTests
{
[Test]
public void TestConstructor() ...
[Test]
public void TestCountGet() ...
[Test]
public void TestIndexerGet() ...
[Test]
public void TestAdd() ...
...
}](https://image.slidesharecdn.com/sevenineffectivecodinghabitsofmanyprogrammers-150302052912-conversion-gate02/75/Seven-Ineffective-Coding-Habits-of-Many-Programmers-78-2048.jpg)
![namespace RecentlyUsedList_spec
{
[TestFixture]
public class A_new_list
{
[Test] public void Is_empty()
}
[TestFixture]
public class An_empty_list
{
[Test] public void Retains_a_single_addition()
[Test] public void Retains_unique_additions_in_stack_order()
}
[TestFixture]
public class A_non_empty_list
{
[Test] public void Is_unchanged_when_head_item_is_readded()
[Test] public void Moves_non_head_item_to_head_when_it_is_readded()
}
[TestFixture]
public class Any_list_rejects
{
[Test] public void Addition_of_null_items()
[Test] public void Indexing_past_its_end()
[Test] public void Negative_indexing()
}
}](https://image.slidesharecdn.com/sevenineffectivecodinghabitsofmanyprogrammers-150302052912-conversion-gate02/75/Seven-Ineffective-Coding-Habits-of-Many-Programmers-80-2048.jpg)
![namespace RecentlyUsedList_spec
{
[TestFixture]
public class A_new_list
{
[Test] public void Is_empty()
}
[TestFixture]
public class An_empty_list
{
[Test] public void Retains_a_single_addition()
[Test] public void Retains_unique_additions_in_stack_order()
}
[TestFixture]
public class A_non_empty_list
{
[Test] public void Is_unchanged_when_head_item_is_readded()
[Test] public void Moves_non_head_item_to_head_when_it_is_readded()
}
[TestFixture]
public class Any_list_rejects
{
[Test] public void Addition_of_null_items()
[Test] public void Indexing_past_its_end()
[Test] public void Negative_indexing()
}
}](https://image.slidesharecdn.com/sevenineffectivecodinghabitsofmanyprogrammers-150302052912-conversion-gate02/75/Seven-Ineffective-Coding-Habits-of-Many-Programmers-81-2048.jpg)
![public class RecentlyUsedList
{
private List<string> items;
public RecentlyUsedList()
{
items = new List<string>();
}
public void Add(string newItem)
{
if (items.Contains(newItem))
{
int position = items.IndexOf(newItem);
string existingItem = items[position];
items.RemoveAt(position);
items.Insert(0, existingItem);
}
else
{
items.Insert(0, newItem);
}
}
public int Count
{
get
{
int size = items.Count;
return size;
}
}
public string this[int index]
{
get
{
int position = 0;
foreach (string item in items)
{
if (position == index)
return item;
++position;
}
throw new ArgumentOutOfRangeException();
}
}
}](https://crownmelresort.com/image.slidesharecdn.com/sevenineffectivecodinghabitsofmanyprogrammers-150302052912-conversion-gate02/75/Seven-Ineffective-Coding-Habits-of-Many-Programmers-8-2048.jpg)
![public class RecentlyUsedList
{
private List<string> items;
public RecentlyUsedList()
{
items = new List<string>();
}
public void Add(string newItem)
{
if (items.Contains(newItem))
{
int position = items.IndexOf(newItem);
string existingItem = list[position];
items.RemoveAt(position);
items.Insert(0, existingItem);
}
else
{
items.Insert(0, newItem);
}
}
public int Count
{
get
{
int size = items.Count;
return size;
}
}
public string this[int index]
{
get
{
int position = 0;
foreach (string value in items)
{
if (position == index)
return value;
++position;
}
throw new ArgumentOutOfRangeException();
}
}
}
public class RecentlyUsedList
{
private List<string> items = new List<string>();
public void Add(string newItem)
{
items.Remove(newItem);
items.Add(newItem);
}
public int Count
{
get
{
return items.Count;
}
}
public string this[int index]
{
get
{
return items[Count - index - 1];
}
}
}](https://crownmelresort.com/image.slidesharecdn.com/sevenineffectivecodinghabitsofmanyprogrammers-150302052912-conversion-gate02/75/Seven-Ineffective-Coding-Habits-of-Many-Programmers-9-2048.jpg)
![public class RecentlyUsedList
{
private IList<string> items = new List<string>();
public int Count
{
get
{
return items.Count;
}
}
public string this[int index]
{
get
{
return items[index];
}
}
public void Add(string newItem)
{
if(newItem == null)
throw new ArgumentNullException();
items.Remove(newItem);
items.Insert(0, newItem);
}
...
}](https://crownmelresort.com/image.slidesharecdn.com/sevenineffectivecodinghabitsofmanyprogrammers-150302052912-conversion-gate02/75/Seven-Ineffective-Coding-Habits-of-Many-Programmers-62-2048.jpg)
![public class RecentlyUsedList
{
private IList<string> items = new List<string>();
public int Count
{
get
{
return items.Count;
}
}
public string this[int index]
{
get
{
return items[Count – index - 1];
}
}
public void Add(string newItem)
{
if(newItem == null)
throw new ArgumentNullException();
items.Remove(newItem);
items.Add(newItem);
}
...
}](https://crownmelresort.com/image.slidesharecdn.com/sevenineffectivecodinghabitsofmanyprogrammers-150302052912-conversion-gate02/75/Seven-Ineffective-Coding-Habits-of-Many-Programmers-63-2048.jpg)
![public class RecentlyUsedList
{
...
public RecentlyUsedList() ...
public int Count
{
get...
}
public string this[int index]
{
get...
}
public void Add(string newItem) ...
...
}](https://crownmelresort.com/image.slidesharecdn.com/sevenineffectivecodinghabitsofmanyprogrammers-150302052912-conversion-gate02/75/Seven-Ineffective-Coding-Habits-of-Many-Programmers-77-2048.jpg)
![[TestFixture]
public class RecentlyUsedListTests
{
[Test]
public void TestConstructor() ...
[Test]
public void TestCountGet() ...
[Test]
public void TestIndexerGet() ...
[Test]
public void TestAdd() ...
...
}](https://crownmelresort.com/image.slidesharecdn.com/sevenineffectivecodinghabitsofmanyprogrammers-150302052912-conversion-gate02/75/Seven-Ineffective-Coding-Habits-of-Many-Programmers-78-2048.jpg)
![namespace RecentlyUsedList_spec
{
[TestFixture]
public class A_new_list
{
[Test] public void Is_empty()
}
[TestFixture]
public class An_empty_list
{
[Test] public void Retains_a_single_addition()
[Test] public void Retains_unique_additions_in_stack_order()
}
[TestFixture]
public class A_non_empty_list
{
[Test] public void Is_unchanged_when_head_item_is_readded()
[Test] public void Moves_non_head_item_to_head_when_it_is_readded()
}
[TestFixture]
public class Any_list_rejects
{
[Test] public void Addition_of_null_items()
[Test] public void Indexing_past_its_end()
[Test] public void Negative_indexing()
}
}](https://crownmelresort.com/image.slidesharecdn.com/sevenineffectivecodinghabitsofmanyprogrammers-150302052912-conversion-gate02/75/Seven-Ineffective-Coding-Habits-of-Many-Programmers-80-2048.jpg)
![namespace RecentlyUsedList_spec
{
[TestFixture]
public class A_new_list
{
[Test] public void Is_empty()
}
[TestFixture]
public class An_empty_list
{
[Test] public void Retains_a_single_addition()
[Test] public void Retains_unique_additions_in_stack_order()
}
[TestFixture]
public class A_non_empty_list
{
[Test] public void Is_unchanged_when_head_item_is_readded()
[Test] public void Moves_non_head_item_to_head_when_it_is_readded()
}
[TestFixture]
public class Any_list_rejects
{
[Test] public void Addition_of_null_items()
[Test] public void Indexing_past_its_end()
[Test] public void Negative_indexing()
}
}](https://crownmelresort.com/image.slidesharecdn.com/sevenineffectivecodinghabitsofmanyprogrammers-150302052912-conversion-gate02/75/Seven-Ineffective-Coding-Habits-of-Many-Programmers-81-2048.jpg)
Uploaded byKevlin Henney
Seven Ineffective Coding Habits of Many Programmers
The document discusses seven ineffective coding habits: noisy code with unnecessary comments, unsustainable spacing that reduces readability, underabstraction, lego naming conventions that are difficult to understand, unencapsulated state, getters and setters that expose implementation details, and uncohesive tests that do not clearly specify requirements. It provides examples of clearer, more effective alternatives for each habit.
More Related Content
Similar to Seven Ineffective Coding Habits of Many Programmers
![ANPARA THERMAL POWER STATION[1] sangam.pdf](https://cdn.slidesharecdn.com/ss_thumbnails/anparathermalpowerstation1sangam-251121115219-9261cde4-thumbnail.jpg?width=640&height=640&fit=bounds)
Seven Ineffective Coding Habits of Many Programmers
- 1. Seven Ineffective Coding Habitsof Many Programmers @KevlinHenney
- 2. It turns outthat style matters in programming for the same reason that it matters in writing. It makes for better reading. Douglas Crockford JavaScript: The Good Parts
- 4.
- 5. Signal-to-noise ratio (oftenabbreviated SNR or S/N) is a measure used in science and engineering that compares the level of a desired signal to the level of background noise. Signal-to-noise ratio is sometimes used informally to refer to the ratio of useful information to false or irrelevant data in a conversation or exchange. http://en.wikipedia.org/wiki/Signal_to_noise_ratio
- 6. To be, ornot to be: that is the question: Whether 'tis nobler in the mind to suffer The slings and arrows of outrageous fortune, Or to take arms against a sea of troubles, And by opposing end them? William Shakespeare Hamlet
- 7. Continuing existence orcessation of existence: those are the scenarios. Is it more empowering mentally to work towards an accommodation of the downsizings and negative outcomes of adversarial circumstance, or would it be a greater enhancement of the bottom line to move forwards to a challenge to our current difficulties, and, by making a commitment to opposition, to effect their demise? Tom Burton Long Words Bother Me
- 8. public class RecentlyUsedList { privateList<string> items; public RecentlyUsedList() { items = new List<string>(); } public void Add(string newItem) { if (items.Contains(newItem)) { int position = items.IndexOf(newItem); string existingItem = items[position]; items.RemoveAt(position); items.Insert(0, existingItem); } else { items.Insert(0, newItem); } } public int Count { get { int size = items.Count; return size; } } public string this[int index] { get { int position = 0; foreach (string item in items) { if (position == index) return item; ++position; } throw new ArgumentOutOfRangeException(); } } }
- 9. public class RecentlyUsedList { privateList<string> items; public RecentlyUsedList() { items = new List<string>(); } public void Add(string newItem) { if (items.Contains(newItem)) { int position = items.IndexOf(newItem); string existingItem = list[position]; items.RemoveAt(position); items.Insert(0, existingItem); } else { items.Insert(0, newItem); } } public int Count { get { int size = items.Count; return size; } } public string this[int index] { get { int position = 0; foreach (string value in items) { if (position == index) return value; ++position; } throw new ArgumentOutOfRangeException(); } } } public class RecentlyUsedList { private List<string> items = new List<string>(); public void Add(string newItem) { items.Remove(newItem); items.Add(newItem); } public int Count { get { return items.Count; } } public string this[int index] { get { return items[Count - index - 1]; } } }
- 12. Comments A delicate matter,requiring taste and judgement. I tend to err on the side of eliminating comments, for several reasons. First, if the code is clear, and uses good type names and variable names, it should explain itself. Second, comments aren't checked by the compiler, so there is no guarantee they're right, especially after the code is modified. A misleading comment can be very confusing. Third, the issue of typography: comments clutter code. Rob Pike, "Notes on Programming in C"
- 13. There is afamously bad comment style: i=i+1; /* Add one to i */ and there are worse ways to do it: /********************************** * * * Add one to i * * * **********************************/ i=i+1; Don't laugh now, wait until you see it in real life. Rob Pike, "Notes on Programming in C"
- 14. A common fallacyis to assume authors of incomprehensible code will somehow be able to express themselves lucidly and clearly in comments. Kevlin Henney https://twitter.com/KevlinHenney/status/381021802941906944
- 16.
- 17. To be, ornot to be: that is the question: Whether 'tis nobler in the mind to suffer The slings and arrows of outrageous fortune, Or to take arms against a sea of troubles, And by opposing end them? William Shakespeare Hamlet
- 18. Continuing existence orcessation of existence: those are the scenarios. Is it more empowering mentally to work towards an accommodation of the downsizings and negative outcomes of adversarial circumstance, or would it be a greater enhancement of the bottom line to move forwards to a challenge to our current difficulties, and, by making a commitment to opposition, to effect their demise? Tom Burton Long Words Bother Me
- 19. Continuing existence orcessation of existence: those are the more empowe to work towa accommodati downsizings outcomes of circumstance a greater enh the bottom li forwards to a our current d by making a opposition, t demise?
- 20. How many programmerslay out their code Column 80
- 21.
- 22. To answer thequestion "What is clean design?" most succinctly: a clean design is one that supports visual thinking so people can meet their informational needs with a minimum of conscious effort. Daniel Higginbotham ∙ "Clean Up Your Mess — A Guide to Visual Design for Everyone" ∙ http://www.visualmess.com/
- 23. You convey informationby the way you arrange a design's elements in relation to each other. This information is understood immediately, if not consciously, by the people viewing your designs. Daniel Higginbotham ∙ "Clean Up Your Mess — A Guide to Visual Design for Everyone" ∙ http://www.visualmess.com/
- 24. This is greatif the visual relationships are obvious and accurate, but if they're not, your audience is going to get confused. They'll have to examine your work carefully, going back and forth between the different parts to make sure they understand. Daniel Higginbotham ∙ "Clean Up Your Mess — A Guide to Visual Design for Everyone" ∙ http://www.visualmess.com/
- 25. public int howNotToLayoutAMethodHeader(intfirstArgument, String secondArgument) public int ensureArgumentsAreAlignedLikeThis( int firstArgument, String secondArgument) public int orEnsureArgumentsAreGroupedLikeThis( int firstArgument, String secondArgument) public int butNotAlignedLikeThis(int firstArgument, String secondArgument)
- 26. int doNotFormat =likeThis(someArgumentOrExpression, anotherArgumentOrExpression); int insteadFormat = somethingLikeThis( someArgumentOrExpression, anotherArgumentOrExpression); int orFormat = somethingLikeThis( someArgumentOrExpression, anotherArgumentOrExpression);
- 27. int asItIs =unstable(someArgumentOrExpression, anotherArgumentOrExpression); int butThisIs = stable( someArgumentOrExpression, anotherArgumentOrExpression); int andThisIs = stable( someArgumentOrExpression, anotherArgumentOrExpression);
- 28. public ResultType arbitraryMethodName(FirstArgumentTypefirs SecondArgumentType sec ThirdArgumentType thir LocalVariableType localVariable = method(firstArgument, secondArgument) if (localVariable.isSomething(thirdArgument, SOME_SHOUTY_CONSTANT)) { doSomethingWith(localVariable); } return localVariable.getSomething(); }
- 29. public ResultType arbitraryMethodName( FirstArgumentTypefirstArgument, SecondArgumentType secondArgument, ThirdArgumentType thirdArgument) { LocalVariableType localVariable = method(firstArgument, secondArgument); if (localVariable.isSomething( thirdArgument, SOME_SHOUTY_CONSTANT)) { doSomething(localVariable); } return localVariable.getSomething(); }
- 30. XXXXXX XXXXXXXXXX XXXXXXXXXXXXXXXXXXX XXXXXXXXXXXXXXXXXXXXXXXXXXXXXX XXXXXXXXXXXXXXXXXX XXXXXXXXXXXXXX XXXXXXXXXXXXXXXXX XXXXXXXXXXXXX XXXXXXXXXXXXXXXXX XXXXXXXXXXXXX XXXXXX XXXXXXXXXXXXX XXXXXXXXXXXXXX XX XXXXXXXXXXXXX XXXXXXXXXXX XXXXXXXXXXXXX XXXXXXXXXXXXXXXXXXXX XXXXXXXXXXX XXXXXXXXXXXXX XXXXXX XXXXXXXXXXXXX XXXXXXXXXXXX
- 31. public ResultType arbitraryMethodName( FirstArgumentTypefirstArgument, SecondArgumentType secondArgument, ThirdArgumentType thirdArgument) { LocalVariableType localVariable = method(firstArgument, secondArgument); if (localVariable.isSomething( thirdArgument, SOME_SHOUTY_CONSTANT)) { doSomething(localVariable); } return localVariable.getSomething(); }
- 32. XXXXXX XXXXXXXXXX XXXXXXXXXXXXXXXXXXX XXXXXXXXXXXXXXXXXXXXXXXXXXXXXX XXXXXXXXXXXXXXXXXX XXXXXXXXXXXXXX XXXXXXXXXXXXXXXXX XXXXXXXXXXXXX XXXXXXXXXXXXXXXXX XXXXXXXXXXXXX XXXXXX XXXXXXXXXXXXX XXXXXXXXXXXXXX XX XXXXXXXXXXXXX XXXXXXXXXXX XXXXXXXXXXXXX XXXXXXXXXXXXXXXXXXXX XXXXXXXXXXX XXXXXXXXXXXXX XXXXXX XXXXXXXXXXXXX XXXXXXXXXXXX
- 33. public ResultType arbitraryMethodName( FirstArgumentTypefirstArgument, SecondArgumentType secondArgument, ThirdArgumentType thirdArgument) { LocalVariableType localVariable = method(firstArgument, secondArgument); if (localVariable.isSomething( thirdArgument, SOME_SHOUTY_CONSTANT)) { doSomething(localVariable); } return localVariable.getSomething(); }
- 34. XXXXXX XXXXXXXXXX XXXXXXXXXXXXXXXXXXX XXXXXXXXXXXXXXXXXXXXXXXXXXXXXX XXXXXXXXXXXXXXXXXX XXXXXXXXXXXXXX XXXXXXXXXXXXXXXXX XXXXXXXXXXXXX XXXXXXXXXXXXXXXXX XXXXXXXXXXXXX XXXXXX XXXXXXXXXXXXX XXXXXXXXXXXXXX XX XXXXXXXXXXXXX XXXXXXXXXXX XXXXXXXXXXXXX XXXXXXXXXXXXXXXXXXXX XXXXXXXXXXX XXXXXXXXXXXXX XXXXXX XXXXXXXXXXXXX XXXXXXXXXXXX
- 36.
- 37. Agglutination is aprocess in linguistic morphology derivation in which complex words are formed by stringing together morphemes, each with a single grammatical or semantic meaning. Languages that use agglutination widely are called agglutinative languages. http://en.wikipedia.org/wiki/Agglutination
- 38.
- 40. public interface ConditionChecker { booleancheckCondition(); }
- 41. public interface Condition { booleanisTrue(); }
- 42. public Connection createConnection(Provider...) throwsConnectionFailureException ...
- 43. public Connection connectTo(Provider...) throwsConnectionFailure ...
- 44.
- 45.
- 46.
- 47.
- 48.
- 49. Omit needless words. WilliamStrunk and E B White The Elements of Style
- 51.
- 52.
- 53.
- 54. if (portfolioIdsByTraderId.get(trader.getId()) .containsKey(portfolio.getId())) { ... } Dan North,"Code in the Language of the Domain" 97 Things Every Programmer Should Know
- 55. if (trader.canView(portfolio)) { ... } Dan North,"Code in the Language of the Domain" 97 Things Every Programmer Should Know
- 57.
- 58. public class RecentlyUsedList { privateList<string> items = new List<string>(); public List<string> Items { get { return items; } } public void Add(string newItem) { if(newItem == null) throw new ArgumentNullException(); items.Remove(newItem); items.Insert(0, newItem); } ... }
- 59. public class RecentlyUsedList { privateList<string> items = new List<string>(); public List<string> Items { get { return items; } } public void Add(string newItem) { if(newItem == null) throw new ArgumentNullException(); items.Remove(newItem); items.Insert(0, newItem); } ... }
- 60. public class RecentlyUsedList { privateList<string> items = new List<string>(); public List<string> Items { get { return items; } } public void Add(string newItem) { if(newItem == null) throw new ArgumentNullException(); items.Remove(newItem); items.Insert(0, newItem); } ... } var list = new RecentlyUsedList(); list.Add("Hello, World!"); Console.WriteLine(list.Items.Count); list.Items.Add("Hello, World!"); Console.WriteLine(list.Items.Count); list.Items.Add(null);
- 61. Don't ever invitea vampire into your house, you silly boy. It renders you powerless.
- 62. public class RecentlyUsedList { privateIList<string> items = new List<string>(); public int Count { get { return items.Count; } } public string this[int index] { get { return items[index]; } } public void Add(string newItem) { if(newItem == null) throw new ArgumentNullException(); items.Remove(newItem); items.Insert(0, newItem); } ... }
- 63. public class RecentlyUsedList { privateIList<string> items = new List<string>(); public int Count { get { return items.Count; } } public string this[int index] { get { return items[Count – index - 1]; } } public void Add(string newItem) { if(newItem == null) throw new ArgumentNullException(); items.Remove(newItem); items.Add(newItem); } ... }
- 65.
- 70. public class Moneyimplements ... { ... public int getUnits() ... public int getHundredths() ... public Currency getCurrency() ... ... public void setUnits(int newUnits) ... public void setHundredths(int newHundredths) ... public void setCurrency(Currency newCurrency) ... ... }
- 71. public final classMoney implements ... { ... public int getUnits() ... public int getHundredths() ... public Currency getCurrency() ... ... }
- 72. public final classMoney implements ... { ... public int units() ... public int hundredths() ... public Currency currency() ... ... }
- 73. When it isnot necessary to change, it is necessary not to change. Lucius Cary
- 75.
- 76. Everybody knows thatTDD stands for Test Driven Development. However, people too often concentrate on the words "Test" and "Development" and don't consider what the word "Driven" really implies. For tests to drive development they must do more than just test that code performs its required functionality: they must clearly express that required functionality to the reader. That is, they must be clear specifications of the required functionality. Tests that are not written with their role as specifications in mind can be very confusing to read. Nat Pryce and Steve Freeman "Are Your Tests Really Driving Your Development?"
- 77. public class RecentlyUsedList { ... publicRecentlyUsedList() ... public int Count { get... } public string this[int index] { get... } public void Add(string newItem) ... ... }
- 78. [TestFixture] public class RecentlyUsedListTests { [Test] publicvoid TestConstructor() ... [Test] public void TestCountGet() ... [Test] public void TestIndexerGet() ... [Test] public void TestAdd() ... ... }
- 79.
- 80. namespace RecentlyUsedList_spec { [TestFixture] public classA_new_list { [Test] public void Is_empty() } [TestFixture] public class An_empty_list { [Test] public void Retains_a_single_addition() [Test] public void Retains_unique_additions_in_stack_order() } [TestFixture] public class A_non_empty_list { [Test] public void Is_unchanged_when_head_item_is_readded() [Test] public void Moves_non_head_item_to_head_when_it_is_readded() } [TestFixture] public class Any_list_rejects { [Test] public void Addition_of_null_items() [Test] public void Indexing_past_its_end() [Test] public void Negative_indexing() } }
- 81. namespace RecentlyUsedList_spec { [TestFixture] public classA_new_list { [Test] public void Is_empty() } [TestFixture] public class An_empty_list { [Test] public void Retains_a_single_addition() [Test] public void Retains_unique_additions_in_stack_order() } [TestFixture] public class A_non_empty_list { [Test] public void Is_unchanged_when_head_item_is_readded() [Test] public void Moves_non_head_item_to_head_when_it_is_readded() } [TestFixture] public class Any_list_rejects { [Test] public void Addition_of_null_items() [Test] public void Indexing_past_its_end() [Test] public void Negative_indexing() } }
- 82. A test caseshould be just that: it should correspond to a single case.
- 83. At some level thestyle becomes the substance.