我有一个颜色数组:
public RGBImage(int width, int height, RGBColor background) {
pixel = new RGBColor[width][height];
this.w = width;
this.h = height;
this.b = background;
for(x = 0; x < width; x++){
for(y = 0; y < height; y++){
pixel[x][y] = b;
}
}
我想旋转它,对吧,由于@Oblivion Creations,代码在平方矩阵方面已经做得很好了,但当使用它非平方矩阵时,我得到了界外错误
public void rotateRight() {
RGBColor[][] mirror = new RGBColor[h][w];
for(int i = 0 ; i < h; i++){
for(int j = 0 ; j < w; j++){
mirror[i][j] = pixel[j][w-i-1];
}
}
pixel = mirror;
}
I认为问题是mirror = pixel;
共享相同的引用。。。因此,它在旋转时改变了自己,导致它做出了奇怪而美妙的事情。
我的建议是从像素复制到镜像,然后在for循环后将镜像分配给像素,如下所示:
public void rotateLeft() {
RGBColor[][] mirror = new RGBColor[w][h];
if(w == h){
for(int i = 0 ; i < h; i++){
for(int j = 0 ; j < h; j++){
mirror[i][j] = pixel[h-j-1][i];
}
}
pixel = mirror;
}
}
编辑:
对于新的RotateRight,使用的是宽度变量,而不是未旋转像素阵列上的高度。试试这个:
public void rotateRight() {
RGBColor[][] mirror = new RGBColor[h][w];
for(int i = 0 ; i < h; i++){
for(int j = 0 ; j < w; j++){
mirror[i][j] = pixel[j][h-i-1];
}
}
pixel = mirror;
}
问题是mirror = pixel
指定mirror
引用与pixel
相同的数组。它不复制数组。因此,在循环中,pixel[i][j] = mirror[h-j-1][i]
将像素从阵列的一个单元复制到同一阵列的另一个单元。