[英]Java 2D array rectangle error
我之前提到的問題還有一個補充。 我有一個2D數組,我需要根據周圍的4個元素得到每個元素的大小。 周圍是上,下,左,右。 如果周圍元素中的一個或多個超出數組邊界,即當前元素位於邊緣上,則將超出邊界元素視為當前元素。 我的程序僅在陣列為正方形,4 x 4,5 x 5等時才有效。 但是當它是直腸時,4 x 5,5 x 6等,我得到一個錯誤。 我相信這是因為x和y的array.length不再相同。 我不知道如何糾正這個錯誤,任何幫助將不勝感激! 這是我目前的代碼:
public class ArrayTest
{
public static int[][] buildE(int[][] array)
{
int [][] arrayE = new int[array.length][array.length];
int up;
int down;
int left;
int right;
for (int y = 0; y < array.length; y++)
{
for (int x = 0; x < array[y].length; x++)
{
//if element is on the top left
if (y == 0 && x == 0)
{
up = array[y][x];
down = array[y + 1][x];
left = array[y][x];
right = array[y][x + 1];
}
//if element is on bottom right
else if (y == array.length - 1 && x == array.length - 1)
{
up = array[y - 1][x];
down = array[y][x];
left = array[y][x - 1];
right = array[y][x];
}
//if element is on top right
else if(y == 0 && x == array.length - 1)
{
up = array[y][x];
down = array[y + 1][x];
left = array[y][x - 1];
right = array[y][x];
}
//if element is on bottom left
else if (y == array.length - 1 && x == 0)
{
up = array[y - 1][x];
down = array[y][x];
left = array[y][x];
right = array[y][x + 1];
}
//if element is on top
else if (y == 0)
{
up = array[y][x];
down = array[y + 1][x];
left = array[y][x - 1];
right = array[y][x + 1];
}
//if element is on left
else if (x == 0)
{
up = array[y - 1][x];
down = array[y + 1][x];
left = array[y][x];
right = array[y][x + 1];
}
//if element is on bottom
else if(y == array.length - 1)
{
up = array[y - 1][x];
down = array[y][x];
left = array[y][x - 1];
right = array[y][x + 1];
}
//if element is on right
else if (x == array.length - 1)
{
up = array[y - 1][x];
down = array[y + 1][x];
left = array[y][x - 1];
right = array[y][x];
}
//if element is not on an edge
else
{
up = array[y - 1][x];
down = array[y + 1][x];
left = array[y][x - 1];
right = array[y][x + 1];
}
int element = array[y][x];
int magnitude = Math.abs(element - up) + Math.abs(element - down) + Math.abs(element - left) + Math.abs(element - right);
System.out.println();
System.out.print("#####################################");
System.out.println();
System.out.println("Array Element: " + array[y][x]);
System.out.println("Up: " + up);
System.out.println("Down: " + down);
System.out.println("Left: " + left);
System.out.println("Right: " + right);
System.out.println("Magnitude: " + magnitude);
System.out.println("X: " + x);
System.out.println("Y: " + y);
System.out.println("Array Length: " + array.length);
arrayE[y][x] = magnitude;
}
}
return arrayE;
}
public static void outputArray(int[][] array)
{
for(int row = 0; row < array.length; row ++)
{
for (int column = 0; column < array[row].length; column++)
System.out.printf("%d ", array[row][column]);
System.out.println();
}
}
public static void main(String[] args)
{
int [][] myArray = {{1, 12, 13, 14, 15}, {2, 22, 23, 24, 25}, {3, 32, 33, 34, 35}, {4, 42, 43, 44, 45}, {5, 52, 53, 54, 55}, {6, 62, 63, 64, 65}};
outputArray(myArray);
outputArray(buildE(myArray));
}
}
首先,您的問題是您正在為兩個索引使用array.length
。 您應該使用array[0].length
作為y
坐標。 你在很多地方這樣做,這就是為什么它不適用於矩形。
但是,對於這種問題采取OO方法會更好。 嘗試這個:
public class ArrayTest {
public static enum Direction {
LEFT, RIGHT, UP, DOWN, SELF;
public int getValue(int[][] array, int yIndex, int xIndex) {
switch(this) {
case LEFT:
if (xIndex == 0) return array[yIndex][xIndex];
return array[yIndex][xIndex - 1];
case RIGHT:
if (xIndex == array[yIndex].length - 1) return array[yIndex][xIndex];
return array[yIndex][xIndex + 1];
case UP:
if (yIndex == 0) return array[yIndex][xIndex];
return array[yIndex - 1][xIndex];
case DOWN:
if (yIndex == array.length - 1) return array[yIndex][xIndex];
return array[yIndex + 1][xIndex];
default:
return array[yIndex][xIndex];
}
}
}
public static int[][] buildE(int[][] array) {
int [][] arrayE = new int[array.length][array[0].length];
for (int y = 0; y < array.length; y++) {
System.out.println("y = " + y);
for (int x = 0; x < array[y].length; x++) {
arrayE[y][x] = 0;
for (Direction d : Direction.values()) {
arrayE[y][x] += d.getValue(array, y, x);
}
}
}
return arrayE;
}
public static void outputArray(int[][] array) {
for(int row = 0; row < array.length; row ++) {
for (int column = 0; column < array[row].length; column++)
System.out.printf("%d ", array[row][column]);
System.out.println();
}
}
public static void main(String[] args) {
int [][] myArray = {{1, 12, 13, 14, 15}, {2, 22, 23, 24, 25}, {3, 32, 33, 34, 35}, {4, 42, 43, 44, 45}, {5, 52, 53, 54, 55}, {6, 62, 63, 64, 65}};
outputArray(myArray);
outputArray(buildE(myArray));
}
}
你是否看到讓對象為你做的工作如何消除了大量的復制粘貼代碼和if語句? 教一個對象如何做一個工作,然后說MyObject.doJob()
而不是讓你的主對象完成這項工作。 代表!
聲明:本站的技術帖子網頁,遵循CC BY-SA 4.0協議,如果您需要轉載,請注明本站網址或者原文地址。任何問題請咨詢:yoyou2525@163.com.