Skip to content

Commit

Permalink
link to "Data Clumps" added
Browse files Browse the repository at this point in the history
  • Loading branch information
pfichtner authored Nov 6, 2024
1 parent 25bd6ab commit 3ac2681
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions _learning_hours/refactoring/parallel_change.md
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ Reference - Danilo Sato has published [an article](https://martinfowler.com/blik
### Concrete: Code Review
Ask everyone to take a look at this code: [Bingo Refactoring Kata](https://github.com/sammancoaching/Bingo-Refactoring-Kata). What code smells do they see (regardless of whether they can name them or not)? What would be a better structure?

You are hoping they will spot that there are data clumps and [Primitive Obsession](/code_smells/primitive_obsession.html) - in particular these primitives:
You are hoping they will spot that there are [Data Clumps](/code_smells/data_clumps.html) and [Primitive Obsession](/code_smells/primitive_obsession.html) - in particular these primitives:

String[][] cells
boolean[][] marked
Expand All @@ -59,4 +59,4 @@ While the first parallel change focuses on swapping the internal data structure


# Acknowlegements
This learning hour was first published elsewhere: [Parallel Change](https://github.com/atruvia/samman-coaching-website/blob/lh-additions/_learning_hours/refactoring/parallel-change.md)
This learning hour was first published elsewhere: [Parallel Change](https://github.com/atruvia/samman-coaching-website/blob/lh-additions/_learning_hours/refactoring/parallel-change.md)

0 comments on commit 3ac2681

Please sign in to comment.