iluwatar#84 More work on cake baking

iluwatar · iluwatar · commit aa04e7c85664 · 2015-08-13T20:35:13.000+03:00
diff --git a/layers/src/main/java/com/iluwatar/layers/App.java b/layers/src/main/java/com/iluwatar/layers/App.java
@@ -16,12 +16,13 @@ public static void main(String[] args) {
 		service.getAllToppings().stream().forEach((topping) -> System.out.println(topping));
 
 		CakeInfo cakeInfo = new CakeInfo(new CakeToppingInfo("candies", 0),
-				Arrays.asList(new CakeLayerInfo("chocolate", 0), new CakeLayerInfo("chocolate", 0),
-						new CakeLayerInfo("chocolate", 0)));
+				Arrays.asList(new CakeLayerInfo("chocolate", 0), new CakeLayerInfo("banana", 0),
+						new CakeLayerInfo("strawberry", 0)));
 		try {
 			service.bakeNewCake(cakeInfo);
 		} catch (CakeBakingException e) {
 			e.printStackTrace();
 		}
+		service.getAllCakes().stream().forEach((cake) -> System.out.println(cake));
 	}
 }
diff --git a/layers/src/main/java/com/iluwatar/layers/Cake.java b/layers/src/main/java/com/iluwatar/layers/Cake.java
@@ -1,10 +1,11 @@
 package com.iluwatar.layers;
 
-import java.util.ArrayList;
-import java.util.List;
+import java.util.HashSet;
+import java.util.Set;
 
 import javax.persistence.CascadeType;
 import javax.persistence.Entity;
+import javax.persistence.FetchType;
 import javax.persistence.GeneratedValue;
 import javax.persistence.Id;
 import javax.persistence.OneToMany;
@@ -17,14 +18,14 @@ public class Cake {
 	@GeneratedValue
 	private Long id;
 
-	@OneToOne(cascade = CascadeType.ALL)
+	@OneToOne(cascade=CascadeType.REMOVE)
 	private CakeTopping topping;
 	
-	@OneToMany(cascade = CascadeType.ALL)
-	private List<CakeLayer> layers;
+	@OneToMany(cascade=CascadeType.REMOVE, fetch=FetchType.EAGER)
+	private Set<CakeLayer> layers;
 	
 	public Cake() {		
-		setLayers(new ArrayList<>());
+		setLayers(new HashSet<>());
 	}
 
 	public Long getId() {
@@ -43,15 +44,20 @@ public void setTopping(CakeTopping topping) {
 		this.topping = topping;
 	}
 
-	public List<CakeLayer> getLayers() {
+	public Set<CakeLayer> getLayers() {
 		return layers;
 	}
 
-	public void setLayers(List<CakeLayer> layers) {
+	public void setLayers(Set<CakeLayer> layers) {
 		this.layers = layers;
 	}
 	
 	public void addLayer(CakeLayer layer) {
 		this.layers.add(layer);
 	}
+	
+	@Override
+	public String toString() {
+		return String.format("id=%s topping=%s layers=%s", id, topping, layers.toString());
+	}	
 }
diff --git a/layers/src/main/java/com/iluwatar/layers/CakeBakingService.java b/layers/src/main/java/com/iluwatar/layers/CakeBakingService.java
@@ -5,6 +5,8 @@
 public interface CakeBakingService {
 	
 	void bakeNewCake(CakeInfo cakeInfo) throws CakeBakingException;
+	
+	List<CakeInfo> getAllCakes();
 
 	void saveNewTopping(CakeToppingInfo toppingInfo);
 
diff --git a/layers/src/main/java/com/iluwatar/layers/CakeBakingServiceImpl.java b/layers/src/main/java/com/iluwatar/layers/CakeBakingServiceImpl.java
@@ -1,9 +1,11 @@
 package com.iluwatar.layers;
 
 import java.util.ArrayList;
+import java.util.HashSet;
 import java.util.Iterator;
 import java.util.List;
 import java.util.Optional;
+import java.util.Set;
 import java.util.stream.Collectors;
 
 import org.springframework.context.support.AbstractApplicationContext;
@@ -30,7 +32,7 @@ public void bakeNewCake(CakeInfo cakeInfo) throws CakeBakingException {
 			throw new CakeBakingException(String.format("Topping %s is not available", cakeInfo.cakeToppingInfo.name));
 		}
 		List<CakeLayer> allLayers = getAllLayerEntities();
-		List<CakeLayer> foundLayers = new ArrayList<>();
+		Set<CakeLayer> foundLayers = new HashSet<>();
 		for (CakeLayerInfo info: cakeInfo.cakeLayerInfos) {
 			Optional<CakeLayer> found = allLayers.stream().filter((layer) -> layer.getName().equals(info.name)).findFirst();
 			if (!found.isPresent()) {
@@ -43,14 +45,16 @@ public void bakeNewCake(CakeInfo cakeInfo) throws CakeBakingException {
 		CakeTopping topping = toppingBean.findOne(matchingToppings.iterator().next().id.get());
 		CakeDao cakeBean = context.getBean(CakeDao.class);
 		Cake cake = new Cake();
-		cake = cakeBean.save(cake);
 		cake.setTopping(topping);
-		topping.setCake(cake);
 		cake.setLayers(foundLayers);
+		cakeBean.save(cake);
+		topping.setCake(cake);
+		toppingBean.save(topping);
+		CakeLayerDao layerBean = context.getBean(CakeLayerDao.class);
 		for (CakeLayer layer: foundLayers) {
 			layer.setCake(cake);
+			layerBean.save(layer);
 		}
-		cakeBean.save(cake);
 	}
 
 	@Override
@@ -108,4 +112,23 @@ public List<CakeLayerInfo> getAllLayers() {
 		}
 		return result;
 	}
+
+	@Override
+	public List<CakeInfo> getAllCakes() {
+		CakeDao cakeBean = context.getBean(CakeDao.class);
+		List<CakeInfo> result = new ArrayList<>();
+		Iterator<Cake> iterator = cakeBean.findAll().iterator();
+		while (iterator.hasNext()) {
+			Cake cake = iterator.next();
+			CakeToppingInfo cakeToppingInfo = new CakeToppingInfo(cake.getTopping().getId(),
+					cake.getTopping().getName(), cake.getTopping().getCalories());
+			ArrayList<CakeLayerInfo> cakeLayerInfos = new ArrayList<CakeLayerInfo>();
+			for (CakeLayer layer: cake.getLayers()) {
+				cakeLayerInfos.add(new CakeLayerInfo(layer.getId(), layer.getName(), layer.getCalories()));
+			}
+			CakeInfo cakeInfo = new CakeInfo(cake.getId(), cakeToppingInfo, cakeLayerInfos);
+			result.add(cakeInfo);
+		}
+		return result;
+	}
 }
diff --git a/layers/src/main/java/com/iluwatar/layers/CakeInfo.java b/layers/src/main/java/com/iluwatar/layers/CakeInfo.java
@@ -20,4 +20,9 @@ public CakeInfo(CakeToppingInfo cakeToppingInfo, List<CakeLayerInfo> cakeLayerIn
 		this.cakeToppingInfo = cakeToppingInfo;
 		this.cakeLayerInfos = cakeLayerInfos;
 	}
+	
+	@Override
+	public String toString() {
+		return String.format("CakeInfo id=%d topping=%s layers=%s", id.get(), cakeToppingInfo, cakeLayerInfos);
+	}
 }
diff --git a/layers/src/main/java/com/iluwatar/layers/CakeLayer.java b/layers/src/main/java/com/iluwatar/layers/CakeLayer.java
@@ -54,7 +54,7 @@ public void setCalories(int calories) {
 	
 	@Override
 	public String toString() {
-		return String.format("name: %s calories: %d", name, calories);
+		return String.format("id=%s name=%s calories=%d", id, name, calories);
 	}
 
 	public Cake getCake() {
diff --git a/layers/src/main/java/com/iluwatar/layers/CakeLayerInfo.java b/layers/src/main/java/com/iluwatar/layers/CakeLayerInfo.java
@@ -22,6 +22,6 @@ public CakeLayerInfo(String name, int calories) {
 	
 	@Override
 	public String toString() {
-		return String.format("CakeLayerInfo name: %s calories: %d", name, calories);
+		return String.format("CakeLayerInfo id=%d name=%s calories=%d", id.get(), name, calories);
 	}
 }
diff --git a/layers/src/main/java/com/iluwatar/layers/CakeTopping.java b/layers/src/main/java/com/iluwatar/layers/CakeTopping.java
@@ -54,7 +54,7 @@ public void setCalories(int calories) {
 	
 	@Override
 	public String toString() {
-		return String.format("name: %s calories: %d", name, calories);
+		return String.format("id=%s name=%s calories=%d", name, calories);
 	}
 
 	public Cake getCake() {
diff --git a/layers/src/main/java/com/iluwatar/layers/CakeToppingInfo.java b/layers/src/main/java/com/iluwatar/layers/CakeToppingInfo.java
@@ -22,6 +22,6 @@ public CakeToppingInfo(String name, int calories) {
 	
 	@Override
 	public String toString() {
-		return String.format("CakeToppingInfo name: %s calories: %d", name, calories);
+		return String.format("CakeToppingInfo id=%d name=%s calories=%d", id.get(), name, calories);
 	}
 }

Original file line number	Diff line number	Diff line change
`@@ -16,12 +16,13 @@ public static void main(String[] args) {`
`16`	`16`	`service.getAllToppings().stream().forEach((topping) -> System.out.println(topping));`
`17`	`17`
`18`	`18`	`CakeInfo cakeInfo = new CakeInfo(new CakeToppingInfo("candies", 0),`
`19`		`- Arrays.asList(new CakeLayerInfo("chocolate", 0), new CakeLayerInfo("chocolate", 0),`
`20`		`- new CakeLayerInfo("chocolate", 0)));`
	`19`	`+ Arrays.asList(new CakeLayerInfo("chocolate", 0), new CakeLayerInfo("banana", 0),`
	`20`	`+ new CakeLayerInfo("strawberry", 0)));`
`21`	`21`	`try {`
`22`	`22`	`service.bakeNewCake(cakeInfo);`
`23`	`23`	`} catch (CakeBakingException e) {`
`24`	`24`	`e.printStackTrace();`
`25`	`25`	`}`
	`26`	`+ service.getAllCakes().stream().forEach((cake) -> System.out.println(cake));`
`26`	`27`	`}`
`27`	`28`	`}`
Original file line number	Diff line number	Diff line change
`@@ -20,4 +20,9 @@ public CakeInfo(CakeToppingInfo cakeToppingInfo, List<CakeLayerInfo> cakeLayerIn`
`20`	`20`	`this.cakeToppingInfo = cakeToppingInfo;`
`21`	`21`	`this.cakeLayerInfos = cakeLayerInfos;`
`22`	`22`	`}`
	`23`	`+`
	`24`	`+ @Override`
	`25`	`+ public String toString() {`
	`26`	`+ return String.format("CakeInfo id=%d topping=%s layers=%s", id.get(), cakeToppingInfo, cakeLayerInfos);`
	`27`	`+ }`
`23`	`28`	`}`
Original file line number	Diff line number	Diff line change
`@@ -54,7 +54,7 @@ public void setCalories(int calories) {`
`54`	`54`
`55`	`55`	`@Override`
`56`	`56`	`public String toString() {`
`57`		`- return String.format("name: %s calories: %d", name, calories);`
	`57`	`+ return String.format("id=%s name=%s calories=%d", id, name, calories);`
`58`	`58`	`}`
`59`	`59`
`60`	`60`	`public Cake getCake() {`
Original file line number	Diff line number	Diff line change
`@@ -22,6 +22,6 @@ public CakeLayerInfo(String name, int calories) {`
`22`	`22`
`23`	`23`	`@Override`
`24`	`24`	`public String toString() {`
`25`		`- return String.format("CakeLayerInfo name: %s calories: %d", name, calories);`
	`25`	`+ return String.format("CakeLayerInfo id=%d name=%s calories=%d", id.get(), name, calories);`
`26`	`26`	`}`
`27`	`27`	`}`
Original file line number	Diff line number	Diff line change
`@@ -22,6 +22,6 @@ public CakeToppingInfo(String name, int calories) {`
`22`	`22`
`23`	`23`	`@Override`
`24`	`24`	`public String toString() {`
`25`		`- return String.format("CakeToppingInfo name: %s calories: %d", name, calories);`
	`25`	`+ return String.format("CakeToppingInfo id=%d name=%s calories=%d", id.get(), name, calories);`
`26`	`26`	`}`
`27`	`27`	`}`