Formatted all files to the same standard

abstract-factory/src/main/java/com/iluwatar/KingdomFactory.java

@@ -8,7 +8,9 @@ package com.iluwatar;
 public interface KingdomFactory {
 
 	Castle createCastle();
+
 	King createKing();
+
 	Army createArmy();
 
 }

adapter/src/main/java/com/iluwatar/GnomeEngineer.java

@@ -2,9 +2,8 @@ package com.iluwatar;
 
 /**
  * 
- * Adapter class. Adapts the interface of the device
+ * Adapter class. Adapts the interface of the device (GoblinGlider) into
- * (GoblinGlider) into Engineer interface expected 
+ * Engineer interface expected by the client (GnomeEngineeringManager).
- * by the client (GnomeEngineeringManager).
  * 
  */
 public class GnomeEngineer implements Engineer {

adapter/src/main/java/com/iluwatar/GnomeEngineeringManager.java

@@ -2,8 +2,7 @@ package com.iluwatar;
 
 /**
  * 
- * GnomeEngineering manager uses Engineer to
+ * GnomeEngineering manager uses Engineer to operate devices.
- * operate devices.
  * 
  */
 public class GnomeEngineeringManager implements Engineer {

bridge/src/main/java/com/iluwatar/App.java

@@ -10,19 +10,22 @@ package com.iluwatar;
 public class App {
 
 	public static void main(String[] args) {
-        BlindingMagicWeapon blindingMagicWeapon = new BlindingMagicWeapon(new Excalibur());
+		BlindingMagicWeapon blindingMagicWeapon = new BlindingMagicWeapon(
+				new Excalibur());
 		blindingMagicWeapon.wield();
 		blindingMagicWeapon.blind();
 		blindingMagicWeapon.swing();
 		blindingMagicWeapon.unwield();
 
-        FlyingMagicWeapon flyingMagicWeapon = new FlyingMagicWeapon(new Mjollnir());
+		FlyingMagicWeapon flyingMagicWeapon = new FlyingMagicWeapon(
+				new Mjollnir());
 		flyingMagicWeapon.wield();
 		flyingMagicWeapon.fly();
 		flyingMagicWeapon.swing();
 		flyingMagicWeapon.unwield();
 
-        SoulEatingMagicWeapon soulEatingMagicWeapon = new SoulEatingMagicWeapon(new Stormbringer());
+		SoulEatingMagicWeapon soulEatingMagicWeapon = new SoulEatingMagicWeapon(
+				new Stormbringer());
 		soulEatingMagicWeapon.wield();
 		soulEatingMagicWeapon.swing();
 		soulEatingMagicWeapon.eatSoul();

bridge/src/main/java/com/iluwatar/Excalibur.java

@@ -19,7 +19,8 @@ public class Excalibur extends BlindingMagicWeaponImp {
 
 	@Override
 	public void blindImp() {
-		System.out.println("bright light streams from Excalibur blinding the enemy");
+		System.out
+				.println("bright light streams from Excalibur blinding the enemy");
 	}
 
 }

bridge/src/main/java/com/iluwatar/Mjollnir.java

@@ -19,7 +19,8 @@ public class Mjollnir extends FlyingMagicWeaponImp {
 
 	@Override
 	public void flyImp() {
-		System.out.println("Mjollnir hits the enemy in the air and returns back to the owner's hand");
+		System.out
+				.println("Mjollnir hits the enemy in the air and returns back to the owner's hand");
 	}
 
 }

builder/src/main/java/com/iluwatar/App.java

@@ -20,23 +20,18 @@ public class App {
 	public static void main(String[] args) {
 
 		Hero mage = new HeroBuilder(Profession.MAGE, "Riobard")
-                .withHairColor(HairColor.BLACK)
+				.withHairColor(HairColor.BLACK).withWeapon(Weapon.DAGGER)
-                .withWeapon(Weapon.DAGGER)
 				.build();
 		System.out.println(mage);
 
 		Hero warrior = new HeroBuilder(Profession.WARRIOR, "Amberjill")
 				.withHairColor(HairColor.BLOND)
-                .withHairType(HairType.LONG_CURLY)
+				.withHairType(HairType.LONG_CURLY).withArmor(Armor.CHAIN_MAIL)
-                .withArmor(Armor.CHAIN_MAIL)
+				.withWeapon(Weapon.SWORD).build();
-                .withWeapon(Weapon.SWORD)
-                .build();
 		System.out.println(warrior);
 
 		Hero thief = new HeroBuilder(Profession.THIEF, "Desmond")
-                .withHairType(HairType.BALD)
+				.withHairType(HairType.BALD).withWeapon(Weapon.BOW).build();
-                .withWeapon(Weapon.BOW)
-                .build();
 		System.out.println(thief);
 
 	}

builder/src/main/java/com/iluwatar/Armor.java

@@ -7,11 +7,19 @@ public enum Armor {
 	@Override
 	public String toString() {
 		String s = "";
-		switch(this) {
+		switch (this) {
-		case CLOTHES: s = "clothes"; break;
+		case CLOTHES:
-		case LEATHER: s = "leather armor"; break;
+			s = "clothes";
-		case CHAIN_MAIL: s = "chain mail"; break;
+			break;
-		case PLATE_MAIL: s = "plate mail"; break;
+		case LEATHER:
+			s = "leather armor";
+			break;
+		case CHAIN_MAIL:
+			s = "chain mail";
+			break;
+		case PLATE_MAIL:
+			s = "plate mail";
+			break;
 		}
 		return s;
 	}

builder/src/main/java/com/iluwatar/HairColor.java

@@ -7,12 +7,22 @@ public enum HairColor {
 	@Override
 	public String toString() {
 		String s = "";
-		switch(this) {
+		switch (this) {
-		case WHITE: s = "white"; break;
+		case WHITE:
-		case BLOND: s = "blond"; break;
+			s = "white";
-		case RED: s = "red"; break;
+			break;
-		case BROWN: s = "brown"; break;
+		case BLOND:
-		case BLACK: s = "black"; break;
+			s = "blond";
+			break;
+		case RED:
+			s = "red";
+			break;
+		case BROWN:
+			s = "brown";
+			break;
+		case BLACK:
+			s = "black";
+			break;
 		}
 		return s;
 	}

builder/src/main/java/com/iluwatar/HairType.java

@@ -7,12 +7,22 @@ public enum HairType {
 	@Override
 	public String toString() {
 		String s = "";
-		switch(this) {
+		switch (this) {
-		case BALD: s = "bold"; break;
+		case BALD:
-		case SHORT: s = "short"; break;
+			s = "bold";
-		case CURLY: s = "curly"; break;
+			break;
-		case LONG_STRAIGHT: s = "long straight"; break;
+		case SHORT:
-		case LONG_CURLY: s = "long curly"; break;
+			s = "short";
+			break;
+		case CURLY:
+			s = "curly";
+			break;
+		case LONG_STRAIGHT:
+			s = "long straight";
+			break;
+		case LONG_CURLY:
+			s = "long curly";
+			break;
 		}
 		return s;
 	}

builder/src/main/java/com/iluwatar/Hero.java

@@ -94,7 +94,8 @@ public class Hero {
 
 		public HeroBuilder(Profession profession, String name) {
 			if (profession == null || name == null) {
-				throw new NullPointerException("profession and name can not be null");
+				throw new NullPointerException(
+						"profession and name can not be null");
 			}
 			this.profession = profession;
 			this.name = name;

builder/src/main/java/com/iluwatar/Profession.java

@@ -7,11 +7,19 @@ public enum Profession {
 	@Override
 	public String toString() {
 		String s = "";
-		switch(this) {
+		switch (this) {
-		case WARRIOR: s = "Warrior"; break;
+		case WARRIOR:
-		case THIEF: s = "Thief"; break;
+			s = "Warrior";
-		case MAGE: s = "Mage"; break;
+			break;
-		case PRIEST: s = "Priest"; break;
+		case THIEF:
+			s = "Thief";
+			break;
+		case MAGE:
+			s = "Mage";
+			break;
+		case PRIEST:
+			s = "Priest";
+			break;
 		}
 		return s;
 	}

builder/src/main/java/com/iluwatar/Weapon.java

@@ -7,12 +7,22 @@ public enum Weapon {
 	@Override
 	public String toString() {
 		String s = "";
-		switch(this) {
+		switch (this) {
-		case DAGGER: s = "dagger"; break;
+		case DAGGER:
-		case SWORD: s = "sword"; break;
+			s = "dagger";
-		case AXE: s = "axe"; break;
+			break;
-		case WARHAMMER: s = "warhammer"; break;
+		case SWORD:
-		case BOW: s = "bow"; break;
+			s = "sword";
+			break;
+		case AXE:
+			s = "axe";
+			break;
+		case WARHAMMER:
+			s = "warhammer";
+			break;
+		case BOW:
+			s = "bow";
+			break;
 		}
 		return s;
 	}

chain/src/main/java/com/iluwatar/App.java

@@ -14,7 +14,8 @@ public class App {
 
 		OrcKing king = new OrcKing();
 		king.makeRequest(new Request(RequestType.DEFEND_CASTLE, "defend castle"));
-        king.makeRequest(new Request(RequestType.TORTURE_PRISONER, "torture prisoner"));
+		king.makeRequest(new Request(RequestType.TORTURE_PRISONER,
+				"torture prisoner"));
 		king.makeRequest(new Request(RequestType.COLLECT_TAX, "collect tax"));
 
 	}

chain/src/main/java/com/iluwatar/RequestType.java

@@ -2,8 +2,6 @@ package com.iluwatar;
 
 public enum RequestType {
 
-	DEFEND_CASTLE,
+	DEFEND_CASTLE, TORTURE_PRISONER, COLLECT_TAX
-	TORTURE_PRISONER,
-	COLLECT_TAX
 
 }

command/src/main/java/com/iluwatar/Size.java

@@ -2,9 +2,7 @@ package com.iluwatar;
 
 public enum Size {
 
-	SMALL,
+	SMALL, NORMAL, LARGE;
-	NORMAL,
-	LARGE;
 
 	@Override
 	public String toString() {
@@ -27,5 +25,4 @@ public enum Size {
 		return s;
 	}
 
-	
 }

command/src/main/java/com/iluwatar/Target.java

@@ -26,6 +26,7 @@ public abstract class Target {
 	public abstract String toString();
 
 	public void printStatus() {
-		System.out.println(String.format("%s, size=%s visibility=%s", this, getSize(), getVisibility()));
+		System.out.println(String.format("%s, size=%s visibility=%s", this,
+				getSize(), getVisibility()));
 	}
 }

command/src/main/java/com/iluwatar/Visibility.java

@@ -2,8 +2,7 @@ package com.iluwatar;
 
 public enum Visibility {
 
-	VISIBLE,
+	VISIBLE, INVISIBLE;
-	INVISIBLE;
 
 	@Override
 	public String toString() {

composite/src/main/java/com/iluwatar/LetterComposite.java

@@ -26,7 +26,7 @@ public abstract class LetterComposite {
 
 	public void print() {
 		printThisBefore();
-		for (LetterComposite letter: children) {
+		for (LetterComposite letter : children) {
 			letter.print();
 		}
 		printThisAfter();

composite/src/main/java/com/iluwatar/Messenger.java

@@ -10,15 +10,20 @@ public class Messenger {
 
 		List<Word> words = new ArrayList<Word>();
 
-		words.add(new Word(Arrays.asList(new Letter('W'), new Letter('h'), new Letter('e'), new Letter('r'), new Letter('e'))));
+		words.add(new Word(Arrays.asList(new Letter('W'), new Letter('h'),
-		words.add(new Word(Arrays.asList(new Letter('t'), new Letter('h'), new Letter('e'), new Letter('r'), new Letter('e'))));
+				new Letter('e'), new Letter('r'), new Letter('e'))));
+		words.add(new Word(Arrays.asList(new Letter('t'), new Letter('h'),
+				new Letter('e'), new Letter('r'), new Letter('e'))));
 		words.add(new Word(Arrays.asList(new Letter('i'), new Letter('s'))));
 		words.add(new Word(Arrays.asList(new Letter('a'))));
-		words.add(new Word(Arrays.asList(new Letter('w'), new Letter('h'), new Letter('i'), new Letter('p'))));
+		words.add(new Word(Arrays.asList(new Letter('w'), new Letter('h'),
-		words.add(new Word(Arrays.asList(new Letter('t'), new Letter('h'), new Letter('e'), new Letter('r'), new Letter('e'))));
+				new Letter('i'), new Letter('p'))));
+		words.add(new Word(Arrays.asList(new Letter('t'), new Letter('h'),
+				new Letter('e'), new Letter('r'), new Letter('e'))));
 		words.add(new Word(Arrays.asList(new Letter('i'), new Letter('s'))));
 		words.add(new Word(Arrays.asList(new Letter('a'))));
-		words.add(new Word(Arrays.asList(new Letter('w'), new Letter('a'), new Letter('y'))));
+		words.add(new Word(Arrays.asList(new Letter('w'), new Letter('a'),
+				new Letter('y'))));
 
 		return new Sentence(words);
 
@@ -28,12 +33,18 @@ public class Messenger {
 
 		List<Word> words = new ArrayList<Word>();
 
-		words.add(new Word(Arrays.asList(new Letter('M'), new Letter('u'), new Letter('c'), new Letter('h'))));
+		words.add(new Word(Arrays.asList(new Letter('M'), new Letter('u'),
-		words.add(new Word(Arrays.asList(new Letter('w'), new Letter('i'), new Letter('n'), new Letter('d'))));
+				new Letter('c'), new Letter('h'))));
-		words.add(new Word(Arrays.asList(new Letter('p'), new Letter('o'), new Letter('u'), new Letter('r'), new Letter('s'))));
+		words.add(new Word(Arrays.asList(new Letter('w'), new Letter('i'),
-		words.add(new Word(Arrays.asList(new Letter('f'), new Letter('r'), new Letter('o'), new Letter('m'))));
+				new Letter('n'), new Letter('d'))));
-		words.add(new Word(Arrays.asList(new Letter('y'), new Letter('o'), new Letter('u'), new Letter('r'))));
+		words.add(new Word(Arrays.asList(new Letter('p'), new Letter('o'),
-		words.add(new Word(Arrays.asList(new Letter('m'), new Letter('o'), new Letter('u'), new Letter('t'), new Letter('h'))));
+				new Letter('u'), new Letter('r'), new Letter('s'))));
+		words.add(new Word(Arrays.asList(new Letter('f'), new Letter('r'),
+				new Letter('o'), new Letter('m'))));
+		words.add(new Word(Arrays.asList(new Letter('y'), new Letter('o'),
+				new Letter('u'), new Letter('r'))));
+		words.add(new Word(Arrays.asList(new Letter('m'), new Letter('o'),
+				new Letter('u'), new Letter('t'), new Letter('h'))));
 
 		return new Sentence(words);
 

composite/src/main/java/com/iluwatar/Sentence.java

@@ -5,7 +5,7 @@ import java.util.List;
 public class Sentence extends LetterComposite {
 
 	public Sentence(List<Word> words) {
-		for (Word w: words) {
+		for (Word w : words) {
 			this.add(w);
 		}
 	}
@@ -20,5 +20,4 @@ public class Sentence extends LetterComposite {
 		System.out.print(".");
 	}
 
-
 }

composite/src/main/java/com/iluwatar/Word.java

@@ -5,7 +5,7 @@ import java.util.List;
 public class Word extends LetterComposite {
 
 	public Word(List<Letter> letters) {
-		for (Letter l: letters) {
+		for (Letter l : letters) {
 			this.add(l);
 		}
 	}

decorator/src/main/java/com/iluwatar/Hostile.java

@@ -3,6 +3,7 @@ package com.iluwatar;
 public interface Hostile {
 
 	void attack();
+
 	void fleeBattle();
 
 }

double-checked-locking/src/main/java/com/iluwatar/App.java

@@ -19,7 +19,8 @@ public class App {
 			executorService.execute(new Runnable() {
 				@Override
 				public void run() {
-                    while (inventory.addItem(new Item()));
+					while (inventory.addItem(new Item()))
+						;
 				}
 			});
 		}

double-checked-locking/src/main/java/com/iluwatar/Inventory.java

@@ -5,7 +5,6 @@ import java.util.List;
 import java.util.concurrent.locks.Lock;
 import java.util.concurrent.locks.ReentrantLock;
 
-
 public class Inventory {
 
 	private int inventorySize;
@@ -17,16 +16,16 @@ public class Inventory {
 		this.items = new ArrayList<Item>(inventorySize);
 	}
 
-	public boolean addItem(Item item){
+	public boolean addItem(Item item) {
-		if(items.size()<inventorySize){
+		if (items.size() < inventorySize) {
 			lock.lock();
-			try{
+			try {
-				if(items.size()<inventorySize){
+				if (items.size() < inventorySize) {
 					items.add(item);
 					System.out.println(Thread.currentThread());
 					return true;
 				}
-			}finally{
+			} finally {
 				lock.unlock();
 			}
 		}

facade/src/main/java/com/iluwatar/DwarvenGoldmineFacade.java

@@ -15,20 +15,20 @@ public class DwarvenGoldmineFacade {
 	}
 
 	public void startNewDay() {
-		for (DwarvenMineWorker worker: workers) {
+		for (DwarvenMineWorker worker : workers) {
 			worker.wakeUp();
 			worker.goToMine();
 		}
 	}
 
 	public void digOutGold() {
-		for (DwarvenMineWorker worker: workers) {
+		for (DwarvenMineWorker worker : workers) {
 			worker.work();
 		}
 	}
 
 	public void endDay() {
-		for (DwarvenMineWorker worker: workers) {
+		for (DwarvenMineWorker worker : workers) {
 			worker.goHome();
 			worker.goToSleep();
 		}

factory-method/src/main/java/com/iluwatar/WeaponType.java

@@ -7,13 +7,18 @@ public enum WeaponType {
 	@Override
 	public String toString() {
 		String s = "";
-		switch(this) {
+		switch (this) {
-		case SHORT_SWORD: s = "short sword"; break;
+		case SHORT_SWORD:
-		case SPEAR: s = "spear"; break;
+			s = "short sword";
-		case AXE: s = "axe"; break;
+			break;
+		case SPEAR:
+			s = "spear";
+			break;
+		case AXE:
+			s = "axe";
+			break;
 		}
 		return s;
 	}
 
-	
 }

flyweight/src/main/java/com/iluwatar/AlchemistShop.java

@@ -43,13 +43,13 @@ public class AlchemistShop {
 
 		System.out.println("Enumerating top shelf potions\n");
 
-		for (Potion p: topShelf) {
+		for (Potion p : topShelf) {
 			p.drink();
 		}
 
 		System.out.println("\nEnumerating bottom shelf potions\n");
 
-		for (Potion p: bottomShelf) {
+		for (Potion p : bottomShelf) {
 			p.drink();
 		}
 

flyweight/src/main/java/com/iluwatar/HealingPotion.java

@@ -4,7 +4,8 @@ public class HealingPotion implements Potion {
 
 	@Override
 	public void drink() {
-		System.out.println("You feel healed. (Potion=" + System.identityHashCode(this) + ")");
+		System.out.println("You feel healed. (Potion="
+				+ System.identityHashCode(this) + ")");
 	}
 
 }

flyweight/src/main/java/com/iluwatar/HolyWaterPotion.java

@@ -4,7 +4,8 @@ public class HolyWaterPotion implements Potion {
 
 	@Override
 	public void drink() {
-		System.out.println("You feel blessed. (Potion=" + System.identityHashCode(this) + ")");
+		System.out.println("You feel blessed. (Potion="
+				+ System.identityHashCode(this) + ")");
 	}
 
 }

flyweight/src/main/java/com/iluwatar/InvisibilityPotion.java

@@ -4,7 +4,8 @@ public class InvisibilityPotion implements Potion {
 
 	@Override
 	public void drink() {
-		System.out.println("You become invisible. (Potion=" + System.identityHashCode(this) + ")");
+		System.out.println("You become invisible. (Potion="
+				+ System.identityHashCode(this) + ")");
 	}
 
 }

flyweight/src/main/java/com/iluwatar/PoisonPotion.java

@@ -4,7 +4,8 @@ public class PoisonPotion implements Potion {
 
 	@Override
 	public void drink() {
-		System.out.println("Urgh! This is poisonous. (Potion=" + System.identityHashCode(this) + ")");
+		System.out.println("Urgh! This is poisonous. (Potion="
+				+ System.identityHashCode(this) + ")");
 	}
 
 }

flyweight/src/main/java/com/iluwatar/PotionType.java

@@ -2,10 +2,6 @@ package com.iluwatar;
 
 public enum PotionType {
 
-	HEALING,
+	HEALING, INVISIBILITY, STRENGTH, HOLY_WATER, POISON;
-	INVISIBILITY,
-	STRENGTH,
-	HOLY_WATER,
-	POISON;
 
 }

flyweight/src/main/java/com/iluwatar/StrengthPotion.java

@@ -4,7 +4,8 @@ public class StrengthPotion implements Potion {
 
 	@Override
 	public void drink() {
-		System.out.println("You feel strong. (Potion=" + System.identityHashCode(this) + ")");
+		System.out.println("You feel strong. (Potion="
+				+ System.identityHashCode(this) + ")");
 	}
 
 }

interpreter/src/main/java/com/iluwatar/App.java

@@ -25,22 +25,28 @@ public class App {
 			if (isOperator(s)) {
 				Expression rightExpression = stack.pop();
 				Expression leftExpression = stack.pop();
-                System.out.println(String.format("popped from stack left: %d right: %d",
+				System.out
-                        leftExpression.interpret(), rightExpression.interpret()));
+						.println(String.format(
+								"popped from stack left: %d right: %d",
+								leftExpression.interpret(),
+								rightExpression.interpret()));
 				Expression operator = getOperatorInstance(s, leftExpression,
 						rightExpression);
 				System.out.println(String.format("operator: %s", operator));
 				int result = operator.interpret();
 				NumberExpression resultExpression = new NumberExpression(result);
 				stack.push(resultExpression);
-                System.out.println(String.format("push result to stack: %d", resultExpression.interpret()));
+				System.out.println(String.format("push result to stack: %d",
+						resultExpression.interpret()));
 			} else {
 				Expression i = new NumberExpression(s);
 				stack.push(i);
-                System.out.println(String.format("push to stack: %d", i.interpret()));
+				System.out.println(String.format("push to stack: %d",
+						i.interpret()));
 			}
 		}
-        System.out.println(String.format("result: %d", stack.pop().interpret()));
+		System.out
+				.println(String.format("result: %d", stack.pop().interpret()));
 	}
 
 	public static boolean isOperator(String s) {

interpreter/src/main/java/com/iluwatar/MultiplyExpression.java

@@ -5,7 +5,8 @@ public class MultiplyExpression extends Expression {
 	private Expression leftExpression;
 	private Expression rightExpression;
 
-	public MultiplyExpression(Expression leftExpression, Expression rightExpression) {
+	public MultiplyExpression(Expression leftExpression,
+			Expression rightExpression) {
 		this.leftExpression = leftExpression;
 		this.rightExpression = rightExpression;
 	}

iterator/src/main/java/com/iluwatar/ItemType.java

@@ -2,9 +2,6 @@ package com.iluwatar;
 
 public enum ItemType {
 
-	ANY,
+	ANY, WEAPON, RING, POTION
-	WEAPON,
-	RING,
-	POTION
 
 }

iterator/src/main/java/com/iluwatar/TreasureChestItemIterator.java

@@ -39,7 +39,8 @@ public class TreasureChestItemIterator implements ItemIterator {
 				tempIdx = -1;
 				break;
 			}
-			if (type.equals(ItemType.ANY) || items.get(tempIdx).getType().equals(type)) {
+			if (type.equals(ItemType.ANY)
+					|| items.get(tempIdx).getType().equals(type)) {
 				break;
 			}
 		}

mediator/src/main/java/com/iluwatar/PartyImpl.java

@@ -13,7 +13,7 @@ public class PartyImpl implements Party {
 
 	@Override
 	public void act(PartyMember actor, Action action) {
-		for (PartyMember member: members) {
+		for (PartyMember member : members) {
 			if (member != actor) {
 				member.partyAction(action);
 			}

memento/src/main/java/com/iluwatar/Star.java

@@ -63,6 +63,7 @@ public class Star {
 
 	@Override
 	public String toString() {
-		return String.format("%s age: %d years mass: %d tons", type.toString(), ageYears, massTons);
+		return String.format("%s age: %d years mass: %d tons", type.toString(),
+				ageYears, massTons);
 	}
 }

memento/src/main/java/com/iluwatar/StarMementoInternal.java

@@ -14,18 +14,23 @@ public class StarMementoInternal implements StarMemento {
 	public StarType getType() {
 		return type;
 	}
+
 	public void setType(StarType type) {
 		this.type = type;
 	}
+
 	public int getAgeYears() {
 		return ageYears;
 	}
+
 	public void setAgeYears(int ageYears) {
 		this.ageYears = ageYears;
 	}
+
 	public int getMassTons() {
 		return massTons;
 	}
+
 	public void setMassTons(int massTons) {
 		this.massTons = massTons;
 	}

memento/src/main/java/com/iluwatar/StarType.java

@@ -2,11 +2,7 @@ package com.iluwatar;
 
 public enum StarType {
 
-	SUN,
+	SUN, RED_GIANT, WHITE_DWARF, SUPERNOVA, DEAD;
-	RED_GIANT,
-	WHITE_DWARF,
-	SUPERNOVA,
-	DEAD;
 
 	@Override
 	public String toString() {

model-view-presenter/src/main/java/com/iluwatar/FileLoader.java

@@ -5,8 +5,8 @@ import java.io.File;
 import java.io.FileReader;
 
 /**
- * Every instance of this class represents the Model component
+ * Every instance of this class represents the Model component in the
- * in the Model-View-Presenter architectural pattern.
+ * Model-View-Presenter architectural pattern.
  * 
  * It is responsible for reading and loading the contents of a given file.
  */
@@ -27,11 +27,12 @@ public class FileLoader {
 	 */
 	public String loadData() {
 		try {
-			BufferedReader br = new BufferedReader(new FileReader(new File(this.fileName)));
+			BufferedReader br = new BufferedReader(new FileReader(new File(
+					this.fileName)));
 			String text = "";
 			String line = "";
 
-			while( (line = br.readLine()) != null ) {
+			while ((line = br.readLine()) != null) {
 				text += line + "\n";
 			}
 
@@ -41,7 +42,7 @@ public class FileLoader {
 			return text;
 		}
 
-		catch(Exception e) {
+		catch (Exception e) {
 			e.printStackTrace();
 		}
 
@@ -51,7 +52,8 @@ public class FileLoader {
 	/**
 	 * Sets the path of the file to be loaded, to the given value.
 	 * 
-	 * @param fileName The path of the file to be loaded.
+	 * @param fileName
+	 *            The path of the file to be loaded.
 	 */
 	public void setFileName(String fileName) {
 		this.fileName = fileName;

model-view-presenter/src/main/java/com/iluwatar/FileSelectorJFrame.java

@@ -14,10 +14,11 @@ import javax.swing.JTextArea;
 import javax.swing.JTextField;
 
 /**
- * This class is the GUI implementation of the View component
+ * This class is the GUI implementation of the View component In the
- * In the Model-View-Presenter pattern.
+ * Model-View-Presenter pattern.
  */
-public class FileSelectorJFrame extends JFrame implements FileSelectorView, ActionListener {
+public class FileSelectorJFrame extends JFrame implements FileSelectorView,
+		ActionListener {
 
 	/**
 	 * Default serial version ID.
@@ -45,8 +46,7 @@ public class FileSelectorJFrame extends JFrame implements FileSelectorView, Acti
 	private JLabel contents;
 
 	/**
-	 * The text field for giving the name of the file
+	 * The text field for giving the name of the file that we want to open.
-	 * that we want to open.
 	 */
 	private JTextField input;
 
@@ -142,13 +142,13 @@ public class FileSelectorJFrame extends JFrame implements FileSelectorView, Acti
 
 	@Override
 	public void actionPerformed(ActionEvent e) {
-		if(e.getSource() == this.OK) {
+		if (e.getSource() == this.OK) {
 			this.fileName = this.input.getText();
 			presenter.fileNameChanged();
 			presenter.confirmed();
 		}
 
-		else if(e.getSource() == this.cancel) {
+		else if (e.getSource() == this.cancel) {
 			presenter.cancelled();
 		}
 	}

model-view-presenter/src/main/java/com/iluwatar/FileSelectorPresenter.java

@@ -1,10 +1,11 @@
 package com.iluwatar;
 
 /**
- * Every instance of this class represents the Presenter component
+ * Every instance of this class represents the Presenter component in the
- * in the Model-View-Presenter architectural pattern.
+ * Model-View-Presenter architectural pattern.
  * 
- * It is responsible for reacting to the user's actions and update the View component.
+ * It is responsible for reacting to the user's actions and update the View
+ * component.
  */
 public class FileSelectorPresenter {
 
@@ -21,7 +22,8 @@ public class FileSelectorPresenter {
 	/**
 	 * Constructor
 	 * 
-	 * @param view The view component that the presenter will interact with.
+	 * @param view
+	 *            The view component that the presenter will interact with.
 	 */
 	public FileSelectorPresenter(FileSelectorView view) {
 		this.view = view;
@@ -30,7 +32,8 @@ public class FileSelectorPresenter {
 	/**
 	 * Sets the FileLoader object, to the value given as parameter.
 	 * 
-	 * @param loader The new FileLoader object(the Model component).
+	 * @param loader
+	 *            The new FileLoader object(the Model component).
 	 */
 	public void setLoader(FileLoader loader) {
 		this.loader = loader;
@@ -52,12 +55,12 @@ public class FileSelectorPresenter {
 	}
 
 	public void confirmed() {
-		if(loader.getFileName() == null || loader.getFileName().equals("")) {
+		if (loader.getFileName() == null || loader.getFileName().equals("")) {
 			view.showMessage("Please give the name of the file first!");
 			return;
 		}
 
-		if(loader.fileExists()) {
+		if (loader.fileExists()) {
 			String data = loader.loadData();
 			view.displayData(data);
 		}

model-view-presenter/src/main/java/com/iluwatar/FileSelectorStub.java

@@ -1,14 +1,14 @@
 package com.iluwatar;
 
 /**
- * Every instance of this class represents the Stub component in
+ * Every instance of this class represents the Stub component in the
- * the Model-View-Presenter architectural pattern.
+ * Model-View-Presenter architectural pattern.
  * 
- * The stub implements the View interface and it is useful when
+ * The stub implements the View interface and it is useful when we want the test
- * we want the test the reaction to user events, such as mouse clicks.
+ * the reaction to user events, such as mouse clicks.
  * 
- * Since we can not test the GUI directly, the MVP pattern provides
+ * Since we can not test the GUI directly, the MVP pattern provides this
- * this functionality through the View's dummy implementation, the Stub.
+ * functionality through the View's dummy implementation, the Stub.
  */
 public class FileSelectorStub implements FileSelectorView {
 

model-view-presenter/src/main/java/com/iluwatar/FileSelectorView.java

@@ -1,9 +1,8 @@
 package com.iluwatar;
 
 /**
- * This interface represents the View component in the 
+ * This interface represents the View component in the Model-View-Presenter
- * Model-View-Presenter pattern. It can be implemented 
+ * pattern. It can be implemented by either the GUI components, or by the Stub.
- * by either the GUI components, or by the Stub.
  */
 public interface FileSelectorView {
 
@@ -25,7 +24,8 @@ public interface FileSelectorView {
 	/**
 	 * Sets the presenter component, to the one given as parameter.
 	 * 
-	 * @param presenter The new presenter component.
+	 * @param presenter
+	 *            The new presenter component.
 	 */
 	public void setPresenter(FileSelectorPresenter presenter);
 
@@ -37,7 +37,8 @@ public interface FileSelectorView {
 	/**
 	 * Sets the file's name, to the value given as parameter.
 	 * 
-	 * @param name The new name of the file.
+	 * @param name
+	 *            The new name of the file.
 	 */
 	public void setFileName(String name);
 
@@ -49,14 +50,16 @@ public interface FileSelectorView {
 	/**
 	 * Displays a message to the users.
 	 * 
-	 * @param message The message to be displayed.
+	 * @param message
+	 *            The message to be displayed.
 	 */
 	public void showMessage(String message);
 
 	/**
 	 * Displays the data to the view.
 	 * 
-	 * @param data The data to be written.
+	 * @param data
+	 *            The data to be written.
 	 */
 	public void displayData(String data);
 }

model-view-presenter/src/test/java/com/iluwatar/FileSelectorPresenterTest.java

@@ -6,8 +6,8 @@ import org.junit.Before;
 import org.junit.Test;
 
 /**
- * This test case is responsible for testing our application
+ * This test case is responsible for testing our application by taking advantage
- * by taking advantage of the Model-View-Controller architectural pattern.
+ * of the Model-View-Controller architectural pattern.
  */
 public class FileSelectorPresenterTest {
 
@@ -26,7 +26,6 @@ public class FileSelectorPresenterTest {
 	 */
 	private FileLoader loader;
 
-	
 	/**
 	 * Initializes the components of the test case.
 	 */
@@ -64,8 +63,8 @@ public class FileSelectorPresenterTest {
 	}
 
 	/**
-	 * Tests if we receive a confirmation when we attempt to open a file
+	 * Tests if we receive a confirmation when we attempt to open a file that
-	 * that it's name is null or an empty string.
+	 * it's name is null or an empty string.
 	 */
 	@Test
 	public void fileConfirmationWhenNameIsNull() {
@@ -80,8 +79,8 @@ public class FileSelectorPresenterTest {
 	}
 
 	/**
-	 * Tests if we receive a confirmation when we attempt to open a file
+	 * Tests if we receive a confirmation when we attempt to open a file that it
-	 * that it doesn't exist.
+	 * doesn't exist.
 	 */
 	@Test
 	public void fileConfirmationWhenFileDoesNotExist() {

observer/src/main/java/com/iluwatar/Hobbits.java

@@ -6,7 +6,8 @@ public class Hobbits implements WeatherObserver {
 	public void update(WeatherType currentWeather) {
 		switch (currentWeather) {
 		case COLD:
-			System.out.println("The hobbits are shivering in the cold weather.");
+			System.out
+					.println("The hobbits are shivering in the cold weather.");
 			break;
 		case RAINY:
 			System.out.println("The hobbits look for cover from the rain.");
@@ -15,7 +16,8 @@ public class Hobbits implements WeatherObserver {
 			System.out.println("The happy hobbits bade in the warm sun.");
 			break;
 		case WINDY:
-			System.out.println("The hobbits hold their hats tightly in the windy weather.");
+			System.out
+					.println("The hobbits hold their hats tightly in the windy weather.");
 			break;
 		default:
 			break;

observer/src/main/java/com/iluwatar/Weather.java

@@ -5,8 +5,8 @@ import java.util.List;
 
 /**
  * 
- * Weather can be observed by implementing WeatherObserver
+ * Weather can be observed by implementing WeatherObserver interface and
- * interface and registering as listener.
+ * registering as listener.
  * 
  */
 public class Weather {
@@ -49,7 +49,7 @@ public class Weather {
 	}
 
 	private void notifyObservers() {
-		for (WeatherObserver obs: observers) {
+		for (WeatherObserver obs : observers) {
 			obs.update(currentWeather);
 		}
 	}

observer/src/main/java/com/iluwatar/WeatherType.java

@@ -2,10 +2,7 @@ package com.iluwatar;
 
 public enum WeatherType {
 
-	SUNNY,
+	SUNNY, RAINY, WINDY, COLD;
-	RAINY,
-	WINDY,
-	COLD;
 
 	public String toString() {
 		return this.name().toLowerCase();

prototype/src/main/java/com/iluwatar/App.java

@@ -15,7 +15,8 @@ public class App {
 		Warlord warlord;
 		Beast beast;
 
-        factory = new HeroFactoryImpl(new ElfMage(), new ElfWarlord(), new ElfBeast());
+		factory = new HeroFactoryImpl(new ElfMage(), new ElfWarlord(),
+				new ElfBeast());
 		mage = factory.createMage();
 		warlord = factory.createWarlord();
 		beast = factory.createBeast();
@@ -23,7 +24,8 @@ public class App {
 		System.out.println(warlord);
 		System.out.println(beast);
 
-        factory = new HeroFactoryImpl(new OrcMage(), new OrcWarlord(), new OrcBeast());
+		factory = new HeroFactoryImpl(new OrcMage(), new OrcWarlord(),
+				new OrcBeast());
 		mage = factory.createMage();
 		warlord = factory.createWarlord();
 		beast = factory.createBeast();

singleton/src/main/java/com/iluwatar/IvoryTower.java

@@ -9,7 +9,8 @@ public class IvoryTower {
 
 	private static IvoryTower instance = new IvoryTower();
 
-	private IvoryTower() {}
+	private IvoryTower() {
+	}
 
 	public static IvoryTower getInstance() {
 		return instance;

strategy/src/main/java/com/iluwatar/ProjectileStrategy.java

@@ -4,7 +4,8 @@ public class ProjectileStrategy implements DragonSlayingStrategy {
 
 	@Override
 	public void execute() {
-		System.out.println("You shoot the dragon with the magical crossbow and it falls dead on the ground!");
+		System.out
+				.println("You shoot the dragon with the magical crossbow and it falls dead on the ground!");
 	}
 
 }

strategy/src/main/java/com/iluwatar/SpellStrategy.java

@@ -4,7 +4,8 @@ public class SpellStrategy implements DragonSlayingStrategy {
 
 	@Override
 	public void execute() {
-		System.out.println("You cast the spell of disintegration and the dragon vaporizes in a pile of dust!");
+		System.out
+				.println("You cast the spell of disintegration and the dragon vaporizes in a pile of dust!");
 	}
 
 }

template-method/src/main/java/com/iluwatar/SubtleMethod.java

@@ -9,12 +9,14 @@ public class SubtleMethod extends StealingMethod {
 
 	@Override
 	protected void confuseTarget(String target) {
-		System.out.println("Approach the " + target + " with tears running and hug him!");
+		System.out.println("Approach the " + target
+				+ " with tears running and hug him!");
 	}
 
 	@Override
 	protected void stealTheItem(String target) {
-		System.out.println("While in close contact grab the " + target + "'s wallet.");
+		System.out.println("While in close contact grab the " + target
+				+ "'s wallet.");
 	}
 
 }

visitor/src/main/java/com/iluwatar/App.java

@@ -11,9 +11,9 @@ public class App {
 
 	public static void main(String[] args) {
 
-        Commander commander = new Commander(
+		Commander commander = new Commander(new Sergeant(new Soldier(),
-                new Sergeant(new Soldier(), new Soldier(), new Soldier()),
+				new Soldier(), new Soldier()), new Sergeant(new Soldier(),
-                new Sergeant(new Soldier(), new Soldier(), new Soldier()));
+				new Soldier(), new Soldier()));
 		commander.accept(new SoldierVisitor());
 		commander.accept(new SergeantVisitor());
 		commander.accept(new CommanderVisitor());

visitor/src/main/java/com/iluwatar/Commander.java

@@ -2,7 +2,7 @@ package com.iluwatar;
 
 public class Commander extends Unit {
 
-	public Commander(Unit ... children) {
+	public Commander(Unit... children) {
 		super(children);
 	}
 

visitor/src/main/java/com/iluwatar/Sergeant.java

@@ -2,7 +2,7 @@ package com.iluwatar;
 
 public class Sergeant extends Unit {
 
-	public Sergeant(Unit ... children) {
+	public Sergeant(Unit... children) {
 		super(children);
 	}
 

visitor/src/main/java/com/iluwatar/Soldier.java

@@ -2,7 +2,7 @@ package com.iluwatar;
 
 public class Soldier extends Unit {
 
-	public Soldier(Unit ... children) {
+	public Soldier(Unit... children) {
 		super(children);
 	}
 

visitor/src/main/java/com/iluwatar/Unit.java

@@ -9,12 +9,12 @@ public abstract class Unit {
 
 	private Unit[] children;
 
-	public Unit(Unit ... children) {
+	public Unit(Unit... children) {
 		this.children = children;
 	}
 
 	public void accept(UnitVisitor visitor) {
-		for (Unit child: children) {
+		for (Unit child : children) {
 			child.accept(visitor);
 		}
 	}

visitor/src/main/java/com/iluwatar/UnitVisitor.java

@@ -8,7 +8,9 @@ package com.iluwatar;
 public interface UnitVisitor {
 
 	void visitSoldier(Soldier soldier);
+
 	void visitSergeant(Sergeant sergeant);
+
 	void visitCommander(Commander commander);
 
 }