文字コードの違いによる、ファイルの読み込みデータの違いにつきまして。

Question

文字コードの違いによる、ファイルの読み込みデータの違いにつきまして、やりたいこととしましてはプロパティ設定ファイル(テキストファイル)を読み込み、対応する値を出力させたいのですが、以下のコードがその処理となります。

package test;

public class Main {

	public static void main(String [] args) {
		//プロパティファイル読み込み
		Properties properties =Properties.getInstance("settings/p.txt");

		//値を取得し、出力
		System.out.println(properties.getValue("aaa"));
		System.out.println(properties.getValue("bbb"));
		System.out.println(properties.getValue("ccc"));
	}
}

package test;

import java.io.BufferedReader;
import java.io.File;
import java.io.FileReader;
import java.io.IOException;
import java.util.ArrayList;
import java.util.HashMap;
import java.util.List;
import java.util.Map;

/**
 * プロパティファイル読み込み処理
 */
public class Properties {

	/** ファイルごとのプロパティインスタンス */
	private static Map<String, Properties> propertiesInstance;

	/** プロパティファイルの中身 */
	private Map<String , String> property;

	/** keyとvalueを分ける文字 */
	public static final String KEY_VALUE_SEPARATOR = "=";

	/**
	 * コンストラクタ
	 * @param filePath
	 */
	public Properties(String filePath) {

		try {

			//filePathの内容を読み込む
			String [] fileContent = fileRead(filePath);

			//プロパティHashmap初期化
			this.property = new HashMap<String, String>();

			for(String line : fileContent) {

				if(line.indexOf(KEY_VALUE_SEPARATOR) == -1) {
					//行中に = を含まない場合は次の行に進む
					continue;
				}
				// = で分割し、左側をkey 右側をvalueとして入れる
				String [] keyValue = line.split(KEY_VALUE_SEPARATOR);
				this.property.put(keyValue[0], keyValue[1]);
			}

		} catch (IOException e) {
			e.printStackTrace();
		}
	}

	/**
	 * インスタンス取得(シングルトン)
	 * @param filePath
	 * @return
	 */
	public static Properties getInstance(String filePath) {
		if(propertiesInstance == null) {
			//プロパティインスタンス保持Hashmap初期化
			propertiesInstance = new HashMap<String, Properties>();
		}
		if(propertiesInstance.containsKey(filePath) == true) {
			//すでにファイルを読み込んだ場合
			return propertiesInstance.get(filePath);
		} else {
			//新しくファイルを読み込む場合
			Properties newInstance = new Properties(filePath);
			propertiesInstance.put(filePath, newInstance);
			return newInstance;
		}
	}

	/**
	 * プロパティフィルのkey値に対するvalue値取得
	 * @param key
	 * @return
	 */
	public String getValue(String key) {
		return this.property.get(key);
	}

	/**
	 * ファイル読み込み処理
	 * @param filePath
	 * @return
	 * @throws IOException
	 */
	public String [] fileRead(String filePath) throws IOException {

		//ファイルの中身を１行ずつ読み込み、Stringの配列に変換する
		File file = new File(filePath);
		FileReader fileReader = new FileReader(file);
		BufferedReader bufferedReader = new BufferedReader(fileReader);

		List<String> readData = new ArrayList<String>();

		String line;

		while((line = bufferedReader.readLine()) != null) {
			readData.add(line);
		}

		fileReader.close();

		return readData.toArray(new String[] {});
	}
}

プロパティ設定ファイルのデータは下記の通りです。

aaa=111
bbb=222
ccc=Settings

プロパティ設定ファイルのデータをShift-JISで保存して実行した場合の実行結果は下記の通りとなり、期待通りの結果となります。

---

111
222
Settings

---

しかし、読み込みデータをUTF-8で保存した場合の実行結果は、

---

null
222
Settings

---

となり、一番最初の行のデータのみ値の取得ができません。(読み込みデータの先頭に１行空白を入れればすべて読み込めるのですが…)

Linux上で動かしたいので、設定ファイルの文字コードはUTF-8としたいのですが、どのようにすれば１行目から正常に読み込めるようになるのでしょうか。

よろしくお願いいたします。

Comments

[umyu]

UTF-8で保存するときに、BOM(バイトオーダーマーク）が付くメモ帳で保存してませんかー？

[y_oota]

BOMが付かないように、個人的に一番信頼しているTeraPadを使用していたのですが、一番最初の情報がnullとなってしまっていました。 Windowsが勝手につけているのでしょうか・・・

Answer 1

つjava.util.Properties

Comments

[退会済みユーザー]

私もJava標準のプロパティで実装したほうが良いと思います。

[y_oota]

回答していただきまして、ありがとうございます。
propertyファイルはフレームワーク上でしか見たことがなかったので、まさか普通にあったとは思いもしませんでした。。
試してみましたところ正常にいけそうなので、この方式を採用しようかと思います。

Answer 2

UTF-8をBOM付きにしたところ、不具合が再現しました。
BOM無しで保存するようにするのが最も簡潔かと思います。

あと、getInstanceがあるのにコンストラクタがpublicであることがすごく気になります。

Comments

[umyu]

メモ帳のUTF８BOMは初見トラップすぎですねー。こちらはgetInstanceで同期を取っていないことが気になりました。

[LouiS0616]

そもそもシングルトンにする必要があるのかどうかもちょっと微妙な気がします。

[y_oota]

回答していただきまして、ありがとうございます。
BOMが付かないように、個人的に一番信頼しているTeraPadを使用していたのですが、一番最初の情報がnullとなってしまっていました。

Windowsが標準で搭載しているものは基本的にBom付きUTF-8になるというのを学生時代に聞いた覚えがあります。
また、BOMを付けないと、例えば、CSVファイルをExcelで開くときにBOMを付けないと文字化けするような事象が起こり、度々悩まされます。

シングルトンにしようとしたのは、ファイル読み込み処理を何度も発生させたくなかったのでこのような形式としたのですが、いままであまり意識したことはなく正解はいまいちわかりません。。。
コンストラクタがpublicなのは、普通にミスでした。指摘していただきまして、ありがとうございます。