[JProgressBar] Raster deaktivieren

[multiholle]

Wenn ich eine JProgressBar verwende ist standardmäßig ein Raster aktiviert. Sie kann z. B. nur die werte 10, 20, 30, 40, ... darstellen, nicht aber 23. Wie kann ich dieses Raster deaktivieren?

 

[Marco13]

Macht eigentlich nicht viel Sinn... dann würde man einfach eine JProgressBar(0,10) nehmen, und nicht 20, 30, 40... anzeigen, sondern 2,3,4... Vielleicht könnte man was basteln,wenn man ihr im Konstruktor ein passendes BoundedRangeModel (Java 2 Platform SE 5.0) übergibt...!?

 

[multiholle]

Ich möchte die Progressbar als Fortschrittsanzeige bei der Musikwiedergabe nutzen. Da sieht es echt doof aus, wenn die Anzeige auf Grund des Rasters ständig springt. Welche Eigenschaft des BoundedRangeModels ist denn für das Raster zuständig?

 

[Marco13]

Naja ... eigentlich sollte man das ja darüber regeln können, dass man "setValue" von der Progressbar nur mit den passenden Werten aufruft, aber... der Gedanke mit dem BoundedRangeModel war mir nur spontan gekommen... man könnte ja ein BoundedRangeModel erstellen, das lügt, wenn man es nach seiner value fragt... Hab' gerade mal (nur testweise) in dem ProgressBar-Beispiel von http://java.sun.com/docs/books/tuto...moProject/src/components/ProgressBarDemo.java das Erstellen der ProgressBar geändert, von vorher
progressBar = new JProgressBar(0, 100);
auf

        DefaultBoundedRangeModel test = new DefaultBoundedRangeModel(0,0,0,100)
        {
            public int getValue()
            {
                return (super.getValue() / 10) * 10;
            }
        };

        progressBar = new JProgressBar(test);

Das sieht erstmal befremdlich aus, aber ... ist vielleicht eine Möglichkeit ... je nachdem, was genau man erreichen will...

 

[Illuvatar]

Marco, ich glaub du verstehst das hier gerade falschrum
multiholle will, dass die JProgressBar eben nicht springt.

Allerdings ist das normale Verhalten eben, dass sie das nicht macht. Du müsstest uns mehr Code zeigen, damit wir rausfinden können, warum sie das tut.

Edit: Kurzer "Beweiscode" :
[HIGHLIGHT="Java"]import javax.swing.*;

public class Test extends JFrame implements ProgressListener
{
private JProgressBar progress;

public Test()
{
super("Test");

progress = new JProgressBar(JProgressBar.HORIZONTAL, 0, 100);
progress.setStringPainted(true);
add(progress);

pack();
setLocationRelativeTo(null);
setDefaultCloseOperation(EXIT_ON_CLOSE);

setVisible(true);
}

@Override
public void progress(final int value)
{
if (value == 100) {
dispose();
} else {
SwingUtilities.invokeLater(new Runnable() {
@Override
public void run()
{
Test.this.progress.setValue(value);
}
});
}
}

public static void main(String[] args)
{
Test test = new Test();
Worker worker = new Worker(test);
new Thread(worker).start();
}
}

interface ProgressListener
{
public void progress(int value);
}

class Worker implements Runnable
{
private final ProgressListener listener;

public Worker (ProgressListener listener)
{
this.listener = listener;
}

@Override
public void run()
{
int timeToSleep = new java.util.Random().nextInt(200) + 50;
for (int i = 0; i <= 100; i++) {
try {
Thread.sleep(timeToSleep);
listener.progress(i);
} catch (InterruptedException e) {
return;
}
}
}
}[/HIGHLIGHT]

 

[Marco13]

@Illuvatar: Jetzt (nach 13 Stunden Schlaf) frage ich mich auch, was ich da "gestern Abend" gelesen habe...?! ???:L

 

[multiholle]

Dieser Code erstellt eine Progressbar, die die Werte von 0 bis 10000 darstellt. Trotzdem zeigt sie nicht 0,03 % sondern 0 % und der nächste Schritt ist dann 1 %. Ich möchte aber jeden Wert darstellen.

[highlight="java"]
import javax.swing.*;

public class Test extends JFrame implements ProgressListener
{
private JProgressBar progress;

public Test()
{
super("Test");

progress = new JProgressBar(JProgressBar.HORIZONTAL, 0, 10000);
progress.setStringPainted(true);
add(progress);

pack();
setLocationRelativeTo(null);
setDefaultCloseOperation(EXIT_ON_CLOSE);

setVisible(true);
}

@Override
public void progress(final int value)
{
if (value == 10000) {
dispose();
} else {
SwingUtilities.invokeLater(new Runnable() {
@Override
public void run()
{
Test.this.progress.setValue(value);
}
});
}
}

public static void main(String[] args)
{
Test test = new Test();
Worker worker = new Worker(test);
new Thread(worker).start();
}
}

interface ProgressListener
{
public void progress(int value);
}

class Worker implements Runnable
{
private final ProgressListener listener;

public Worker (ProgressListener listener)
{
this.listener = listener;
}

@Override
public void run()
{
int timeToSleep = new java.util.Random().nextInt(20);
for (int i = 0; i <= 10000; i++) {
try {
Thread.sleep(timeToSleep);
listener.progress(i);
} catch (InterruptedException e) {
return;
}
}
}
}
[/highlight]

 

[Marco13]

Ach so, das ist ja nochmal was anderes. Dafür kann man getString überschreiben

          progress = new JProgressBar(JProgressBar.HORIZONTAL, 0, 10000)
          {
              public String getString()
              {
                  return String.format("%.3f%%", (float)progress.getValue() / progress.getMaximum()); // Oder was auch immer....
              }
          };

 

[multiholle]

Das ist auch nicht das, was ich meine. Ich möchte das die Progressbar die Werte ausgibt, die die Konsole im Beispielcode ausgibt.

[highlight="java"]
import javax.swing.*;

public class Test extends JFrame implements ProgressListener
{
private JProgressBar progress;

public Test()
{
super("Test");

progress = new JProgressBar(JProgressBar.HORIZONTAL, 0, 10000) {
@Override
public String getString() {
return String.format("%.3f%%", (float)progress.getValue() / progress.getMaximum() * 100);
}
};
progress.setStringPainted(true);
add(progress);

pack();
setLocationRelativeTo(null);
setDefaultCloseOperation(EXIT_ON_CLOSE);

setVisible(true);
}

@Override
public void progress(final int value)
{
if (value == 10000) {
dispose();
} else {
SwingUtilities.invokeLater(new Runnable() {
@Override
public void run()
{
Test.this.progress.setValue(value);
System.out.println(String.format("%.3f%%", (float) value / 100));
}
});
}
}

public static void main(String[] args)
{
Test test = new Test();
Worker worker = new Worker(test);
new Thread(worker).start();
}
}

interface ProgressListener
{
public void progress(int value);
}

class Worker implements Runnable
{
private final ProgressListener listener;

public Worker (ProgressListener listener)
{
this.listener = listener;
}

@Override
public void run()
{
int timeToSleep = new java.util.Random().nextInt(20);
for (int i = 0; i <= 10000; i++) {
try {
Thread.sleep(timeToSleep);
listener.progress(i);
} catch (InterruptedException e) {
return;
}
}
}
}
[/highlight]

 

[max40]

[HIGHLIGHT="Java"]SwingUtilities.invokeLater(new Runnable() {
@Override
public void run() {

Test.this.progress.setValue(value); // Wert setzen

String s = String.format("%.3f%%",(float) value / 100);
Test.this.progress.setString(s); // Text akt.
System.out.println(s);
}
});
[/HIGHLIGHT]

 

[Marco13]

Waha - böse Falle :autsch: Was soll das denn? ???:L Wenn man "setValue" macht, sollte man ja eigentlich annehmen, dass auch neu gezeichnet wird. Wird aber nicht. Und es sollte IMHO auch nicht notwendig sein. Nach dem Durchpflügen der Sources von ProgressBar & Co konnte ich es zurückverfolgen auf BasicProgressBarUI.Handler, wo tatsächlich steht

    public void stateChanged(ChangeEvent e) {
            ...
	    if (newPercent != oldPercent) {
		setCachedPercent(newPercent);
		progressBar.repaint();
	    }

Es wird also tatsächlich nur repaintet, wenn sich die Prozentzahl ändert :autsch: (Sieht man auch, wenn man die ProgressBar mit der Maus vergrößert und verkleinert: Dann auch zwischendurch repaintet, und es wird immer der richtige Text angezeigt - auch die Zwischenschritte)

Um deine ursprüngliche Frage also (hoffentlich) zu beantworten: Dieses scheinbare(!) "Raster" kommt daher, dass nur dann ein repaint gemacht wird, wenn sich die Prozentzahl ändert - warum auch immer :bahnhof:

Eine Möglichkeit ist, wie max40 schon geschrieben hat, den TEXT per Hand zu setzen (dann wird auch repaintet) - ich find's aber unschön, dass das notwendig ist: Dort, wo man irgendeine Zahl setzt, will (und sollte) man sich keine Gedanken drum machen müssen, wie (oder wann :autsch: ) der Text in der GUI dargestellt wird... eine in diesem Sinne "sauberere" Lösung wäre IMHO sowas wie

          progress = new JProgressBar(JProgressBar.HORIZONTAL, 0, 10000)
          {
              @Override
              public String getString()
              {
                  return String.format("%.3f%%", (float) getValue() / 100);
              }
              protected void fireStateChanged()
              {
                  super.fireStateChanged();
                  repaint();
              }
          };

Aber IMHO grenzt diese Sache, dass nur bei Prozent-Grenzen neu gezeichnet wird, schon fast an einen Bug....

 

[max40]

Also wenn ich eine ProgressBar habe wo nur drin stehen soll "Bitte warten sie" oder sowas, finde ich das richtig und gut das er kein repaint macht! Und wenn du den Wert setzt, ist es ja nicht so viel aufwand nochmal setString aufzurufen!

 

[Marco13]

Naja, bei "Bitte warten" wäre die ProgressBar vermutlich "indeterminate", und selbst wenn, wäre der Aufwand zu Neuzeichnen ja nicht dramatisch. Der Aufwand, den String per Hand zu setzen ist natürlich auch nicht so hoch - ich finde es nur irgendwie unschön...

 

[multiholle]

@Marco13
Deine Methode funktioniert wunderbar. Danke Was macht eigentlich die Funktion fireStateChanged(), wann wird sie aufgerufen?

 

[Marco13]

Die wird immer aufgerufen, wenn sich das darunterliegende Datenmodell (d.h. das BoundedRangeModel) ändert. An das jeweilige Modell wird ein ChangeListener gehängt, der nichts anderes macht, als diese Methode aufzurufen (die wiederum ChangeListener benachrichtigt, die an der JProgressBar hängen). Standardmäßig hängt an der ProgressBar der oben angesprochene BasicProgressBarUI$Handler als ChangeListener, der bei vollen Prozenten das repaint auslöst. Jetzt wird das repaint eben bei jeder Änderung am Datenmodell aufgerufen.