Swing JFormattedTextfield markiert nicht gesamten Inhalt

[Skrodde]

Hallo zusammen,
ich habe mir eine Klasse von

JFormattedTextField

abgeleitet, in der ich direkt im Constructor einen Focus Listener setze, welcher den gesamten Text markieren soll, sobald der Focus an das Feld geht. Hier die Klasse:

import java.awt.event.FocusEvent;
import java.awt.event.FocusListener;
import java.text.Format;

import javax.swing.JFormattedTextField;
import javax.swing.JTextField;

public class JFormattedTextFieldHA extends JFormattedTextField {

	public JFormattedTextFieldHA() {
		setFocusListener();
	}

	public JFormattedTextFieldHA(Object arg0) {
		super(arg0);
		setFocusListener();
	}

	public JFormattedTextFieldHA(Format arg0) {
		super(arg0);
		setFocusListener();
	}

	public JFormattedTextFieldHA(AbstractFormatter arg0) {
		super(arg0);
		setFocusListener();
	}

	public JFormattedTextFieldHA(AbstractFormatterFactory arg0) {
		super(arg0);
		setFocusListener();
	}

	public JFormattedTextFieldHA(AbstractFormatterFactory arg0, Object arg1) {
		super(arg0, arg1);
		setFocusListener();
	}

	private void setFocusListener() {
		FocusListener focus_highlightAll = new FocusListener() {
			@Override
			public void focusGained(FocusEvent arg0) {
				((JTextField)arg0.getComponent()).selectAll();
			}
			@Override
			public void focusLost(FocusEvent arg0) {	
			}
		};
		this.addFocusListener(focus_highlightAll);
	}

}

Wenn ich nun folgende main-Methode ausführe:

public static void main(String[] args) throws Exception {
		
		JFrame testFrame = new JFrame( );
		testFrame.setLayout( new FlowLayout() );		
		JFormattedTextFieldHA formatted1 = new JFormattedTextFieldHA(NumberFormat.getInstance());
		formatted1.setText("formatted1");
		testFrame.add( formatted1 );
		JFormattedTextFieldHA formatted2 = new JFormattedTextFieldHA(NumberFormat.getInstance());
		formatted2.setText("formatted2");
		testFrame.add( formatted2 );
		testFrame.pack();
		testFrame.setVisible(true);
		}

Dann klappt das markieren beim ersten Mal anwählen ganz gut, allerdings habe ich noch folgendes Problem: Gibt es Nutzer eine Zahl ein und klickt erneut in das Textfeld, wird rein gar nichts mehr markiert ;(
Kann mir jemand von euch sagen, woran das liegt?
Danke, Gruß, Skrodde

 

[Michael...]

Grundsätzlich sollte man von einer Klasse nur erben, wenn man deren Funktionsumfang erweitert. Bei einer Registrierung eines FocusListeners, der das Focusverhalten ändert, ist das m.M. nicht unbedingt gerechtfertigt. Egal.

Zum eigentlichen Problem: vermutlich beieinflusst des Setzen des Formatters ebenfalls das Focus Verhalten. Eine Lösung dazu weis ich allerdings auch nicht.

 

[SlaterB]

so sieht eine Methode in JFormattedTextField aus:

    /**
     * Processes any focus events, such as
     * <code>FocusEvent.FOCUS_GAINED</code> or
     * <code>FocusEvent.FOCUS_LOST</code>.
     *
     * @param e the <code>FocusEvent</code>
     * @see FocusEvent
     */
    protected void processFocusEvent(FocusEvent e) {
        super.processFocusEvent(e);

	// ignore temporary focus event
	if (e.isTemporary()) {
	    return;
	}

        if (isEdited() && e.getID() == FocusEvent.FOCUS_LOST) {
	    InputContext ic = getInputContext();
	    if (focusLostHandler == null) {
		focusLostHandler = new FocusLostHandler();
	    }
	    
	    // if there is a composed text, process it first
	    if ((ic != null) && composedTextExists) {
		ic.endComposition();
		EventQueue.invokeLater(focusLostHandler);
	    } else {
		focusLostHandler.run();
	    }
        }
        else if (!isEdited()) {
            // reformat
            setValue(getValue(), true, true);
        }
    }

der Text wird evtl. nach deinem selectAll() nochmal neu gesetzt und die Selektion geht verloren,
habs noch nicht ganz durchdrungen, ein PropertyChangeEvent in setValue() gibt es wohl doch nicht,

mit nochmaligen Überschreiben:

    protected void processFocusEvent(FocusEvent e)
    {
        super.processFocusEvent(e);
        System.out.println("nach pfe");
        selectAll();
    }

in deiner Klasse klappt es aber glaube ich, dann kannst du deinen FocusListener, der eh zu früh drankommt, wohl einsparen

 

[Skrodde]

Grundsätzlich sollte man von einer Klasse nur erben, wenn man deren Funktionsumfang erweitert. Bei einer Registrierung eines FocusListeners, der das Focusverhalten ändert, ist das m.M. nicht unbedingt gerechtfertigt. Egal.

Ich habe mir folgendes dabei gedacht: Ich setze in unterschiedlichen Klassen in meinem Code immer wieder vereinzelt Textfelder ein, bei denen ich dieses "markiere beim Fokus alles" Feature gerne hätte. Wenn ich nun an jeder Stelle im Code hingehe, einen Focus-Listener erzeuge, konfiguriere und setze, dann ist das wesentlich unlesbarer Code, als wenn ich direkt eine neue Klasse mit diesem Feature ausrüste.
Oder gibt es da eine bessere Lösung?

mit nochmaligen Überschreiben:

    protected void processFocusEvent(FocusEvent e)
    {
        super.processFocusEvent(e);
        System.out.println("nach pfe");
        selectAll();
    }

in deiner Klasse klappt es aber glaube ich, dann kannst du deinen FocusListener, der eh zu früh drankommt, wohl einsparen

So funktioniert es wunderbar :toll: Vielen Dank!

 

[SlaterB]

Oder gibt es da eine bessere Lösung?

die allgemeine Alternative ist eine Factory,

ob du nun
JTextField x = new MyFocusField();
oder
JTextField x = Gui.createFocusField();
schreibst, macht kaum Unterschied,

schlimm ist eine Vererbung aber auch nicht, im Zweifel später auch noch gut zu korrigieren,
für den Moment nun ja eh nötig
wobei noch eine anonyme geerbte Klasse in der statischen Methode auch ginge