closeEntry()って、.putNextEntry()ごとに閉じるんじゃないの？

.close()はfinallyでやるべきだし。

よく分からんけど、csvファイルの内容をwrite()しないで一体何をwrite()してるのか？

それに、closeEntry()の場所も違うみたい。

まず、objBosの内容がfileName.csvに書き出されるの？　なにしろ、よーわからん。

いや、DBから取ったcsvがobjBosのbyte[]に書き出されるから、それをresponseにかぶせたZipのOutputStreamへ流す。

よく見ればcloseEntry()だけじゃなくclose()もループの外だなあ。


>lineData.toString()
resultはList<String>なんだから、toString()するまでもなくStringだと思われ。

あ、さらに。
Writerをclose()すれば、普通はその中のStreamもclose()される。

掲示板に掲載用にソースの省略をした際に省略しすぎました。済みません。
実際には

ZipOutputStream objZos
 = new ZipOutputStream(response.getOutputStream());
 ByteArrayOutputStream objBos = null;
 OutputStreamWriter objOsw = null;

try {
 // ContentTypeを設定
 response.setContentType"application/zip; charset=Shift_JIS");response.setHeader( "Content-Disposition"," attachment;filename=\"data.zip\"");

 for (Iterator<String> itr = tableList.iterator(); itr.hasNext();) {
   objBos = new ByteArrayOutputStream();
   objOsw = new OutputStreamWriter(objBos, "Shift_JIS");
     # (略)
     # ﾃｰﾌﾞﾙごとに検索し、
     # resultに格納する処理
     # (略)
       for (int i = 0; i < result.size(); i++) {
                        lineData = result.get(i);
          objOsw.write(lineData.toString() + "\n");
       }
       ZipEntry objZe = new ZipEntry(fileName + ".csv");
       objZe.setMethod(ZipOutputStream.DEFLATED);
       objZos.putNextEntry(objZe);
       byte[] aryByt = objBos.toByteArray();
       objZos.write(aryByt, 0, aryByt.length);
       //objZos.closeEntry(); //掲示板質問前には
       //     この位置でﾃｽﾄ済み→結果は同じでした
     }
     objZos.closeEntry();
  } catch (Exception e){
   ・
   ・
  } finally {
    if (objOsw != null) {
       objOsw.close();
    }
    if (objBos != null) {
       objBos.close();
    }
    if (objZos != null) {
       objZos.close();
    }
  }


という具合です。

objZos.closeEntry();の位置については、私も皆様と同様に考え、最初はループの中に入れていましたが、うまく行かなかったため、一度外に出し、結果が全く同じであったためそのままにしておりました。
ドキュメントには、
--------------------------------------
public void putNextEntry(ZipEntry e)
                  throws IOException
･･･新しい ZIP ファイルエントリの書き込みを開始し、エントリデータの開始位置にストリームを配置します。現在のエントリがアクティブである場合はそれを閉じます。
--------------------------------------
とありましたので、必ずしも毎回closeEntry()する必要はないのかな？と思っておりましたし。
やっぱりマズいでしょうか？

ちなみに、
? closeEntry()をループ内に配置 → 結果は変わらず
? objOsw.close()をループ内に配置 → 結果は変わらず
? objBos.close()をループ内に配置 → 結果は変わらず
という結果になりました。

>   byte[] aryByt = objBos.toByteArray();
>   objZos.write(aryByt, 0, aryByt.length);
このあたりが怪しいのでしょうか？

あと一点、
>   lineData = result.get(i);
>   objOsw.write(lineData.toString() + "\n");
の部分につきましても、
実際には、lineData は StringBuffer で、
resultをあれこれ編集してCSV形式の1行にしたものと
なっております。
中途半端な省略(書き込み)を行ったために、余計な
疑問を抱かせてしまい申し訳ございません。

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

今、うまく行きました！
  
----------------------
  //(ループ内)
  objOsw.close(); // ←【ここに持ってきた】
  ZipEntry objZe = new ZipEntry(fileName + ".csv");
  objZe.setMethod(ZipOutputStream.DEFLATED);
  objZos.putNextEntry(objZe);
  byte[] aryByt = objBos.toByteArray();
  objZos.write(aryByt, 0, aryByt.length);
} //ループ終了
objZos.closeEntry();
----------------------
objOsw.close(); の位置が悪かったようです。
皆様のアドバイスのおかげです。
本当にありがとうございました。
なお、うまく行ったとは言え、直すべきだという箇所に
ついてもさらにご教示いただければありがたく思います。

要するに出力をflush（完全吐き出し）してなかったってことね。処理のストーリーがよく分かるコードを最初から投稿してくれたら、こっちも最初に気づいたでしょう。最初の時点では、なんだかよーわからんコードでした。

>処理のストーリーがよく分かるコードを最初から投稿し>てくれたら
はい。
以後、気をつけます。