elhacker.net cabecera Bienvenido(a), Visitante. Por favor Ingresar o Registrarse
¿Perdiste tu email de activación?.

 

 


Tema destacado: Recopilación Tutoriales y Manuales Hacking, Seguridad, Privacidad, Hardware, etc


+  Foro de elhacker.net
|-+  Programación
| |-+  Programación General
| | |-+  Java
| | | |-+  Unir contenido de dos txt en un txt nuevo
0 Usuarios y 1 Visitante están viendo este tema.
Páginas: [1] Ir Abajo Respuesta Imprimir
Autor Tema: Unir contenido de dos txt en un txt nuevo  (Leído 3,630 veces)
thebus4k

Desconectado Desconectado

Mensajes: 39


Ver Perfil
Unir contenido de dos txt en un txt nuevo
« en: 9 Mayo 2020, 18:03 pm »

Hola a todos.
Tengo un problema en un programa que debe unir el contenido de dos archivos txt en un archivo nuevo. El programa que tengo es que me crea bien el archivo, pero no copia el contenido de los dos archivos iniciales.

Código:

import java.io.*;
import javax.swing.JOptionPane;
public class Ejercicio3{
 
    public static void main(String[] args) {
 
        //Introducimos los datos
        String rutaFichero1=JOptionPane.showInputDialog("Indica la ruta del primer fichero");
        String rutaFichero2=JOptionPane.showInputDialog("Indica la ruta del segundo fichero");
        String rutaDestino=JOptionPane.showInputDialog("Indica la ruta donde quieres guardarlo");
 
        //Creamos dos objetos File para que nos sea mas sencillo manejarlos
        File fichero1=new File(rutaFichero1);
        File fichero2=new File(rutaFichero2);
 
        //Troceamos el el nombre del primer fichero para que se quede sin extension
        String primerFichero=fichero1.getName().substring(0, fichero1.getName().length()-4);
 
        //Crear el nombre de salida del fichero
        String nombreFicheroFinal=primerFichero+"_"+fichero2.getName();
 
        rutaDestino+=nombreFicheroFinal;
 
        File destino=new File(rutaDestino);
 
        UneFicheros(fichero1, fichero2, destino);
 
    }
 
    public static void UneFicheros (File fich1, File fich2, File destino){
 
        try (BufferedReader br=new BufferedReader(new FileReader(fich1));
            BufferedReader br2=new BufferedReader(new FileReader(fich2))){
 
            int eleccion=-1;
 
            if(destino.exists()){
                eleccion=JOptionPane.showConfirmDialog(null, "El fichero ya existe, ¿Quieres sobrescribir el fichero "+destino.getName()+"?",
                        "Sobrescribir",
                        JOptionPane.OK_CANCEL_OPTION, JOptionPane.INFORMATION_MESSAGE);
            }
            if(eleccion!=JOptionPane.CANCEL_OPTION){
 
                /*
                 * Lo creamos aquí, ya que si lo hacemos arriba
                 * siempre existira porque se crea al abrir el Stream
                 */
 
                BufferedWriter bw=new BufferedWriter(new FileWriter(destino));
 
                //Copiamos el contenido al fichero destino
                copiar(bw, br);
                copiar(bw, br2);
 
            }
        }catch(IOException e){
            JOptionPane.showMessageDialog(null, e.getMessage(), "Error", JOptionPane.ERROR_MESSAGE);
        }
 
    }
 
    private static void copiar(BufferedWriter bw, BufferedReader br) throws IOException{
 
            String linea=br.readLine();
            while(linea!=null){
 
                bw.write(linea);
 
                linea=br.readLine();
            }
 
    }
}



Espero que alguien sepa ayudarme, un saludo.


En línea

EdePC
Moderador Global
***
Desconectado Desconectado

Mensajes: 2.165



Ver Perfil
Re: Unir contenido de dos txt en un txt nuevo
« Respuesta #1 en: 9 Mayo 2020, 21:08 pm »

Saludos,

- Te falta cerrar tu Buffer:

Cita de: Del libro adjunto, pág.: 159
Es importante ejecutar close() después de realizar la escritura; de esta manera nos
aseguramos que se graba toda la información en el disco.

Libro Aprende Java con más de 200 ejercicios (Luis José Sánchez González, 2016)
https://www.academia.edu/36088785/Aprende_java_con_ejercicios



En línea

rub'n


Desconectado Desconectado

Mensajes: 1.217


(e -> λ("live now")); tatuar -> λ("α");


Ver Perfil WWW
Re: Unir contenido de dos txt en un txt nuevo
« Respuesta #2 en: 10 Mayo 2020, 04:23 am »

* Ligero cambio, y es verdad cierra los recursos asi como haces con BufferedReader dentro del try-with-resources
* Crea el fichero destino si no existe
* Hardcodear las rutas asi es un fastidio, usa mejor JFileChooser


Código
  1. public class Ejercicio3 {
  2.  
  3.    public static void main(String[] args) {
  4.  
  5.        //Harcodear la ruta no esta bien, evitar en lo posible a menos que sea una ruta fija siempre
  6.  
  7.        //Introducimos los datos
  8.        String rutaFichero1 = JOptionPane.showInputDialog("Indica la ruta del primer fichero");
  9.        String rutaFichero2 = JOptionPane.showInputDialog("Indica la ruta del segundo fichero");
  10.        String rutaDestino = JOptionPane.showInputDialog("Indica la ruta donde quieres guardarlo");
  11.  
  12.        //Creamos dos objetos File para que nos sea mas sencillo manejarlos
  13.        File fichero1 = new File(rutaFichero1);
  14.        File fichero2 = new File(rutaFichero2);
  15.  
  16.        //Troceamos el el nombre del primer fichero para que se quede sin extension
  17.        String primerFichero = fichero1.getName().substring(0, fichero1.getName().length() - 4);
  18.  
  19.        //Crear el nombre de salida del fichero
  20.        String nombreFicheroFinal = primerFichero + "_" + fichero2.getName();
  21.  
  22.        //invocando a .replaceAll() quitando null en caso que no coloquen nombre de Archivos
  23.        rutaDestino += nombreFicheroFinal.replace("null", "");
  24.  
  25.        //ruta escrita por el usuario
  26.        File destino = new File(rutaDestino.replaceAll("null", ""));
  27.        uneFicheros(fichero1, fichero2, destino);
  28.  
  29.    }
  30.  
  31.    public static void uneFicheros(File fich1, File fich2, File destino) {
  32.  
  33.        try (BufferedReader br = new BufferedReader(new FileReader(fich1));
  34.             BufferedReader br2 = new BufferedReader(new FileReader(fich2))) {
  35.  
  36.            int eleccion = -1;
  37.  
  38.            if (destino.exists()) {
  39.                eleccion = JOptionPane.showConfirmDialog(null, "El fichero ya existe, ¿Quieres sobrescribir el fichero " + destino.getName() + "?",
  40.                        "Sobrescribir",
  41.                        JOptionPane.OK_CANCEL_OPTION, JOptionPane.INFORMATION_MESSAGE);
  42.            }
  43.            if (eleccion != JOptionPane.CANCEL_OPTION) {
  44.  
  45.                /**
  46.                  * creando el fichero destino si no existe
  47.                  */
  48.                Files.createFile(destino.toPath());
  49.  
  50.  
  51.                /*
  52.                  * Lo creamos aquí, ya que si lo hacemos arriba
  53.                  * siempre existira porque se crea al abrir el Stream
  54.                  */
  55.                copiar(br, br2, destino.toPath());
  56.            }
  57.        } catch (IOException e) {
  58.            JOptionPane.showMessageDialog(null, e.getMessage(), "Error", JOptionPane.ERROR_MESSAGE);
  59.        }
  60.  
  61.    }
  62.  
  63.    private static void copiar(final BufferedReader br, final BufferedReader br2, final Path destino) {
  64.  
  65.        //Recorremos el Stream para hacer una reduccion en String con el separador del sistema
  66.        String lines = br.lines().collect(Collectors.joining(System.lineSeparator()));
  67.        String lines2 = br2.lines().collect(Collectors.joining(System.lineSeparator()));
  68.  
  69.        try (final BufferedWriter bw = Files.newBufferedWriter(destino)) {
  70.  
  71.            //escribimos la concatenacion de ambos String
  72.            bw.write(lines.concat(lines2));
  73.  
  74.        } catch (IOException ex) {
  75.            ex.printStackTrace();
  76.        }
  77.  
  78.    }
  79. }
  80.  
« Última modificación: 10 Mayo 2020, 04:32 am por rub'n » En línea

rubn0x52.com KNOWLEDGE  SHOULD BE FREE.
If you don't have time to read, you don't have the time (or the tools) to write, Simple as that. Stephen king
Páginas: [1] Ir Arriba Respuesta Imprimir 

Ir a:  

WAP2 - Aviso Legal - Powered by SMF 1.1.21 | SMF © 2006-2008, Simple Machines