The textarea is not getting value

[budsy remo]

import javax.swing.*;
import java.awt.event.*;
import static java.lang.System.*;
import java.awt.*;
import javax.swing.border.*;
import java.io.*;
import java.util.*;
public class CodeObfuscatorGui
{
	public static void main(String... args)
	{
		EventQueue.invokeLater(new Runnable()
		{
			public void run()
			{
				TFrame tf1=new TFrame();
				tf1.setVisible(true);
				tf1.setDefaultCloseOperation(tf1.EXIT_ON_CLOSE);
			}
			
		});
	}
}
class TFrame extends JFrame
{
	private JTextArea jta1,jta2;
	private JEditorPane jep1;
	private JFileChooser jfc1;
	private String path;
	private Font f1;
	TFrame()
	{
		
		Toolkit kit=Toolkit.getDefaultToolkit();
		Dimension d=kit.getScreenSize();
		out.println(d.height);
		setSize(d);
		//new Dimension(800,600));
		setLayout(null);
		
		JMenuBar jmb1=new JMenuBar();
		setJMenuBar(jmb1);
		Dimension d1=calculateSize(d);
		if(path==null)
		out.println("hello");
		
		JMenu file=new JMenu("File");
		jmb1.add(file);
		
		JMenuItem open_menu=new JMenuItem("Open",'O');
		file.add(open_menu);
		//jmi1.setEnabled(false);
		JMenuItem get_menu=new JMenuItem("get");
		file.add(get_menu);
		
		
		JPanel jp1=new JPanel();
		jp1.setLayout(new BorderLayout());
		Border b1=BorderFactory.createEtchedBorder();
		Border b2=BorderFactory.createTitledBorder(b1,"Original Code");
		jp1.setBorder(b2);
		add(jp1);
		jp1.setBounds(0,0,d1.width-10,d1.height-90);
		jp1.setBackground(Color.GRAY);
		out.println(d1.width);
		
		jep1 = new JEditorPane();
		
		JPanel jp2=new JPanel();
		jp2.setLayout(new BorderLayout());
		b2=BorderFactory.createTitledBorder(b1,"Obfuscated Code");
		jp2.setBackground(Color.GRAY);
		jp2.setBorder(b2);
		add(jp2);
		
		jp2.setBounds(d1.width,0,d1.width-10,d1.height-90);
		
		
		f1=new Font("SansSerif",Font.PLAIN,15);
		
		jta1=new JTextArea();
		jta1.setFont(f1);
		jta1.setBounds(0,0,d1.width-10,d1.height-90);
		jta1.setLineWrap(true);
		JScrollPane jsp1=new JScrollPane(jta1);
		jp1.add(jep1,BorderLayout.CENTER);
		
		jta2=new JTextArea();
		jta2.setLineWrap(true);
		JScrollPane jsp2=new JScrollPane(jta2);
		jta2.setFont(f1);
		jp2.add(jsp2,BorderLayout.CENTER);
		
		/*addComponentListener(new ComponentListener()
		{
		});*/


		/* Adding Menu Items*/
		jfc1=new JFileChooser();
		jfc1.setCurrentDirectory(new File("."));
		
		open_menu.addActionListener(new ActionListener()
		{
			public void actionPerformed(ActionEvent e)
			{
				int result=jfc1.showOpenDialog(TFrame.this);
				if(result==jfc1.APPROVE_OPTION)
				path=jfc1.getSelectedFile().getPath();
			}
		});


		

		get_menu.addActionListener(new ActionListener()
		{
			public void actionPerformed(ActionEvent e)
			{	

				String textdata=jta1.getText();
				out.println(textdata);
				//Separator sep=new Separator(textdata);
				//ArrayList<String> list=(ArrayList<String>)sep.getProgramLines();
				//we need a separate thread to use this coz otherwise the application will become very slow.
				

				//KeywordId kid=new KeywordId(list);
				
				//FunctionIdentification fi1=new FunctionIdentification(list);
			}
		});


		
		
	}
	public Dimension calculateSize(Dimension d)
	{
		return new Dimension(d.width/2,d.height);
	}
}

My problem is that the textarea in the local inner class is not getting the value that i type when i run the code and click the get from the menu . It is supposed to take the value from the textarea and then print it on command line .

[Darryl Burke]

1. Always, always, to get better help sooner, post a SSCCE (Short, Self Contained, Compilable and Executable) example that demonstrates the problem. That means eliminating all unrelated stuff such as setting fonts and borders, and removing unnecessary vertical white space. Remember, the shorter it is, the more members will read it.

2. Learn to use layouts correctly. That means no calls to setBounds(...). I'm sure you've been told this before.

3. Use meaningful variable names. Names like jta1, jep1, jfc1 aren't.

4. I don't see anywhere in the posted code that jta1 is added to the visible component hierarchy, so why does it surprise you that its text remains empty?

[john price]

Here is the bare minimum (doesn't work, but allows to see the problem) :

import javax.swing.*;
import java.awt.event.*;
import java.awt.*;
public class CodeObfuscatorGui {
	public static void main(String... args) {
		EventQueue.invokeLater(new Runnable() {
			public void run()
			{
				TFrame tf1=new TFrame();
				tf1.setVisible(true);
			}
		});
	}
}
private class TFrame extends JFrame {
	private JTextArea jta1;
	
	TFrame() {
		
		JMenuItem get_menu=new JMenuItem();
		
		jta1=new JTextArea();

                JScrollPane jsp1=new JScrollPane(jta1);

		get_menu.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent e) {
				System.out.println(jta1.getText());
			}
		});
	}
}

[Darryl Burke]

And the problem is that once again, the JTextArea jta1 isn't added to a visible component hierarchy. Isn't that obvious to you?

[budsy remo]

@ Darryl Don't you think that i added the textarea through a JScrollPane ?

[Darryl Burke]

1. In the first code posted, that JScrollPane isn't added to any container.

2. In the second code posted, there's no JScrollPane.