Mittels Streams aus Strings A B C den String A, B und C machen

[LimDul]

Gibt es eine einfache, schlanke und lesbare Möglichkeit, wenn ich einen Stream von Strings habe die zu einem String zu joinen, so dass die einzelnen Elemente mit einem Komma getrennt werden - außer dem letzten Element, wo ein und stehen soll. Sprich bei der Eingabe
[A, B, C, D] soll der String "A, B, C und D" rauskommen.

mittels Collectors.joining geht das leider nicht.

Ich kann das natürlich in eine Liste sammeln und das ausprogrammieren, das ist kein Akt, aber mich fuchst es gerade das mir keine Lösung direkt im Stream einfällt.

 

[httpdigest]

stream.collect(Collectors.joining(","))
Ach so ups, ich hätte bis zu Ende lesen sollen.

 

[LimDul]

Dann kommt A, B, C, D raus - ich will aber beim letzten kein Komma sondern ein und haben. Sprich A, B, C und D.

 

[httpdigest]

Schnell zusammengehacked:

import static java.util.Arrays.stream;
import static java.util.stream.Collectors.joining;
import static java.util.stream.Stream.of;
import java.util.stream.Stream;

public class StringStreamJoin {
    public static String join(Stream<String> stream) {
        return stream(stream
                .reduce(new String[0], (a, s) -> a.length == 0 ?
                        new String[] {s} : a.length == 1 ?
                        new String[] {a[0], s} :
                        new String[] {a[0] + ", " + a[1], s}, (a, b) -> null))
                .collect(joining(" und "));
    }

    public static void main(String[] args) {
        System.out.println(join(of("a", "b", "c")));
    }
}

 

[LimDul]

Bei der Anforderung verständlich bzw. lesbar würde ich die Lösung zwar nicht als ideal sehen, aber immerhin ist es eine Stream-Lösung

 

[Xyz1]

import java.util.*;
import java.util.stream.*;

public class MyClass {
    public static String getString(String[] sa) {
        return Arrays.stream(sa).limit(sa.length -
1).collect(Collectors.joining(", ")) + " und " +
Arrays.stream(sa).reduce((f,s) -> s).get();
    }

    public static void main(String args[]) {
      System.out.println(getString(new String[] {"Hallo", "du", "da", "!"}));
    }
}

 

[kneitzel]

Die Lösung von Tobias ist ja gut, aber den letzten Part würde ich nicht mehr als Stream machen. Da geht es ja nur um das letzte Elemente des Arrays. Und das hat man ja im direkten Zugriff und braucht da überhaupt nichts mehr drumherum: sa[sa.length-1].

Dann würde ich den code maximal noch ergänzen um Prüfungen auf die Länge von sa:
sa null oder 0 Elemente: "" zurück geben.
sa hat 1 Element: Das Element zurück geben.
sa hast > 1 Element: Die Stream-Lösung mit eben dem letzten Part nicht über Streams.

 

[Xyz1]

Danke, aber eigentlich eignen sich Streams nicht ganz so gut für diese Aufgabe. Oder wenn, dann auch mal einen Blick auf googles guava werfen.

 

[httpdigest]

Ich würde sagen, die Prämisse der Aufgabenstellung ist aber nicht erfüllt:

Gibt es eine einfache, schlanke und lesbare Möglichkeit, wenn ich einen Stream von Strings habe die zu einem String zu joinen

Man kann natürlich den Stream<String> erstmal wieder zu einem Array machen.

 

[kneitzel]

Danke, aber eigentlich eignen sich Streams nicht ganz so gut für diese Aufgabe. Oder wenn, dann auch mal einen Blick auf googles guava werfen.

Wobei ich das Arrays.stream(sa).limit(sa.length - 1).collect(Collectors.joining(", ")) durchaus verständlich und akzeptabel finde.

Ich denke, bei einem Code Review würde ich das auf jeden Fall direkt durchwinken. Und einen Code ohne Streams kann man natürlich auch bauen, aber da wäre mein aktueller Ansatz wie von httpdigest erwähnt erst einmal die Umwandlung weg von einem Stream um dann das Array zu bearbeiten.

Also ich denke, die Stream Lösung dürfte da sogar die bevorzugte Lösung sein...

 

[thecain]

Das Problem ist halt das man für sa.length sowieso den Stream in ein Array umwandeln muss.

 

[kneitzel]

Das Problem ist halt das man für sa.length sowieso den Stream in ein Array umwandeln muss.

Oh, stimmt. War mir beim drüber schauen gar nicht aufgefallen. Da beißt sich die Katze in den eigenen Schwanz

 

[Xyz1]

Also - ich habe nochmal etwas auf SO nachgelesen und für diese Aufgabenstellung(en) sind Streams eher ungeeignet.
java stream all elements except one
java stream last element

Soll heißen, mach es so wie bisher auch.