Question

我将端口扫描程序类复制到我的android项目而不更改源代码中的任何内容。 Eclipse没有抱怨代码，但我的代码似乎在android中似乎没有正常工作。这是我的整个portscanner代码。

import java.net.InetSocketAddress;
import java.net.Socket;
import java.util.ArrayList;
import java.util.Collection;
import java.util.LinkedList;
import java.util.concurrent.ExecutorService;
import java.util.concurrent.Executors;
import java.util.concurrent.Future;


public class PortScanner {

    private final String ip;
    private final int sPort, ePort, timeout, poolSize;
    private ArrayList<Integer> openPorts = new ArrayList();
    private final ExecutorService es;
    private Collection<Future<?>> futures = new LinkedList<>();


    public PortScanner(String ip, int sPort, int ePort, int timeout, int poolSize) {
        this.ip = ip;
        this.sPort = sPort;
        this.ePort = ePort;
        this.timeout = timeout;
        this.poolSize = poolSize;
        es = Executors.newFixedThreadPool(this.poolSize);

    }

    public ArrayList<Integer> getPorts() {
        return openPorts;
    }

    public void runScanner() {

        for (int startPort = sPort; startPort <= ePort; startPort++) {
            futures.add(es.submit(new Check(ip, startPort, timeout)));
        }

        es.shutdown();

    }

    private class Check implements Runnable { //  BEGININ OF INNER CLASS

        private String ip;
        private int port, timeout;

        private Check(String ip, int port, int timeout) {
            this.ip = ip;
            this.port = port;
            this.timeout = timeout;
        }

        @Override
        public void run() {

            try {
                Socket socket = new Socket();
                socket.connect(new InetSocketAddress(ip, port), timeout);
                socket.close();
                openPorts.add(port);
            } catch (Exception ex) {
            }
        }
    }                                       // END OF INNER CLASS
}

我用几种不同的方式测试了我的代码，并确保只有内部类运行方法才能正常工作，因为当我手动将一些端口添加到数组列表时它们会打印，但是当我拉动扫描仪时添加它们它总是空的。相同的端口扫描代码在桌面上运行良好。有谁知道我做错了什么？

Answer 1

不确定这是否是您遇到的问题，但是您正在从多个线程向ArrayList添加一些不是线程安全的东西，而没有任何同步。这将导致不稳定的行为。

此外，您不会显示如何以及何时显示此列表的内容。调用代码无法知道扫描何时结束，因此如果您在调用runScanner()之后立即询问列表，则很可能没有任何扫描任务已完成执行。顺便说一句，如果你从不使用这个清单，为什么要将期货添加到期货清单？

Answer 2

你检查过例外吗？如果在添加到openPorts的套接字之前抛出异常，您将获得一个空数组列表。并且在捕获异常时没有打印异常堆栈 - 当我忘记在异常补丁中打印错误日志时，我也遇到了类似的问题。

        try {
            Socket socket = new Socket();
            socket.connect(new InetSocketAddress(ip, port), timeout);
            socket.close();
            openPorts.add(port);
        } catch (Exception ex) {
            //It's better to do something at least print an error log
            ex.printStackTrace();
        }

android项目中的J2ME类

2 个答案: