Während unserer coding night, zu der mit Sicherheit noch ein separater Blogeintrag folgt, setzen wir voll auf Spring Technologien. Dabei hat sich eine vermutlich simple Anforderung als ziemlich halsbrecherisch herausgestellt.
Wie kann man mit Spring MVC eine Select box darstellen, bei der man multiple Elemente auswählen kann, welche dann in eine Collection der Bean hinzugefügt werden?
Wir implementiere eine einfache Zeiterfassung. In dem Domänenmodell gibt es Projekte, denen Mitarbeiter zugeordnet sind. Zudem können Projekte Aufgaben haben, denen auch Mitarbeiter zugeordnet sein können. In der View zum erstellen der Aufgaben benötigen wir also eine Selectbox aus allen verfügbaren Mitarbeitern.
Modell
Hier das wichtigste der Aufgaben und Mitarbeiter:
Task
1@Entity 2public class Task implements Serializable { 3 @Id 4 @GeneratedValue 5 private long id; 6 7 @ManyToMany 8 private Set<Staff> staffs = new HashSet<Staff>(0); 9 10 //... 11}
Staff
1@NamedQuery(name = "staff.activeStaff", query = "select s from Staff s where s.disabled = false") 2@Entity 3@DiscriminatorValue("STAFF") 4public class Staff extends Person { 5 6 //... 7}
Person
1@NamedQuery(name = "person.findByUsername", query = "from Person p where p.login.username = :username") 2@Entity 3@Inheritance(strategy = InheritanceType.SINGLE_TABLE) 4@DiscriminatorColumn(name = "type", discriminatorType = DiscriminatorType.STRING) 5public abstract class Person implements Serializable { 6 7 @Id 8 @GeneratedValue 9 private long id; 10 11 //... 12}
View
In der view, in der die Tasks erstellt werden sollen, brauchen wir also eine select box, welche alle verfügbaren Mitarbeiter anzeigt. Die ausgewählten Mitarbeiter sollen dann der Aufgabe hinzugefügt werden. Die View ist mit der Task hinterlegt, und fügt alle aktiven Mitarbeiter zu einem Attribut hinzu:
1@Controller
2@SessionAttributes("project")
3public class ProjectController {
4 @RequestMapping(method = RequestMethod.GET, value = "/project/createTask.action")
5 public void createTaskView(@ModelAttribute Task task, Model model) {
6 List<Staff> activeStaff = timetrackingService.getActiveStaff();
7 model.addAttribute("activeStaff", activeStaff);
8 }
9
10 //...
Die entsprechende JSP sieht folgendermaßen aus:
1<td><label for="staffs">Chose Staff: </label></td> 2<td><form:select path="staffs" multiple="true" items="${activeStaff}" itemLabel="fullName" itemValue="id"/></td> 3<td><form:errors path="staffs" /></td>
Das Problem, sobald man die Form submitted, bekommt man eine ServletRequestBindingException, denn Spring weiß nicht wie man aus einem String (bzw. String[] wenn mehrere Personen markiert waren) ein Set erstellt.
1org.springframework.web.bind.ServletRequestBindingException: Errors binding onto object 'task'; nested exception is org.springframework.validation.BindException: org.springframework.validation.BeanPropertyBindingResult: 1 errors 2Field error in object 'task' on field 'staffs': rejected value [3]; codes [typeMismatch.task.staffs,typeMismatch.staffs,typeMismatch.java.util.Set,typeMismatch]; arguments [org.springframework.context.support.DefaultMessageSourceResolvable: codes [task.staffs,staffs]; arguments []; default message [staffs]]; default message [Failed to convert property value of type [java.lang.String] to required type [java.util.Set] for property 'staffs'; nested exception is java.lang.IllegalArgumentException: Cannot convert value of type [java.lang.String] to required type [de.codecentric.timetracking.model.Staff] for property 'staffs[0]': no matching editors or conversion strategy found]
Zudem ist der generierte HTML-code lückenhaft, er enthält keine id für die einzelnen Mitarbeiter!
1<select id="staffs" multiple="multiple" name="staffs"> 2<option value="">firstname lastname</option> 3<option value="">aaaa aaaa</option> 4<option value="">firstname lastName</option> 5<option value="">firstname aaaa</option> 6<option value="">firstname lastname</option> 7</select> 8<input type="hidden" value="1" name="_staffs"/>
Id als String
Um die ID des Mitarbeiters in das value-Attribut der option zu bekommen, kann man einen eigenen Getter auf der Person implementieren, und diesen dann in der JSP statt der ID verwenden:
1public abstract class Person implements Serializable {
2 //...
3 public String getIdAsString() {
4 return new Long(id).toString();
5 }
6}
1<form:select path="staffs" itemValue="idAsString" multiple="true" items="${activeStaff}" itemLabel="fullName"/>
InitBinder
Die Lösung für das Binding-Problem ist, dass wir für das Attribut ’staffs‘ einen eigenen PropertyEditor registrieren müssen. Spring bringt eine Reihe eigener PropertyEditoren mit, in diesem Falle können wir den CustomCollectionEditor wiederverwenden. Um von der String-ID wieder auf den Mitarbeiter mappen zu können, müssen wir uns auch noch eine Map initialisieren, die dieses Mapping vorhält.
1public class ProjectController {
2
3private Map<String, Staff> staffCache;
4
5@RequestMapping(method = RequestMethod.GET, value = "/project/createTask.action")
6public void createTaskView(@ModelAttribute Task task, Model model) {
7 List<Staff> activeStaff = timetrackingService.getActiveStaff();
8 staffCache = new HashMap<String, Staff>();
9 for (Staff staff : activeStaff) {
10 staffCache.put(staff.getIdAsString(), staff);
11 }
12 model.addAttribute("activeStaff", activeStaff);
13}
14
15@InitBinder
16protected void initBinder(WebDataBinder binder) throws Exception {
17 binder.registerCustomEditor(Set.class, "staffs", new CustomCollectionEditor(Set.class) {
18 protected Object convertElement(Object element) {
19 if (element instanceof Staff) {
20 System.out.println("Converting from Staff to Staff: " + element);
21 return element;
22 }
23 if (element instanceof String) {
24 Staff staff = staffCache.get(element);
25 System.out.println("Looking up staff for id " + element + ": " + staff);
26 return staff;
27 }
28 System.out.println("Don't know what to do with: " + element);
29 return null;
30 }
31 });
32}
Der Code enthält noch etwas Debug-output nach System.out, der natürlich noch entfernt werden muss. Er zeigt aber sehr schön, dass der Code sehr (zu?) häufig aufgerufen wird. Außerdem wird erwartetn, dass die Property in beide Richtungen konvertiert werden kann!
Allein wenn die select box angezeit werden soll, steht folgendes im Log:
1Looking up staff for id 1: Staff(firstname lastname) 2Looking up staff for id 1: Staff(firstname lastname) 3Converting from Staff to Staff: Staff(firstname lastname) 4Looking up staff for id 2: Staff(aaaa aaaa) 5Looking up staff for id 2: Staff(aaaa aaaa) 6Converting from Staff to Staff: Staff(aaaa aaaa) 7Looking up staff for id 3: Staff(firstname lastName) 8Looking up staff for id 3: Staff(firstname lastName) 9Converting from Staff to Staff: Staff(firstname lastName) 10Looking up staff for id 4: Staff(firstname aaaa) 11Looking up staff for id 4: Staff(firstname aaaa) 12Converting from Staff to Staff: Staff(firstname aaaa) 13Looking up staff for id 5: Staff(firstname lastname) 14Looking up staff for id 5: Staff(firstname lastname) 15Converting from Staff to Staff: Staff(firstname lastname)
Wenn man einen Mitarbeiter aus der Select-Box auswählt und die Form submitted, findet sich dann aber wie erwartet nur ein Eintrag im Log:
1Looking up staff for id 3: Staff(firstname lastName)
Fazit
Angesichts der Masse an Webframeworks, die es gibt, finde ich es erschreckend Kompliziert so etwas einfaches wie eine multiple select Box mit Spring MVC zu implementieren. Da ich mir Spring MVC aber auch erst seit gestern genauer angesehen habe, verstehe ich es vielleicht noch nicht richtig, von daher bin ich sehr für Vorschläge zu haben, wie man das beschriebene Szenario mit Spring MVC-Mitteln eleganter und einfacher implementieren kann.
War dieser Beitrag hilfreich?
Weitere Beiträge
von Andreas Ebbert-Karroum
18.11.2020 Dein Job bei codecentric?
Jobs
Agile Developer und Consultant (w/d/m)
Alle Standorte
Weitere Artikel in diesem Themenbereich
Entdecke spannende weiterführende Themen und lass dich von der codecentric Welt inspirieren.
Gemeinsam bessere Projekte umsetzen.
Wir helfen deinem Unternehmen.
Du stehst vor einer großen IT-Herausforderung? Wir sorgen für eine maßgeschneiderte Unterstützung. Informiere dich jetzt.
Hilf uns, noch besser zu werden.
Wir sind immer auf der Suche nach neuen Talenten. Auch für dich ist die passende Stelle dabei.
Blog-Autor*in
Andreas Ebbert-Karroum
Agile Principal Consultant
Du hast noch Fragen zu diesem Thema? Dann sprich mich einfach an.
Du hast noch Fragen zu diesem Thema? Dann sprich mich einfach an.