FTPClient上传文件内容为空/损坏/缺失

项目场景：

项目场景：本地项目联调OA系统的时候，在发送审批时会传送相关附件，该附件由本地项目上传至FTP，OA系统会根据我们提供的路径和文件名去FTP中找到该文件

---

问题描述：

同事写了一个FTPClient的工具类，侧重看一下连接部分的代码，主要出的问题也是在这一块内容上

public static boolean connect(FTPClient ftpClient,String host,String user,String password, String encoding){
        boolean flag = false;
        try {
            ftpClient.connect(host);
            ftpClient.setFileType(FTPClient.BINARY_FILE_TYPE);
            ftpClient.setControlEncoding(encoding);
            ftpClient.enterLocalPassiveMode();
            if (ftpClient.login(user,password)){
                flag = true;


                log.info("ftp connected");
            }
            else{
                log.info("ftp connect failed : user or password error");
                ftpClient.disconnect();
            }
        } catch (IOException e) {
            log.error("ftp connect failed , error {[]}",e);
            e.printStackTrace();
        }
        return flag;
    }

---

原因分析：

这段代码也是很多网上给出的代码，问题也就随之而来了，传txt文件完全没有问题，那么其它文件如图片、pdf等等的时候就会出现问题了。
做了很多测试，最后将问题定位在了ftpClient.setFileType(FTPClient.BINARY_FILE_TYPE);
进入setFileType()方法，断点如下图位置
程序运行后会发现并没有通过if判断，evaluate一下type(fileType)的值，会发现得到的值是530
然后我们继续进入isPositiveCompletion方法

那么530一定是不满足这个条件的，所以导致了我们的文件内容为空或损坏，FTPClient默认的传输格式是ASCII形式的
这里思考一下为什么是530返回，仔细看ftpClient.setFileType(FTPClient.BINARY_FILE_TYPE);的位置，是在connect以后，login以前，530在ftp中有未登录的意思
这样子问题就很明显了，大家自己去打断点的时候会发现，一共会经历三个码值，530、200、230，第二个是不是200我也记不太清了，这篇文章是解决问题的后两天写

---

解决方案：

问题找到以后，解决的方法也很简单了，把代码的顺序换一下就可以了

public static boolean connect(FTPClient ftpClient,String host,String user,String password, String encoding){
        boolean flag = false;
        try {
            ftpClient.connect(host);
            
            if (ftpClient.login(user,password)){
                flag = true;
                ftpClient.setFileType(FTPClient.BINARY_FILE_TYPE);
                ftpClient.setControlEncoding(encoding);
                ftpClient.enterLocalPassiveMode();

                log.info("ftp connected");
            }
            else{
                log.info("ftp connect failed : user or password error");
                ftpClient.disconnect();
            }
        } catch (IOException e) {
            log.error("ftp connect failed , error {[]}",e);
            e.printStackTrace();
        }
        return flag;
    }

不仔细看的话很难发现差别

ftpClient.setFileType(FTPClient.BINARY_FILE_TYPE);
ftpClient.setControlEncoding(encoding);
ftpClient.enterLocalPassiveMode();

主要就是这三行代码的位置要在login后面才会生效
到此为止，问题解决了
说点题外话，看看这三行代码
ftpClient.setFileType(FTPClient.BINARY_FILE_TYPE);这个就是用来配置文件传输形式的，默认ASCII，我们一般都是配置用二进制形式传送
ftpClient.setControlEncoding(encoding);这个是用来配置编码的，其实除了上面的传输形式没什么问题，这个我是有点不太明白的，为什么需要在登录后配置才有效，可能涉及到其它问题，猜测：FTPClient登录后会产生新的对象，这时候引用地址会改变，也就导致了登录前的配置无效
ftpClient.enterLocalPassiveMode();这行代码倒是没有太大的感受，留着和删掉暂时没有影响，网上说是被动和主动，不过我没有去深究，感兴趣可以自行学习