使用JSlider调整图像亮度：滑块双向滑动均触发亮度提升的问题

你的代码现在不管滑块往左还是往右滑，图像一个劲变亮，我帮你分析下原因，再给你修正方案：

为啥会出问题？ #

核心原因有两个：

1. 没存原始图像的备份：你每次调用brighter()都是在当前已经变亮的图像上继续操作——哪怕滑块往左滑，相当于想调暗，结果还是在亮的基础上又加亮，可不就越来越亮嘛！
2. 没判断滑块滑的方向：你压根没记录滑块之前的数值，根本不知道用户是想往右加亮还是往左调暗。
   另外还有个小坑：你在像素循环里每改一个像素就刷一次界面，这会让界面卡到飞起，完全没必要～

怎么改？ #

我们要做这几件事：

• 存一份原始图像的副本，每次调亮度都从原始图出发，不能在修改过的图上叠加操作。
• 跟踪滑块的上一次数值，判断是往右滑（要加亮）还是往左滑（不用动，或者如果想支持调暗也可以加逻辑）。
• 把刷新界面的代码移到循环外面，等所有像素都改完再刷一次。

修正后的代码示例 #

首先，在你的类里加两个成员变量：

// 存原始图像的副本，绝对不能改它！
private BufferedImage originalImage;
// 记录滑块上一次的数值，用来判断滑动方向
private int previousSliderValue;

然后在初始化图像和滑块的时候，给这俩变量赋值：

// 假设你已经加载好了显示用的image，把它复制一份存到originalImage里
originalImage = new BufferedImage(image.getWidth(), image.getHeight(), image.getType());
Graphics g = originalImage.getGraphics();
g.drawImage(image, 0, 0, null);
g.dispose();

// 把滑块初始值设为上一次数值
previousSliderValue = slider.getValue();

最后改你的ChangeListener：

slider.addChangeListener(new ChangeListener() {
    @Override
    public void stateChanged(ChangeEvent e) {
        int currentVal = slider.getValue();
        
        // 只有当滑块往右滑（当前值比上一次大）的时候，才加亮
        if (currentVal > previousSliderValue) {
            // 从原始图像复制一份临时图来改，别碰原始图
            BufferedImage tempImage = new BufferedImage(originalImage.getWidth(), originalImage.getHeight(), originalImage.getType());
            Graphics g = tempImage.getGraphics();
            g.drawImage(originalImage, 0, 0, null);
            g.dispose();
            
            // 计算要加亮几次（滑块滑了多少步就加几次）
            int brightnessSteps = currentVal - previousSliderValue;
            for (int step = 0; step < brightnessSteps; step++) {
                for (int x = 0; x < tempImage.getWidth(); x++) {
                    for (int y = 0; y < tempImage.getHeight(); y++) {
                        Color pixelColor = new Color(tempImage.getRGB(x, y));
                        pixelColor = pixelColor.brighter();
                        tempImage.setRGB(x, y, pixelColor.getRGB());
                    }
                }
            }
            
            // 把改好的图赋值给显示用的image
            image = tempImage;
            // 所有像素改完再刷新界面，不卡！
            frame.repaint();
        }
        
        // 更新上一次的滑块数值，下次判断用
        previousSliderValue = currentVal;
    }
});

进阶优化建议 #

上面的代码是适配你原来的brighter()方法，但这个方法的亮度步长是固定的，不够灵活。如果想更精准控制亮度，比如滑块往左滑能调暗，往右滑能加亮，可以自己写个亮度调整方法：

// 自定义亮度调整：brightnessFactor大于1是加亮，小于1是变暗
private BufferedImage adjustBrightness(BufferedImage src, float brightnessFactor) {
    BufferedImage dest = new BufferedImage(src.getWidth(), src.getHeight(), src.getType());
    for (int x = 0; x < src.getWidth(); x++) {
        for (int y = 0; y < src.getHeight(); y++) {
            Color color = new Color(src.getRGB(x, y));
            // 计算每个通道的亮度，别超过255（不然会变成白色）
            int r = Math.min(255, (int)(color.getRed() * brightnessFactor));
            int g = Math.min(255, (int)(color.getGreen() * brightnessFactor));
            int b = Math.min(255, (int)(color.getBlue() * brightnessFactor));
            dest.setRGB(x, y, new Color(r, g, b).getRGB());
        }
    }
    return dest;
}

然后在ChangeListener里，让滑块数值对应亮度系数，比如滑块范围设0-200，100是原始亮度：

slider.addChangeListener(new ChangeListener() {
    @Override
    public void stateChanged(ChangeEvent e) {
        int currentVal = slider.getValue();
        // 100对应原始亮度，大于100加亮，小于100变暗
        float brightnessFactor = currentVal / 100.0f;
        image = adjustBrightness(originalImage, brightnessFactor);
        frame.repaint();
        previousSliderValue = currentVal;
    }
});

这样不管滑块往哪个方向滑，都能精准控制亮度，体验会好很多～

内容的提问来源于stack exchange，提问作者Nudd3