CSVファイルを読むサンプルコードは以下のとおり．CSVReader#readNext()で1行読み込んで，行を区切った結果をStringの配列で返してくれる．

CSVReader reader = new CSVReader(new FileReader("hoge.csv"));
String[] line;
while ((line = reader.readNext()) != null) {
    System.out.println(line[0] + line[1] + "etc...");
}

まずは現状の説明から．EJB層からの戻り値となるvalue objectでは，EJB層で検出するひとつのエラーに対して，ひとつの～Flgというvalue objectのプロパティを定義している．

public class FooResult {
    private boolean hogeCodeErrorFlg;
    private boolean gehoCodeErrorFlg;

    public boolean getHogeCodeErrorFlg() {
        return hogeCodeErrorFlg;
    }
    public void setHogeCodeErrorFlg(boolean value) {
        this.hogeCodeErrorFlg = value;
    }

    public boolean getGehoCodeErrorFlg() {
        return gehoCodeErrorFlg;
    }
    public void setGehoCodeErrorFlg(boolean value) {
        this.gehoCodeErrorFlg = value;
    }
}

EJB層ではメソッドの戻り値として返すFooResultに対して，エラーがあった場合はそれぞれのプロパティにfalseを，なかった場合にはtrueを設定している．

public FooResult foo(...) throws Exception {
    FooResult result = new FooResult();

    if (validateHogeCode(...)) {
        result.setHogeCodeErrorFlg(true);
    } else {
        result.setHogeCodeErrorFlg(false);
    }

    if (validateGehoCode(...)) {
        result.setGehoCodeErrorFlg(true);
    } else {
        result.setGehoCodeErrorFlg(false);
    }

    /// do something...

    return result;
}

JSP/Servlet層では，このvalue objectのget～Flg()をそれぞれ調べて，falseの場合はエラーを表示している．以下の例ではstrutsのタグを使ってエラーを表示している．

<logic:equal name="fooResult" property="hogeCodeErrorFlg" value="false">
    <%= "hogeコードがおかしいですよ．" %>
</logic:equal>

<logic:equal name="fooResult" property="gehoCodeErrorFlg" value="false">
    <%= "gehoコードがおかしいですよ．" %>
</logic:equal>

このコードにはいくつかの問題点がある．

まず大きな問題として，エラーの追加という変更に対して非常に弱いということである．EJB層のfoo()メソッドで行うエラーチェックをひとつ増やすと，上で示した3つのコードのすべての箇所で修正が必要である．データ・ロジック・表示の修正が必要と，まさにダメ三冠王である．例えばFooResultではエラー内容をListなどに格納しておき，JSP/Servlet層ではそのListの中身を順番に表示する，といった形にすればデータと表示に関してはエラーの追加による影響を減らすことができる．

次に気になるのはプロパティ名である．boolean型のプロパティ名に～Flg，～Flagなどと名づけるのは基本的にはご法度である．hogeCodeErrorFlgといわれても，エラーがあった場合の値がtrueなのかfalseなのかわからない．実際私は上のコードを書いていて何度かtrueとfalseを間違えてしまった．boolean型のプロパティ名は，is～，has～，can～のどれかの名前にしないといけない，と思っておけば問題ない．例えばisValidHogeCodeやisNotValidHogeCodeといったプロパティ名にしておけば，エラーがあった場合の値がtrueなのかfalseなのかはプロパティ名だけから推測できる．

改善案の一例を以下に示しておく．エラー情報保持用のValidationErrorクラスを定義し，FooResultがリストでエラーを保持できる形にする．なお，この例では簡単のため，エラーメッセージをValidateionErrorのプロパティとして持たせているが，idとエラーの原因となった入力値だけをプロパティに持たせておいて，JSP/Servlet層でそれらの値からメッセージを組み立てるようにした方がよいこともある．

/// なにかしらのエラー情報を持つクラス
public class ValidationError {
    /// エラーの種別を表すID
    private String id;
    /// エラーメッセージ
    private String message;

    public ValidationError(String id, String message) {
        this.id = id;
        this.message = message;
    }
    public String getId() {
        return id;
    }
    public String getMessage() {
        return message;
    }
}

/// エラー情報をリストで保持
public class FooResult {
    private List<ValidationError> errors;
    public FooResult() {
        errors = new ArrayList<ValidationError>();
    }
    public List<ValidationError> getErrors() {
        return errors;
    }
}

EJB層では，エラーがあったときにValidationErrorオブジェクトを生成して，FooResultのerrorsプロパティに追加する．

public FooResult foo(...) throws Exception {
    FooResult result = new FooResult();

    if (!validateHogeCode(...)) {
        result.getErrors().add(
            new ValidationError("error.isNotValidHogeCode", "hogeコードがおかしいですよ．");
        );
    }

    if (!validateGehoCode(...)) {
        result.getErrors().add(
            new ValidationError("error.isNotValidGehoCode", "gehoコードがおかしいですよ．");
        );
    }

    /// do something...

    return result;
}

JSP/Servlet層では，FooResultのerrorsプロパティに格納されているValidationErrorのmessageプロパティを順次表示する．

<logic:iterate id="error" name="fooResult" property="errors">
    <bean:write name="error" property="message"/>
</logic:iterate>

これでエラーチェックを追加する場合でも，EJB層のfoo()メソッドのコードの修正だけで対応できるようになる．

Googleで「csv java」で検索するとStringTokenizerを使ってCSV形式の文字列を処理するいくつかの例が見つかるが，これらの例には注意しないといけない点がある．具体的には，"foo,,baz"といった，一部の列に値が存在しないために「,」が続くような文字列が入力された場合，"foo"と"baz"だけが処理対象となってしまう．一般的に期待するのは"foo"と""(空文字)と"bar"が処理対象となることだろう．

Googleで見つかるStringTokenizerを使ったCSV処理の例から，列の分割処理だけを切り出して整えるとだいたい以下のようなコードになる．

private static String[] splitLineWithComma(String line) {
    if (line == null) {
        return new String[0];
    }

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

    StringTokenizer tokenizer = new StringTokenizer(line, ",");
    while (tokenizer.hasMoreTokens()) {
        ret.add(tokenizer.nextToken());
    }

    return (String[]) ret.toArray(new String[ret.size()]);
}

このコードは"foo,bar,baz"といったすべての列に値が入っている文字列に対しては期待通りに"foo"と"bar"と"baz"が入った配列を返す．しかし，"foo,,baz"といった，一部の列に値が存在しないために「,」が続くような文字列に対しては期待通りの結果を返さない．StringTokenizerが「,」と「,」の間の""をトークンとはみなせないために，"foo"と"bar"だけが入った配列を返してしまう．

単純にカンマで区切るだけでよいのならJDK 1.4以上であれば line.split(",", -1)とすればよいし，それ以前のバージョンのJDKであれば以下のようなコードでよい．

private static String[] splitLineWithComma(String line) {
    if (line == null) {
        return new String[0];
    }

    List ret = new ArrayList();

    int colStart = 0;
    int colEnd = 0;
    while ((colEnd = line.indexOf(',', colStart)) != -1) {
        ret.add(line.substring(colStart, colEnd));
        colStart = colEnd + 1;
    }

    ret.add(line.substring(colStart));

    return (String[]) ret.toArray(new String[ret.size()]);
}

CSVファイルの一行を列に分解するには「"」に囲まれていない「,」で行を分割して，その後各列から余分な「"」を省けばよい．以下でこうした処理を正規表現を使って実現する方法の一例を示す．

まず「"」に囲まれていない「,」で行を分割することを考える．正規表現で表現するために，「"」に囲まれていない「,」，を，後ろに偶数個の「"」が現れる「,」，と読み替える．これを正規表現で書くと，「,(?=(([^"]*"){2})*[^"]*$)」となる．この正規表現で区切って行を分割すればよい．

次に各列から余分な「"」を省くことを考える．余分な「"」を省くには，各列を囲んでいる「"」を削除し，エスケープされた「""」を「"」に戻せばよい．それぞれ「^"|"$」にマッチすれば「」(空文字)に置換，「""」にマッチすれば「"」に置換という処理を行えばよい．

以上をまとめてJavaのコードにしたものを以下に示す．なお，この例では「"」が閉じられていないなど，CSVとして不正な形式の文字列が入力されたときのことは考慮していない．

/** 後ろに偶数個の「"」が現れる「,」にマッチする正規表現． */
private static final Pattern separatingCommaPattern = Pattern.compile(",(?=(([^\"]*\"){2})*[^\"]*$)");
/** 最初と最後の「"」にマッチする正規表現． */
private static final Pattern surroundingDoubleQuatationPattern = Pattern.compile("^\"|\"$");
/** 「""」にマッチする正規表現． */
private static final Pattern escapedDoubleQuoatationPattern = Pattern.compile("\"\"");

/** lineをカンマ区切りで分割して不要な'"'を取り除いたStringの配列を返す． */
public static String[] splitLineWithComma(String line) {
    /// '"'で囲まれていない','でlineを分割
    String[] cols = separatingCommaPattern.split(line, -1);
    String[] ret = new String[cols.length];
    for (int i = 0, len = cols.length; i < len; i++) {
        String col = cols[i].trim();

        /// 最初と最後に'"'があれば削除
        Matcher matcher = surroundingDoubleQuatationPattern.matcher(col);
        col = matcher.replaceAll("");

        /// エスケープされた'"'を戻す
        matcher = escapedDoubleQuoatationPattern.matcher(col);
        col = matcher.replaceAll("\"");

        ret[i] = col;
    }
    return ret;
}