Параметр передан от вызываемого, но равен нулю в определении метода

Question

Я в основном пытаюсь создать функцию распаковки.Я вызвал функцию с параметрами в блоке ниже:

UnzipUtility unzipUtility = new UnzipUtility();
    try {
        unzipUtility.unzip(localFilePath, parentPath);
    } catch (IOException e) {
        e.printStackTrace();
    }

Определение метода в классе UnzipUtility, код следующий:

    public void unzip(String zipFilePath, String destDirectory) throws IOException {
    File destDir = new File(destDirectory);
    if (!destDir.exists()) {
        destDir.mkdir();
    }
    ZipInputStream zipIn = new ZipInputStream(new FileInputStream(zipFilePath));
    ZipEntry entry = zipIn.getNextEntry();
    // iterates over entries in the zip file
    while (entry != null) {
        String filePath = destDirectory + File.separator + entry.getName();
        if (!entry.isDirectory()) {
            // if the entry is a file, extracts it
            extractFile(zipIn, filePath);
        } else {
            // if the entry is a directory, make the directory
            File dir = new File(filePath);
            dir.mkdir();
        }
        zipIn.closeEntry();
        entry = zipIn.getNextEntry();
    }
    zipIn.close();
}

Но во время выполнения, хотя параметрыправильно передаются в основной класс, значения отображаются как нулевые в методе распаковки.

Пожалуйста, помогите с этим

Основной класс выглядит следующим образом:

package com.example.sftpconnection;
import android.annotation.SuppressLint;
import android.os.AsyncTask;
import android.support.v7.app.AppCompatActivity;
import android.os.Bundle;
import android.util.Log;
import com.jcraft.jsch.Channel;
import com.jcraft.jsch.ChannelSftp;
import com.jcraft.jsch.JSch;
import com.jcraft.jsch.Session;
import com.jcraft.jsch.JSchException;
import com.jcraft.jsch.SftpException;


//import java.io.BufferedOutputStream;
import java.io.File;
//import java.io.FileOutputStream;
import java.io.IOException;
//import java.io.OutputStream;
import java.util.List;
import com.example.sftpconnection.UnzipUtility;

public class SFTPActivity extends AppCompatActivity {

    private String fileName = "1234.zip";
    private String localFilePath;
    private String parentPath;
    @SuppressLint("StaticFieldLeak")
    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_sftp);


        new AsyncTask<Void, Void, List<String>>() {
            @Override
            protected List<String> doInBackground(Void... params) {
                try {
                    Downloader(fileName);
                } catch (Exception e) {
                    e.printStackTrace();
                }
                return null;
            }

        }.execute();

        UnzipUtility unzipUtility = new UnzipUtility();
        try {
            unzipUtility.unzip(localFilePath, parentPath);
        } catch (IOException e) {
            e.printStackTrace();
        }
    }

    public void Downloader(String fileName) {

        String user = "1234";
        String pass = "1234";
        String host = "1234";
        int portNum = 22;

        JSch jsch = new JSch();
        Session session;

        try {

            session = jsch.getSession(user,host,portNum);
            session.setConfig("StrictHostKeyChecking", "no");
            session.setPassword(pass);
            session.connect();

            Channel channel = session.openChannel("sftp");
            channel.connect();
            ChannelSftp sftpChannel = (ChannelSftp) channel;


            File localFile = File.createTempFile("1234",".zip");
            sftpChannel.get(fileName,localFile.getAbsolutePath());
            //sftpChannel.get(fileName);

            Log.d(fileName, " has been downloaded");

            sftpChannel.exit();
            session.disconnect();
            localFilePath = localFile.getAbsolutePath();
            parentPath = localFile.getParent();
        } catch (JSchException | IOException | SftpException e) {
            e.printStackTrace();
        }
    }


}

Я редактировал поля данных по соображениям безопасности.

Answer 1

Причина, по которой ваш код не работает, заключается в том, что на момент запуска этой строки:

unzipUtility.unzip(localFilePath, parentPath);

переменные localFilePath и parentPath не были установлены.

Вы можете утверждать, что они установлены в методе Downloader, который вызывается перед строкой unzip.К сожалению, это не так.В этом случае выполнение кода не является линейным, потому что вы используете AsyncTask.Материал в асинхронной задаче выполняется одновременно со строками, которые идут после нее.

Ваш вызов на загрузку не завершится до вызова unzipUtility.unzip, поскольку загрузка занимает огромное количество времени по сравнению с созданиемnew UnzipUtility object.

Вот почему localFilePath и parentPath равны нулю.

Один из способов решения этой проблемы - также переместить логику распаковки в асинхронную задачу:

new AsyncTask<Void, Void, List<String>>() {
    @Override
    protected List<String> doInBackground(Void... params) {
        try {
            Downloader(fileName);

            UnzipUtility unzipUtility = new UnzipUtility();
            unzipUtility.unzip(localFilePath, parentPath);
        } catch (Exception e) {
            e.printStackTrace();
        }
        return null;
    }

}.execute();

Другой способ - также переопределить onPostExecute в AsyncTask:

new AsyncTask<Void, Void, List<String>>() {
     // doInBackground goes here...

     @Override
     protected void onPostExecute(Long result) {
         try {
            UnzipUtility unzipUtility = new UnzipUtility();
            unzipUtility.unzip(localFilePath, parentPath);
         } catch (Exception e) {
            e.printStackTrace();
         }
     }
}

Answer 2

Можно вырезать следующие строки:

UnzipUtility unzipUtility = new UnzipUtility();
try {
    unzipUtility.unzip(localFilePath, parentPath);
} catch (IOException e) {
    e.printStackTrace();
}

вставить чуть ниже следующей строки в классе SFTPActivity :

Downloader(fileName); 

метода doInBackground.На самом деле метод doInBackground() выполняется в потоке, отличном от потока, в котором выполняется метод onCreate.Вы пытаетесь использовать переменные до того, как метод Downloader(fileName) сможет выполнить свою работу.Это причина, почему вы видите нулевые значения в переменных, таких как: localFilePath и parentPath