回答編集履歴
1
ファイル名の拡張子についてアドバイスを書き漏らし追記するためと、表現を少し手直し。
answer
CHANGED
|
@@ -1,15 +1,21 @@
|
|
|
1
1
|
問題は次の点にあります。
|
|
2
|
+
|
|
2
|
-
|
|
3
|
+
まずは、`bos` の `write` メソッドを使っている点です。
|
|
3
4
|
ここは、 `gzos` の `write` メソッドを使いましょう。
|
|
4
5
|
そうしないと、`GZIPOutputStream` が基底クラスの `write` メソッドをオーバーライドして実現しようとした処理が実行されず、期待通りの出力を得られなくなります。
|
|
5
6
|
|
|
6
|
-
ついでに言うと、`System.out.println` で読み込んだ値を `char` 型にキャストして表示していますが、`FileInputStream` クラスの `read` メソッドはファイルから１バイトずつ読み込みますので、
|
|
7
|
-
複数バイトで表現される文字(漢字など)の場合、１バイト読むごとに `char` にキャストしてもまともな文字としては表示されません。1バイトで表現される文字であれば結果的にうまくいくのですが、論理的には考慮がかけたコードになっていると言えます。
|
|
7
|
+
ついでに言うと、`System.out.println` で読み込んだ値を `char` 型にキャストして表示していますが、`FileInputStream` クラスの `read` メソッドはファイルから１バイトずつ読み込みますので、複数バイトで表現される文字(漢字など)の場合、１バイト読むごとに `char` にキャストしてもまともな文字としては表示されません。1バイトで表現される文字であれば結果的にうまくいくのですが、論理的には考慮がかけたコードになっていると言えます。
|
|
8
8
|
|
|
9
9
|
`char` 型の `c` を `gzos` の `write` メソッドで書き出しても結果的に期待通りに動きますが、不必要なキャストはやめて、 `gzos.write(fisi)；` とするので十分です。
|
|
10
10
|
|
|
11
|
+
出力するファイル名の拡張子には `.gz` をつけると良いでしょう。
|
|
12
|
+
展開ツールなどがファイルの圧縮形式を GZIP として扱ってくれます。
|
|
13
|
+
`.gz` をつけるときは`"~FileCopy2.txt.gz"` とすると、展開後に `~FileCopy2.txt` というファイル名になります。
|
|
14
|
+
|
|
15
|
+
なお、出来上がったファイルをテキストエディター等で、圧縮したまま開いてもまともに表示されないのでご注意を。
|
|
16
|
+
|
|
11
17
|
最後に、 正常に処理できた時には、`gzos.close()` を実行しましょう。
|
|
12
|
-
そうすることで `bos` の `flush` と `close` は不要
|
|
18
|
+
そうすることで `bos` の `flush` と `close` は不要というよりも、むしろ実行すべきではないと考えた方が良いです。
|
|
13
19
|
`GZIPOutputStream` の親クラス `DeflaterOutputStream` の `close` メソッドの Javadoc を見ると次のように書いてありますので、 `gzos` に対して `flush` は実行しなくて良いです。 (`GZIPOutputStream` は `close` メソッドをオーバーライド指定ないので、メソッドの使用を確認するには親クラスを見る必要があります。)
|
|
14
20
|
|
|
15
21
|
> public void close()
|