Problems with InputStream ()

Question

hi there,
i have a litle server, that sends a file to the client. the problem is that availaible() is always 0 and i cant read out the things. i use java 1.4.1.
SERVER CODE
import javax.net.ServerSocketFactory;
import java.net.Socket;
import java.net.ServerSocket;
import java.io.*;
public class Listener extends Thread {
    ServerSocket socket;
    ServerSocketFactory socketFactory;
    public Listener(int port) {
        try {
            socket = ServerSocketFactory.getDefault().createServerSocket(port);
            while (true) {
                /* connecting */
                System.out.print("waiting");
                Socket client = socket.accept();
                System.out.println("         [accepted]");
                BufferedInputStream bis = new BufferedInputStream(client.getInputStream());
                BufferedOutputStream bos = new BufferedOutputStream(client.getOutputStream());
                String file = "";
                bos.write(0);
                System.out.println("available            "+bis.available());
                while (bis.available()!=-1) {
                    int c = bis.read();
                    System.out.println("reading: "+(char)c);
                    if(c == -1) break;
                    file += (char) c;
                System.out.println("sending file: " + file);
                /* sending file */
                file = "\\test.jpg";
                if (new File(file).exists()) {
                    FileInputStream fis = new FileInputStream(file);
                    System.out.println("filesize: " + fis.available());
                    if (file.endsWith(".jpg") || file.endsWith(".gif")) {
                        while (true) {
                            int c = fis.read();
                            System.out.println(c);
                            if(c < 0) break;
                            System.out.print(".");
                            bos.write(fis.read());
                        bos.flush();
                        fis.close();
                    System.out.println("");
                    System.out.println("OK..");
                else{
                    bos.write(0);
                /* closing streams*/
                bos.close();
                bis.close();
                client.close();
        } catch (Exception e) {
            e.printStackTrace();
}CLIENT CODE
import java.net.Socket;
import java.net.InetAddress;
import java.io.*;
public class StreamConnector {
    private String file;
    private InetAddress host;
    private int port;
    public StreamConnector(String file, InetAddress host, int port) {
        this.file = file;
        this.host = host;
        this.port = port;
    public boolean getImageAndSave(String newFilename) throws IOException {
        try {
            /* connect */
            Socket socket = new Socket(host, port);
            BufferedInputStream bis = new BufferedInputStream(socket.getInputStream());
            BufferedOutputStream bos = new BufferedOutputStream(socket.getOutputStream());
            /* sending path*/
            char[] sendFile = file.toCharArray();
            for (int i = 0; i < sendFile.length; i++) {
                bos.write((int)sendFile);
System.out.println("writing: "+sendFile[i]);
bos.flush();
/* getting file and save */
FileOutputStream fop = new FileOutputStream(newFilename);
int i=0;
while(true){
System.out.print(".");
int c = bis.read();
System.out.println(c);
if(c < 0) break;
fop.write(c);
i++;
if(i == 0 || i == 1 || i == 2){
new File(newFilename).delete();
socket.close();
return false;
File f = new File(newFilename);
if(!f.exists()){
socket.close();
return false;
socket.close();
} catch (IOException e) {
e.printStackTrace();
throw e;
return true;
i have this problem since a few days and realy need help. thank you for your time. jules

Answer 1

I don't think available() ever returns -1. If you want to read the file name byte by byte, you will need to send something to signal the end of the string. But really, it would be much easier to use a Reader and a Writer instead of streams. They are designed for handling text. If you used a PrintWriter and a BufferedReader for the file name, you could read and write the whole thing at once.