JAVA。我嵌套的for循环,我从另一个数组插入数组中的整数跳过整个部分,除了第一个和最后一个值

问题描述 投票:0回答:3

对于我的作业,我们必须创建一个Grid类,其中一个方法是setCells(int rows [],int cols [],int vals []),其中用户可以指定用户想要设置的行和列的索引带有一些特定值(在vals数组中提供)。请注意,rows,cols和vals数组的大小应相同。

这是我到目前为止所得到的。

public void setCells(int rowsArray[ ], int columnsArray[ ], int valuesArray[ ]) {
    for (int rows = 0; rows < rowsArray.length; rows++) {
        for (int columns = 0; columns < columnsArray.length; columns++) {
            for (int values = 0; values < valuesArray.length; values++) {
                setValue(rows, columns, valuesArray[values]);
            }
        }
    }
}

总的来说,我已经检查了这段是否有效。这是代码的一部分:

int[] valuesArray = new int[]{1, 222, 3, 45, 4, 99};
Grid lol = new Grid(11);

    lol.setCells(new int[5], new int[5], valuesArray);
    lol.getValue(4,4);
    lol.getValue(3,3);
    lol.getValue(2,2);
    lol.getValue(1,1);
    lol.getValue(0,0);

这是输出。

99 99 99 99 1

我认为嵌套循环的逻辑有问题。这是我编程的第一年,所以我一直在考虑这个问题3个小时。如果可以帮我找到代码中的错误,我会很高兴的。

编辑。全网格类:

    public class Grid {
private int value;
private int[][] grid;

public Grid(int height, int width) {
    if (height == width) {
        grid = new int[height][width];
    } else{
        System.out.println("Height and width should be equal");
    }
}

public Grid(int size) {
      grid = new int[size][size];
}

public boolean isEmpty() {
    boolean empty = true;
    for (int i = 0; i < grid.length; i++) {
        for (int p = 0; p < grid.length; p++) {
            if (grid[i][p] != 0) {
                empty = false;
                break;
            }
        }

        if (!empty) {
            System.out.println("Grid is not empty");
            return false;
        }
    }
    if (true)
        System.out.println("Grid is empty");
        return true;
}

public void Clear() {
    for (int i = 0; i < grid.length; i++) {
        for (int p = 0; p < grid.length; p++) {
            grid[i][p] = 0;
        }
    }
}

public boolean isValid(int rows, int columns) {
    if(grid[rows][columns] == 0){
        return true;
    } else {
        return false;
    }
}

public void setValue(int rows, int columns, int value) {
    if (isValid(rows, columns)) {
        grid[rows][columns] = value;
        } else if(!isValid(rows, columns) && rows < grid.length && columns < grid.length) {
            grid[rows+1][columns+1] = value;
        }
    }

public int getValue(int rows, int columns){
    if(rows >= 0 && rows <= grid.length && columns >= 0 && columns <= grid.length ){
        System.out.println(grid[rows][columns]);
    }
    return value;
}

public void setCells(int rowsArray[], int columnsArray[], int valuesArray[]) {
    for (int rows = 0; rows < rowsArray.length; rows++) {
        for (int columns = 0; columns < columnsArray.length; columns++) {
            for (int values = 0; values < valuesArray.length; values++) {
                setValue(rows, columns, valuesArray[values]);
            }
        }
    }
}

全部主要:

public class GridTest {
public static void main(String[] args) {

    int[] valuesArray = new int[]{1, 222, 3, 45, 4, 99};
    Grid kek = new Grid(10,10);
    Grid lol = new Grid(11);

    kek.isEmpty();
    lol.isEmpty();

    if(kek.isValid(4,5)){
        System.out.println("\nTrue");
    }
    if(lol.isValid(6,1)){
        System.out.println("\nTrue");
    }

    kek.setValue(4, 5, 52);
    kek.getValue(4,5);
    if(!kek.isValid(4,5)){
        System.out.println("False");
    }
    kek.setValue(4,5,5);
    kek.getValue(5,6);

    System.out.println();
    kek.isEmpty();

    kek.Clear();
    if(kek.isValid(4,5)){
        System.out.println("\nTrue");
    }
    kek.isEmpty();

    System.out.println();
    lol.setCells(new int[5], new int[5], valuesArray);
    lol.getValue(4,4);
    lol.getValue(3,3);
    lol.getValue(2,2);
    lol.getValue(1,1);
    lol.getValue(0,0);

全输出:

网格为空网格为空

真正

真52假5

网格不是空的

True Grid是空的

99 99 99 99 1

1

进程以退出代码0结束

java arrays multidimensional-array nested-loops
3个回答
0
投票

我想你打算这样做:

public void setCells(int rowsArray[ ], int columnsArray[ ], int valuesArray[ ]) {
    for (int i= 0; i< rowsArray.length; i++) {                  
        setValue(rowsArray[i], columnsArray[i], valuesArray[i]);    
    }
}

同样在您的测试代码中,rowsArray和valuesArray都填充了零,因为您没有设置任何值。

我希望测试代码如下:

int[] valuesArray = new int[] {12, 222, 31, 45, 42, 99};
int[] rowsArray = new int[] {5, 4, 3, 2, 1, 0};
int[] columnsArray = new int[] {5, 4, 3, 2, 1, 0};
Grid lol = new Grid(11);    
lol.setCells(rowsArray , columnsArray, valuesArray);
lol.getValue(5,5);
lol.getValue(4,4);
lol.getValue(3,3);
lol.getValue(2,2);
lol.getValue(1,1);
lol.getValue(0,0);

BTW一种更好的模型建模方法是使用一个单独的类来表示一组用户输入:

class UserInput {
    private final int row;
    private final int col;
    private final int value;

    UserInput (int row, int col, int value) { 
        this.row = row;
        this.col = col;
        this.val = val;
    }

    // getters and setters
}

每次用户输入新的row / col / val时,您都会创建一个新的UserInput实例并将其添加到列表中:

List<UserInput> inputs = new UserInput<>();    

// for each set of user input do:

inputs.add(new UserInput(row, col, val));

然后设置网格中的条目:

for (UserInput i: inputs) {                  
    setValue(i.getRow(), i.getCol(), i.getValue());    
}

您可以删除2-args构造函数,因为您的网格必须始终为方形。

你的代码非常有缺陷。我认为isValid也应该检查网格大小。什么是setValue方法应该做什么?如果单元格已填满,请增加行索引和列索引?如果那个细胞也已被填满怎么办?你不需要第二个!isValid检查else(你刚刚在if中检查了isValid)。

请花些时间考虑一下代码的逻辑。

下一次,立即发布完整的代码以节省我们所有的时间。


0
投票

在你最里面的循环中,你将valuesArray的所有值写入同一个没有意义的单元格,即当行为2且列为3时,首先将该单元格设置为1然后设置为222,依此类推,以99结尾。


0
投票

肯定这个输出是预期的,因为在你的setCells()方法中,你在同一个单元格上写了valuesArray[]的所有值。

假设rows = 0和columns = 0的计数器值,并且在3rd for循环中,它将迭代所有值并将值放在相同的单元格上,即grid [0] [0] = 1则grid [0] [0] = 222这样直到最后一个值将它替换为grid [0] [0] = 99。

因此每次每个单元格只有最后一个值,即99

© www.soinside.com 2019 - 2024. All rights reserved.