迭代2D数组错误

时间:2016-04-21 02:13:13

标签: java arrays multidimensional-array

我在使用HackerRank挑战时遇到了困难。我的代码在大多数情况下运行,但在其他情况下运行。

挑战在于找到一个2D阵列中的Max Sum,其形状为一个跨越6 x 6阵列的沙漏形状。约束是整数值-9到+9。

示例:

0 2 4 5 1 2 0 2 3 3 2 0 1 4 0 8 6 4 With 8 6 4 0 2 1 4 7 1 7 = 8 + 6 + 4 + 7 + 6 + 2 + 7 = 40 5 0 3 6 2 7 6 2 7 6 3 2 2 0 1

当我用负整数运行我的代码时,我的返回语句为0。

这是我的代码:

import java.io.*;
import java.util.*;
import java.text.*;
import java.math.*;
import java.util.regex.*;

public class Solution {
    public static int maxSumValue;
    public static int y;
    public static int maxSumHolder;


    public static void main(String[] args) {
    Scanner in = new Scanner(System.in);
    int arr[][] = new int[6][6];
    for (int i = 0; i < 6; i++) {
        for (int j = 0; j < 6; j++) {
            arr[i][j] = in.nextInt();
        }
    }
    for (int x = 0; x < 4; x++) {
        for (int y = 0; y < 4; y++){
            maxSumHolder = arr[x][y] + arr[x][y + 1] + arr[x][y + 2]
                    + arr[x + 1][y + 1] + arr[x + 2][y] + arr[x + 2][y + 1] + arr[x + 2][y + 2];
            if (maxSumHolder > maxSumValue || maxSumHolder == maxSumValue){
                maxSumValue = maxSumHolder;
            }
        }
    }
    System.out.println(maxSumValue);
}
}

欢迎任何建议,提示和/或解决方案!

2 个答案:

答案 0 :(得分:0)

您说您对替代解决方案感兴趣。为了您的兴趣,这里大量使用Java 8流。它比你的解决方案更长(效率更低),但可以说是封装逻辑而不是将它嵌入到数组计算中。

class Position {
    public static final int SIZE = 6;
    private final int row;
    private final int col;

    private Position(int row, int col) {
        this.row = row;
        this.col = col;
    }

    public static Stream<Position> all() {
        return IntStream.range(0, SIZE).boxed()
                .flatMap(row -> IntStream.range(0, SIZE)
                        .mapToObj(col -> new Position(row, col)));
    }

    public static Stream<Position> allNonEdge() {
        return all().filter(Position::notOnEdge);
    }

    private boolean notOnEdge() {
        return row > 0 && col > 0 && row < SIZE - 1 || col < SIZE - 1;
    }

    public int shapeSum(int[][] array) {
        return all().filter(this::isInShape)
                .mapToInt(pos -> pos.getVal(array))
                .sum();
    }

    private boolean isInShape(Position other) {
        int rowdiff = Math.abs(this.row - other.row);
        int coldiff = Math.abs(this.col - other.col);
        return rowdiff == 0 && coldiff == 0 || rowdiff == 1 && coldiff <= 1;
    }

    public int getVal(int[][] array) {
        return array[row][col];
    }

    public void setVal(int[][] array, int val) {
        array[row][col] = val;
    }    
}

以下是一些显示如何使用它的代码:

    Random rand = new Random();
    int[][] array = new int[Position.SIZE][Position.SIZE];
    Position.all().forEach(pos -> pos.setVal(array, rand.nextInt(100)));
    Position.allNonEdge()
            .mapToInt(pos -> pos.shapeSum(array))
            .max()
            .ifPresent(System.out::println);

答案 1 :(得分:0)

问题似乎正在发生,因为如果添加负值,那么它永远不会大于maxSumValue的原始值,它从零开始(Java将其初始化为零)默认,因为它从未被初始化为任何东西)。这里的一个简单修复是,在将maxSumHoldermaxSumValue进行比较时,只需取if (maxSumHolder > maxSumValue || maxSumHolder == maxSumValue) 的绝对值,以便将负值考虑在内。此,

if (Math.abs(maxSumHolder) > maxSumValue || Math.abs(maxSumHolder) == maxSumValue)

应改为

maxSumValue

然而,如果你的目标是找不到具有最大幅度的总和,并且你确实认为较小的正总和具有较大的权重,那么一个巨大的负数,那么我的建议是初始化{{1 Java可以容纳的最小数量。变化

public static int maxSumValue;

public static int maxSumValue = -Integer.MAX_VALUE;