Problem in repaint

[Neeba Rebbaca]

When i click on the button from bottom panel, top panel need to be redrawn. But redrawing is not happening. It happens only when resizing the applet. Please someone guide me in fixing this issue.TIA.

public class Graphics_Chart(){

public init(){
	JScrollPane topPane = new JScrollPane(new ImagePanel2());
	topPane.setHorizontalScrollBarPolicy(JScrollPane.HORIZONTAL_SCROLLBAR_NEVER);
	topPane.setVerticalScrollBarPolicy(JScrollPane.VERTICAL_SCROLLBAR_NEVER);

	JScrollPane bottomPane = new JScrollPane();
	bottomPane.getViewport().add(new ImagePanel());//,BorderLayout.CENTER);

	JSplitPane main = new JSplitPane(JSplitPane.VERTICAL_SPLIT,topPane,bottomPane);
	main.setDividerLocation(60);
	main.setDividerSize(0);
	getContentPane().add(main);

	JScrollBar  topBar      = topPane.getHorizontalScrollBar();
	JScrollBar  bottomBar   = bottomPane.getHorizontalScrollBar();

	MyChangeListener listener = new MyChangeListener(topBar.getModel());
	bottomBar.getModel().addChangeListener( listener );
}
private final class ImagePanel extends JPanel{
//Some code goes here....
	@Override
    protected void paintComponent(Graphics g) {
		super.paintComponent(g);
		g.create();
		Graphics2D g2 = (Graphics2D) g;
		g2.setRenderingHint(RenderingHints.KEY_ANTIALIASING, RenderingHints.VALUE_ANTIALIAS_ON);
		this.setBackground(new Color(240,242,247));
		g2.setComposite(makeComposite(1F));
	}
	public void actionPerformed(ActionEvent e) {

		if (e.getActionCommand().equalsIgnoreCase("Show")) {
			weeklyView = true;
			new ImagePanel2(weeklyView);//Calling the panel on click of button
			weeklyView_option_Panel.setVisible(false);
			getData(value);
			mi_save.setEnabled(true);
			setPreferredSize(new Dimension(1270, getHeight()));
			revalidate();
			repaint();
		}
}
}
private final class ImagePanel2 extends JPanel{
//some code goes here
public ImagePanel2(boolean view){
	weeklyView = view;
	panel();
    validate();
}
public void panel(){
	setPreferredSize(new Dimension(850, 50));
}
@Override
protected void paintComponent(Graphics g) {
	super.paintComponent(g);
	Graphics2D g2 = (Graphics2D) g;
	g2.setRenderingHint(RenderingHints.KEY_ANTIALIASING, RenderingHints.VALUE_ANTIALIAS_ON);

	System.out.println("repainting");
	if (weeklyView) {
		drawWeekly_Calendar(g2);
	} else {
		drawCalendar(g2);
	}
}
}

[Rob Camick]

new ImagePanel2(weeklyView);

The above line of code does nothing. You create a panel, but that panel just sits in memory because you didn't add the panel to the frame.

repaint();  

The repaint() will only paint the panel that contains the button. If you also want to repaint a different panel then you probably need to invoke repaint() on the parent panel that contains the two panels. Something like:

getParent().repaint();

[Neeba Rebbaca]

ImagePanel2() is already added to the parent. now i want it to reload based on changes.
changed the code,

ImagePanel2 obj = new ImagePanel2();
obj.repaint();

even after that the panel is not getting refreshed. but its refreshing if i resize the panel.

[Rob Camick]

The updated code you posted still does nothing. The panel is still not added to the frame. However, you should NOT even be trying to create a new panel.

now i want it to reload based on changes

Then add "setter" methods to change the properties of your panel.

[Neeba Rebbaca]

is has been already added in init method. I have modified my code for easy understanding and given here.

/*
 * To change this template, choose Tools | Templates
 * and open the template in the editor.
 */
package jscc_serialPort;

import java.awt.*;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import javax.swing.*;

public class MainPanel extends JApplet {

    boolean changeColor;
    @Override
    public void init() {
        setPreferredSize(new Dimension(1500, 600));
        
        JScrollPane topPane = new JScrollPane(new Panel1());//Panel1 added to scrollpane(toppane)
        topPane.setHorizontalScrollBarPolicy(JScrollPane.HORIZONTAL_SCROLLBAR_NEVER);
        topPane.setVerticalScrollBarPolicy(JScrollPane.VERTICAL_SCROLLBAR_NEVER);
        
        JScrollPane bottomPane = new JScrollPane(new Panel2());//Panel2 added to scrollpane(bottompane)
        //.getViewport().add(new Panel2());//
        
        JSplitPane main = new JSplitPane(JSplitPane.VERTICAL_SPLIT,true,topPane,bottomPane);//Both top and bottom pane added to splitpane
        main.setDividerLocation(50);
        getContentPane().add(main);//split pane is added to applet
        
        JScrollBar  topBar      = topPane.getHorizontalScrollBar();
        JScrollBar  bottomBar   = bottomPane.getHorizontalScrollBar();
        
        MyChangeListener listener = new MyChangeListener(topBar.getModel());
        bottomBar.getModel().addChangeListener( listener );
    }
    class Panel1 extends JPanel{
        public void panel() {
            setPreferredSize(new Dimension(2100,150));
        }
        Panel1() {
            //setLayout(null);
            panel();
        }
        
        @Override
        protected void paintComponent(Graphics g) {
            super.paintComponent(g);
            g.create();
            Graphics2D g2 = (Graphics2D) g;
            g2.setRenderingHint(RenderingHints.KEY_ANTIALIASING, RenderingHints.VALUE_ANTIALIAS_ON);
            //g2.setColor(getBackground());
            this.setBackground(new Color(240,242,247));
            if(changeColor){//changeColor is set to true on button click present in panel2
                g2.setColor(Color.red);
            }
            else{
                g2.setColor(Color.BLUE);
            }
            g2.drawString("ss", 30, 10);
            g2.drawString("ss", 2000, 10);
        }
   }
    class Panel2 extends JPanel  implements ActionListener{
        public void panel() {
            //setLayout(null);
            setPreferredSize(new Dimension(2100, 1800));
        }
        Panel2() {
            panel();
            addButton();
        }
        @Override
        protected void paintComponent(Graphics g) {
            super.paintComponent(g);
            g.create();
            Graphics2D g2 = (Graphics2D) g;
            g2.setRenderingHint(RenderingHints.KEY_ANTIALIASING, RenderingHints.VALUE_ANTIALIAS_ON);
            //g2.setColor(getBackground());
            this.setBackground(new Color(240,242,247));
            g2.setColor(Color.red);
            g2.drawString("dd", 30, 10);
        }
        void addButton(){
            JButton button  = new JButton("Click");
            button.setLocation(0, 10);
            button.addActionListener(this);
            this.add(button);
            
        }
        @Override
        public void actionPerformed(ActionEvent e) {
            if(e.getActionCommand().equals("Click")){
                System.out.println("Clicked....");
                changeColor = true;
                Panel1 obj = new Panel1();
                obj.repaint();
            }
        }
    }

}

[Neeba Rebbaca]

Any leads?? TIA.

[Paul Clapham]

Any leads? You seem to have been ignoring the perfectly good leads which Rob Camick gave you. As for the revised code you posted, all I can see is that you have changed the names of your panel classes. That just makes it harder to figure out your problem, which by the way your last post didn't describe.

[Neeba Rebbaca]

Ok, i will remove the new code added.

[Neeba Rebbaca]

removed code.and continue with previous one.