Zusätzliche Parameter für Methode hinzufügen?

[Pago83]

Hallo zusammen,

ich möchte die folgende Methode etwas anpassen:

    public void exportFile(String localFile, String user, String host, String remoteFile, Session session)
    //public void exportFile(String localFile, String localFileFtp, String user, String host, String remoteFile, String remoteFileFtp, Session session)
    {
        FileInputStream fis = null;
        try
        {
            String command = "scp -p -t " + remoteFile;
            Channel channel = session.openChannel("exec");
            ((ChannelExec) channel).setCommand(command);

            OutputStream out = channel.getOutputStream();
            InputStream in = channel.getInputStream();

            channel.connect();

            if (checkAck(in) != 0)
            {
                System.exit(0);
            }

            long filesize = (new File(localFile)).length();
            command = "C0644 " + filesize + " ";
            if (localFile.lastIndexOf('/') > 0)
            {
                command += localFile.substring(localFile.lastIndexOf('/') + 1);
            }
            else
            {
                command += localFile;
            }
            command += "\n";
            out.write(command.getBytes());
            out.flush();
            if (checkAck(in) != 0)
            {
                System.exit(0);
            }

            fis = new FileInputStream(localFile);
            byte[] buf = new byte[1024];
            while (true)
            {
                int len = fis.read(buf, 0, buf.length);
                if (len <= 0)
                    break;
                out.write(buf, 0, len);
            }
            fis.close();
            fis = null;
            buf[0] = 0;
            out.write(buf, 0, 1);
            out.flush();
            if (checkAck(in) != 0)
            {
                System.exit(0);
            }
            out.close();

            channel.disconnect();
        }
        catch (Exception e)
        {
            System.out.println(e);
            try
            {
                if (fis != null)
                    fis.close();
            }
            catch (Exception ee)
            {
                System.out.println(ee);
            }
        }
    }

Die Methode soll zwei zusätzliche Parameter übergeben bekommen (siehe Kommentar). Der String localFileFtp soll sich wie localFile verhalten, ebenso der String remoteFileFtp zu remoteFile.

Wie kann ich die Methode am elegantesten umstellen, so dass localFileFtp (remoteFileFtp) sich genauso verhält wie localFile (remoteFile)?

Hinweise sind erwünscht ;-)

MfG Pago

 

[SlaterB]

'sich Verhalten wie' ist etwas merkwürdig, das musst du näher erläutern,
meinst du vielleicht

String command = "scp -p -t " + remoteFile; 
if (remoteFileFtp != null) {
  command += " "+remoteFileFtp 
}
statt
String command = "scp -p -t " + remoteFile;

oder soll remoteFileFtp das ursprüngliche remoteFile ersetzen?
oder oder, muss man alles programmieren,

 

[Pago83]

Also,

das localFileFtp soll genau das machen wie das localFile (ist halt das selbe in grün, nur für ein FTP File).

Bei remoteFileFtp ebenso. Es soll genau das machen, was remoteFile macht.

Hoffe, es ist jetzt klar, was ich erreichen möchte.

 

[SlaterB]

dann bin ich für

public void exportFileNormal(String localFile, String user, String host, String remoteFile, Session session) {
  ...
}

public void exportFileFTP(String localFileFtp, String user, String host,  String remoteFileFtp, Session session) 
    { 
exportFileNormal(..);
}

oder, wenn exportFileFTP() wirklich beide Strings übergeben bekommt,
dann exportFileNormal() zweimal aufrufen,

oder in exportFileFTP prüfen, welche Strings sinnvoll befüllt sind
und abhängig davon exportFileNormal() mit den richtigen Strings aufrufen

 

[Pago83]

Habe jetzt mal folgende Variante probiert:

    public void exportFiles(String localFileFtp, String user, String host, String remoteFile, Session session)
    {
        exportFile(localFile, user, host, remoteFile, session);
    }

Bekomme die Fehlermeldung, localFile cannot be resolved.

Hier noch einmal die Ausgangsmethode:

    public void exportFile(String localFile, String user, String host, String remoteFile, Session session)
    {
        FileInputStream fis = null;
        try
        {
            String command = "scp -p -t " + remoteFile;
            Channel channel = session.openChannel("exec");
            ((ChannelExec) channel).setCommand(command);

            OutputStream out = channel.getOutputStream();
            InputStream in = channel.getInputStream();

            channel.connect();

            if (checkAck(in) != 0)
            {
                System.exit(0);
            }

            long filesize = (new File(localFile)).length();
            command = "C0644 " + filesize + " ";
            if (localFile.lastIndexOf('/') > 0)
            {
                command += localFile.substring(localFile.lastIndexOf('/') + 1);
            }
            else
            {
                command += localFile;
            }
            command += "\n";
            out.write(command.getBytes());
            out.flush();
            if (checkAck(in) != 0)
            {
                System.exit(0);
            }

            fis = new FileInputStream(localFile);
            byte[] buf = new byte[1024];
            while (true)
            {
                int len = fis.read(buf, 0, buf.length);
                if (len <= 0)
                    break;
                out.write(buf, 0, len);
            }
            fis.close();
            fis = null;
            buf[0] = 0;
            out.write(buf, 0, 1);
            out.flush();
            if (checkAck(in) != 0)
            {
                System.exit(0);
            }
            out.close();

            channel.disconnect();
        }
        catch (Exception e)
        {
            System.out.println(e);
            try
            {
                if (fis != null)
                    fis.close();
            }
            catch (Exception ee)
            {
                System.out.println(ee);
            }
        }
    }

Muss ich localFile global anlegen oder was muss ich ändern?

MfG Pago

 

[Pago83]

Dieses Posting soll nur zur Info dienen, sonst merkt ja keiner, dass sich was geändert hat ;-)

 

[AlArenal]

Habe jetzt mal folgende Variante probiert:

    public void exportFiles(String localFileFtp, String user, String host, String remoteFile, Session session)
    {
        exportFile(localFile, user, host, remoteFile, session);
    }

Und was wundert dich daran? Immerhin gibts dort keine Variable localFile...

 

[Pago83]

Das ist mir schon klar, ich möchte ja das localFile aus der exportFile Methode haben.

Muss ich das zu diesem Zwecke global anlegen?

 

[mikachu]

...kann weg... doppelpost :roll:

 

[mikachu]

public void exportFiles(String localFileFtp, String user, String host, String remoteFile, Session session)

und

public void exportFiles(String localFile, String user, String host, String remoteFile, Session session)

in einer klasse zusammen geht ?!?

ich glaube nicht

 

[SlaterB]

Das ist mir schon klar, ich möchte ja das localFile aus der exportFile Methode haben.

Muss ich das zu diesem Zwecke global anlegen?

der erste Satz macht keinen Sinn, was willst du erreichen?

 

[Pago83]

Also,

bisher sah meine Methode für die beiden Exports so aus:

    public void exportFiles(String localFile, String localFileFtp, String user, String host, 
                            String remoteFile, String remoteFileFtp, Session session)
    {
        exportFile(localFile, user, host, remoteFile, session);
        exportFile(localFileFtp, user, host, remoteFileFtp, session);
    }

Ist aber etwas unübersichtlich, das muss (und wird sicherlich) ja auch anders gehen.

 

[mikachu]

iss doch gut

 

[Pago83]

OK, dann vertrau ich mal auf deinen Rat ;-)

 

[merlin2]

iss doch gut

Immer diese Meinung, man könnte alle Probleme durch Essen lösen. :roll: :wink:

 

[Leroy42]

iss doch gut

Immer diese Meinung, man könnte alle Probleme durch Essen lösen. :roll: :wink:

Trink doch gut

 

[merlin2]

Auch nicht besser.

 

[Leroy42]

Auch nicht besser.

Aber rein subjektiv ein gewaltiger Unterschied! :meld: ( )

 

[merlin2]

Was ich nicht bestreite.

 

[merlin2]

Aber immerhin: Besser iss doch gut als ess doch gut!