refactor(demo): improve demo readibility

Author: javier-godoy

src/test/java/com/flowingcode/vaadin/addons/chipfield/BinderDemo.java

@@ -7,9 +7,9 @@
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
- * 
+ *
  *      http://www.apache.org/licenses/LICENSE-2.0
- * 
+ *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
  * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
@@ -19,35 +19,35 @@
  */
 package com.flowingcode.vaadin.addons.chipfield;
 
+import java.util.Arrays;
+
 import com.vaadin.flow.component.button.Button;
 import com.vaadin.flow.component.notification.Notification;
 import com.vaadin.flow.component.notification.Notification.Position;
 import com.vaadin.flow.component.orderedlayout.VerticalLayout;
 import com.vaadin.flow.data.binder.Binder;
-import java.util.Arrays;
-import java.util.stream.Collectors;
 
 @SuppressWarnings("serial")
 public class BinderDemo extends VerticalLayout {
 
 	public BinderDemo() {
+
 		Planet p = new Planet("A new planet");
-		ChipField<String> chf = new ChipField<>(
-				"Choose planet features (Binder demo, try with: 'Rings', 'Moons', 'Water', etc.)");
+
+		ChipField<String> chf = new ChipField<>("Choose planet features (try with: 'Rings', 'Moons', 'Water', etc.)");
 		chf.setWidthFull();
 		chf.setItems(Arrays.asList("Rings", "Moons", "Water", "Rocks", "Lava", "Ice", "Cold", "Heat", "Atmosphere"));
 		Binder<Planet> binder = new Binder<>();
-		binder.bind(chf, Planet::getConfiguration, Planet::setConfiguration);
+		binder.bind(chf, Planet::getFeatures, Planet::setFeatures);
 		binder.setBean(p);
-		Button show = new Button("Show planet configuration");
-		show.addClickListener(
-				event -> Notification.show(
-						"Planet: " + p.getName() + ", features: "
-								+ p.getConfiguration().stream().collect(Collectors.joining(",")),
-						5000, Position.BOTTOM_START));
+
 		chf.addValueChangeListener(
-				newItem -> Notification.show("Items: " + newItem.getValue(), 5000, Position.BOTTOM_START));
+				newItem -> Notification.show("Current value: " + newItem.getValue(), 5000, Position.BOTTOM_START));
+
+		add(chf);
+		add(new Button("Show planet features",
+				e -> Notification.show("Features: " + p.getFeatures(), 5000, Position.BOTTOM_START)));
 
-		add(chf, show);
 	}
+
 }

src/test/java/com/flowingcode/vaadin/addons/chipfield/DataProviderDemo.java

@@ -7,9 +7,9 @@
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
- * 
+ *
  *      http://www.apache.org/licenses/LICENSE-2.0
- * 
+ *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
  * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
@@ -19,45 +19,38 @@
  */
 package com.flowingcode.vaadin.addons.chipfield;
 
+import java.util.stream.Collectors;
+
 import com.vaadin.flow.component.button.Button;
 import com.vaadin.flow.component.notification.Notification;
 import com.vaadin.flow.component.notification.Notification.Position;
 import com.vaadin.flow.component.orderedlayout.HorizontalLayout;
 import com.vaadin.flow.component.orderedlayout.VerticalLayout;
 import com.vaadin.flow.data.provider.ListDataProvider;
-import java.util.ArrayList;
-import java.util.Arrays;
-import java.util.List;
-import java.util.stream.Collectors;
 
 @SuppressWarnings("serial")
 public class DataProviderDemo extends VerticalLayout {
 
 	public DataProviderDemo() {
-		List<Planet> availablePlanets = new ArrayList<>(
-				Arrays.asList(new Planet("Mercury"), new Planet("Venus"), new Planet("Earth"), new Planet("Mars"),
-						new Planet("Jupiter"), new Planet("Saturn"), new Planet("Uranus"), new Planet("Neptune")));
-		ListDataProvider<Planet> ldp = new ListDataProvider<>(availablePlanets);
 
-		ChipField<Planet> chf = new ChipField<>("Select some planets (Mercury, Venus, Earth, etc.)",
-				planet -> planet.getName());
+		ListDataProvider<Planet> ldp = new ListDataProvider<>(Planet.all());
+
+		ChipField<Planet> chf = new ChipField<>("Select some planets (Mercury, Venus, Earth, etc.)", planet -> planet.getName());
 		chf.setWidthFull();
 		chf.setDataProvider(ldp);
 		chf.setClosable(true);
 		chf.setNewItemHandler(label -> new Planet(label));
 
-		Button b = new Button("Obtain selected planets");
-		b.addClickListener(event -> Notification.show(
-				"Planets: " + chf.getValue().stream().map(planet -> planet.getName()).collect(Collectors.joining(",")),
-				5000, Position.BOTTOM_START));
+		HorizontalLayout buttons = new HorizontalLayout();
+		buttons.add(new Button("Obtain selected planets", ev -> Notification
+				.show("Planets: " + chf.getValue().stream().map(Planet::getName).collect(Collectors.joining(",")), 5000, Position.BOTTOM_START)));
 
-		Button b2 = new Button("Add random planet");
-		b2.addClickListener(event -> {
-			Planet p = new Planet("Planet" + Math.round(Math.random() * 10000));
-			availablePlanets.add(p);
+		buttons.add(new Button("Add random planet", ev -> {
+			Planet newPlanet = Planet.random();
+			ldp.getItems().add(newPlanet);
 			ldp.refreshAll();
-			chf.addSelectedItem(p);
-		});
+			chf.addSelectedItem(newPlanet);
+		}));
 
 		chf.addChipCreatedListener(
 				ev -> Notification.show("Chip: " + ev.getChipLabel() + " Created by client: " + ev.isFromClient() + "!",
@@ -68,7 +61,7 @@ public DataProviderDemo() {
 		chf.addChipClickedListener(
 				ev -> Notification.show("Chip: " + ev.getChipLabel() + " Clicked!", 5000, Position.BOTTOM_END));
 
-		VerticalLayout vl = new VerticalLayout(chf, new HorizontalLayout(b, b2));
-		add(vl);
+		add(new VerticalLayout(chf, buttons));
 	}
+
 }

src/test/java/com/flowingcode/vaadin/addons/chipfield/Planet.java

@@ -7,9 +7,9 @@
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
- * 
+ *
  *      http://www.apache.org/licenses/LICENSE-2.0
- * 
+ *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
  * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
@@ -19,12 +19,13 @@
  */
 package com.flowingcode.vaadin.addons.chipfield;
 
-import java.util.ArrayList;
 import java.util.List;
+import java.util.stream.Collectors;
+import java.util.stream.Stream;
 
 public class Planet {
 	private String name;
-	private List<String> configuration = new ArrayList<>();
+	private List<String> features;
 
 	public Planet(String name) {
 		this.name = name;
@@ -38,12 +39,22 @@ public void setName(String name) {
 		this.name = name;
 	}
 
-	public List<String> getConfiguration() {
-		return configuration;
+	public List<String> getFeatures() {
+		return features;
 	}
 
-	public void setConfiguration(List<String> configuration) {
-		this.configuration = configuration;
+	public void setFeatures(List<String> features) {
+		this.features = features;
 	}
+	}
+
+	public static List<Planet> all() {
+		return Stream.of("Mercury", "Venus", "Earth", "Mars", "Jupiter", "Saturn", "Uranus", "Neptune").map(Planet::new).collect(Collectors.toList());
+	}
+
+	public static Planet random() {
+		return new Planet("Planet " + Integer.toString((int) Math.round(Math.random() * 36 * 36 * 36), 36).toUpperCase());
+	}
+
 }