MyException - 我的异常网
当前位置:我的异常网» J2SE » 关于多线程有关问题。

关于多线程有关问题。

www.MyException.Cn  网友分享于:2015-01-21  浏览:0次
关于多线程问题。。。。。。。。。。。。。。。。。。。。。。。。。。。。。。。。。。。。。。。
之前学java的时候多线程就走马观花,现在回头看看,准备写个多线程读写文件。发现读写都没问题,但是对于可执行文件或者pdf之类的,写好文件后大小都一样,就是不能执行,是不是写的时候那里出问题了。以下是我的源码。麻烦各位前辈指点下。


package sfbc;

import java.io.File;
import java.io.IOException;
import java.io.RandomAccessFile;
import java.util.Arrays;
import java.util.concurrent.BlockingQueue;
import java.util.concurrent.ExecutorService;
import java.util.concurrent.Executors;
import java.util.concurrent.LinkedBlockingQueue;
import java.util.concurrent.locks.Lock;
import java.util.concurrent.locks.ReentrantLock;

public class CopyT {
private BlockingQueue<Object[]> offBlock;
private BlockingQueue<Object[]> taskBlock;
private File frmFile;//源文件
private File toFile;//目标文件
private long frmFileSize;//源文件大小
private long taskSize = 10;//默认线程个数

{
this.offBlock = 
new LinkedBlockingQueue<Object[]>();
this.taskBlock = 
new LinkedBlockingQueue<Object[]>();
}
public CopyT(File frmFile, File toFile) {
this.frmFile = frmFile;
this.toFile = toFile;
this.frmFileSize = frmFile.length();
}

//计算开始读写的位置
public void Offset(){
long c = 0;
for (int i = 0; i < this.taskSize; i++) {
try {
long block = this.frmFileSize / this.taskSize;
long lastBlock = this.frmFileSize % this.taskSize;
if (i == 0 && lastBlock != 0) {
block += lastBlock;
}
c+=block;
long off = i * block;
this.offBlock.put(new Object[]{off,block});
} catch (InterruptedException e) {
e.printStackTrace();
}
}
System.out.println("this.frmFileSize:" + this.frmFileSize + ",c:" + c);
}

public void readT() {
RRunnable RRT = null;
ExecutorService RRTES = Executors.newCachedThreadPool();
for (int i = 0; i < this.taskSize; i++) {
try {
Object[] offBlock = this.offBlock.take();
long off = (Long) offBlock[0];
long block = (Long) offBlock[1];
RRT =  new RRunnable(this.frmFile, this.taskBlock,off, block);
RRTES.execute(RRT);
} catch (InterruptedException e) {
e.printStackTrace();
}
}
RRTES.shutdown();
}

public void writeT() {
Lock lock = new ReentrantLock();
ExecutorService WRTES = Executors.newCachedThreadPool();
WRunnable WRT = null;
for (int i = 0; i < this.taskSize; i++) {
try {
Object[] taskBlock = this.taskBlock.take();
long off = (Long) taskBlock[0];
byte[] b = (byte[]) taskBlock[1];
WRT = new WRunnable(off, b, this.toFile,lock);
WRTES.execute(WRT);
} catch (InterruptedException e) {
e.printStackTrace();
}
}
WRTES.shutdown();
}

public void starTask(){
Offset();
readT();
writeT();
}

//读线程
class RRunnable implements Runnable{
private File frmFile;//源文件
private BlockingQueue<Object[]> taskBlock;
private RandomAccessFile randomAccessFile;
private long off;
private long block;

public RRunnable(File frmFile, BlockingQueue<Object[]> taskBlock,long off,long block) {
this.frmFile = frmFile;
this.taskBlock = taskBlock;
this.off = off;
this.block = block;
}
public void run() {
try {
this.randomAccessFile = new RandomAccessFile(this.frmFile, "r");
this.randomAccessFile.seek(this.off);
byte[] b = new byte[(int) this.block];
//this.randomAccessFile.readFully(b, 0, b.length);
this.randomAccessFile.read(b, 0, b.length);
this.taskBlock.put(new Object[]{this.off,b});
} catch (Exception e) {
e.printStackTrace();
}finally{
try {
this.randomAccessFile.close();
} catch (IOException e) {
e.printStackTrace();
}
}
}
}

//写线程
class WRunnable implements Runnable{
private File toFile;
private RandomAccessFile randomAccessFile;
private long off;
private byte[] b;
private Lock lock;
public WRunnable(long off,byte[] b,File toFile,Lock lock) {
this.off = off;
this.b = b;
this.toFile = toFile;
this.lock = lock;
}
public void run() {
try {
this.lock.lock();
this.randomAccessFile = new RandomAccessFile(this.toFile, "rw");
this.randomAccessFile.seek(this.off);
this.randomAccessFile.write(b, 0, b.length);
} catch (Exception e) {
e.printStackTrace();
}finally{
try {
this.randomAccessFile.close();
this.lock.unlock();
} catch (IOException e) {
e.printStackTrace();
}
}
}
}

/**
 * @return the taskSize
 */
public long getTaskSize() {
return taskSize;
}

/**
 * @param taskSize the taskSize to set
 */
public void setTaskSize(long taskSize) {
this.taskSize = taskSize;
}

public static void main(String[] args) {
File frmFile = new File("C:\\Users\\trfi\\Desktop\\金山清理.exe");
File toFile = new File("C:\\Users\\trfi\\Desktop\\1\\金山清理.exe");
CopyT ct = new CopyT(frmFile, toFile);
ct.setTaskSize(20);
ct.starTask();
}

}


------解决思路----------------------
你确定你原来的 C:\\Users\\trfi\\Desktop\\金山清理.exe 这个能执行吗?

还有这个 金山清理.exe  不会是破解版的一个桌面链接吧?

最后你就这么改下:
File toFile = new File("C:\\Users\\trfi\\Desktop\\金山清理1.exe");
------解决思路----------------------
你是不是流没关闭和flush。
------解决思路----------------------
bug太多了~~
------解决思路----------------------
引用:
Quote: 引用:

bug太多了~~

求指点下。
 只举几个bug, 代码要改进的地方就更多了, 看我给你加的注释
import java.io.File;
import java.io.IOException;
import java.io.RandomAccessFile;
import java.util.Arrays;
import java.util.concurrent.BlockingQueue;
import java.util.concurrent.ExecutorService;
import java.util.concurrent.Executors;
import java.util.concurrent.LinkedBlockingQueue;
import java.util.concurrent.locks.Lock;
import java.util.concurrent.locks.ReentrantLock;
 
public class CopyT {
    private BlockingQueue<Object[]> offBlock;
    private BlockingQueue<Object[]> taskBlock;
    private File frmFile;
    private File toFile;
    private long frmFileSize;
    private long taskSize = 10;
     
    {
        this.offBlock = 
                new LinkedBlockingQueue<Object[]>();
        this.taskBlock = 
                new LinkedBlockingQueue<Object[]>();
    }
    public CopyT(File frmFile, File toFile) {
        this.frmFile = frmFile;
        this.toFile = toFile;
        this.frmFileSize = frmFile.length();
    }
     
    public void Offset(){
        long c = 0;
        for (int i = 0; i < this.taskSize; i++) {
            try {
                long block = this.frmFileSize / this.taskSize;
                long lastBlock = this.frmFileSize % this.taskSize;
                if (i == 0 && lastBlock != 0) {
                    block += lastBlock;
                }
                c+=block;
                long off = i * block;
                this.offBlock.put(new Object[]{off,block});
            } catch (InterruptedException e) {
                e.printStackTrace();
            }
        }
        System.out.println("this.frmFileSize:" + this.frmFileSize + ",c:" + c);
    }
     
    public void readT() {
        RRunnable RRT = null;
        ExecutorService RRTES = Executors.newCachedThreadPool();
        for (int i = 0; i < this.taskSize; i++) {
            try {
                Object[] offBlock = this.offBlock.take();
                long off = (Long) offBlock[0];
                long block = (Long) offBlock[1];
                RRT =  new RRunnable(this.frmFile, this.taskBlock,off, block);
                RRTES.execute(RRT);
            } catch (InterruptedException e) {
                e.printStackTrace();
            }
        }
        RRTES.shutdown();
    }
     
    public void writeT() {
        Lock lock = new ReentrantLock();
        ExecutorService WRTES = Executors.newCachedThreadPool();
        WRunnable WRT = null;
        for (int i = 0; i < this.taskSize; i++) {
            try {
                Object[] taskBlock = this.taskBlock.take();
                long off = (Long) taskBlock[0];
                byte[] b = (byte[]) taskBlock[1];
                WRT = new WRunnable(off, b, this.toFile,lock);
                WRTES.execute(WRT);
            } catch (InterruptedException e) {
                e.printStackTrace();
            }
        }
        WRTES.shutdown();
    }
     
    public void starTask(){
        Offset();
        readT();
        writeT();
    }
     
    class RRunnable implements Runnable{
        private File frmFile;
        private BlockingQueue<Object[]> taskBlock;
        private RandomAccessFile randomAccessFile;
        private long off;
        private long block;
         
        public RRunnable(File frmFile, BlockingQueue<Object[]> taskBlock,long off,long block) {
            this.frmFile = frmFile;
            this.taskBlock = taskBlock;
            this.off = off;
            this.block = block;
        }
        public void run() {
            try {
                this.randomAccessFile = new RandomAccessFile(this.frmFile, "r");
                this.randomAccessFile.seek(this.off);
                byte[] b = new byte[(int) this.block];//bug1 把long转成int好吗?
                this.randomAccessFile.read(b, 0, b.length);//bug2 要是文件很大, 一次性全部读到内存好吗? bug3 api中说这个read方法是保证读取b.length个字节吗?
                this.taskBlock.put(new Object[]{this.off,b});
            } catch (Exception e) {
                e.printStackTrace();
            }finally{
                try {
                    this.randomAccessFile.close();//bug4 不做非空判断, 要是new RandomAccessFile失败, 这里不会导致空指针吗?
                } catch (IOException e) {
                    e.printStackTrace();
                }
            }
        }
    }
     
    class WRunnable implements Runnable{
        private File toFile;
        private RandomAccessFile randomAccessFile;
        private long off;
        private byte[] b;
        private Lock lock;
        public WRunnable(long off,byte[] b,File toFile,Lock lock) {
            this.off = off;
            this.b = b;
            this.toFile = toFile;
            this.lock = lock;
        }
        public void run() {
            try {
                this.lock.lock();
                this.randomAccessFile = new RandomAccessFile(this.toFile, "rw");
                this.randomAccessFile.seek(this.off);
                this.randomAccessFile.write(b, 0, b.length);
            } catch (Exception e) {
                e.printStackTrace();
            }finally{
                try {
                    this.randomAccessFile.close();
                    this.lock.unlock();//bug5 且不说这个锁的意义何在, 上面的close要是抛错, 下面这个锁还能得到释放吗? 
                } catch (IOException e) {
                    e.printStackTrace();
                }
            }
        }
    }
 
    /**
     * @return the taskSize
     */
    public long getTaskSize() {
        return taskSize;
    }
 
    /**
     * @param taskSize the taskSize to set
     */
    public void setTaskSize(long taskSize) {
        this.taskSize = taskSize;
    }
 
    public static void main(String[] args) {
        File frmFile = new File("C:\\Users\\trfi\\Desktop\\金山清理.exe");
        File toFile = new File("C:\\Users\\trfi\\Desktop\\1\\金山清理.exe");
        CopyT ct = new CopyT(frmFile, toFile);
        ct.setTaskSize(20);
        ct.starTask();
    }
 
}

------解决思路----------------------
引用:
Quote: 引用:

Quote: 引用:

Quote: 引用:

bug太多了~~

求指点下。
 只举几个bug, 代码要改进的地方就更多了, 看我给你加的注释
import java.io.File;
import java.io.IOException;
import java.io.RandomAccessFile;
import java.util.Arrays;
import java.util.concurrent.BlockingQueue;
import java.util.concurrent.ExecutorService;
import java.util.concurrent.Executors;
import java.util.concurrent.LinkedBlockingQueue;
import java.util.concurrent.locks.Lock;
import java.util.concurrent.locks.ReentrantLock;
 
public class CopyT {
    private BlockingQueue<Object[]> offBlock;
    private BlockingQueue<Object[]> taskBlock;
    private File frmFile;
    private File toFile;
    private long frmFileSize;
    private long taskSize = 10;
     
    {
        this.offBlock = 
                new LinkedBlockingQueue<Object[]>();
        this.taskBlock = 
                new LinkedBlockingQueue<Object[]>();
    }
    public CopyT(File frmFile, File toFile) {
        this.frmFile = frmFile;
        this.toFile = toFile;
        this.frmFileSize = frmFile.length();
    }
     
    public void Offset(){
        long c = 0;
        for (int i = 0; i < this.taskSize; i++) {
            try {
                long block = this.frmFileSize / this.taskSize;
                long lastBlock = this.frmFileSize % this.taskSize;
                if (i == 0 && lastBlock != 0) {
                    block += lastBlock;
                }
                c+=block;
                long off = i * block;
                this.offBlock.put(new Object[]{off,block});
            } catch (InterruptedException e) {
                e.printStackTrace();
            }
        }
        System.out.println("this.frmFileSize:" + this.frmFileSize + ",c:" + c);
    }
     
    public void readT() {
        RRunnable RRT = null;
        ExecutorService RRTES = Executors.newCachedThreadPool();
        for (int i = 0; i < this.taskSize; i++) {
            try {
                Object[] offBlock = this.offBlock.take();
                long off = (Long) offBlock[0];
                long block = (Long) offBlock[1];
                RRT =  new RRunnable(this.frmFile, this.taskBlock,off, block);
                RRTES.execute(RRT);
            } catch (InterruptedException e) {
                e.printStackTrace();
            }
        }
        RRTES.shutdown();
    }
     
    public void writeT() {
        Lock lock = new ReentrantLock();
        ExecutorService WRTES = Executors.newCachedThreadPool();
        WRunnable WRT = null;
        for (int i = 0; i < this.taskSize; i++) {
            try {
                Object[] taskBlock = this.taskBlock.take();
                long off = (Long) taskBlock[0];
                byte[] b = (byte[]) taskBlock[1];
                WRT = new WRunnable(off, b, this.toFile,lock);
                WRTES.execute(WRT);
            } catch (InterruptedException e) {
                e.printStackTrace();
            }
        }
        WRTES.shutdown();
    }
     
    public void starTask(){
        Offset();
        readT();
        writeT();
    }
     
    class RRunnable implements Runnable{
        private File frmFile;
        private BlockingQueue<Object[]> taskBlock;
        private RandomAccessFile randomAccessFile;
        private long off;
        private long block;
         
        public RRunnable(File frmFile, BlockingQueue<Object[]> taskBlock,long off,long block) {
            this.frmFile = frmFile;
            this.taskBlock = taskBlock;
            this.off = off;
            this.block = block;
        }
        public void run() {
            try {
                this.randomAccessFile = new RandomAccessFile(this.frmFile, "r");
                this.randomAccessFile.seek(this.off);
                byte[] b = new byte[(int) this.block];//bug1 把long转成int好吗?
                this.randomAccessFile.read(b, 0, b.length);//bug2 要是文件很大, 一次性全部读到内存好吗? bug3 api中说这个read方法是保证读取b.length个字节吗?
                this.taskBlock.put(new Object[]{this.off,b});
            } catch (Exception e) {
                e.printStackTrace();
            }finally{
                try {
                    this.randomAccessFile.close();//bug4 不做非空判断, 要是new RandomAccessFile失败, 这里不会导致空指针吗?
                } catch (IOException e) {
                    e.printStackTrace();
                }
            }
        }
    }
     
    class WRunnable implements Runnable{
        private File toFile;
        private RandomAccessFile randomAccessFile;
        private long off;
        private byte[] b;
        private Lock lock;
        public WRunnable(long off,byte[] b,File toFile,Lock lock) {
            this.off = off;
            this.b = b;
            this.toFile = toFile;
            this.lock = lock;
        }
        public void run() {
            try {
                this.lock.lock();
                this.randomAccessFile = new RandomAccessFile(this.toFile, "rw");
                this.randomAccessFile.seek(this.off);
                this.randomAccessFile.write(b, 0, b.length);
            } catch (Exception e) {
                e.printStackTrace();
            }finally{
                try {
                    this.randomAccessFile.close();
                    this.lock.unlock();//bug5 且不说这个锁的意义何在, 上面的close要是抛错, 下面这个锁还能得到释放吗? 
                } catch (IOException e) {
                    e.printStackTrace();
                }
            }
        }
    }
 
    /**
     * @return the taskSize
     */
    public long getTaskSize() {
        return taskSize;
    }
 
    /**
     * @param taskSize the taskSize to set
     */
    public void setTaskSize(long taskSize) {
        this.taskSize = taskSize;
    }
 
    public static void main(String[] args) {
        File frmFile = new File("C:\\Users\\trfi\\Desktop\\金山清理.exe");
        File toFile = new File("C:\\Users\\trfi\\Desktop\\1\\金山清理.exe");
        CopyT ct = new CopyT(frmFile, toFile);
        ct.setTaskSize(20);
        ct.starTask();
    }
 
}

谢谢你的指点,我会注意的,现在我最大的Bug就是比如可执行之类的文件读写不成功,这个麻烦帮忙看看。
bug1和bug3结合起来就会导致你现在的bug, 会漏掉字节

文章评论

程序员周末都喜欢做什么?
程序员周末都喜欢做什么?
5款最佳正则表达式编辑调试器
5款最佳正则表达式编辑调试器
程序员的鄙视链
程序员的鄙视链
程序员应该关注的一些事儿
程序员应该关注的一些事儿
“懒”出效率是程序员的美德
“懒”出效率是程序员的美德
聊聊HTTPS和SSL/TLS协议
聊聊HTTPS和SSL/TLS协议
初级 vs 高级开发者 哪个性价比更高?
初级 vs 高级开发者 哪个性价比更高?
60个开发者不容错过的免费资源库
60个开发者不容错过的免费资源库
编程语言是女人
编程语言是女人
团队中“技术大拿”并非越多越好
团队中“技术大拿”并非越多越好
如何区分一个程序员是“老手“还是“新手“?
如何区分一个程序员是“老手“还是“新手“?
Java程序员必看电影
Java程序员必看电影
旅行,写作,编程
旅行,写作,编程
我是如何打败拖延症的
我是如何打败拖延症的
代码女神横空出世
代码女神横空出世
一个程序员的时间管理
一个程序员的时间管理
每天工作4小时的程序员
每天工作4小时的程序员
Java 与 .NET 的平台发展之争
Java 与 .NET 的平台发展之争
漫画:程序员的工作
漫画:程序员的工作
2013年中国软件开发者薪资调查报告
2013年中国软件开发者薪资调查报告
Web开发者需具备的8个好习惯
Web开发者需具备的8个好习惯
程序员必看的十大电影
程序员必看的十大电影
程序员都该阅读的书
程序员都该阅读的书
“肮脏的”IT工作排行榜
“肮脏的”IT工作排行榜
老程序员的下场
老程序员的下场
总结2014中国互联网十大段子
总结2014中国互联网十大段子
程序员和编码员之间的区别
程序员和编码员之间的区别
亲爱的项目经理,我恨你
亲爱的项目经理,我恨你
10个帮程序员减压放松的网站
10个帮程序员减压放松的网站
看13位CEO、创始人和高管如何提高工作效率
看13位CEO、创始人和高管如何提高工作效率
 程序员的样子
程序员的样子
程序猿的崛起——Growth Hacker
程序猿的崛起——Growth Hacker
不懂技术不要对懂技术的人说这很容易实现
不懂技术不要对懂技术的人说这很容易实现
Web开发人员为什么越来越懒了?
Web开发人员为什么越来越懒了?
为什么程序员都是夜猫子
为什么程序员都是夜猫子
鲜为人知的编程真相
鲜为人知的编程真相
十大编程算法助程序员走上高手之路
十大编程算法助程序员走上高手之路
我的丈夫是个程序员
我的丈夫是个程序员
10个调试和排错的小建议
10个调试和排错的小建议
程序员眼里IE浏览器是什么样的
程序员眼里IE浏览器是什么样的
那些争议最大的编程观点
那些争议最大的编程观点
当下全球最炙手可热的八位少年创业者
当下全球最炙手可热的八位少年创业者
程序员的一天:一寸光阴一寸金
程序员的一天:一寸光阴一寸金
做程序猿的老婆应该注意的一些事情
做程序猿的老婆应该注意的一些事情
程序员最害怕的5件事 你中招了吗?
程序员最害怕的5件事 你中招了吗?
中美印日四国程序员比较
中美印日四国程序员比较
Google伦敦新总部 犹如星级庄园
Google伦敦新总部 犹如星级庄园
要嫁就嫁程序猿—钱多话少死的早
要嫁就嫁程序猿—钱多话少死的早
写给自己也写给你 自己到底该何去何从
写给自己也写给你 自己到底该何去何从
软件开发程序错误异常ExceptionCopyright © 2009-2015 MyException 版权所有