Java variable defined inside a loop doesn't seem to be recognized outside the loop?

后端 未结 4 1282
北荒
北荒 2021-01-20 10:54

I have a section of code that is puzzling me. I define an integer array inside an if/else statement because the length of the array depends on the length of 2 inputs to the

相关标签:
4条回答
  • 2021-01-20 11:12

    Well, you have already got the solution, but I would like to point out that, you can reduce your methods, to avoid duplicating codes, which you are currently doing.

    You can make use of conditional operators to create the array according to the result of L1 > L2. And rather than iterating till L1 or L2, you should iterate till the length of array output.

    So, you can try using the below code: -

    public String addArrays(int [] arg1, int [] arg2) {
        int L1 = arg1.length;
        int L2 = arg2.length;
    
        int[] output = L1 > L2 ? new int[L2]: new int[L1];
    
        for (int i = 0; i < output.length; i++) {
            output[i] = arg1[i] + arg2[i];
        }
    
        return Arrays.toString(output);
    }
    

    And please follow Java Naming Conventions. Method name should start with lowercase alphabets.

    0 讨论(0)
  • 2021-01-20 11:18

    Define output before if statement. Like this:

    int[] output;
    int L1 = arg1.length;
    int L2 = arg2.length;
    if (L1 > L2) {
        output = new int[L2];
        for (int i = 0; i < L2; i++) {
            output[i] = arg1[i] + arg2[i];
        }
    } else {
        output = new int[L1];
        for (int i = 0; i < L2; i++) {
            output[i] = arg1[i] + arg1[i];
        }
    }
    String result = Arrays.toString(output);
    return result;
    }
    

    When you declared output inside the if statement it simply had only that block scope.

    0 讨论(0)
  • 2021-01-20 11:19

    Scope of a variable is always the next enclosing { }.

    Of cause starting at its declaration (not at the {)

    0 讨论(0)
  • 2021-01-20 11:20

    You are declaring the output variable as a local variable inside each of the if/else statements. To fix this, declare it first outside then adjust it and return the results. This keeps it in the scope between the brackets { }.

    public String AddArrays(int [] arg1, int [] arg2) {
        int L1 = arg1.length;
        int L2 = arg2.length;
        int[] output;
        if (L1 > L2) {
            output = new int[L2];
            for (int i = 0; i < L2; i++) {
                output[i] = arg1[i] + arg2[i];
            }
        } else {
            output = new int[L1];
            for (int i = 0; i < L2; i++) {
                output[i] = arg1[i] + arg1[i];
            }
        }
        String result = Arrays.toString(output);
        return result;
    }
    
    0 讨论(0)
提交回复
热议问题