JAVAでテキスト中の文字の登場回数を数える。

Question

###前提・実現したいこと
テキスト中の<year></year>で囲まれた年数について、それぞれ何回登場したかを表示するプログラムを書きたいと思っています。
2005~2016が対象なので、それぞれに対してMapに割当し、回数をカウントした後に出力するプログラムにする予定です。
###発生している問題・エラーメッセージ

今のところ、空のテキストファイルが作成されるのみになっています。
PatternとMatcherの問題ではないと思うのですが、Mapの使い方が間違っているのでしょうか？

###該当のソースコード

import java.io.*;
import java.util.regex.Pattern;
import java.util.HashMap;
import java.util.Map;
import java.util.regex.Matcher;

public class CountePaper2 {
		void dataInOut() throws IOException{
		String Year = "<year>(.*?)</year>";
		
		//パターンの作成
		Pattern pY = Pattern.compile(Year);
		//入力ファイルと出力ファイルの指定、読み込み
		BufferedReader din = new BufferedReader(new FileReader("Output.txt"));
		BufferedWriter dout = new BufferedWriter(new FileWriter("Counted.txt"));
		
		//パターンとマッチしたらカウントを１増やす。
		String buf;
		Matcher mY;	
		Map<String, Integer> map = new HashMap<>();
		while((buf = din.readLine()) != null){
			mY = pY.matcher(buf);
			if(mY.find()){
				if (map.containsKey(mY.group(0))) {
                    int count = map.get(mY.group(0)) + 1;
                    map.put(mY.group(0), count);
                } else {
                    map.put(mY.group(0), 1);
			}
		}
	}
		for(int y = 2005; y <= 2016; y++){
			if(map.containsKey(y)){
				dout.write( y + " (" + map.get(y) + ")\n");
			}
		}
		din.close();
		dout.close();
	}
	//実行
	public static void main(String[] args){
		CountePaper2 t = new CountePaper2();
		try{
			t.dataInOut();
		}catch(IOException error) {System.out.println("IOエラー");}
		System.out.println("動作終了");
	}
}

###試したこと
課題に対してアプローチしたことを記載してください

###補足情報(言語/FW/ツール等のバージョンなど)
より詳細な情報

Answer 1

for (int y = 2005; y <= 2016; y++) {
  if (map.containsKey(y)) { // <- containsKeyの引数にIntegerが渡されている
    ...
  }
}

mapのキーの型がStringなのですが、containsKeyに渡すキーの型がInteger(※)になっており型が不一致なためどんな値を指定しても「一致するキーは存在しない」とみなされているのが原因です。

※: 変数yの型はintなのですがauto boxingという言語仕様によってInteger型へ自動変換されます。
本件の原因とは直接関係しません。

対処方法はキーにアクセスしている箇所の型をStringまたはIntegerに統一すればよいです。とりあえず修正箇所を少なくするなら上のコードで以下のようにint->Stringへ変換すると期待通りの結果となると思います。
if (map.containsKey(String.valueOf(y))) {

Comments

[KSwordOfHaste]

Matcher.groupの部分は他の方が指定されていたので本回答から省きました。と思ったらそちらの回答に既に自分の回答と同じコメントがついてましたね・・・
map.containsKeyにStringを指定してないのにコンパイルエラーになってくれない件は標準ライブラリーのインターフェースがジェネリクスなのに引数の型がObjectになっているせいですが、こうしたインターフェースには悲しくなることがたまにあります…

[退会済みユーザー]

ありがとうございました！解決しました。

Answer 2

おそらくMapのkeyに格納されているのが2005などの数値ではないからです。System.out.printlnなどでmY.group(0)を出力してみてください。
Matcher#groupの引数0だと<year>xxxx</year>という、マッチした文字列全体がとれます。()部分だけ欲しいなら引数は1を指定すべきです。引数は0だと全体、1以上だとn番目のカッコ部分、ということになります。

Comments

[KiyoshiMotoki]

横から失礼します。

> （Matcher#groupの）引数は1を指定すべき

に加えて、

　for(int y = 2005; y <= 2016; y++){
　　　if(map.containsKey(y)){
　　　　　dout.write( y + " (" + map.get(y) + ")\n");
　　　}
　}

部分にも修正が必要ですね。

変数 map のキーは String型であるにも関わらず
containsKey() やget() の引数に int型の変数を指定しているため、
この部分も正常に動作していませんので。

[退会済みユーザー]

ありがとうございました！解決しました。