Kann Joiner (Collector, Akkumulator) über die Funktion mit Java Guava angewendet werden?

Question

Ist es möglich, Strings über Sammlung während der Gruppierung zu sammeln? Das ist, wie es funktioniert in Java 8:

Map<String, String> discountOptions = p.getDiscountOptions().Stream() 
    .collect(groupingBy(
     this::getDiscountName, 
     Collectors.mapping(this::getValue, Collectors.joining(",")))); 

Ich bin neugierig, gibt es prägnante Art und Weise es in Google Guava zu tun? Das ist, wie ich versuche, es zu replizieren in Guava:

Map<String, Collection<String>> stringCollectionMap = Multimaps.transformValues(
    Multimaps.index(p.getDiscountOptions(), 
     new Function<DiscountOption, String>() { 
      @Override 
      public String apply(DiscountOption d) { 
       return getDiscountName(d); 
      } 
     }), 
    new Function<DiscountOption, String>() { 
     @Override 
     public String apply(DiscountOption d) { 
      return getValue(d); 
     } 
    }).asMap(); 

Map<String, String> discountOptions = Maps.transformValues(
    stringCollectionMap, 
    new Function<Collection<String>, String>() { 
     @Override 
     public String apply(Collection<String> strings) { 
      return Joiner.on(",").join(strings); 
     } 
    }); 

Answer 1

Sie sind nicht gehen zu etwas prägnanter als die Java 8 Streams API bekommen, da der Grund, es existiert, ist diese Art von Operationen zu verbessern.

Pre-Java 8 funktionale Programmierung Jury-rigged mit Guava funktionalen Dienstprogramme sein kann, aber as they warn:

Ab Java 7, funktionale Programmierung in Java kann nur durch umständliche und ausführliche Verwendung von anonymen Klassen angenähert wird .... Übermäßige Verwendung der funktionalen Programmierung Idiome der Guava kann ausführlich, verwirrend, unlesbar führen, und ineffiziente Code .... Imperative Code sollte Ihr Standard, Ihre erste Wahl als Java 7.

sein

Hier ist ein Imperativ Übersetzung des Codes:

private static final Joiner COMMA_JOINER = Joiner.on(","); 

ListMultimap<String, String> groupedByDiscountName = ArrayListMultimap.create(); 
for (DiscountOption option : p.getDiscountOptions()) { 
    groupedByDiscountName.put(getDiscountName(option), getValue(option)); 
} 

ImmutableMap.Builder<String, String> builder = ImmutableMap.builder(); 
for (Entry<String, Collection<String>> e : groupedByDiscountName.asMap().entrySet()) { 
    builder.put(e.getKey(), COMMA_JOINER.join(e.getValues()); 
} 
Map<String, String> discountOptions = builder.build(); 

Es ist kürzer und leichter zu lesen. In Anbetracht Ihrer aktuellen API ist dies etwa das Beste, was Sie mit Java tun 7.

Das heißt Sie können die erneute Prüfung Ihrer API berücksichtigen - insbesondere ist es seltsam, dass Sie statische Methoden verwenden würde (getDiscountName(), getValue()) Daten zu extrahieren von Ihrer DiscountOption Klasse - diese scheinen klare Kandidaten für Instanzmethoden zu sein. Ähnlich können Sie eine Klasse Discounts erstellen, die eine oder mehrere DiscountOption-Instanzen enthält und eine toString() bereitstellt, die Ihre durch Komma getrennte Zeichenfolge zurückgibt. Dann müssen Sie nur Ihre Objekte konstruieren und Sie sind gut zu gehen.