java - 在 java 中编辑像素值

Question

我有一个 Pixel 类，它有方法 getRed、getBlue、getGreen、setRed、setBlue、setGreen (r,g,b)到目前为止，一切都单独工作，但是当我把它放在一起时，setGreen 和 setBlue 不起作用。我在屏蔽方面做错了什么吗？

public class Pixel {
    int pixel;
    public Pixel (int pixel) {
        this.pixel = pixel;
    }
    public int getRed() {
        int red = pixel >> 16;
        red = red & (0xFF);
        return red;   
    }
    public int getGreen() {
        int green = pixel >> 8;
        green = green & (0xFF);
        return green;
    }
    public int getBlue() {
        int blue = pixel;
        blue = blue & (0xFF);
        return blue;
    }
    public void setRed(int red) {
        pixel = (pixel & ~(0xFFFF0000)) << 16;
        pixel= ((red << 16))|pixel ;   
    }
    public void setGreen(int value) {
        pixel = (pixel & ~(0xFF00FF00)) << 8;
        pixel= (value << 8) |pixel; 
    }
    public void setBlue(int value) {
        pixel = (pixel & ~(0xFFFFFF00));
        pixel= (value) |pixel; 
    }
    public static void main(String[] args) {
        Pixel p3 = new Pixel(0xFF000000);
        System.out.printf("rgb = (%d, %d, %d)\n", p3.getRed(), p3.getGreen(), p3.getBlue());

        p3.setRed(42);
        p3.setGreen(18);
        p3.setBlue(225);
        System.out.printf("rgb = (%d, %d, %d)\n", p3.getRed(), p3.getGreen(), p3.getBlue());

        p3.setRed(-1);
        p3.setGreen(500);
        p3.setBlue(1000);
        System.out.printf("rgb = (%d, %d, %d)\n", p3.getRed(), p3.getGreen(), p3.getBlue());
    }   
}

我的结果应该是这样的:

rgb = (0, 0, 0)
rgb = (42, 18, 225)
rgb = (255, 244, 232)

但这就是我得到的

rgb = (0, 0, 0)
rgb = (0, 0, 225)
rgb = (0, 3, 232)

Answer 1

例如考虑您的 setGreen方法:

public void setGreen(int value) {
    pixel = (pixel & ~(0xFF00FF00)) << 8;
    pixel = (value << 8) | pixel; 
}

最初，您的 pixel是AARRGGBB .首先，您要设置 GG当前值 pixel 的一部分正确地为 0(暂时忽略 alpha)。 (pixel & ~(0xFF00FF00)) == (AARRGGBB & 0x00FF00FF) == 00RR00BB .然而，接下来你将该值向左移动 8 位，所以你得到 RR00BB00 .接下来，您(正确地)将作为参数获得的绿色值左移。你会得到 (000000GG << 8) == 0000GG00 .最后，您将其与之前的值进行或运算:(RR00BB00 | 0000GG00) == RR00XX00 X在哪里es 是一些垃圾，它们源自旧蓝色与新绿色位的“或”运算。你能看到要修复什么吗？

setBlue 中还有另一个问题方法。 (pixel & ~(0xFFFFFF00)) == (AARRGGBB & 000000FF) == 000000BB .将其与 setGreen 的(正确)掩码进行比较以上。

最后，您的方法似乎应该优雅地处理无效值 (–1, 500, 1000)。因此，您应该将参数的最后 8 位以外的所有位都设置为 0。我只是建议这样做，因为您所需的输出似乎要求这样做。在编写良好的 Java 程序中，最好在传递超出范围的值时抛出异常，而不是默默地忽略无效位。或者，您可以更改方法以接受 byte而不是 int .这将是最好的解决方案，因为现在完全不可能传递无效参数。

将这些放在一起，这就是它的工作原理。我还稍微完善了代码，希望我们能更容易地看到发生了什么。移动 0 位当然是无用的，但出于光学原因我将它们包括在内。

public final class Pixel {

    private int value;

    public Pixel (int rgb) {
        this.value = value;
    }

    public int getRed() {
        return (this.value >> 16) & 0xFF;
    }

    public int getGreen() {
        return (this.value >>  8) & 0xFF;
    }

    public int getBlue() {
        return (this.value >>  0) & 0xFF;
    }

    public void setRed(final int r) {
        this.value = (this.value & 0xFF00FFFF) | ((r & 0xFF) << 16);
    }

    public void setGreen(final int g) {
        this.value = (this.value & 0xFFFF00FF) | ((g & 0xFF) <<  8);
    }

    public void setBlue(final int b) {
        this.value = (this.value & 0xFFFFFF00) | ((b & 0xFF) <<  0);
    }

    @Override
    public String toString() {
        return String.format("rgb = (%3d, %3d, %3d)",
                             this.getRed(),
                             this.getGreen(),
                             this.getBlue());
    }

    public static void main(String[] args) {
        final Pixel pixel = new Pixel(0);
        System.out.println(pixel);
        pixel.setRed(42);
        pixel.setGreen(18);
        pixel.setBlue(225);
        System.out.println(pixel);
        pixel.setRed(-1);
        pixel.setGreen(500);
        pixel.setBlue(1000);
        System.out.println(pixel);
    }
}

运行上述程序的输出:

rgb = (  0,   0,   0)
rgb = ( 42,  18, 225)
rgb = (255, 244, 232)