Merge pull request #6 from jeffmorin/tech-review-ch06

Proofed chapter 6

Author: davecom

CCSPiJ/src/chapter6/Album.java

@@ -50,10 +50,10 @@ public static void main(String[] args) {
 		KMeans<Album> kmeans = new KMeans<>(2, albums);
 		List<KMeans<Album>.Cluster> clusters = kmeans.run(100);
 		for (int clusterIndex = 0; clusterIndex < clusters.size(); clusterIndex++) {
-			System.out.println("Cluster " + clusterIndex + " Avg Length " +
-					clusters.get(clusterIndex).centroid.dimensions.get(0) +
-					" Avg Tracks " + clusters.get(clusterIndex).centroid.dimensions.get(1)
-					+ ": " + clusters.get(clusterIndex).points);
+			System.out.printf("Cluster %d Avg Length %f Avg Tracks %f: %s%n",
+					clusterIndex, clusters.get(clusterIndex).centroid.dimensions.get(0),
+					clusters.get(clusterIndex).centroid.dimensions.get(1),
+					clusters.get(clusterIndex).points);
 		}
 	}
 

CCSPiJ/src/chapter6/Governor.java

@@ -92,8 +92,7 @@ public static void main(String[] args) {
 		KMeans<Governor> kmeans = new KMeans<>(2, governors);
 		List<KMeans<Governor>.Cluster> govClusters = kmeans.run(100);
 		for (int clusterIndex = 0; clusterIndex < govClusters.size(); clusterIndex++) {
-			System.out.println("Cluster " + clusterIndex + ": "
-					+ govClusters.get(clusterIndex).points);
+			System.out.printf("Cluster %d: %s%n", clusterIndex, govClusters.get(clusterIndex).points);
 		}
 	}
 

CCSPiJ/src/chapter6/KMeans.java

@@ -117,7 +117,7 @@ private void generateCentroids() {
 			}
 			List<Double> means = new ArrayList<>();
 			for (int i = 0; i < cluster.points.get(0).numDimensions; i++) {
-				final int dimension = i; // needed to use in scope of closure
+				int dimension = i; // needed to use in scope of closure
 				Double dimensionMean = cluster.points.stream()
 						.mapToDouble(x -> x.dimensions.get(dimension)).average().getAsDouble();
 				means.add(dimensionMean);