Slider addChangeListener - StateChange question

Question

I'm having a hard time wording my problem, Please let me know if I don't make sense,
Below I have three snippets of code the first set of code works and achives what I want to do but I feel like it is redunedent.
The second set of code is what i have tried based off of what can be done with checkboxes and the third set of code actually uses the checkboxes to select the event source.
I feel like I am repeating my Slider Listeners over and over when I should be using a generic Listener which would have a statement inside it to compare where my selection came from. Is there a way to use event.getSource() method to compare which combobox was selected like the last piece of code below that uses checkboxes.
Snippet One
      //Start Slider Listeners
     //this is inside my constructor right now??
      sliderRed.addChangeListener(
        new ChangeListener() {
           public void stateChanged( ChangeEvent e )
              canvas.setNumber( sliderRed.getValue() );
              canvas.setRed(sliderRed.getValue());
              canvas.setGreen(sliderGreen.getValue());
              canvas.setBlue(sliderBlue.getValue());             
              displayRed.setText( String.valueOf( sliderRed.getValue() ) );
      sliderGreen.addChangeListener(
        new ChangeListener() {
           public void stateChanged( ChangeEvent e )
              canvas.setNumber( sliderGreen.getValue() );
              canvas.setRed(sliderRed.getValue());
              canvas.setGreen(sliderGreen.getValue());
              canvas.setBlue(sliderBlue.getValue());
              displayGreen.setText( String.valueOf( sliderGreen.getValue() ) );
      sliderBlue.addChangeListener(
        new ChangeListener() {
           public void stateChanged( ChangeEvent e )
              canvas.setNumber( sliderBlue.getValue() );
              canvas.setRed(sliderRed.getValue());
              canvas.setGreen(sliderGreen.getValue());
              canvas.setBlue(sliderBlue.getValue());             
              displayBlue.setText( String.valueOf( sliderBlue.getValue() ) );
      //Start Display Listeners
      displayRed = new JTextField( "0", 5 );
      displayRed.addActionListener(
         new ActionListener() {
            public void actionPerformed( ActionEvent e )
               int v = Integer.parseInt( displayRed.getText() );
               if ( v < sliderRed.getMinimum() || v > sliderRed.getMaximum() )
                  return;
               canvas.setNumber( v );
               sliderRed.setValue( v );
      displayGreen = new JTextField( "0", 5 );
      displayGreen.addActionListener(
         new ActionListener() {
            public void actionPerformed( ActionEvent e )
               int v = Integer.parseInt( displayGreen.getText() );
               if ( v < sliderGreen.getMinimum() || v > sliderGreen.getMaximum() )
                  return;
               canvas.setNumber( v );
               sliderGreen.setValue( v );
      displayBlue = new JTextField( "0", 5 );
      displayBlue.addActionListener(
         new ActionListener() {
            public void actionPerformed( ActionEvent e )
               int v = Integer.parseInt( displayBlue.getText() );
               if ( v < sliderBlue.getMinimum() || v > sliderBlue.getMaximum() )
                  return;
               canvas.setNumber( v );
               sliderBlue.setValue( v );
      //            End Display Listeners
      //********************************************Snippet Two
      ComboBoxHandler handler = new ComboBoxHandler();
      shapeJComboBox.addItemListener( handler );
  public ComboBoxHandler implements ItemListener
    public void itemStateChanged(ItemEvent e)
  }Snippet 3
   // private inner class for ItemListener event handling
   private class CheckBoxHandler implements ItemListener
      private int valBold = Font.PLAIN; // controls bold font style
      private int valItalic = Font.PLAIN; // controls italic font style
      // respond to checkbox events
      public void itemStateChanged( ItemEvent event )
         // process bold checkbox events
         if ( event.getSource() == boldJCheckBox )
            valBold =
               boldJCheckBox.isSelected() ? Font.BOLD : Font.PLAIN;
         // process italic checkbox events
         if ( event.getSource() == italicJCheckBox )
            valItalic =
               italicJCheckBox.isSelected() ? Font.ITALIC : Font.PLAIN;
         // set text field font
         textField.setFont(
            new Font( "Serif", valBold + valItalic, 14 ) );
      } // end method itemStateChanged
   } // end private inner class CheckBoxHandler

Answer 1

camickr Thanks for the tip about SSCCE
I am building a drawing program in this code I have a drawstring where shape objects would be. In the program I am running I have a JFrame just like this which places three panels at the north center and south, when you change the slider bars the text color changes which is what I want it to do, I just want to know if I'm wasting space and or memory by repeating the slider and display listeners. Are there shorter ways to code this? ie. the listeners
import javax.swing.*;
import javax.swing.event.*;
import java.awt.*;
import java.awt.event.*;
import java.awt.Color;
public class RGBSlider extends JFrame {  
   private DrawCanvas canvas;
   private OvalPanel oval;
   private JCheckBox fillCheck;
   private JComboBox selectShapes;
   //shows user the color value of slider
   private JTextField displayRed, displayGreen, displayBlue;  
   //Slider Labels
   private JLabel labelRed, labelGreen, labelBlue;  
   //Sliders
   private JSlider sliderRed, sliderGreen, sliderBlue;  
   private JPanel p;  
   private JPanel topPanel;
   public RGBSlider()
      super( "DrawValue" );     
      getContentPane().add( oval, BorderLayout.CENTER );
      oval.setVisible(false);
      //labels for sliders show color
      labelRed = new JLabel( "Red:" );
      labelGreen = new JLabel( "Green:" );
      labelBlue = new JLabel( "Blue:" );
      topPanel = new JPanel();
      topPanel.setLayout( new GridLayout(1,2));     
      p = new JPanel();
      p.setLayout( new GridLayout( 3, 3 ) );
      canvas = new DrawCanvas();
      sliderRed = new JSlider( SwingConstants.HORIZONTAL,
                              0, 255, 1 );
      sliderGreen = new JSlider( SwingConstants.HORIZONTAL,
                              0, 255, 1 );
      sliderBlue = new JSlider( SwingConstants.HORIZONTAL,
                              0, 255, 1 );
      sliderRed.setPaintTicks(true);
      sliderGreen.setPaintTicks(true);
      sliderBlue.setPaintTicks(true);
      sliderRed.setMajorTickSpacing( 1 );
      sliderGreen.setMajorTickSpacing( 1 );
      sliderBlue.setMajorTickSpacing( 1 );
  //**************  Question       ***********
      //  Is this a waste of space can I condense these listeners into one
     //  and use something like event.getSource to change the actions
      sliderRed.addChangeListener(
        new ChangeListener() {
           public void stateChanged( ChangeEvent e )
              canvas.setNumber( sliderRed.getValue() );
              canvas.setRed(sliderRed.getValue());
              canvas.setGreen(sliderGreen.getValue());
              canvas.setBlue(sliderBlue.getValue());             
              displayRed.setText( String.valueOf( sliderRed.getValue() ) );
      sliderGreen.addChangeListener(
        new ChangeListener() {
           public void stateChanged( ChangeEvent e )
              canvas.setNumber( sliderGreen.getValue() );
              canvas.setRed(sliderRed.getValue());
              canvas.setGreen(sliderGreen.getValue());
              canvas.setBlue(sliderBlue.getValue());
              displayGreen.setText( String.valueOf( sliderGreen.getValue() ) );
      sliderBlue.addChangeListener(
        new ChangeListener() {
           public void stateChanged( ChangeEvent e )
              canvas.setNumber( sliderBlue.getValue() );
              canvas.setRed(sliderRed.getValue());
              canvas.setGreen(sliderGreen.getValue());
              canvas.setBlue(sliderBlue.getValue());             
              displayBlue.setText( String.valueOf( sliderBlue.getValue() ) );
      //Start Display Listeners
      displayRed = new JTextField( "0", 5 );
      displayRed.addActionListener(
         new ActionListener() {
            public void actionPerformed( ActionEvent e )
               int v = Integer.parseInt( displayRed.getText() );
               if ( v < sliderRed.getMinimum() || v > sliderRed.getMaximum() )
                  return;
               canvas.setNumber( v );
               sliderRed.setValue( v );
      displayGreen = new JTextField( "0", 5 );
      displayGreen.addActionListener(
         new ActionListener() {
            public void actionPerformed( ActionEvent e )
               int v = Integer.parseInt( displayGreen.getText() );
               if ( v < sliderGreen.getMinimum() || v > sliderGreen.getMaximum() )
                  return;
               canvas.setNumber( v );
               sliderGreen.setValue( v );
      displayBlue = new JTextField( "0", 5 );
      displayBlue.addActionListener(
         new ActionListener() {
            public void actionPerformed( ActionEvent e )
               int v = Integer.parseInt( displayBlue.getText() );
               if ( v < sliderBlue.getMinimum() || v > sliderBlue.getMaximum() )
                  return;
               canvas.setNumber( v );
               sliderBlue.setValue( v );
      //            End Display Listeners
      //add components
      p.add( labelRed );
      p.add(sliderRed);
      p.add( displayRed );
      p.add( labelGreen );
      p.add(sliderGreen);
      p.add( displayGreen );
      p.add( labelBlue );
      p.add(sliderBlue);
      p.add( displayBlue );
      topPanel.add(selectShapes);
      topPanel.add(fillCheck);
      getContentPane().add(topPanel, BorderLayout.NORTH);
      getContentPane().add( canvas, BorderLayout.CENTER );
      getContentPane().add( p, BorderLayout.SOUTH );
      setSize( 500, 500 );
      show();
   public static void main( String args[] )
      RedSlider app = new RedSlider();
      app.addWindowListener(
         new WindowAdapter() {
            public void windowClosing( WindowEvent e )
               System.exit( 0 );
//          Canvas Class
class DrawCanvas extends JPanel {
   private int number;
   private Font f;
   private int red;
   private int green;
   private int blue;
   private int shape;
   private int fill;
   private Color color;
   public DrawCanvas()
      setSize( 200, 200 );
      f = new Font( "Serif", Font.BOLD, 33 );
   public void setRed( int c){
       red = c;
       repaint();
   public int getRed( int r){return red;}
   public void setGreen( int c){
       green = c;
       repaint();
   public int getGreen( int r){return green;}
   public void setBlue( int c){
       blue = c;      
       repaint();
   public int getBlue( int r)   {return blue;}
   public void setNumber( int n ){
      number = n;
      repaint();
   public void setShape(int n){
       shape = n;
   public void setColor( int n){
       fill = n;
   public void paintComponent( Graphics g )
      super.paintComponent( g );
      color = new Color( red,green, blue);
      g.setColor (color);
      g.drawString( String.valueOf( number ),
                    getSize().width / 2 - 40,
                    getSize().height / 2 );
}