Skip to main content
1 de 6

hay muchos problemas con tu codigo. empezemos por el lado del Servidor:

...
ServerSocket servidor=new ServerSocket(4500);
Socket cliente=servidor.accept();
FileInputStream file=new FileInputStream("C:\\archivoservidor.txt");
byte b[]=new byte[20002]; 

Aqui vemos un problema byte b[]=new byte[20002]; este "buffer" de bytes es Exesivo o al contrario es insuficiente. no es recomendable hacer un "buffer" de memoria sin conocimiento de cuanto realmente se va a necesitar, en este caso se recomienda y se utilizara Files.readAllBytes(java.nio.file.Path)

file.read(b,0,b.length);
    OutputStream out=cliente.getOutputStream();
} catch (IOException ex) {
    Logger.getLogger(Socket6.class.getName()).log(Level.SEVERE, null, ex);
}
}
}

y eso es todo! donde Envia los datos al out? por tanto aqui el problema es: no se esta transmitiendo NADA debemos enviar los bytes que estan en b hacia out mediante out.write() otro problema GRAVE es que no se estan cerrando los Streams! no cierras el Stream de lectura file ni el socket (ya sea el de servidor o el Cliente!)

por tanto un codigo Mejorado Y correcto seria:

import java.io.IOException;
import java.io.OutputStream;
import java.net.ServerSocket;
import java.net.Socket;
import java.nio.file.Files;
import java.nio.file.Paths;
import java.util.logging.Level;
import java.util.logging.Logger;

public class Socket6 {

public static void main(String[] args) {
//try con recursos (try con autoclose) ver link
    try (ServerSocket servidor = new ServerSocket(4500)) {
        try (Socket SocketCliente = servidor.accept()) {
            byte filebytes[] = null;
            try {
                //se lee TODOS los bytes del archivo a un Byte Array. 
                filebytes = Files.readAllBytes(Paths.get("archivoservidor.txt"));
            } catch (IOException fileex) {
                Logger.getLogger(Socket6.class.getName()).log(Level.SEVERE, "Error en la Lectura del archivo Datos", fileex);
            }
            //si filebytes es nulo significa que la lectura fallo o el archivo esta vacio! 
            if (filebytes != null) {
                //el paso anterior puede tardar mucho necesitamos asegurar que el socket (cliente) aun este disponible(connectado)
                if (!SocketCliente.isClosed() && SocketCliente.isConnected()) {
                    try {
                        OutputStream outstream = SocketCliente.getOutputStream();
                        //tomar los bytes y escribirlos al stream (NOTA esto no los ENVIA! para enviarlos devemos usar Flush!
                        outstream.write(filebytes);
                        //flush (enviar los datos al cliente)
                        outstream.flush();
                    } catch (IOException outex) {
                        Logger.getLogger(Socket6.class.getName()).log(Level.SEVERE, "Error en la Transmisión de Datos", outex);
                    }
                } else {
                    Logger.getLogger(Socket6.class.getName()).log(Level.SEVERE, "El Socket esta Cerrado o desconectado");
                }
            }
            System.out.println("done");
        //nota si no usamos try with resource no necesitamos hacer close() 
        //PERO! si usamos un "try normal" debemos hacer SocketCliente.close() en el Finally Statement... de **este** try
        //SocketCliente.close();
        }//SocketCliente close implisito
    } catch (IOException ex) {
        Logger.getLogger(Socket6.class.getName()).log(Level.SEVERE, "Exception con el Socket?", ex);
    }// servidor close implisito
    }

}

try con recursos (try con autoclose)

El Cliente posee errores en Similares, no cierras los Streams! y no haces Flush para mandar los datos al Archivo! en cada loop.

Ademas de esto, las lineas: in.read(b,0,b.length); y file.write(b,0,b.length); solo sucede 1 ves(no esta dentro de un loop). Osea si 20002 bytes no esta todos los datos del archivo solo guardas esos 20002 bytes y el resto del archivo? queda por fuera o viseversa. si el archivo es de 496 bytes (suponiendo es solo texto en ansii son 62 letras)? el archivo va a tener datos invalidos (el resto de los bytes en 0 \0) por tanto el resultado puede ser incositente con lo esperado. para evitar eso debes usar loops. ver: guia sobre loops

y por tanto para que el cliente realize la funcion de forma correcta

import java.io.IOException;
import java.io.InputStream;
import java.io.OutputStream;
import java.net.Socket;
import java.nio.file.Files;
import java.nio.file.Paths;
import java.util.logging.Level;
import java.util.logging.Logger;

public class Cliente {

    public static void main(String[] args) {
        byte readbytes[] = new byte[1000];//20002-> this is unesesary and a waste of memory un HEAP! 
        //try with resources socket & inputsstream 
        try (Socket cliente = new Socket("localhost", 4500); InputStream in = cliente.getInputStream()) {
            //try with resourse for the file Write operation
            try (OutputStream FileToFill = Files.newOutputStream(Paths.get("archivocliente.txt"))) {
                //loop para lectura del archivo usamos un for para poder
                //declarar una variable & hacer la verificacion podriamos 
                //usar un While pero necesitariamos 1 variable extra.
                for (int read=-1;(read = in.read(readbytes)) >= 0;) {
                    //se llena los datos a guardar con los Bytes del socket
                    FileToFill.write(readbytes, 0, read);
                    //se imprimimen los bytes leidos (CUIDADO) SE asume que son el mismo formato que la PC <CHARSET>
                    System.out.print(new String(readbytes, 0, read));
                    //se graban los bytes al archivo!
                    FileToFill.flush();
                }
            }//implisit file close. 
        } catch (IOException ex) {
            Logger.getLogger(Cliente.class.getName()).log(Level.SEVERE, null, ex);
        }//implisit Socket close & input close

    }
}