JTextField

[gancio]

Hello is there any method that select/highlight the text of a JtextField?
I mean without using the mouse ofc :D

[PhHein]

setSelectionStart and setSelectionEnd.
JTextComponent (Java Platform SE 6))

[gancio]

setSelectionStart and setSelectionEnd.

No, it doesn't highlight the selected text like it could be done with mouse.

[r035198x]

Of course it does. Did you try it?
Post the code you used that did not work.

[gancio]

Which code? Just try yourself, create a simple JtextField and use those methods; you will see that nothing is highlighted even if you use textfield.selectAll().

[r035198x]

selectAll? The methods suggested were setSelectionStart and setSelectionEnd. Read about them in the API docs and use them. I don't have to create any code to test them. I already know that they work.

[Fubarable]

Agree. The onus of creating and posting code that shows that the suggested methods don't work is on the original poster. gancio, the ball's in your court.

[gancio]

argh.. with a sample JTextField code it works but in my program doesn't work.. maybe cause it's associated with a button event.. :confused:
I tried also with repaint but nothing.

[r035198x]

And we all can't see your code so we can't really say much about it.

[gancio]

Here the code:

class MyDocumentListener implements DocumentListener {
        public int x,y;
        public MyDocumentListener(int posx,int posy) {
            x=posx;
            y=posy;
        }
            public void insertUpdate(DocumentEvent e) {
            try {
                if (cellCheck(x,y)) {
                    errore=false;
                    tf[x][y].setBackground(Color.LIGHT_GRAY);
                }
                else {
                    JOptionPane.showMessageDialog(null, "error");
                    error=true;
                    tf[x][y].setBackground(Color.RED); //it works
                    tf[x][y].select(0,1);                     //doesn't works
                }

            } catch (NumberFormatException nfe) {
                JOptionPane.showMessageDialog(null, "error");
                errore=true;
                tf[x][y].setBackground(Color.RED);
            }   
        }
{

tf are JTextField of course.

Maybe it doesn't work cause there's already a mouse selection so the caret is already setted. :confused:

[Fubarable]

What if you delay the select call slightly by queuing it up on the event queue:

   else {
       JOptionPane.showMessageDialog(null, "error");
       error=true;
       tf[x][y].setBackground(Color.RED); //it works
       SwingUtilities.invokeLater(new Runnable() {
          public void run() {
             tf[x][y].select(0,1);  // try it here
          }
       });
       //tf[x][y].select(0,1);   // comment this out
   }

?

[Fubarable]

Also a suggestion: when posting code for us to analyze and debug, it is almost always better to post a small compilable program that demonstrates the problem, called a "Short, Self Contained, Correct (Compilable), Example" or SSCCE. For example, this is the SSCCE that I created to test your problem:

import java.awt.Color;
import java.util.Random;
import javax.swing.*;
import javax.swing.event.DocumentEvent;
import javax.swing.event.DocumentListener;

public class SwingFu {

  private static void createAndShowGUI() {
    final Random random = new Random();
    final JTextField textfield = new JTextField(10);
    textfield.getDocument().addDocumentListener(
        new DocumentListener() {

          public void insertUpdate(DocumentEvent e) {
            if (random.nextBoolean()) {
              textfield.setBackground(Color.LIGHT_GRAY);
            } else {
              JOptionPane.showMessageDialog(null, "error");
              textfield.setBackground(Color.RED); // it works
              SwingUtilities.invokeLater(new Runnable() {
                public void run() {
                  textfield.select(0, 1); // does works
                }
              });
              //textfield.select(0, 1); // doesn't works
            }
          }

          public void changedUpdate(DocumentEvent arg0) {}
          public void removeUpdate(DocumentEvent arg0) {}
        });

    JFrame frame = new JFrame("SwingFu Application");
    frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    frame.getContentPane().add(textfield);
    frame.pack();
    frame.setLocationRelativeTo(null);
    frame.setVisible(true);
  }

  public static void main(String[] args) {
    javax.swing.SwingUtilities.invokeLater(new Runnable() {
      public void run() {
        createAndShowGUI();
      }
    });

  }
}

[r035198x]

What if you delay the select call slightly by queuing it up on the event queue:

Not really necessary:

//What a funny name for a class!
class SwingFu {

	private static void createAndShowGUI() {
		final JTextField textfield = new JTextField("Some text", 10);
		textfield.getDocument().addDocumentListener(new DocumentListener() {
			public void insertUpdate(DocumentEvent e) {
				textfield.select(0, 1); //works
			}

			public void changedUpdate(DocumentEvent e) {
			}

			public void removeUpdate(DocumentEvent e) {
			}
		});

		JFrame frame = new JFrame("SwingFu Application");
		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		frame.getContentPane().add(textfield);
		frame.pack();
		frame.setLocationRelativeTo(null);
		frame.setVisible(true);
	}

	public static void main(String[] args) {
		javax.swing.SwingUtilities.invokeLater(new Runnable() {
			public void run() {
				createAndShowGUI();
			}
		});

	}
}

[Fubarable]

On my system, it is necessary; don't know why.

[gancio]

Eureka!
It works.
Thank you very much. ;)

What if you delay the select call slightly by queuing it up on the event queue:

   else {
       JOptionPane.showMessageDialog(null, "error");
       error=true;
       tf[x][y].setBackground(Color.RED); //it works
       SwingUtilities.invokeLater(new Runnable() {
          public void run() {
             tf[x][y].select(0,1);  // try it here
          }
       });
       //tf[x][y].select(0,1);   // comment this out
   }

?

[r035198x]

On my system, it is necessary; don't know why.

You don't think that the random.nextBoolean() has anything to do with it?
If you look closely at my code you will see that I did not put that weird piece of code in it.

[Fubarable]

It's not the next boolean. I don't know why but your program doesn't work for me.

[r035198x]

It's not the next boolean. I don't know why but your program doesn't work for me.

By it doesn't work you mean that you fire it up and they start to type on the textfield that gets displayed and the first character doesn't get highlighted every time you type in there?

[Fubarable]

I'm sorry for not being clear: it's the highlighting. It will not highlight the first char of the String ever if I don't queue the method call on the EDT.

[r035198x]

I was just asking to check that you are actually typing some text onto the textfield because the highlight will only occur if you insert new text into the textfield.