Verbesserungfrage zu einem MP3Player Slider - Ebenius kannst du helfen bitte ?

[osix]

Ich hatte gefragt, wie man einen JSlider zu einer ziehbaren Progressbar verwenden kann, für einen MP3Player unter Java.

Das funktioniert auch prinzipiell gut, doch eines ist verbesserungswürdig.

Ich möchte, daß erst im Moment des Loslassens des Mausknopfes, die Musik auf die gewählte Stelle des Sliders springt. Denn sonst verzögert die Musik, während man den Slider schiebt.

Ich hab eine "unschöne" Lösung gefunden, und zwar koppele ich einen MouseListener an den JSlider, und arbeite mit einem "sliderChangedFlag". Wenn man also den Silder verschiebt, wird das sliderChangedFlag gesetzt, und im MouseListener wird genau das abgefragt, ob es true ist, WENN man den Mausknopf losläßt (mouseBottonReleased)

Das ist unschön, ich hab doch extra das schöne "MediaSliderModel", das Ebenius so schön entworfen hat.
Kann ich da nicht noch irgendwie die Sache mit dem Mausknopf auch noch mit reinweben ?

Wer hat eine Idee ? Ebenius, du vielleicht ?

 

[Noctarius]

Sonst mach doch einen Deltawert. Erst wenn der Deltawert größer x ist springst du mit der Wiedergabe zu der passenden Stelle im Song. Dann hast du kleinere Sprünge, was nach Suchen im Stück klingt, du willst beim Sliden ja ungefähr wissen wo du dich befindest.

 

[osix]

Ne, vom Bediengefühl her möchte man einfach frei den Slider ziehen können, und erst wenn man losläßt, springt die Musik dahin.

Ich hab auch schon eine Idee, ich hänge ans MediaSliderModell noch einen Mouselistern, doch der reagiert irgendwie nicht.

import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.MouseEvent;
import java.awt.event.MouseListener;

import javax.swing.DefaultBoundedRangeModel;

public class MediaSliderModel extends DefaultBoundedRangeModel implements
        MouseListener
            
{
    
    public MediaSliderModel()
    {
        
    }

    
    /**
     * Called by the JSlider if the user drags the slider. ...
     */
    @Override
    public void setValue(int n)
    {
        super.setValue(n);
        fireActionEvent(new ActionEvent(this, ActionEvent.ACTION_PERFORMED, "sliderValueChanged"));
    }

    /**
     * Call this method for media progress change. ...
     */
    public void updateMediaProgress(int n)
    {
        super.setValue(n);
        fireActionEvent(new ActionEvent(this, ActionEvent.ACTION_PERFORMED, "mediaProgressChanged"));
    }

    /**
     * Fires the {@link ActionEvent} to all listeners.
     * @param event the event to fire
     */
    protected void fireActionEvent(ActionEvent event)
    {
        final Object[] list = listenerList.getListenerList();
        for (int idx = 0; idx < list.length; idx += 2)
        {
            if (list[idx] == ActionListener.class)
            {
                ((ActionListener) list[idx + 1]).actionPerformed(event);
            }
        }
    }

    /**
     * Adds the {@link ActionListener} to <code>this</code> instance.
     * @param listener the listener to add
     */
    public void addActionListener(ActionListener listener)
    {
        listenerList.add(ActionListener.class, listener);
    }

    /**
     * Removes the {@link ActionListener} from <code>this</code> instance.
     * @param listener the listener to remove
     */
    public void removeActionListener(ActionListener listener)
    {
        listenerList.remove(ActionListener.class, listener);
    }

    public void mouseClicked(MouseEvent arg0)
    {
        // TODO Auto-generated method stub

    }

    public void mouseEntered(MouseEvent arg0)
    {
        // TODO Auto-generated method stub

    }

    public void mouseExited(MouseEvent arg0)
    {
        // TODO Auto-generated method stub

    }

    public void mousePressed(MouseEvent arg0)
    {
        // TODO Auto-generated method stub

    }

    public void mouseReleased(MouseEvent arg0)
    {
        fireActionEvent(new ActionEvent(this, ActionEvent.ACTION_PERFORMED, "sliderValueChangedAndMouseReleased"));
    }

Ich schaffe es irgendwie nicht, den Mouselistener scharf zu machen, er meldet sich nicht.

Ich hab schon versucht ihn mit this.addMouselistener im Konstrukur anzuhängen, aber das klappt auch nicht.
Es fehlt nicht mehr viel, glaub ich, wer weiß den Trick ?

 

[osix]

Ich stochere im Nebel, aber probier es mal.
MIST, warum klappt das hier nicht.
Ich mach doch alles richtig. Jedes neue Objekt krieg im Konstruktor gleich einen Mouselistener verpaßt, der auch korrekt in der Klasse drin ist. UNd im MouseReleased Teil wird die Action abgefeuert. Was fehlt hier noch ?

package de.proton.MP3SofaSortNeu;

import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.MouseEvent;
import java.awt.event.MouseListener;

import javax.swing.DefaultBoundedRangeModel;

public class MediaSliderModel extends DefaultBoundedRangeModel implements
        MouseListener
            
{    
    public MediaSliderModel()
    {
        this.addMouseListener(this);
    }

    
    /**
     * Called by the JSlider if the user drags the slider. ...
     */
    @Override
    public void setValue(int n)
    {
        super.setValue(n);
        fireActionEvent(new ActionEvent(this, ActionEvent.ACTION_PERFORMED, "sliderValueChanged"));
    }

    /**
     * Call this method for media progress change. ...
     */
    public void updateMediaProgress(int n)
    {
        super.setValue(n);
        fireActionEvent(new ActionEvent(this, ActionEvent.ACTION_PERFORMED, "mediaProgressChanged"));
    }

    /**
     * Fires the {@link ActionEvent} to all listeners.
     * @param event the event to fire
     */
    protected void fireActionEvent(ActionEvent event)
    {
        final Object[] list = listenerList.getListenerList();
        for (int idx = 0; idx < list.length; idx += 2)
        {
            if (list[idx] == ActionListener.class)
            {
                ((ActionListener) list[idx + 1]).actionPerformed(event);
            }
        }
    }
    
    /**
     * Adds the {@link ActionListener} to <code>this</code> instance.
     * @param listener the listener to add
     */
    public void addActionListener(ActionListener listener)
    {
        listenerList.add(ActionListener.class, listener);
    }

    public void addMouseListener(MouseListener listener)
    {
        listenerList.add(MouseListener.class, listener);
    }
    
    public void removeMouseListener(MouseListener listener)
    {
        listenerList.remove(MouseListener.class, listener);
    }
    
    /**
     * Removes the {@link ActionListener} from <code>this</code> instance.
     * @param listener the listener to remove
     */
    public void removeActionListener(ActionListener listener)
    {
        listenerList.remove(ActionListener.class, listener);
    }

    public void mouseClicked(MouseEvent arg0)
    {
        // TODO Auto-generated method stub

    }

    public void mouseEntered(MouseEvent arg0)
    {
        // TODO Auto-generated method stub

    }

    public void mouseExited(MouseEvent arg0)
    {
        // TODO Auto-generated method stub

    }

    public void mousePressed(MouseEvent arg0)
    {
        // TODO Auto-generated method stub

    }

    public void mouseReleased(MouseEvent arg0)
    {
        fireActionEvent(new ActionEvent(this, ActionEvent.ACTION_PERFORMED, "sliderValueChangedAndMouseReleased"));
    }
}

 

[Ebenius]

Da hat doch Sun schon für Dich mitgedacht. ;-) Jedes BoundedRangeModel hat eine Eigenschaft namens [c]valueIsAdjusting[/c]. Diese kann man abfragen. Entsprechend kann man einen Zuhörer an das Modell hängen der diese Eigenschaft beachtet. Hier mal ein Minibeispiel direkt zum ausprobieren:

final JSlider slider = new JSlider();
final JLabel label = new JLabel("Slide the slider");
slider.getModel().addChangeListener(new ChangeListener() {

  public void stateChanged(ChangeEvent e) {
    final BoundedRangeModel model = (BoundedRangeModel) e.getSource();
    if (model.getValueIsAdjusting()) {
      label.setText(label.getText() + '.');
    } else {
      label.setText("Slider Value: " + model.getValue());
    }
  }
});
final JPanel contentPane = new JPanel(new BorderLayout(6, 6));
contentPane.add(slider, BorderLayout.NORTH);
contentPane.add(label, BorderLayout.CENTER);

final JFrame f = new JFrame("Test Frame: JSliderFun"); //$NON-NLS-1$
f.setContentPane(contentPane);
f.pack();
f.setSize(new Dimension(400, f.getHeight()));
f.setLocationRelativeTo(null);
f.setDefaultCloseOperation(WindowConstants.EXIT_ON_CLOSE);
f.setVisible(true);

Fertig. :-D

Ebenius

 

[osix]

Aber dann verstehe ich nicht, warum du mir extra das schöne MediaSliderModel empfohlen hast. (siehe mein letzer Beitrag vor ein paar Tagen.)

Dadurch nutze ich doch dann statt dem ChangeListener einen ActionListener, der die neuen Actions "Mediaprogresschanged" und "sliderValuechanged" verwendet.

Der ChangeListener wird doch wieder nicht unterscheiden können zwischen "Slider gezogen" und "Slider per setValue verändert, weil die Musik weiterspielt.

Oder muß ich zusätzlcih noch den ChangeListener verwenden ?

 

[Ebenius]

Ich dachte, Du kombinierst die beiden Sachen selbständig. ;-) Zum Beispiel sowas...
[java=20] /**
* Called by the JSlider if the user drags the slider. ...
*/
@Override
public void setValue(int n)
{
super.setValue(n);
if (!getValueIsAdjusting) {
fireActionEvent(new ActionEvent(this, ActionEvent.ACTION_PERFORMED, "sliderValueChanged"));
} else {
fireActionEvent(new ActionEvent(this, ActionEvent.ACTION_PERFORMED, "sliderValueChangedAdjusting"));
}
}
[/code]
Die Eigenschaft [c]valueIsAdjusting[/c] ist nicht immer [c]true[/c] wenn der Nutzer etwas auf dem Slider macht. Sie ist immer dann [c]true[/c], wenn der Nutzer noch am SliderKnob zieht.

Ebenius

 

[osix]

Komisch, ich hab es genauso probiert und eingefügt.

Sicherheitshalber lasse ich mir noch den Wert von getValueIsAdjusting() ausgeben.

Der ist IMMER true, egal ob ich am Slider ziehe oder den Mausknopf loslasse.

Somit geht er auch immer nur in den "sliderValueChangedAdjusting" Zweig rein.

Mach ich da noch was falsch ?

 

[osix]

Ebenius, ich kriegs immer noch nicht zusammen, ich hab lange drüber nachgedacht, viel probiert es klappt alles nicht.

Mein Verständnis ist, daß du doch extra das MediaSliderModel entworfen hast, damit wir KEINEN ChangeStateListener mehr brauchen - die Abfrage läuft doch dann über einen ActionListener, oder ?

getValueIsAdjusting hat aber wieder was spezielles mit dem ChangeListener zu tun, und den verwende ich doch dann garnicht mehr, und deswegen klappt die Abfrage auch nicht im MediaSliderModel. In einem ChangeStateListener tut die Abfrage wunderbar.

Oder versteh ich wieder was nicht richtig ?

 

[Ebenius]

Also in meinem Code oben habe ich zwei Klammern vergessen: [c]()[/c]. Wenn Du die einfügst, dann fehlt noch eine Kleinigkeit ([c]setValueIsAdjusting(boolean)[/c] muss man auch noch überschreiben). :-( Die MouseListener-Geschichte muss aus dem Modell wieder heraus, ein Modell hört niemals auf einen Controller! Mach mal das Modell genau so:

public static class MediaSliderModel extends DefaultBoundedRangeModel {

  @Override
  public void setValueIsAdjusting(boolean b) {
    if (getValueIsAdjusting() != b) {
      super.setValueIsAdjusting(b);
      if (b) {
        fireActionEvent(new ActionEvent(this, ActionEvent.ACTION_PERFORMED,
              "sliderValueChangedAdjusting"));
      } else {
        fireActionEvent(new ActionEvent(this, ActionEvent.ACTION_PERFORMED,
              "sliderValueChanged"));
      }
    }
  }

  /**
   * Called by the JSlider if the user drags the slider. ...
   */
  @Override
  public void setValue(int n) {
    super.setValue(n);
    if (!getValueIsAdjusting()) {
      fireActionEvent(new ActionEvent(this, ActionEvent.ACTION_PERFORMED,
            "sliderValueChanged"));
    } else {
      fireActionEvent(new ActionEvent(this, ActionEvent.ACTION_PERFORMED,
            "sliderValueChangedAdjusting"));
    }
  }

  /**
   * Call this method for media progress change. ...
   */
  public void updateMediaProgress(int n) {
    super.setValue(n);
    fireActionEvent(new ActionEvent(this, ActionEvent.ACTION_PERFORMED,
          "mediaProgressChanged"));
  }

  /**
   * Fires the {@link ActionEvent} to all listeners.
   * 
   * @param event the event to fire
   */
  protected void fireActionEvent(ActionEvent event) {
    final Object[] list = listenerList.getListenerList();
    for (int idx = 0; idx < list.length; idx += 2) {
      if (list[idx] == ActionListener.class) {
        ((ActionListener) list[idx + 1]).actionPerformed(event);
      }
    }
  }

  /**
   * Adds the {@link ActionListener} to <code>this</code> instance.
   * 
   * @param listener the listener to add
   */
  public void addActionListener(ActionListener listener) {
    listenerList.add(ActionListener.class, listener);
  }

  public void addMouseListener(MouseListener listener) {
    listenerList.add(MouseListener.class, listener);
  }

  public void removeMouseListener(MouseListener listener) {
    listenerList.remove(MouseListener.class, listener);
  }

  /**
   * Removes the {@link ActionListener} from <code>this</code> instance.
   * 
   * @param listener the listener to remove
   */
  public void removeActionListener(ActionListener listener) {
    listenerList.remove(ActionListener.class, listener);
  }
}

Und teste mal mit diesem hübschen Testprogramm, so sollte alles stimmen:

/* A label to report actions to */
final JLabel label = new JLabel("Slide the slider");

/* model with action listener appended */
final MediaSliderModel model = new MediaSliderModel();
model.addActionListener(new ActionListener() {

  public void actionPerformed(ActionEvent e) {
    final BoundedRangeModel model = (BoundedRangeModel) e.getSource();
    final Integer value = Integer.valueOf(model.getValue());
    if ("sliderValueChangedAdjusting".equals(e.getActionCommand())) {
      label.setText("User drags the knob: " + value);
    } else if ("sliderValueChanged".equals(e.getActionCommand())) {
      label.setText("User changed the value: " + value);
    } else if ("mediaProgressChanged".equals(e.getActionCommand())) {
      label.setText("Tracker updated progress: " + value);
    }
  }
});

/* slider and tracker simulator */
final JSlider slider = new JSlider(model);
final JButton emuButton =
      new JButton(new AbstractAction("Emulate Tracker") {

        public void actionPerformed(ActionEvent e) {
          new SwingWorker<Object, Integer>() {

            @Override
            protected Object doInBackground() throws Exception {
              for (int i = 10; i < 80; i++) {
                Thread.sleep(50);
                publish(Integer.valueOf(i));
              }
              return null;
            }

            @Override
            protected void process(List<Integer> chunks) {
              model.updateMediaProgress(chunks.get(chunks.size() - 1)
                    .intValue());
            }
          }.execute();
        }
      });

final JPanel contentPane = new JPanel(new BorderLayout(6, 6));
contentPane.add(slider, BorderLayout.NORTH);
contentPane.add(label, BorderLayout.CENTER);
contentPane.add(emuButton, BorderLayout.SOUTH);

final JFrame f = new JFrame("Test Frame: JSliderFun"); //$NON-NLS-1$
f.setContentPane(contentPane);
f.pack();
f.setSize(new Dimension(400, f.getHeight()));
f.setLocationRelativeTo(null);
f.setDefaultCloseOperation(WindowConstants.EXIT_ON_CLOSE);
f.setVisible(true);

Ebenius

 

[osix]

Meeeensch, vielen Dank !

Hab noch keine Zeit gehabt es auszuprobieren, wird auch wahrscheinlich nichts mehr vorm Urlaub, aber ich "seh" schon, daß es so gehen wird.

Ebenius, du bist ja ein "Swing"-Meister und hast das Zeugs komplett verinnerlicht...