Skip to content

Commit

Permalink
Make diagrams smaller
Browse files Browse the repository at this point in the history
  • Loading branch information
damithc committed Sep 14, 2016
1 parent ebb641c commit 9c2f38b
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions doc/LearningOutcomes.md
Original file line number Diff line number Diff line change
Expand Up @@ -100,7 +100,7 @@ Covered by `[LO-Polymorphism]`
* Note how `Logic` class depends on the `StorageFile` class. This is a violation of DIP.
* Modify the implementation as follows so that both `Logic` and `StorageFile` now depend on the abstraction
`Storage`. (Note: The term *abstraction* here is referring to the conception of abstracting, not to the fact that `Storage` class is `abastract`)<br>
<img src="images/LogicStroageFileDIP.png" width="400">
<img src="images/LogicStroageFileDIP.png" width="300">
* Where else in the code do you notice the application of DIP?

------------------------------------------------------------------------------------------------------
Expand All @@ -115,7 +115,7 @@ getting the `StorageFile` class involved? That is a situation where we can use *

* Change the implementation as follows so that we can inject a `StorageStub` when testing the `Logic`
class. <br>
<img src="images/DependencyInjection.png" width="800">
<img src="images/DependencyInjection.png" width="600">

> If you did the exercise in [`LO-DIP`](#apply-dependency-inversion-principle)
already but those changes are in a different branch, you may be able to reuse some of those commits
Expand Down

0 comments on commit 9c2f38b

Please sign in to comment.