JTextAreaの文字列が表示されない

質問

閲覧ありがとうございます。

現在eclipseでjavaの勉強をしており、Apache POIを使ってエクセルの操作をしたいと思い以下のプログラムを組んでみたのですが、ボタンの方は上手く動作したのですがメニューの方が上手く動作しません・・・やりたい事は以下の通りです。

・[ファイル]-[ファイルを作成]をクリックすると、保存ダイアログが表示され指定したディレクトリにtextAreaの内容が1行目、1列目のセルに表示されているxlsファイルを作成する

また、ついでに教えて頂きたいのですが、WindowBuilderを使用しフレームの設計をする際、Button等各コンポーネントのアクセス修飾子はデフォルトだと全てprivateなのですが、この修飾子はプログラム中で変更しても問題無いでしょうか?WindowBuilderはつい最近使い始めたばかりでまだ慣れてないので・・・

以下がソースです。開発環境はeclipse Indigo3.7で、WindowBuilderを使用しています。言語はJavaです。（字数オーバーしたので、import文は省略しています。
■ExcelTest.java
public class ExcelTest implements ActionListener {

private JFrame frame;
private JTextArea textArea;
private JButton btnwo;
private menu menu;

public static void main(String[] args) {
EventQueue.invokeLater(new Runnable() {
public void run() {
try {
ExcelTest window = new ExcelTest();
window.frame.setVisible(true);
} catch (Exception e) {
e.printStackTrace();
}
}
});
}

public ExcelTest() {
initialize();
}

private void initialize() {
frame = new JFrame();
frame.setBounds(100, 100, 450, 300);
frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

textArea = new JTextArea();
frame.getContentPane().add(textArea, BorderLayout.CENTER);

btnwo = new JButton("ファイルを作成");
btnwo.addActionListener(this);
frame.getContentPane().add(btnwo, BorderLayout.WEST);

menu = new menu();
frame.getContentPane().add(menu, BorderLayout.NORTH);
}

　　　　　　//ボタンの動作
// こちらは上手く動作する
@Override
public void actionPerformed(ActionEvent e) {
JFileChooser chooser = new JFileChooser();
int selected = chooser.showSaveDialog(null);

if (selected == JFileChooser.APPROVE_OPTION) {
String path = chooser.getSelectedFile().getPath(); // パスを取得
System.out.println(path); // パスを表示

Workbook wb = new HSSFWorkbook();
Sheet sheet = wb.createSheet();
Row row = sheet.createRow(0); // 1行目を作成
Cell cell = row.createCell(0); // 1行目の1列目にセルを作成
cell.setCellValue(getTextAreaText()); // テキストエリアの文字列を表示
try {
FileOutputStream fo = new FileOutputStream(path + ".xls");
wb.write(fo);
fo.close();
} catch (IOException e1) {
e1.printStackTrace();
}
}
}

public String getTextAreaText() {
return textArea.getText(); // テキストエリアの文字列を取得
}
}

■menu.java
public class menu extends JPanel implements ActionListener {
private static final long serialVersionUID = 1L;
private JMenuItem menuItem;

public menu() {
setLayout(new BorderLayout(0, 0));

JMenuBar menuBar = new JMenuBar();
add(menuBar, BorderLayout.NORTH);

JMenu menu = new JMenu("ファイル");
menuBar.add(menu);

menuItem = new JMenuItem("ファイルを作成");
menuItem.addActionListener(this);
menu.add(menuItem);
}

//メニューの動作
// こちらが上手く動作しない
@Override
public void actionPerformed(ActionEvent e) {
JFileChooser chooser = new JFileChooser();
int selected = chooser.showSaveDialog(null);

if (selected == JFileChooser.APPROVE_OPTION) {
String path = chooser.getSelectedFile().getPath(); // パスを取得
System.out.println(path); // パスを表示

Workbook wb = new HSSFWorkbook();
Sheet sheet = wb.createSheet();
Row row = sheet.createRow(0); // 1行目を作成
Cell cell = row.createCell(0); // 1行目の1列目にセルを作成
ExcelTest excel = new ExcelTest();
cell.setCellValue(excel.getTextAreaText());
try {
FileOutputStream fo = new FileOutputStream(path + ".xls");
wb.write(fo);
fo.close();
} catch (IOException e1) {
e1.printStackTrace();
}
}
}
}

回答No.2

LancerVII · ベストアンサー

こんにちは。

どうしてもこのクラス構成で作りたいのであれば以下のようにすれば良いと思います。
(途中は割愛します)

==== Menu.java(先頭大文字にしました。Javaの推奨クラス名です)
private JMenuItem menuItem;
// ExcelTestを保持する変数
private ExcelTest f_excelTest;
public Menu ( ExcelTest _excelTest ) {
// コンストラクタにExcelTestを受け取るように引数を追加
// 受け取ったExcelTestをフィールド変数へ
f_excelTest = _excelTest
}
public void actionPerformed(ActionEvent e) {
//========= 中略
// TestExcel excel = new TestExcel();
// cell.setCellValue(excel.getTextAreaText());
// 上記では新しくTestExcelクラスを生成しているので取得できない
// なのでコンストラクタで受け取ったExcelTestクラスより取得
cell.setCellValue(f_excelTest.getTextAreaText());
}

==== TestExcel.java
//menu = new menu();
// コンストラクタにExcelTestすなわち自分自身(this)を渡すように修正
menu = new Menu(this);

>Button等各コンポーネントのアクセス修飾子はデフォルトだと全てprivateなのですが
WindowBuilderを使ったことが無いのでちがっていたらすみません。

private JButton button;
private JTextArea textArea;
と定義されるということでしょうか。

変えても良いですが、自分がクラス設計する場合でもそのクラス内で利用されている部品に関してはprivateにします。
外部クラスからのアクセスが必要であればその必要なもののみアクセスできるようにします。
例えばTextAreaの内容を取得したいのであれば
private JTextArea textArea = new JTextArea();

public String getText() {
return textArea.getText();
}
とします。textAreaはあくまでprivateとして外部から参照できるようにpublicのgetText()を用意します。
この辺は「カプセル化」みたいなワードで検索すると有益な情報が出てくると思います。

お礼

成程・・・勉強になりました！ご回答、ありがとうございます。

回答No.1

tyatsumi

ExcelTest excel = new ExcelTest();
cell.setCellValue(excel.getTextAreaText());

となっているので、main で生成されて表示されているExcelTestインスタンスとは違う新しく生成されたExelTestインスタンスのテキストエリアを読んでしまっている。
対策としては、menuにExcelTestの参照を渡しておいて、menu内のメソッドから表示されているExcelTestのテキストエリアを参照できるようにするなどの方法が考えられる。

対策案。
menuクラスは

public class menu extends JPanel implements ActionListener {
private ExcelTest exel;
public menu(ExcelTest exel) {
this.exel = exel;
...
}

のような感じにして、

ExcelTest.initializeでは

menu = new menu(this);

のようにExcelTestインスタンスの参照を渡す。

あとはmenuクラス内の
ExcelTest excel = new ExcelTest();
の行を削除して、excelとしては新しく作ったメンバ private excel; を使うようにする。

これでどうでしょう。

お礼

お陰様で上手く行きました！丁寧なご回答、ありがとうございます。