Help with image editing

Question

i have written this code for changing the brightness of the image.....but the problem is that when i use the slider the change is not taking effect.....it works if i click on the radio button...
import java.awt.*;
import java.awt.event.*;
import java.awt.image.*;
import java.awt.color.ColorSpace;
import javax.swing.border.TitledBorder;
import java.awt.BorderLayout;
import java.awt.Container;
import javax.swing.BoundedRangeModel;
import javax.swing.JFrame;
import javax.swing.JProgressBar;
import javax.swing.JSlider;
import javax.swing.event.ChangeEvent;
import javax.swing.event.ChangeListener;
import java.awt.image.*;
import javax.swing.*;
import java.awt.color.ColorSpace;
import javax.swing.border.TitledBorder;
public class ImageColorEffect extends JFrame implements ActionListener {
DisplayPanel displayPanel;
JRadioButton brightButton, darkButton, contrastIButton, contrastDButton,
reverseButton, changeButton, resetButton;
public ImageColorEffect() {
super("Color Effect Example");
Container container = getContentPane();
displayPanel = new DisplayPanel();
container.add(displayPanel);
JPanel panel = new JPanel();
panel.setLayout(new GridLayout(2, 10));
ButtonGroup group = new ButtonGroup();
brightButton = new JRadioButton("Brightness",false);
panel.add(brightButton);
group.add(brightButton);
brightButton.addActionListener(this);
JSlider aJSlider = new JSlider(JSlider.HORIZONTAL);
container.add(aJSlider, BorderLayout.SOUTH);
ChangeListener aChangeListener = new BoundedChangeListener();
BoundedRangeModel model = aJSlider.getModel();
model.addChangeListener(aChangeListener);
aJSlider.addChangeListener(aChangeListener);
// System.out.println( "valueee="+value);
container.add(BorderLayout.NORTH, panel);
addWindowListener(new WindowEventHandler());
setSize(displayPanel.getWidth(), displayPanel.getHeight() + 50);
show();
class WindowEventHandler extends WindowAdapter {
public void windowClosing(WindowEvent e) {
System.exit(0);
public static void main(String arg[]) {
new ImageColorEffect();
public void actionPerformed(ActionEvent e) {
JRadioButton rbutton = (JRadioButton) e.getSource();
if (rbutton.equals(brightButton)) {
displayPanel.brightenLUT(500);
displayPanel.applyFilter();
displayPanel.repaint();
class DisplayPanel extends JPanel {
Image disImage;
BufferedImage image;
Graphics2D graph;
LookupTable lookup;
DisplayPanel() {
setBackground(Color.black);
loadImage();
setSize(disImage.getWidth(this), disImage.getWidth(this));
createBufferedImage();
public void loadImage() {
disImage = Toolkit.getDefaultToolkit().getImage("1.jpg");
MediaTracker media = new MediaTracker(this);
media.addImage(disImage, 1);
try {
media.waitForAll();
} catch (Exception e) {}
if (disImage.getWidth(this) == -1) {
System.out.println("file not found");
System.exit(0);
public void createBufferedImage() {
image = new BufferedImage(disImage.getWidth(this), disImage
.getHeight(this), BufferedImage.TYPE_INT_ARGB);
graph = image.createGraphics();
graph.drawImage(disImage, 0, 0, this);
public void brightenLUT(int value)
System.out.println(" The brightness amount : "+value);
short brighten[] = new short[256];
for (int i = 0; i < 256; i++)
//System.out.println(i);
short pixelValue = (short) (i+value);
if (pixelValue > 255)
pixelValue = 255;
else if (pixelValue < 0)
pixelValue = 0;
brighten[i] = pixelValue;
lookup = new ShortLookupTable(0, brighten);
applyFilter();
repaint();
public void applyFilter() {
LookupOp lop = new LookupOp(lookup, null);
lop.filter(image, image);
public void update(Graphics g) {
g.clearRect(0, 0, getWidth(), getHeight());
paintComponent(g);
public void paintComponent(Graphics g) {
super.paintComponent(g);
Graphics2D g2D = (Graphics2D) g;
g2D.drawImage(image, 0, 0, this);
//repaint();
/////// The slider control
class BoundedChangeListener extends DisplayPanel implements ChangeListener
public void stateChanged(ChangeEvent changeEvent)
Object source = changeEvent.getSource();
if (source instanceof BoundedRangeModel)
BoundedRangeModel aModel = (BoundedRangeModel) source;
if (!aModel.getValueIsAdjusting())
DisplayPanel d=new DisplayPanel();
brightenLUT(aModel.getValue()) ;
System.out.println("Changed value: " + aModel.getValue());
}}

Answer 1

found the soln maself....
jus the displaypanel object had to be used to call the func.
and had to place the othr funcs in teh ImageColorEffect class!
wohoo!
//CODE
import java.awt.*;
import java.awt.event.*;
import java.awt.image.*;
import java.awt.color.ColorSpace;
import javax.swing.border.TitledBorder;
import java.awt.BorderLayout;
import java.awt.Container;
import javax.swing.BoundedRangeModel;
import javax.swing.JFrame;
import javax.swing.JProgressBar;
import javax.swing.JSlider;
import javax.swing.event.ChangeEvent;
import javax.swing.event.ChangeListener;
import java.awt.image.*;
import javax.swing.*;
import java.awt.color.ColorSpace;
import javax.swing.border.TitledBorder;
public class ImageColorEffect extends JFrame implements ActionListener {
DisplayPanel displayPanel;
JRadioButton brightButton, darkButton, contrastIButton, contrastDButton,
reverseButton, changeButton, resetButton;
public ImageColorEffect() {
super("Color Effect Example");
Container container = getContentPane();
displayPanel = new DisplayPanel();
container.add(displayPanel);
JPanel panel = new JPanel();
panel.setLayout(new GridLayout(2, 10));
ButtonGroup group = new ButtonGroup();
brightButton = new JRadioButton("Brightness",false);
panel.add(brightButton);
group.add(brightButton);
brightButton.addActionListener(this);
JSlider aJSlider = new JSlider(JSlider.HORIZONTAL);
container.add(aJSlider, BorderLayout.SOUTH);
ChangeListener aChangeListener = new BoundedChangeListener();
BoundedRangeModel model = aJSlider.getModel();
model.addChangeListener(aChangeListener);
aJSlider.addChangeListener(aChangeListener);
// System.out.println( "valueee="+value);
container.add(BorderLayout.NORTH, panel);
addWindowListener(new WindowEventHandler());
setSize(displayPanel.getWidth(), displayPanel.getHeight() + 50);
show();
class WindowEventHandler extends WindowAdapter {
public void windowClosing(WindowEvent e) {
System.exit(0);
public static void main(String arg[]) {
new ImageColorEffect();
public void actionPerformed(ActionEvent e) {
JRadioButton rbutton = (JRadioButton) e.getSource();
if (rbutton.equals(brightButton)) {
displayPanel.brightenLUT(10);
displayPanel.applyFilter();
displayPanel.repaint();
class DisplayPanel extends JPanel {
Image disImage;
BufferedImage image;
Graphics2D graph;
LookupTable lookup;
DisplayPanel() {
setBackground(Color.black);
loadImage();
setSize(disImage.getWidth(this), disImage.getWidth(this));
createBufferedImage();
public void loadImage() {
disImage = Toolkit.getDefaultToolkit().getImage("1.jpg");
MediaTracker media = new MediaTracker(this);
media.addImage(disImage, 1);
try {
media.waitForAll();
} catch (Exception e) {}
if (disImage.getWidth(this) == -1) {
System.out.println("file not found");
System.exit(0);
public void createBufferedImage() {
image = new BufferedImage(disImage.getWidth(this), disImage
.getHeight(this), BufferedImage.TYPE_INT_ARGB);
graph = image.createGraphics();
graph.drawImage(disImage, 0, 0, this);
public void brightenLUT(int value)
System.out.println(" The brightness amount : "+value);
short brighten[] = new short[256];
short pixelValue;
for (int i = 0; i < 256; i++)
//System.out.println(i);
pixelValue = (short) (i+value);
if (pixelValue > 255)
pixelValue = 255;
else if (pixelValue < 0)
pixelValue = 0;
brighten[i] = pixelValue;
for (int i = 0; i < 256; i++){
System.out.print(brighten[i]+" ");
lookup = new ShortLookupTable(0, brighten);
public void applyFilter() {
//System.out.print("hajdfh");
LookupOp lop = new LookupOp(lookup, null);
lop.filter(image, image);
public void update(Graphics g) {
g.clearRect(0, 0, getWidth(), getHeight());
paintComponent(g);
public void paintComponent(Graphics g) {
super.paintComponent(g);
Graphics2D g2D = (Graphics2D) g;
g2D.drawImage(image, 0, 0, this);
//repaint();
class BoundedChangeListener extends DisplayPanel implements ChangeListener
public void stateChanged(ChangeEvent changeEvent)
Object source = changeEvent.getSource();
if (source instanceof BoundedRangeModel)
BoundedRangeModel aModel = (BoundedRangeModel) source;
if (!aModel.getValueIsAdjusting())
displayPanel.brightenLUT(aModel.getValue()) ;
displayPanel.applyFilter();
displayPanel.repaint();
System.out.println("\nChanged value: " + aModel.getValue());
}