对于我的作业,我们必须创建一个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结束
我想你打算这样做:
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)。
请花些时间考虑一下代码的逻辑。
下一次,立即发布完整的代码以节省我们所有的时间。
在你最里面的循环中,你将valuesArray的所有值写入同一个没有意义的单元格,即当行为2且列为3时,首先将该单元格设置为1然后设置为222,依此类推,以99结尾。
肯定这个输出是预期的,因为在你的setCells()
方法中,你在同一个单元格上写了valuesArray[]
的所有值。
假设rows = 0和columns = 0的计数器值,并且在3rd for循环中,它将迭代所有值并将值放在相同的单元格上,即grid [0] [0] = 1则grid [0] [0] = 222这样直到最后一个值将它替换为grid [0] [0] = 99。
因此每次每个单元格只有最后一个值,即99