GUI, printing available Locales

Ok so Im trying to print out the available Locales on a machine in a text area


The lie of code output.setText(availableLocales); is giving the trouble.

Eclipse is giving out about it but Im not to sure on the problem why.

Can anyone shed some light please

Code:

---

import java.awt.Container;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.util.Calendar;
import java.util.Locale;

import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.JScrollPane;
import javax.swing.JTextArea;



/**
 * @author B00022881 Mark Walsh
 *
 */
public class localesTest extends JFrame implements ActionListener {

    /**
    *
    */
    private static final long serialVersionUID = 1L;
   
   
    JTextArea output = new JTextArea();
    Locale[]  availableLocales = Calendar.getAvailableLocales( );
   
    public localesTest(){
       
        super("Testing Available Locales");
       
        Container c = getContentPane();
       
        JPanel panel = new JPanel();
       
        output.setText("Available Locales will Appear here..");
        output.setEditable(false);
       
       
        JButton listLocals = new JButton();
        listLocals.setText("List All Locales");
        listLocals.addActionListener(this);
       
       
        JScrollPane outputPane = new JScrollPane(output);
        panel.add(listLocals);
        panel.add(outputPane);
       
        c.add(panel);
       
        setSize(250, 300);
        setVisible(true);
        setResizable(false);
    }
   
   
    public void actionPerformed(ActionEvent e) {
       
        if(e.getActionCommand().equals("List All Locales")){
           

            /**
            * This Line is giving the Problems
            *
            */
            output.setText(availableLocales);
           
        }
       
    }

   
    public static void main(String[] args) {
        localesTest myLocalest = new localesTest();
        myLocalest.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

    }

}

---

Thanks in advance

Quote:

---

This Line is giving the Problems

---

What problems?

Emm here's the exact definition

Quote:

---

The method setText(String) in the type JTextComponent is not applicable for the arguments (Locale[]) localesTest.java

---

There you go. What output do you expect?

Im trying to output the Locales available on the current machine.


Just looking at it some more I should probably have a for loop to go through the array

so I now have the line

Code:

---

for(int i=0; i<availableLocales.length; i++){
           
              output.setText(availableLocales);
        }

---

Same problem

Yes, even if you had the output.setText(availableLocales[i]); part correct, because you'd
a) overwrite the text in every iteration and
b) a Locale is not a String

You can either loop and use Locale's toString() implementation, but remember to append. Or you set the text using Arrays.toString(availableLocales).

Thanks man :D, ill work on that now.

Have fun:)

Got it,

Thanks for the help..

I just used output.append to print out the contents of the array.


Replaced what was in my actionPerformed with

Code:

---

if(e.getActionCommand().equals("List All Locales")){
           
           
            for(int i=0; i<availableLocales.length; i++){
           
               
                output.append(availableLocales[i].getDisplayName()+"\n");
        }
            }

---

So full code reads

Code:

---

//imports

public class localesTest extends JFrame implements ActionListener {

    /**
    *
    */
    private static final long serialVersionUID = 1L;
   
   
    JTextArea output = new JTextArea(20, 22);
    Locale[]  availableLocales = Calendar.getAvailableLocales( );
   
    public localesTest(){
       
        super("Testing Available Locales");
       
        Container c = getContentPane();
       
        JPanel panel = new JPanel();
       
        output.setText("Available Locales will Appear here..");
        output.setEditable(false);
       
       
        JButton listLocals = new JButton();
        listLocals.setText("List All Locales");
        listLocals.addActionListener(this);
       
       
        JScrollPane outputPane = new JScrollPane(output);
        panel.add(listLocals);
        panel.add(outputPane);
       
        c.add(panel);
       
        setSize(250, 300);
        setVisible(true);
        setResizable(false);
    }
   
   
    public void actionPerformed(ActionEvent e) {
       
        if(e.getActionCommand().equals("List All Locales")){
           
           
            for(int i=0; i<availableLocales.length; i++){
           
               
                output.append(availableLocales[i].getDisplayName()+"\n");
        }
            }
       
    }

   
    public static void main(String[] args) {
        localesTest myLocalest = new localesTest();
        myLocalest.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

    }

}

---

Good. Glad you solved it.

EDIT: while we're at it, class names start with a capital letter, so it should be LocalesTest and there is no need to extend JFrame, as you don't alter JFrame's behaviour. Rather have a JFrame instance in your class. And last but not least start your window from the EDT.

Quote:

---

Originally Posted by PhHein

EDIT: while we're at it, class names start with a capital letter, so it should be LocalesTest

---

Cheers Ill remember that from now on.

Quote:

---

Originally Posted by PhHein

and there is no need to extend JFrame, as you don't alter JFrame's behaviour. Rather have a JFrame instance in your class.

---

Usually I would actually, don't know what made me extend this time around

Quote:

---

Originally Posted by PhHein

And last but not least start your window from the EDT.

---

Ok excuse the newwb question but by EDT you mean?

The EDT is the event-dispatching thread. See this example from the tutorials:
http://java.sun.com/docs/books/tutor...FrameDemo.java

Ah was never thought that, cheers again man