diff options
| author | mokha <mokha@cisco.com> | 2019-05-13 21:12:50 -0600 |
|---|---|---|
| committer | mokha <mokha@cisco.com> | 2019-05-13 21:12:50 -0600 |
| commit | 2b469c027a59b1ba4125e2d7ab6a970e3deaa186 (patch) | |
| tree | 0dc5f40c1afc1b5aaa95f7056f40c5d074b4f89f /src | |
| parent | c4001d1614f80ce034a2c2cb49aa1e4b9bf19880 (diff) | |
update discussion
Diffstat (limited to 'src')
| -rw-r--r-- | src/Q10/README.md | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/src/Q10/README.md b/src/Q10/README.md index 8c7175f..112696a 100644 --- a/src/Q10/README.md +++ b/src/Q10/README.md @@ -164,3 +164,7 @@ Tests run: 52, Failures: 0, Errors: 0, Skipped: 0 Test cases are defined in `TaxReturnTest.java`. 5. Discussion: + +There is a lot of conditional logic in `getTax()`. I considered +[Refactor conditional with polymorphism](https://refactoring.com/catalog/replaceConditionalWithPolymorphism.html) +but decided not to make this change. |