Это правильный способ резервного копирования файла с потоками и исключениями?

Question

Итак, я написал простой файл резервной копии в моем java-коде, но когда я тестирую метод в своем тестовом классе и снова проверяю свою папку, я не вижу созданного файла копирования или резервной копии, хотя я получаю сообщение об успешности. Это даже правильно или я что-то упускаю?

import java.io.*; 
import java.util.ArrayList; 
import javax.swing.*; 

public class BasicFile { 

File file1; 
JFileChooser selection; 
File file2 = new File(".", "Backup File"); 

public BasicFile() { 
    selection = new JFileChooser("."); 
} 

public void selectFile() { 
    int status = selection.showOpenDialog(null); 

    try { 
     if (status != JFileChooser.APPROVE_OPTION) { 
      throw new IOException(); 
     } 
     file1 = selection.getSelectedFile(); 

     if (!file1.exists()) { 
      throw new FileNotFoundException(); 
     } 
    } catch (FileNotFoundException e) { 
     JOptionPane.showMessageDialog(null, "File Not Found ", "Error", JOptionPane.INFORMATION_MESSAGE); 
    } catch (IOException e) { 
     System.exit(0); 
    } 
} 

public void backupFile() throws FileNotFoundException { 
    DataInputStream in = null; 
    DataOutputStream out = null; 
    try { 
     in = new DataInputStream(new FileInputStream(file1)); 
     out = new DataOutputStream(new FileOutputStream(file2)); 

     try { 
      while (true) { 
       byte data = in.readByte(); 
       out.writeByte(data); 
      } 
     } catch (EOFException e) { 
      JOptionPane.showMessageDialog(null, "File has been backed up!", 
        "Backup Complete!", JOptionPane.INFORMATION_MESSAGE); 
     } catch (IOException e) { 
      JOptionPane.showMessageDialog(null, "File Not Found ", 
        "Error", JOptionPane.INFORMATION_MESSAGE); 
     } 
    } finally { 
     try { 
      in.close(); 
      out.close(); 
     } catch (Exception e) { 
      display(e.toString(), "Error"); 
     } 
    } 

} 

boolean exists() { 
    return file1.exists(); 
} 

public String toString() { 
    return file1.getName() + "\n" + file1.getAbsolutePath() + "\n" + file1.length() + " bytes"; 
} 

Answer 1

Это правильно, но ужасно неэффективно. Вам также не нужны DataInputStream или DataOutputStream. Канонический способ скопировать поток в Java является:

int count; 
byte[] buffer = new byte[8192]; // or more if you like 
while ((count = in.read(buffer)) > 0) 
{ 
    out.write(buffer, 0, count); 
} 

Этот код не бросает EOFException поэтому вам нужно настроить свой код соответственно.