ftp download

InputStream is = ftpClient.getInputStream();

ser meget suspekt ud.

Hvis det er Commons Net FTPClient, så:

InputStream is = ftpClient.retrieveFileStream(localfilename);

retrieveFile fremfor retrieveFileStream er iøvrigt nogen gange bedre !

Den smider nu en exception.

java.io.IOException: Stream closed

er det en stream jeg ikke får åbnet.. synes nemlig ikke jeg lukker den nogen steder. Har forresten nu ændret den til retrieveFileStream.

Desuden hvad er forskellen på retriveFile og retrieveFileStream?

Hvilken linie giver den exception.

Med retriveFile giver man den bare filnavn på server og en OutputStream så kopierer den selv data d.v.s. at man sparer løkken.

Min linie System.out.println("outside while"); bliver udført men den kommer ikke ind i while løkken så vil skyde på det er der den smider exception

Hvordan ser koden ud nu ?

Og bruger du PASV ?

public synchronized boolean downloadFile (String localfilename)
    {
        try{

                // Use passive mode as default because most of us are
                // behind firewalls these days.
                ftpClient.enterLocalPassiveMode();
             
              InputStream is = ftpClient.retrieveFileStream(localfilename);
              BufferedInputStream bis = new BufferedInputStream(is);

              OutputStream os = new FileOutputStream(localfilename);
              BufferedOutputStream bos = new BufferedOutputStream(os);

              byte[] buffer = new byte[1024];
              int readCount;
                System.out.println("outside while");
              while( (readCount = bis.read(buffer)) > 0)
              {
                  System.out.println("inside while");
                bos.write(buffer, 0, readCount);
              }
              bos.close();
              is.close (); // close the FTP inputstream
          //  this.succMesg = "Downloaded!";
              System.out.println("now we should have succcesided");

              return true;
        }catch(Exception ex)
        {
          StringWriter sw0= new StringWriter ();
          PrintWriter p0= new PrintWriter ( sw0, true );
          System.out.println(""+ex);
          ex.printStackTrace ( p0 );
      //  erMesg = sw0.getBuffer().toString ();

          return false;
        }
    }

her hvor jeg kalder den fra

            FTPConnect ftp = new FTPConnect();
            ftp.connecToServer();
            ftp.cwd("test");
            boolean stuff = ftp.downloadFile("5011 test.xml");

            if(stuff) {
                System.out.println("blaaa success");
            }
            else {
                System.out.println("crap");
            }

            ftp.disconnet();

Og hvad laver:

            ftp.connecToServer();
            ftp.cwd("test");

?

(ovenstående ser umiddelbart OK ud)

her er hele min ftp klasse.

import java.io.BufferedInputStream;
import java.io.BufferedOutputStream;
import java.io.File;
import java.io.FileInputStream;
import java.io.FileOutputStream;
import java.io.IOException;
import java.io.InputStream;
import java.io.OutputStream;
import java.io.PrintWriter;
import java.io.StringWriter;
import org.apache.commons.net.ftp.FTPClient;
import org.apache.commons.net.ftp.FTPReply;

/**
*
* @author mn
*/
public class FTPConnect {
   
    private String server = "XXXX";
    private String user = "XXX";
    private String pass = "XXXX";
    private String account = "XXXX";
    private FTPClient ftpClient = new FTPClient();
    private boolean success = false;

    public FTPConnect() {

    }

    public synchronized void connecToServer() {
        try {
            int reply;
            ftpClient.connect(server);
            System.out.println("Connected to " + server + ".");

            reply = ftpClient.getReplyCode();

            if (!FTPReply.isPositiveCompletion(reply))
            {
                ftpClient.disconnect();
                System.err.println("FTP server refused connection.");
                System.exit(1);
            }
        }
        catch (IOException e) {
            if(ftpClient.isConnected()) {
                try {
                    ftpClient.disconnect();
                }
                catch(IOException f) {
                    f.printStackTrace();
                }
            }
            System.out.println("Could not connect to server");
            e.printStackTrace();
            System.exit(1);
        }
        try {
            if(!ftpClient.login(user, pass, account)) {
                System.out.println("Coldn't login to " + server);
            }
            System.out.println("kommer jeg her til");
        }
        catch(IOException e) {
            e.printStackTrace();
        }
    }

    public synchronized void cwd (String dir) {
        try {
            ftpClient.cwd(dir);
        } catch (IOException ex) {
            ex.printStackTrace();
        }
    }

    public synchronized boolean downloadFile (String localfilename)
    {
        try{

                // Use passive mode as default because most of us are
                // behind firewalls these days.
                ftpClient.enterLocalPassiveMode();
             
              InputStream is = ftpClient.retrieveFileStream(localfilename);
              BufferedInputStream bis = new BufferedInputStream(is);

              OutputStream os = new FileOutputStream(localfilename);
              BufferedOutputStream bos = new BufferedOutputStream(os);

              byte[] buffer = new byte[1024];
              int readCount;
                System.out.println("outside while");
              while( (readCount = bis.read(buffer)) > 0)
              {
                  System.out.println("inside while");
                bos.write(buffer, 0, readCount);
              }
              bos.close();
              is.close (); // close the FTP inputstream
          //  this.succMesg = "Downloaded!";
              System.out.println("now we should have succcesided");

              return true;
        }catch(Exception ex)
        {
          StringWriter sw0= new StringWriter ();
          PrintWriter p0= new PrintWriter ( sw0, true );
          System.out.println(""+ex);
          ex.printStackTrace ( p0 );
      //  erMesg = sw0.getBuffer().toString ();

          return false;
        }
    }

    public synchronized boolean stor(File file, String inputStream) throws IOException {
        if (file.isDirectory()) {
          throw new IOException("SimpleFTP cannot upload a directory.");
        }
        // Use passive mode as default because most of us are
        // behind firewalls these days.
        ftpClient.enterLocalPassiveMode();
       
        String filename = file.getName();

        InputStream in = new FileInputStream(inputStream);
        success = ftpClient.storeFile(filename, in);
//      ftpClient.disconnect();
        return success;
    }

    public synchronized void disconnet() {
        try {
            ftpClient.disconnect();
        }
        catch (IOException ex) {
            System.out.println("Connection closed");
        }
    }
}

Mystisk.

Hvis directpry og fil eksisterer, så virker det hos mig.

Jeg får samme fejl som dig, hvis de ikke eksisterer.

Meget underligt... nu spørger jeg måske dumt men hvad skrev du i boolean stuff = ftp.downloadFile("?????");

hvis jeg udkommenter denne
ftp.cwd("test");

og skriver
boolean stuff = ftp.downloadFile("test.xml");

samt lægger filen test.xml på serveren rod. Så får jeg stadig samme fejl.

bare et eksisterende filnavn

så du får kun denne fejl meddelse hvis filen ikke findes?

java.io.IOException: Stream closed

Ja. Eller hvis det dir man cwd'er til ikke findes.

I det tilfælde returnerer ftpClient.retrieveFileStream(localfilename) nemlig null og bis.read giver så den fejl.

host host host... Sorry det var mig som var lidt halv stiv opdager jeg lige.... skrev test.xml men min fil hedder test.xls

Synes dog ikke helt at jeg får filen ned på computeren. Får dog success når jeg køre programmet. Men kan ikke se filen.
Hvor lagde den sig da du fik det til at virke?

Har prøvet at ændre localfilename til "c:\\test\\test.xls" men ser ikke ud til at virke. Eller rettere får desværre ikke filen ned til computeren.

OutputStream os = new FileOutputStream(localfilename);
              BufferedOutputStream bos = new BufferedOutputStream(os);

OutputStream os = new FileOutputStream("c:\\test\\" + localfilename);

Synes nu også jeg gjorde det samme men nu virker det... Mange tak.. Forresten har du en ide om hvorfor jeg får denne besked når jeg åbner mit excel dokument "test.xls cannot be accessed. The file may be read-only or you may be trying to access a read-only location. Or the server the document is stored on may not be responding."

Kan se at filen har samme størrelse som den jeg har hentet fra serveren. Når jeg bare kopire filen over via et ftp program(GUI) så får jeg samme besked. Så det er ikke pga mit program.

Check permissions på filen og det dir som den ligger i.

har højreklikket på filen og valgt egenskaber men her står der ikke nogen specilet unormalt. den er ikke skrivebeskyttet. Er der andre steder jeg skal kigge. Samt er der nogle rettigheder man kan ændre inden man henter filen ned.

Du skal ind i security tabben.

Hey Arne tror jeg har fundet ud af lidt mere. Det går galt når mit program uploader filen til serveren. Har taget en fil og uploaded den til ftp-serveren via en GUI. Herefter hentede jeg den ned igen via mit program og her virker det helt fint. Kan du se hvad der går galt i mit upload eller har du et eksempel som virker eller er bedre?

Smider gerne 60 point ekstra i det hvis du kan hjælpe mig med dette nye problem:)

Jeg kan ikke umiddelbart se noget problem i upload.

Har du checket med et andet FTP program hvordan den uploadede fil ser ud ?

Har prøvet med bare et alm. ftp program (brugte core ftp) og her virker det helt fint. I min kode bruger jeg SimpleFTP. Kender du et andet som er bedre at bruge?

Kan fejlen opstå hvis jeg ikke uploader filen som binary? Har kigget lidt rundt og ser det måske kunne være en mulighed hvorfor det fejler.

Har fået det til at virke nu. Det viser sig at man skal sende det binært, for ellers korrupter den filen. Smider du et svar

og mange tak for hjælpen:)

ok