Skip to content

Commit b5d4445

Browse files
committed
Made example App a bit easier to understand
1 parent f64ba22 commit b5d4445

File tree

2 files changed

+4
-16
lines changed

2 files changed

+4
-16
lines changed

abstract-factory/src/main/java/com/iluwatar/abstractfactory/App.java

Lines changed: 2 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -52,14 +52,6 @@ public void createKingdom(final KingdomFactory factory) {
5252
setArmy(factory.createArmy());
5353
}
5454

55-
ElfKingdomFactory getElfKingdomFactory() {
56-
return new ElfKingdomFactory();
57-
}
58-
59-
OrcKingdomFactory getOrcKingdomFactory() {
60-
return new OrcKingdomFactory();
61-
}
62-
6355
King getKing(final KingdomFactory factory) {
6456
return factory.createKing();
6557
}
@@ -107,17 +99,13 @@ public static void main(String[] args) {
10799
App app = new App();
108100

109101
System.out.println("Elf Kingdom");
110-
KingdomFactory elfKingdomFactory;
111-
elfKingdomFactory = app.getElfKingdomFactory();
112-
app.createKingdom(elfKingdomFactory);
102+
app.createKingdom(new ElfKingdomFactory());
113103
System.out.println(app.getArmy().getDescription());
114104
System.out.println(app.getCastle().getDescription());
115105
System.out.println(app.getKing().getDescription());
116106

117107
System.out.println("\nOrc Kingdom");
118-
KingdomFactory orcKingdomFactory;
119-
orcKingdomFactory = app.getOrcKingdomFactory();
120-
app.createKingdom(orcKingdomFactory);
108+
app.createKingdom(new OrcKingdomFactory());
121109
System.out.println(app.getArmy().getDescription());
122110
System.out.println(app.getCastle().getDescription());
123111
System.out.println(app.getKing().getDescription());

abstract-factory/src/test/java/com/iluwatar/abstractfactory/AbstractFactoryTest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -36,8 +36,8 @@ public class AbstractFactoryTest {
3636

3737
@Before
3838
public void setUp() {
39-
elfFactory = app.getElfKingdomFactory();
40-
orcFactory = app.getOrcKingdomFactory();
39+
elfFactory = new ElfKingdomFactory();
40+
orcFactory = new OrcKingdomFactory();
4141
}
4242

4343
@Test

0 commit comments

Comments
 (0)