com.justep.doc.AbstractDoc download 方法有缺陷

linxin@msn.com

在 com.justep.doc.AbstractDoc 的 download  方法中。
        public void download(OutputStream outputStream, String versionID, String partType) throws Exception {
                InputStream inputStream = this.download(versionID, partType);
                int bytesRead;
                byte[] buf = new byte[4 * 1024]; //4K buffer
                while ((bytesRead = inputStream.read(buf)) != -1) {
                        outputStream.write(buf, 0, bytesRead);
                }
                outputStream.flush();
                outputStream.close();
        }



没有 关闭 InputStream ， 反而关闭了 outputStream , 如果 应用方需要反复使用 OutputStream 就变得不可行了。  inputStream 是由API 内部构建， 应该由内部提供关闭。 OutputStream 由API 外部提供 应该由外部负责关闭， 责任不能倒置！。

linxin@msn.com

继续 上面的AbstractDoc 类里面的缺陷代码：在里面有一个内部类：
        private class HttpInputStream extends InputStream{
                private InputStream input;
                private HttpMethod method;
                public HttpInputStream(InputStream input,HttpMethod method) {
                        this.input = input;
                        this.method = method;
                }
                @Override
                public int read() throws IOException {
                        return this.input.read();
                }
                @Override
                public void close() throws IOException {
                        try{
                                this.input.close();
                        }finally{
                                if(this.method!=null){
                                        this.method.releaseConnection();
                                }
                        }
                }
               
        }
在这个实现里面 因为没有重载 public int read(byte b[], int off, int len) throws IOException 会导致输出流 1个字节一个字节的读取。效率很差！！！！

比较好的修订应该改为继承 FilterInputStream：
        private class HttpInputStream extends FilterInputStream{
                private HttpMethod method;
                public HttpInputStream(InputStream input,HttpMethod method) {
                        super(input);
                        this.method = method;
                }
               
                @Override
                public void close() throws IOException {
                        try{
                                this.input.close();
                        }finally{
                                if(this.method!=null){
                                        this.method.releaseConnection();
                                }
                        }
                }
               
        }
       


       
       

jishuang

你好楼主，您反馈的这个现象将在后续版本完善，感谢楼主的支持与配合。