黑马程序员技术交流社区

标题: 我的字节流拷贝图片不成功, [打印本页]

作者: serioustd 时间: 2014-4-26 21:17
标题: 我的字节流拷贝图片不成功,
package com.a.io;
import java.io.*;

public class CopyPic {
public static void main(String[] args) {
FileOutputStream fos= null;
FileInputStream fis =null;
try
{
fos=new FileOutputStream("C:\\Users\\Administrator\\Desktop\\eq\\新建文件夹\\6.bmp");
fis=new FileInputStream("C:\\Users\\Administrator\\Desktop\\eq\\新建文件夹\\o.bmp");
int num=fis.available();
byte[] buf=new byte[num];
int len=0;
while((len=fis.read())!=-1)
{
fos.write(buf,0,len);
}
}
catch(IOException e)
{
System.out.println("复制文件失败");
}
finally
{
try
{
if(fis!=null)
fis.close();
}catch(IOException e)
{
System.out.println("关闭读取字节流失败");
}
try
{
if(fos!=null)
fos.close();
}catch(IOException e)
{
System.out.println("关闭写入字节流失败");
}
}
}
}
图片拷贝是拷贝了,但是图片打不开,图片可能损坏.损毁或过大,拷贝的图片内存比实际的内存要大出好多

作者: SyouRai_Tsk 时间: 2014-4-26 21:45
应该是未刷新字节流

作者: serioustd 时间: 2014-4-26 21:47

SyouRai_Tsk 发表于 2014-4-26 21:45
应该是未刷新字节流

字节流是不需要刷新的

作者: 今生无憾 时间: 2014-4-26 21:52
本帖最后由今生无憾于 2014-4-26 21:58 编辑

int num=fis.available();
                     byte[] buf=new byte[num];
                     int len=0;
                     while((len=fis.read())!=-1)
                     {
                              fos.write(buf,0,len);
                     }
问题出现在这一段，首先你用的是available()方法，这是把图片剩下的字节全部读出来，你用byte数组存储，在下面的循环里，循环了很多次，也就是写了很多次num大小的byte数组。自然就会出现复制后的图片大很多。而且无法显示了。
建议你去回顾下自学视频里的IO流，你拷贝图片的方式弄错了。你创建一个byte数组，给定一个大小。512字节。1024字节等等随你。然后每次循环读取byte数组大小的字节，就写入进去。直到读完。就OK了。

作者: serioustd 时间: 2014-4-26 21:58

今生无憾发表于 2014-4-26 21:52
int num=fis.available();
byte[] buf=new byte[num];
...

有点含糊,我没懂

作者: 今生无憾 时间: 2014-4-26 22:02

serioustd 发表于 2014-4-26 21:58
有点含糊,我没懂

                  byte[] buf=new byte[512];
                     int len=0;
                     while((len=fis.read(buf))!=-1)
                     {
                              fos.write(buf);
                     }

作者: serioustd 时间: 2014-4-26 22:05

今生无憾发表于 2014-4-26 22:02
byte[] buf=new byte[512];
int len=0;
...

哦,我找到问题拉,是我buf数组,没有放到while循环里进行判断所导致,3q

作者: 程序爱好者 时间: 2014-4-26 22:12

try
{
fos=new FileOutputStream("C:\\Users\\Administrator\\Desktop\\eq\\新建文件夹\\6.bmp");
fis=new FileInputStream("C:\\Users\\Administrator\\Desktop\\eq\\新建文件夹\\o.bmp");
int num=fis.available();
byte[] buf=new byte[num];
int len=0;
while((len=fis.read()!=-1) //同学你这个read()方法里面漏传个数组了fis.read([color=Red]buf[/color]))

复制代码

作者: 程序爱好者 时间: 2014-4-26 22:14
while((len=fis.read(buf)!=-1)

作者: SyouRai_Tsk 时间: 2014-4-26 22:21

serioustd 发表于 2014-4-26 21:47
字节流是不需要刷新的

未看清楚.具体问题出在那个available()方法

作者: serioustd 时间: 2014-4-26 22:24

SyouRai_Tsk 发表于 2014-4-26 22:21
未看清楚.具体问题出在那个available()方法

跟avaibale()方法没有关系呀,只不过是num数组没有传递,我现在用了avaibale()方法也能成功

作者: 王春蕾 时间: 2014-4-26 22:31
代码出现的问题在这里：

int num=fis.available();
byte[] buf=new byte[num];
int len=0;
while((len=fis.read())!=-1)
{
fos.write(buf,0,len);
}

复制代码

你调用的是read方法，该方法是一次读取一个字节并写入到输出流，你这个len定义的也就有问题了，既然是一次读取一个字节，那么fis.read()应该是一个int类型的数值，而不是长度，可以直接写：fos.write(fis.read);或者定义一个有意义的int变量接收返回值。
这种方式复制图片的效率很低，你可以通过定义缓冲区的方式，这样可以减少while循环执行的次数，提高程序的性能，减少不必要的内存消耗。
希望会对楼主有用吧~

作者: SyouRai_Tsk 时间: 2014-4-26 22:37

serioustd 发表于 2014-4-26 22:24
跟avaibale()方法没有关系呀,只不过是num数组没有传递,我现在用了avaibale()方法也能成功 ...

刚发现，有点感觉自己一下子抽了

作者: ò壞尛孩 时间: 2014-4-26 22:56
搜噶！搜噶！搜噶！搜噶！搜噶！

作者: 小周务商 时间: 2014-4-26 23:15
大哥。能不能加个注释啊。看得好累哦。

作者: eternallove 时间: 2014-4-27 00:07

while ((len=fis.read(buf)!=-1)
{
fos.write(buf,0,len);
}
记得把数组传进去。。。

作者: 你为谁归来 时间: 2014-4-27 00:08
while((len=fis.read())!=-1) 你读的是那个数组没有写进去啊！

作者: ς高眼光の目标 时间: 2014-4-27 03:13
while((len=fis.read())!=-1)//这里写错了，你没把数组传进去
                     {
                              fos.write(buf,0,len);
                     }

欢迎光临黑马程序员技术交流社区 (http://bbs.itheima.com/)

黑马程序员IT技术论坛 X3.2