JSlider wird nicht neugezeichnet

[jperl]

hallo,

wieder einmal habe ich ein problem mit swing.

ich will, dass ab einem gewissen zeitpunkt ein gewisser sliderwert nicht unterschritten werden darf.

deshalb habe ich folgenden code:

		slider.addChangeListener(new ChangeListener()
		{

			@Override
			public void stateChanged(ChangeEvent e)
			{
				if (slider.getValue() < CRVSOMConfig.getStartIt())
				{
					slider.setValue(CRVSOMConfig.getStartIt() + 1);
				}
				sliderLabel.setText(Integer.toString(slider.getValue()));
				repaint();
			}

		});

jedoch wird der slider danach nicht neugezeichnet, d.h. die position des zeigers des sliders stimmt nicht mit dem value überein.
wie kriege ich es hin, dass der slider automatisch neugezeichnet wird?

ist das überhaupt möglich oder ist diese möglichkeit nicht vorgesehen?
gibt es andere bessere möglichkeiten, sprich eine andere gui komponente zu verwenden?
ich würde jedoch ungern die limits des sliders ändern, da ich dann bei einem reset wieder probleme mit der aktualisierung bekomme.

ich hoffe ihr könnt mir helfen.

jperl

 

[André Uhres]

1234567890

 

[jperl]

Versuch's mal so:

slider.addChangeListener(new ChangeListener() {
    @Override
    public void stateChanged(ChangeEvent e) {
        if (slider.getValue() > CRVSOMConfig.getStartIt()) {
            sliderLabel.setText(Integer.toString(slider.getValue()));
        }
        if (!slider.getValueIsAdjusting()) {
            Runnable sliderValue = new Runnable() {
                @Override
                public void run() {
                    if (slider.getValue() < CRVSOMConfig.getStartIt()) {
                        slider.setValue(CRVSOMConfig.getStartIt() + 1);
                    }
                }
            };
            SwingUtilities.invokeLater(sliderValue);
        }
    }
});

optimal. es funktioniert. :toll:
danke. auf die idee mit einem thread wäre ich nicht gekommen.

jperl

 

[Ebenius]

danke. auf die idee mit einem thread wäre ich nicht gekommen.

Ist kein extra Thread. Führt den Code nur im Event Dispatcher Thread später aus.

 

[Gernot_]

Ich habe eine andere Lösung desselben Problems gefunden.
Idee: Beim Loslassen der Maus den Slider aktualisieren
Problem: Wenn ich nur den richtigen Wert auslese und neu hineinschreibe, ändert sich gar nichts.
Lösung: Zuerst einen geringfügig anderen Wert setzen (1 mehr, bei Maximum 1 weniger), dann wieder den richtigen

Im Code ist sld der Slider, für den der Mauslistener registriert wurde:

public void mouseReleased(MouseEvent e) 
 {
  int wert = sld.getValue();
  if(wert < sld.getMaximum()) 
   sld.setValue(wert+1);
  else
   sld.setValue(wert-1);
  sld.setValue(wert);
 }

 

[Ebenius]

Und was, wenn ich den Slider mit der Tastatur bediene? :-O

jperl, magst Du das Thema als erledigt markieren? http://www.java-forum.org/forum-faq-beitraege/86306-thema-erledigt-markieren.html

Ebenius

 

[Gernot_S]

So, jetzt bin ich ordentlich registriert (Ich bin der Gernot_ von oben!).
Ich wollte nur noch sagen, dass ich die angegebene Methode mit dem Runnable-Objekt auch probiert habe. Sie hat bei mir aber nicht funktioniert, deshalb werde ich es wie angegeben mit dem Maus-Release-Ereignis machen.

Noch ein paar Fragen an die Java-Profis: Ist diese Funktionalität der JSlider-Komponente beabsichtigt? Oder handelt es sich um einen Fehler in der Swing-Bibliothek? Gibt es einen einfacheren Weg, den Slider zum korrekten Verhalten zu bringen?

Auf Wunsch poste ich hier den Code eines kompletten getesteten Demo-Programms zu diesem Thema.

 

[Gernot_S]

@Ebenius: Bei Bedienung mit der Tastatur funktioniert ohnehin alles erwartungsgemäß.

Nochmal meine Frage: Weiß jemand, ob dieser "Bug" wirklich einer ist? Wurde das schon an die Java-Entwickler weitergegeben?

Hier ein kleines, aber komplettes Demoprogramm, falls noch jemand weiter damit experimentieren will:

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

// Test für das Swing-Element JSlider
// Gernot Schatzdorfer, 01/2010

// In JSlider gibt es eine Fehlfunktion: Wenn man direkt im Event-Handler für die Slider-Änderung den Wert
// mit setValue korrigiert, wird er zwar korrekt ins Slider-Objekt übernommen, die Zunge bleibt aber am falschen Wert.

// Diese Fehlfunktion gibt es nur beim Ziehen mit der Maus. Wenn man den Slider mit der Tastatur ändert oder mit der
// Maus drüber oder drunter klickt, funktioniert alles normal.

// Lösung in diesem Demoprogramm: Dem Slider einen Maus-Event-Listener zuordnen und beim Loslassen der Maus den
// Wert kurz verändern und dann wieder richtig setzen.

// Im Demoprogramm verwende ich einen Slider mit Bereich von 0 bis 255. Wenn ein Wert 128 übersteigt, soll der Slider
// auf genau 128 springen.

// Dazu kommt noch ein Label zur Anzeige der Sliderwerte.


class Slidertest3                                     // Klasse für das Hauptprogramm
{
 public static void main (String args[]) 
 {
  MeinFenster f = new MeinFenster("Testfenster"); // Hauptprogramm: Nur das Fenster erzeugen 
 }
}

class MeinFenster extends JFrame implements ChangeListener // Klasse für das eingentliche Programmfenster                
{                                                
 JSlider slider1;                                        // Mein Testobjekt
 JLabel anzeige;                                         // Ein Label für die Anzeige des Slider-Wertes
                                               
 MeinFenster(String titel)                        
 {
  super(titel);
  setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); 
  setVisible(true); 
  setSize(400,200);
  slider1 = new JSlider(0,255,64);                       // Slider erzeugen
  anzeige = new JLabel("nix");                           // Label für die Anzeige erzeugen
  setLayout(new GridLayout(0,1));                        // Grid-Layout (Slider und Wertanzeige untereinander)
  add(slider1);
  add(anzeige);
  slider1.addChangeListener(this);                       // Slider-Ereignisauswertung gleich hier   
  
  slider1.addMouseListener(new MausAuswertung());        // Maus-Ereignisauswertung in eigener Klasse
 }

 public void stateChanged(ChangeEvent e)                // Slider-Änderungs-Auswertungs-Listener
 {
  int wert;
  JSlider absender = (JSlider)e.getSource();
  wert = absender.getValue();            
  if(wert>128) absender.setValue(128);                  // Nach Änderung: Wenn > 128 gewaltsam in die Mitte setzen
  wert = absender.getValue();                           // (Ohne die Maus-Released-Funktion unten würde der Slider
  anzeige.setText(" Wert: "+wert);                      //  das aber nicht anzeigen!)
                                                         
 }
 
}

class MausAuswertung implements MouseListener           // Mausereignis-Listener-Klasse
{
 
 
 public void mouseReleased(MouseEvent e)               // Beim Loslassen der Maus auf dem Slider:
 {
  JSlider sld = (JSlider)e.getSource();
  int wert = sld.getValue();
  if(wert < sld.getMaximum()) 
   sld.setValue(wert+1);                               // Wert um 1 erhöhen
  else
   sld.setValue(wert-1);                               // bzw. (bei Maximalwert) um 1 erniedrigen
  sld.setValue(wert);                                  // und wieder richtigstellen
 }

 public void mouseExited(MouseEvent e)  {}             // Die anderen Event-Handler brauche ich nicht
 public void mouseClicked(MouseEvent e) {}
 public void mouseEntered(MouseEvent e) {}
 public void mousePressed(MouseEvent e) {}
}

 

[Ebenius]

Ich halte das nicht für eine Fehlfunktion. Innerhalb des Events der auf die Änderung eines Wertes kommt den selben Wert zu ändern ist nunmal unschön. Das geht in vielen Swing-Komponenten schief. Entweder löst man das über eine eigene Modellimplementierung. Oder man führt den Code aus [c]stateChanged[/c] verspätet per [c]SwingUtilities.invokeLater()[/c] aus.

Ebenius