Problem with JSplitPane and JFrame

[dziubut dziubucinski]

Hi all I'm stuck...
I have a frame and jsplitpane on it. JSplitPane has 2 components: (left)panel with several labels and buttons, and (right) jscrollpane which has panel with bufferedimage on it (i draw img using g2.drawImage(...)). Everything works perfect till i try to resize frame - left jsplitpane's component is tryin to disappear (blinkin) and image is moving to left edge of frame. I set left panel (and also scrollpane) minimum size but it doesnt work. Problem appears only when i touch frame..
I would appreciate for any suggestions.

[Darryl Burke]

To get better help sooner, post a SSCCE that demonstrates your problem.

[dziubut dziubucinski]

   

import java.awt.Dimension;
import java.awt.EventQueue;
import java.awt.Font;
import java.awt.Graphics;
import java.awt.Graphics2D;
import java.awt.RenderingHints;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.image.BufferedImage;
import java.awt.*;
import java.io.File;
import java.io.IOException;
import javax.imageio.ImageIO;
import javax.swing.*;
import javax.swing.filechooser.FileFilter;


public class test {

	
	public static void main(String[] args) {
		
		EventQueue.invokeLater(new Runnable()
				{
		public void run() {
		ImgFrame frame = new ImgFrame();
		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		frame.setVisible(true);

						  }
				}
		);
	}
}

class ImgFrame extends JFrame
{

	private static final int DEFAULT_WIDTH = 800;
	private static final int DEFAULT_HEIGHT = 500;
	private JLabel label;
	private JMenu menu;
	private JMenuBar bar;
	private JMenuItem open;
	private JMenuItem save;
	private JMenuItem end_program;
	private JFileChooser chooser;
	private JButton convert;
	private JComboBox combo;
	private JLabel label1, label2, label3,label4, label5;
	private JPanel panel;
	private JSplitPane split;
	private int iw,ih;
	private long space;
	private BufferedImage img;
	private JScrollPane p;
	private MyPanel panelik;
	
	public ImgFrame()
	{
		
		setTitle("Image Converter");
		setSize(DEFAULT_WIDTH,DEFAULT_HEIGHT);
		
		Dimension mindd = new Dimension(300,500);
		bar=new JMenuBar();
		menu=new JMenu("Plik");
		open=new JMenuItem("Otwórz");
	//	save=new JMenuItem("Zapisz");
		end_program=new JMenuItem("Zakończ");
		split=new JSplitPane(JSplitPane.HORIZONTAL_SPLIT);
		
		menu.add(open);
	//	menu.add(save);
		menu.add(end_program);
		bar.add(menu);
		setJMenuBar(bar);
		add(split);
	
		
		open.addActionListener(new ActionListener() {
			
			public void actionPerformed(ActionEvent e) {
				choose();
			}
		});
	/*	end_program.addActionListener(new ActionListener() {
			
			public void actionPerformed(ActionEvent e) {
				System.exit(0);
			}
		});
	*/
		
	}
	
	public void choose()
	{
		split.removeAll();
		chooser = new JFileChooser();
		//chooser.setCurrentDirectory(new File("C:\\Documents and Settings\\piotrek\\Pulpit"));
		chooser.setFileFilter(new FileFilter() {
			
			public String getDescription() {
				
				return "zdjęcia";
			}
			
			public boolean accept(File f) {
				
				return f.isDirectory() ||
				f.getName().toLowerCase().endsWith("jpg") ||
				f.getName().toLowerCase().endsWith("jpeg") ||
				f.getName().toLowerCase().endsWith("bmp");
			}
		});
		int r=chooser.showOpenDialog(this);
		if(r != JFileChooser.APPROVE_OPTION)
			return;
		
		File file = chooser.getSelectedFile();
		String fname = file.getName();
	
		try
		{
		Dimension mind = new Dimension(300,500);
		img = ImageIO.read(file);
		
		//iw=img.getWidth(observer)
		//ih=img.getHeight();
		space=file.length()/1000;
		System.out.print(iw+" "+ih+" "+space);
		String []imgSizes={"800","1024"};
		panelik = new MyPanel(img);
		p = new JScrollPane(panelik, JScrollPane.VERTICAL_SCROLLBAR_AS_NEEDED,
                JScrollPane.HORIZONTAL_SCROLLBAR_AS_NEEDED);
		panelik.setMinimumSize(mind);
		p.setPreferredSize(mind);
		Font st1 = new Font ("Arial",Font.BOLD,12);
		Font st2 = new Font ("Arial",Font.ITALIC,12);
		
		panel=new JPanel();
		panel.setLayout(null);
		convert=new JButton("Konwertuj obraz");
		convert.setBounds(0, 180, 150, 40);
		convert.addActionListener(new ActionListener() {
			
			public void actionPerformed(ActionEvent e) {
				
			
				if(e.getSource() != null)
				{
					panelik.repaint();
					panelik.res((float)0.5,(float)0.5);
					panelik.repaint();
				}
			}
});
		
		label1=new JLabel("Nazwa bieżącego pliku:");
		label1.setFont(st1);
		label1.setBounds(0, 0, 300, 25);
		label2=new JLabel(fname);
		label2.setFont(st2);
		label2.setBounds(0,30,200,25);
		label4=new JLabel("Rozmiar bieżącego pliku:");
		label4.setFont(st1);
		label4.setBounds(0, 60, 250, 25);
		label5=new JLabel(Long.toString(space)+" kb");
		label5.setFont(st2);
		label5.setBounds(0, 90, 200, 25);
		label3=new JLabel("Wybierz nowy rozmiar dolnej krawędzi");
		label3.setFont(st1);
		label3.setBounds(0,120,250,25);
		combo=new JComboBox(imgSizes);
		combo.setBounds(0, 150, 100, 25);
		
		panel.add(convert);
		panel.add(label1);
		panel.add(label2);
		panel.add(label3);
		panel.add(label4);
		panel.add(label5);
		panel.add(combo);
		
		split.add(panel);
		split.add(p);
		split.setOneTouchExpandable(true); 
		split.setDividerLocation(250);
		split.getLeftComponent().setMinimumSize(mind);
		split.setEnabled(true);
		
		
		}
		catch(IOException e)
		{
			JOptionPane.showMessageDialog(this, e);
		}
		}
		
}
class MyPanel extends JPanel
{
	private float xScaleFactor;
	private float yScaleFactor;
	private int w,h;
    private BufferedImage originalImage;
    public boolean arg;
    
	public MyPanel(BufferedImage img)
	{
		
		w=img.getWidth();
		h=img.getHeight();
		
		originalImage = img;
	}
	public Dimension getPreferredSize() {
        return new Dimension(w,h);
    }
   
    public void paintComponent(Graphics g) {
        
    	super.paintComponent(g);       
        Graphics2D g2 = (Graphics2D)g;
        
        
        int newW = (int)(originalImage.getWidth() * xScaleFactor);
        int newH = (int)(originalImage.getHeight() * yScaleFactor);
        
        if(arg)
        {
        g2.setRenderingHint(RenderingHints.KEY_INTERPOLATION,
                            RenderingHints.VALUE_INTERPOLATION_BILINEAR);
       
        g2.drawImage(originalImage, 0, 0, newW, newH, null);
        arg=false;
        }
        else
        {
        	 g2.drawImage(originalImage,0,0,this);
        }
    	
    }  
    public void res(float xscale, float yscale)
    {
    	xScaleFactor = xscale;
    	yScaleFactor = yscale;
    	arg = true;
    	
    }
}
 

Here it is...Try to load an image (Plik->Otworz) and then grab frame and try to resize..
Btw windows, eclipse JavaSE-1.6

[Rob Camick]

There are a couple of problems:

a) I never like using a null layout because the preferred size is never calculated. Don't know if this is a problem or not.

b) Don't use the add(...) method to add components to the split pane. Use the setLeft/Right component methods.

c) Don't use the removeAll() method. This removes the divider from the split pane. Instead you might just add an empty JPanel.

[dziubut dziubucinski]

The problem was c).
Thank you for advices!
all the best