Why does not my repaint work?

slider2 is not connected to aPanel like slider1 is.
Repaint works well but drawOval reads the slider1 values.
Thereby there are no visual changes during the repaint.

See yourself by adding a repaint counter:

class Apanel extends JPanel
    {
        JSlider slider = null;
        int count = 0;
        Apanel(JSlider s)
        {
            slider = s;
            setSize(100, 100);
            setPreferredSize(new Dimension(100, 100));
            setBorder(new BevelBorder( BevelBorder.LOWERED ) );
            setBackground(Color.red);
        }
        public void paintComponent(Graphics g)
        {
        this.count++;
                g.setColor(Color.red);
                g.fillRect(0, 0, 100, 100);
                g.setColor(Color.black);
                g.drawOval(10, 10, slider.getValue(), slider.getValue());
        g.drawString( Integer.toString(count), 40, 40 );
        }
     
    }

Hello!

Thanks alot!
I did not saw that misstake, and I acctually did it twice, in my real app (to big to post here) and in this example app.

Thanks alot!
You really should post a "svar" so I can reward you mate!
Best regards
Fredrik

Below my new example app:
package test;

import java.awt.Color;
import java.awt.Dimension;
import java.awt.Graphics;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

import javax.swing.JButton;
import javax.swing.JDialog;
import javax.swing.JFrame;
import javax.swing.JOptionPane;
import javax.swing.JPanel;
import javax.swing.JSlider;
import javax.swing.JTextField;
import javax.swing.border.BevelBorder;
import javax.swing.event.ChangeEvent;
import javax.swing.event.ChangeListener;

public class SliderTest extends JFrame implements ActionListener
{
    JPanel panel = new JPanel();
    JSlider slider1 = null;
    JTextField textField1 = null;
    JSlider slider2 = null;
    JButton openDialog = null;
    JDialog dialog = null;
    Apanel aPanel = null;
   
   
    public SliderTest()
    {
        textField1 = new JTextField("0");
        slider1 = getSlider(textField1, this);
        aPanel = new Apanel();
        aPanel.setSize(100, 100);
        openDialog = new JButton("Open dialog");
       
        panel.add(slider1);
        panel.add(textField1);
        panel.add(openDialog);
        panel.add(aPanel);
       
        JOptionPane optionPane = new JOptionPane();

        slider2 = getSlider(textField1, this);
       
        optionPane.setMessage(new Object[] { slider2});

        dialog = optionPane.createDialog(this, "Lets see if this slider can update other window!");

        openDialog.addActionListener(this);
       
        add(panel);
        pack();
       
        setVisible(true);
    }
   
    public void repaint(JSlider slider)
    {
        aPanel.setSlider(slider);
        aPanel.repaint();
    }
   
    public JSlider getSlider(final JTextField tf, final SliderTest sliderTest)
    {
        final JSlider slider = new JSlider(JSlider.VERTICAL, 0, 10, 0);
        slider.setMajorTickSpacing(10);
        slider.setPaintTicks(true);
        slider.setPaintLabels(true);
       
   
        ChangeListener changeListener = new ChangeListener()
        {
            public void stateChanged(ChangeEvent changeEvent)
            {
                JSlider theSlider = (JSlider) changeEvent.getSource();
                tf.setText("" + theSlider.getValue() );
                sliderTest.repaint(slider);
            }
        };
        slider.addChangeListener(changeListener);
        return slider;
    }
   
    public void actionPerformed(ActionEvent e)
    {
        dialog.setVisible(true);
    }
   
    public static void main(String[] args)
    {
        new SliderTest();
    }
   
   
    class Apanel extends JPanel
    {
        JSlider slider = null;
        Apanel()
        {
            setSize(100, 100);
            setPreferredSize(new Dimension(100, 100));
            setBorder(new BevelBorder( BevelBorder.LOWERED ) );
            setBackground(Color.red);
        }
       
        public void setSlider(JSlider s)
        {
            slider = s;
        }
       
        public void paintComponent(Graphics g)
        {

                g.setColor(Color.red);
                g.fillRect(0, 0, 100, 100);
                g.setColor(Color.black);
                if(slider != null)
                {
                    g.drawOval(10, 10, slider.getValue(), slider.getValue());
                }
        }
       
    }

}

Svar