无法在流java中的forEach循环内部打印

Question

方法"组合"；应该组合输入数组。我喜欢得到这个组合流并将其保存到一个文件中。

public void writeDot() {
try (PrintStream out = new PrintStream(path, "UTF-8")) {
out.print("digraph {n");
String[] arr = {"hyo", "ji", "yoo", "mi", "vi", "se", "ari"};
combination(arr, 2, 0, new String[2])
.stream()
.map(a -> Arrays.toString(a).join(" -> "))
.forEach(out::print);
out.println(";n");
out.println("}");
} catch (IOException ioException) {
ioException.printStackTrace();
}
}

组合方法如下：

public List<String[]> combination(String[] arr, int len, int startPosition, String[] result) {
if (len == 0) {
//System.out.println(Arrays.toString(result));
return null;
}
for (int i = startPosition; i <= arr.length - len; i++) {
result[result.length - len] = arr[i];
combination(arr, len - 1, i + 1, result);
list.add(result);
}
return list;
}

我预期的结果是：

digraph {
hyo -> ji;
ji -> hyo;
and so on..
}

但我只得到：

digraph {
;
}

我的代码出了什么问题？请帮帮我。

Answer 1

String.join是一个静态方法，它接受两个参数：分隔符和元素数组。

并没有传递任何元素，所以结果是空字符串。

正确的代码是：

combination(arr, 2, 0, new String[2])
.stream()
.map(a->String.join(" -> ", a))
.forEach(out::print);

Answer 2

您的组合方法似乎存在一些问题，这些问题对您的案例没有帮助，因为您递归调用它时只生成一个新的结果数组。相反，尝试一种更好的组合方法——最好使用一个经过良好测试的库，比如apachecommons或guava。

public static List<String[]> makePairsFromArray(String[] arr) {
List<String[]> list = new ArrayList<>();
for(int i = 0; i < arr.length - 1; i++) {
for(int j = i + 1; j < arr.length; j++) {
String[] pair = new String[2];
pair[0] = arr[i]; 
pair[1] = arr[j];
list.add(pair);
String[] opp = new String[2];
opp[0] = arr[j];
opp[1] = arr[i];
list.add(opp);
}
}
return list;
}

此外，您还需要更新talex所描述的map(join)方法调用。