Swing: JSlider wird nicht angezeigt

[Hutmacher]

Nachdem ich alle AWT-Kapitel in meinem Buch durch bin, widme ich mich jetzt endlich Swing .

Zum Testen wollte ich mal den JSlider ausprobieren, also diese Scrollbars mit Skala.
Mein Code setzt 2 JSlider auf ein Frame. Der rechte Schieber symbolisiert dabei die Width eines Rechtecks und der untere Schieber die Height. Diese Rechteck, was auf dem Bildschirm ausgegeben wird, soll also von den JSlidern abhängig sein
(heißt also, dass immer, wenn der Wert von diesen verändert wird, auch der Wert des Rechtecks verändert wird).

package tests.gui;

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

@SuppressWarnings("serial")
public class SlideTest 
 extends JFrame
 implements ChangeListener
{
	private JSlider   WidthSlider = new JSlider(JSlider.VERTICAL, 0, 100, 0); //Der rechte Width-Schieberegler
	private JSlider   HeightSlider = new JSlider(JSlider.HORIZONTAL, 0, 100, 0); //Der untere Height-Schieberegler
	private Rectangle rect    = new Rectangle(35, 35, 0, 0); //Das zu zeichnende Rechteck
	
	public SlideTest()
	{
		super("Slide-Test!");
		setSize(400, 300);
		setResizable(false);
		
		//WidthSlider-Initialisierungen
		WidthSlider.setMajorTickSpacing(50);
		WidthSlider.setMinorTickSpacing(10);
		WidthSlider.setPaintLabels(true);
		WidthSlider.setPaintTicks(true);
		
		//HeightSlider-Initialisierungen
		HeightSlider.setMajorTickSpacing(50);
		HeightSlider.setMinorTickSpacing(10);
		HeightSlider.setPaintLabels(true);
		HeightSlider.setPaintTicks(true);
		
		//Listener registrieren
		WidthSlider.addChangeListener(this);
		HeightSlider.addChangeListener(this);
		
		//Adden
		Container cp = getContentPane();
		cp.add(WidthSlider, BorderLayout.EAST);
		cp.add(HeightSlider, BorderLayout.SOUTH);
	
		//Fenster anzeigen
		setVisible(true);
	}
	
	@Override
	public void stateChanged(ChangeEvent event)
	{
		JSlider slider = (JSlider)event.getSource();
		if (slider == WidthSlider)
		{
			rect.width = slider.getValue();
		}
		else if (slider == HeightSlider)
		{
			rect.height = slider.getValue();
		}
		
		repaint();
	}

	public void paint(Graphics g)
	{
		g.drawRect(rect.x, rect.y, rect.width, rect.height);
	}

	public static void main(String[] args)
	{
		new SlideTest();
	}
}

Jetzt aber das Problem:
Erstens wird mir nur der rechte, der Width-Slider angezeigt

und zweitens buggt dieser auch noch irgendwie rum ...

Kann mir jemand sagen, was ich falsch mache?

 

[sayang]

Hi,

du überschreibst mit paint(Graphics g) die paint-Methode des ContentPanes des JFrames. Und zwar nur mit dem Zeichnen des Rechtecks, welches zu Beginn der Ausführung eine größe von 0 hat.

Der ContentPane enthält jedoch noch die Slider, welche bei der Ausführung von Paint quasi weggeschmissen werden.

Mach doch ein JPanel, welches du in BorderLayout.CENTER setzt. Und statt repaint() rufst du eine eigene Methode paintRect() oder so auf. Diese Methode malt mit dem Graphics-Objekt des JPanels das Rechteck auf das Panel drauf.

Ungefähr so:

import java.awt.*;
import javax.swing.*;
import javax.swing.event.*;
 
@SuppressWarnings("serial")
public class SlideTest 
 extends JFrame
 implements ChangeListener
{
    private JSlider   WidthSlider = new JSlider(JSlider.VERTICAL, 0, 100, 0); //Der rechte Width-Schieberegler
    private JSlider   HeightSlider = new JSlider(JSlider.HORIZONTAL, 0, 100, 0); //Der untere Height-Schieberegler
    private Rectangle rect    = new Rectangle(35, 35, 0, 0); //Das zu zeichnende Rechteck
    private JPanel panel = new JPanel(); // Das Panel, auf dem das Rechteck gemalt wird
    private Color backgroundColor = Color.white; // Hintergrundfarbe des Panels
    private Color foregroundColor = Color.black; // Vordergrundfarbe des Panels
    
    public SlideTest()
    {
        super("Slide-Test!");
        setSize(400, 300);
        setResizable(false);
        
        //WidthSlider-Initialisierungen
        WidthSlider.setMajorTickSpacing(50);
        WidthSlider.setMinorTickSpacing(10);
        WidthSlider.setPaintLabels(true);
        WidthSlider.setPaintTicks(true);
        
        //HeightSlider-Initialisierungen
        HeightSlider.setMajorTickSpacing(50);
        HeightSlider.setMinorTickSpacing(10);
        HeightSlider.setPaintLabels(true);
        HeightSlider.setPaintTicks(true);
        
        //Listener registrieren
        WidthSlider.addChangeListener(this);
        HeightSlider.addChangeListener(this);
        
        panel.setBackground(backgroundColor);
//        panel.setForeground(foregroundColor);
        
        //Adden
        Container cp = getContentPane();
        cp.setLayout(new BorderLayout());
        cp.add(WidthSlider, BorderLayout.EAST);
        cp.add(HeightSlider, BorderLayout.SOUTH);
        cp.add(panel, BorderLayout.CENTER); // Panel auf CENTER legen
        //Fenster anzeigen
        setVisible(true);
        
    }
    
    public void stateChanged(ChangeEvent event)
    {
        JSlider slider = (JSlider)event.getSource();
        
        clearRect();
        
        if (slider == WidthSlider)
        {
            rect.width = slider.getValue();
        }
        else if (slider == HeightSlider)
        {
            rect.height = slider.getValue();
        }
        
        paintRect();
    }
    
    // erstmal das alte Rechteck "löschen"
    public void clearRect() {
    	Graphics g = panel.getGraphics();
    	g.setColor(backgroundColor);
    	g.drawRect(rect.x, rect.y, rect.width, rect.height);
    }
    
    // statt paint(Graphics g), zum zeichnen des neuen Rechtecks
    public void paintRect()
    {
    	Graphics g = panel.getGraphics();
    	g.setColor(foregroundColor);
    	g.drawRect(rect.x, rect.y, rect.width, rect.height);
    }
 
    public static void main(String[] args)
    {
        new SlideTest();
    }
}

Hier gibts 2 neue Methoden clearRect und paintRect. Was clearRect macht, kannst du ja mal ausprobieren, wenn du es in deiner stateChanged-Methode NICHT aufrufst.

Die Abfrage, welcher Slider das stateChanged auslöst, ist übrigens überflüssig. Setz einfach den aktuellen Wert des HeightSliders auf rect.height und den aktuellen Wert des WidthSliders auf rect.width - egal was passiert

public void stateChanged(ChangeEvent event) {
   clearRect();
   rect.width = slider.getValue();
   rect.height = slider.getValue();
   paintRect();
}

Lg
sayang

 

[Hutmacher]

Ah, danke! Klappt perfekt!

 

[Verjigorm]

Mag klappen, schön ist die Lösung aber nicht

 

[SebiB90]

Man sollte aber generell lieber nicht mit getGraphics arbeiten.
Erstell lieber eine Unterklasse von JPanel und in der überschreibst du die paintComponent Methode, in der du dann zeichnest.

 

[sayang]

Das stimmt natürlich. Wollte nur nicht zuviel auf einmal "vorsetzen"

Lg
sayang