西山和広です。

2点気になったのですが、

At Sun, 9 Feb 2014 16:07:34 +0900,
terasawan wrote:
> 
>     @tmpdir = Dir.mktmpdir

これは自分で削除するよりもブロック付きで呼び出した方が
良いのではないでしょうか。

http://docs.ruby-lang.org/ja/2.1.0/method/Dir/s/mktmpdir.html

>         system("zip -r download.zip download")

これは固定の文字列だから問題は起きなさそうですが、普通は
 system("zip", "-r", "download.zip", "download")
のように配列で渡した方がシェル経由で余計な問題が発生する可能性が
減らせて良いのではないでしょうか。

http://docs.ruby-lang.org/ja/2.1.0/method/Kernel/m/system.html


-- 
|ZnZ(ゼット エヌ ゼット)
|西山和広(Kazuhiro NISHIYAMA)

--
ML: rails@r...
使い方: http://QuickML.com/