gettter、setterの使い方がいまいちよくわかりません…

Question

課題で、テキストファイルから単語を読み込み、その単語の数をカウントして表示するプログラムを作成しています。

例) りんご：３回です。

○単語登録用の配列数は６
○それ以上は登録できない
○アクセス制御とパッケージ化を使用する
○全ての処理をmainメソッドだけで行わない
○staticはmainメソッドのみ使用可

○以下は必ず使用
・メインクラス TextWordCounter
・単語登録用配列 tw[]

・単語登録クラス TextWord
・単語 word (文字列) privateで
・回数 cnt (整数) privateで

○その他必要に応じて適宜

getやsetを上手く使用できず、コンパイルは通りますが、単語の登録も出現回数もぬるぽになってしまいます。
とりあえずcount[]を一時的に作成して、先に単語の登録をどうにかしようと試行錯誤しているのですが、どうすればいいのでしょうか。
アドバイスよろしくお願いします。

---

テキストファイルはこんな感じです↓

りんご
みかん
りんご

ぶどう
りんご
めろん
すいか

きうい
ばなな
みかん
いちご
ざくろ

---

import java.io.*;
import java.util.Scanner;
import java.util.*;
import mypackage.TextWord;



public class Test{
	
	void scan(){
		
		//変数の定義
//		String tw[] = new String[6];
		int i = 0;
		String s = null;
		TextWord tw[]= new TextWord[6];
		int count[] = new int[tw.length];

//		count[i] = tw[i].getCnt();
		
		try{
			
		//ファイルを開く
			Scanner scn = new Scanner ( new File("テキストのファイル名"));
			
			
		//ファイルの読み込み
			while( scn.hasNextLine() ) {
				s =scn.nextLine();
				for ( i = 0; i < tw.length; i++ ) {
					
					//未登録単語の場合
					if ( tw[i] == null) {
						if("".equals( s )){
							//改行はカウントせずにスルーする
						}else {
							tw[i].setWord( s );
							count[i] = 1;
//							tw[i].setCnt(1);
						}
						break;
						
					//登録済み単語の場合
					} else if ( tw[i].equals( s ) ) {
						count[i]++;
//						tw[i].setCnt(tw[i].getCnt()+1);
						break;
					}
				}
				
				i++;
				
			}
			scn.close();
			
		}catch(FileNotFoundException e){
			System.out.println("ファイルが存在しません。");
			
		}

		// 登録リストの表示
		try{
			for ( i = 0 ; tw[i] != null ; i++ ) {
				System.out.println( tw[i].getWord() + ":" + count[i] + "回です。" );
			}
		}catch(ArrayIndexOutOfBoundsException e){
			System.out.println("登録件数は上限です。");
		}

	}


	
	
	//実行
	public static void main ( String [] args ) {
		
		Test t = new Test();
		t.scan();
		
	}
}

package mypackage;

public class TextWord {
	
	//単語 文字列型
	private String word = null;
	
	
	public String getWord(){
		return this.word;
	}
	
	public void setWord(String word){
		this.word = word;
	}
	
	//出現回数 整数型
	private int cnt = 0;
	
	public int getCnt(){
		return this.cnt;
	}
	
	public void setCnt(int cnt){
		this.cnt = cnt;
	}
	
	
}

Answer 1

        TextWord tw[]= new TextWord[6];

これでは、「TextWordが6個入る配列」を作っただけで、本体のTextWordが作られていません。
別途TextWordのコンストラクタを呼び出し、TextWordオブジェクトを作成する必要があります。
例えば、こんな感じ。

//未登録単語の場合
                    if ( tw[i] == null) {
                        if("".equals( s )){
                        }else {
+                            tw[i] = new TextWord();  //TextWord作成して代入
                            tw[i].setWord( s );
                            count[i] = 1;
//                            tw[i].setCnt(1);
                        }
                        break;

                    }

ただ、課題なのかもしれませんが、Mapを使ったほうがわかりやすいのと、
現状のこのコードが、説明するのが面倒になるレベルで結構ひどい惨状になっています。

Comments

[tatatana]

なるほど。配列しか作ってなかったのでぬるぽが出てきたのですね。
MapやList？等は使わずに作ることが課題でした…。
無事に解決できました！ありがとうございます！