有没有更好的方法在Java中编写这个方法 - 随机生成器

Question

我有这个方法，它接受一组微调器值。它获取这些值并将它们添加到具有 9 个随机数字的前缀中。

到目前为止，我有很多与每个微调值相关的前缀。

每个值都会检查它是否大于 0，循环遍历值的数量，然后添加前缀 + 9 位数字。例如，R1 旋转器值 = 10，则数组会以 R1+9DIGITS 吐出 10 次

if 语句太多了，我相信会有更好的方法。您会看到下面的内容极其重复。任何帮助或想法将不胜感激。谢谢！

    Spinner<Integer>[] spinners = new Spinner[]{
                r1Spinner, r2Spinner, r3Spinner, r5Spinner, rd2Spinner, rd8Spinner,m1Spinner,gSpinner, srSpinner, lcSpinner, singleURNSpinner,
                customPrefixSpinner, prefixSpinner};

     public static String[] randomPrefixGenerator(int[] values){

        int R1 = values[0];
        int R2 = values[1];
        int R3 = values[2];
        int R5 = values[3];
        int RD2 = values[4];
        int RD8 = values[5];
        int M1 = values[6];
        int G = values[7];
        int SR = values[8];
        int LC = values[9];

        String[] urnR1 = new String[R1];
        String[] urnR2 = new String[R2];
        String[] urnR3 = new String[R3];
        String[] urnR5 = new String[R5];
        String[] urnRD2 = new String[RD2];
        String[] urnRD8 = new String[RD8];
        String[] urnG = new String[G];
        String[] urnM1 = new String[M1];
        String[] urnLC = new String[LC];
        String[] urnSR = new String[SR];

        if (R1 >= 0){
            for (int i = 0; i < R1 ; i++) {
                urnR1[i] = "R1" + random9digits();
            }
        }
        if (R2 >= 0){
            for (int i = 0; i < R2 ; i++) {
                urnR2[i] = "R2" + random9digits();
            }
        }
        if (R3 >= 0){
            for (int i = 0; i < R3 ; i++) {
                urnR3[i] = "R3" + random9digits();
            }
        }
        if (R5 >= 0){
            for (int i = 0; i < R5 ; i++) {
                urnR5[i] = "R5" + random9digits();
            }
        }
        if (RD2 >= 0){
            for (int i = 0; i < RD2 ; i++) {
                urnRD2[i] = "RD2" + random9digits();
            }
        }
        if (RD8 >= 0){
            for (int i = 0; i < RD8 ; i++) {
                urnRD8[i] = "RD8" + random9digits();
            }
        }
        if (M1 >= 0){
            for (int i = 0; i < M1 ; i++) {
                urnM1[i] = "M1" + random9digits();
            }
        }
        if (G >= 0){
            for (int i = 0; i < G ; i++) {
                urnG[i] = "G" + random9digits();
            }
        }
        if (LC >= 0){
            for (int i = 0; i < LC ; i++) {
                urnLC[i] = "LC" + random9digits();
            }
        }
        if (SR >= 0){
            for (int i = 0; i < SR ; i++) {
                urnSR[i] = "SR" + random9digits();
            }
        }
        return Arrays.stream(mergeArray(urnR1,urnR2, urnR3,urnR5, urnRD2, urnRD8, urnG, urnM1, urnLC, urnSR)).toArray(String[]::new);
    }

Answer 1

我不知道合并数组功能是做什么的。但我认为你可以使用下面的代码稍微优化你的代码：

public static String[] randomPrefixGenerator(int[] values) {
        
        // we can use a map to retrieve your prefixes by their index
        Map<Integer, String> prefixNameByIndex = Map.of(
                0, "R1",
                1, "R2",
                2, "R3",
                3, "R5",
                4, "RD2",
                5, "RD8",
                6, "M1",
                7, "G",
                8, "SR",
                9, "LC"
        );
        
        // this stream goes through value 0 to 9
        return IntStream.rangeClosed(0, 9)
                // we need objects for the flatmap to work
                .boxed()
                // flatmap so we get a single list of strings
                .flatMap(generatePrefixNamesForIndex(values, prefixNameByIndex))
                .collect(Collectors.toList())
                // the to array call as last
                .toArray(String[]::new);
        
}

private static Function<Integer, Stream<? extends String>> generatePrefixNamesForIndex(int[] values, Map<Integer, String> prefixNameByIndex) {
        return index -> {
            // we can retrieve the prefix from the map by its index
            var prefixValue = prefixNameByIndex.get(index);
            // retrieve te number of iterations from parameters given in the method call
            return IntStream.range(0, values[index])
                    .mapToObj(i -> prefixValue + random9digits());
        };
}