Приходит битая картинка с сервера. Как это исправить?

Question

[SuperDuperPuper]

import javax.imageio.ImageIO;
import java.awt.image.BufferedImage;
import java.io.*;
import java.net.ServerSocket;
import java.net.Socket;
import java.util.Date;
import java.util.List;

public class Server extends  Thread{
    Socket socket;
    public static void main (String [] args) {
        try {
            ServerSocket serverSocket = new ServerSocket(8000);
            while (true) {
                new Server(serverSocket.accept());
            }
        }
        catch (IOException ex) {System.out.println("IO error!");}
    }
    Server (Socket socket){
        this.socket = socket;
        setDaemon(true);
        start();
    }
    String getPath(String request) {
        String[] arr = request.split(" ");
        //"/home/nikita/IdeaProjects/Web_Platform/src
        return "./src/web" + arr[1];
    }
    public void run (){
        String response = "";
        try{
            InputStream inputStream = socket.getInputStream();
            OutputStream outputStream = socket.getOutputStream();

            byte [] buf = new byte[1024*1024*5];
            int bytes = inputStream.read(buf); // читает байты и передает в buf байты ,а потом возвращает кол-во байтов
            String request = new String(buf,0,buf.length);
            String path = getPath(request);
            File file = new File (path);

            System.out.println(request + '\n');
            boolean exists = file.exists();
            if (exists) {
                byte[] data ={};
                String content_type = "text/html";
                if (path.contains("png")) {
                    content_type = "image/png";
                    BufferedImage image = ImageIO.read(file);
                    ByteArrayOutputStream byteArrayOutputStream = new ByteArrayOutputStream();
                    ImageIO.write(image, "png", byteArrayOutputStream);
                    data = byteArrayOutputStream.toByteArray();
                }
                else {
                    FileInputStream input = new FileInputStream(file);
                    data = input.readAllBytes();
                }



                response += "HTTP/1.1 200 OK\r\n";
                response+= "Date: "  + new Date() + "\r\n";
                response +="Content-Type: " + content_type + "\r\n";

                response +="Connection: close\r\n";

                response +="Server: Server\r\n";
                response +="Pragma: no-cache\r\n\r\n";
                response+= new String(data);

                //System.out.println(response);
                //outputStream.write(response.getBytes());

            }
            else {
                response += "HTTP/1.1 404 Not Found\n";
                response+= "Date: "  + new Date() + "\n";
                response +="Content-Type: text/plain\n";
                response +="Connection: close\n";

                response +="Server: Server\n";
                response +="Pragma: no-cache\n\n";
                response += "File " + path + " Not Found!\n\nIts not  normal\nS";
                response+="end";

            }

            outputStream.write(response.getBytes());
            outputStream.flush();
            outputStream.close();
            inputStream.close();
            socket.close();
            return;


            }
        catch (IOException ex){
            System.out.println("Error: " + ex.toString());
        }
    }

}

Comments

[Dmitry Roo]

while (true) {
  new Server(serverSocket.accept());
}

Вы уверены насчет этого кода?

[AlexVWill]

"\r\n" - плохая идея использовать для обозначения конца строки, используй System.lineSeparator()
была похожая проблема

[SuperDuperPuper]

Dmitry Roo, да

[SuperDuperPuper]

AlexVWill, не помогло

Answer 1

[SuperDuperPuper]

я понял в чем проблема. Она заключается в этой строке : response+= new String(data);
new String() - искажала байты . Поэтому я создал массив байтов , куда скопировал массив байтов c ответом и массив байтов с картинкой .

byte[] responseBytes  = response.getBytes();
 byte[] finalResponse = new byte[responseBytes.length + data.length];
 System.arraycopy(responseBytes,0,finalResponse, 0,responseBytes.length);
System.arraycopy(data,0,finalResponse, responseBytes.length, data.length);
outputStream.write(finalResponse);

Теперь все заработало