【java】【サーブレットでcsvファイル読み込み】【jspにエラー文を出したい】

Question

###前提・実現したいこと

JSPServletでcsvファイルを読み込むシステムを作っています。
【csvファイルの各セルが各字数制限を超えた場合（200文字以上でエラーです）、jspにエラー文を表示する】機能を実装中に以下の問題が発生しました。

###発生している問題・エラーメッセージ

【エラーした該当のセルを全てjspに表示させたいのですが、最後に読み込んだエラーのセルだけがjspに表示されてしまいます】

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

（Servletです）
for｛i++
for｛j++
csv［i］［j］
if（csv［i］［j］≧ 200）｛
request.setAttribute（”errorMessage”、”エラー文”）
...
...
...
ソースコードはもっと複雑なのですが、該当箇所についてまとめるとこのような記述になっています。

###試したこと
String型で変数を作って、右辺にはあらかじめcsvファイルを収めていたリストを代入しました。

エラー箇所をその都度足していって、最後にまとめてセットしようと判断しました。

String error ;
for｛i++
for｛j++
csv［i］［j］
if（csv［i］［j］≧ 200）｛

★error += list［i］［j］

request.setAttribute（”errorMessage”、”エラー文”）
...
...
...

しかし、+=の部分でエラーになってしまいました。

どなたかアドバイスをよろしくお願いいたします。

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

Comments

[A-pZ]

request.setAttribute(”errorMessage”,”エラー文”); は、request.setAttribute(”errorMessage”,error); で正しいでしょうか。エラーと言われている内容が、Javaの例外が出ているのか、それとも予期したメッセージが出ないで、特に例外が出ていないのか、を書くと、より回答に近づくかも知れません。

[otabeb]

回答ありがとうございます。
正確には
request.setAttribute(”errorMessage”,”csvエラー:” + list［i］［j］);
です。
例外は出ません。
セットアトリビュートのlist［i］［j］の部分がどんどん上書きされて、結局は一番最後のエラー箇所だけが表示される...という事態です。

Answer 1

追記ありがとうございました。確かに request.setAttribute("errorMessage" , エラーメッセージ内容)を毎回上書きしてしまっていますね。

その場合は、エラーメッセージを複数持てるよう、動的配列に持たせてしまうのが良いでしょうか。
一番かんたんなのは、java.util.ArrayListを使います。

ループの中では次のように

List<String> errorMessages = new ArrayList<String>();
...ループ...
errorMessages.add( list[i][j] );
...

...ループ終了
request.setAttribute（”errorMessage”, errorMesssages);

とすることで、複数のメッセージがList型になって errorMessage の名前でリクエスト属性に格納されます。
あとはJSPにて、errorMessage の内容をループ処理で取り出すと良いかと。

Comments

[otabeb]

皆さま回答ありがとうございます！解決しました

Answer 2

コードが断片的なので正確に原因を特定するのは少し難しいですが、

ここに書かれている情報で判断すると、
String errorが初期化されていないために、コンパイルエラーになっていると推測します。
String error = "";とすればコンパイルエラーは無くなると思います。

ただし、Stringをループの中で結合するとパフォーマンスが悪くなりますので、
StringBuilderというクラスを使ったほうが良いです。

StringBuilder errorValues = new StringBulder();
errorValues.append("csvエラー:"); // コンストラクターで指定しても良い

// ...省略

errorValues.append(list[i][j]);

// ...省略

String error = errorValues.toString();
request.setAttribute("errorMessage", error)

...

（追記）

ちょっと問題を読み違えていたところがありましたので追記しておきます。

変数は代入している前提なのですね。上にも書き足しておきます。

request.setAttribute("errorMessage", "エラー文")のところは、
ループでやってしまうと毎回errorMessageに上書きされるので、
上記のとおりループが終わった後で
request.setAttribute("errorMessage", error)とすれば良いです。

Comments

[otabeb]

皆さま回答ありがとうございます！解決しました