0

下面的代码是我用来创建端口扫描器的代码，但是，当我单击“扫描”时，它会执行，但 gui 会冻结，直到完成扫描，有没有可能的方法允许它在扫描时执行其他操作？

import javax.swing.*;
import java.awt.*;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.net.InetAddress;
import java.net.Socket;
import java.net.UnknownHostException;

public class PortScannerGUI extends JFrame implements ActionListener
{
JPanel Panel = new JPanel(null);
JTextField Field = new JTextField();
JButton Button = new JButton();
JTextArea Area = new JTextArea();
JButton limit = new JButton("limit");

public PortScannerGUI()
{
    super();
    setSize(350, 400);
    setDefaultCloseOperation(EXIT_ON_CLOSE);
    setResizable(false);
    setLocationRelativeTo(null);
    LoadUI();
}

public void LoadUI()
{
    Area.setBounds(20, 50, 310, 310);
    Area.setBorder(BorderFactory.createLineBorder(Color.BLUE));
    Area.setEditable(false);

    Field.setBounds(20, 10, 200, 35);

    Button.setBounds(230, 10,100, 35);
    Button.setText("Scan");
    Button.addActionListener(this);

    Panel.add(Field);
    Panel.add(Area);
    Panel.add(Button);
    add(Panel);
    setVisible(true);
}

public static void main(String[] args)
{
    PortScannerGUI Main = new PortScannerGUI();
}

@Override
public void actionPerformed(ActionEvent arg0)
{
    Thread Runner = new Thread();
    Runner.start();
    int j=0;
    String str = Field.getText();
    for(int i=1;i<str.length();i++)
    {
        if(str.substring(i-1, i).equals("."))
        {
            j++;
        }
    }
    if(!str.equals("") || j==4 || j==6)
    {
        try
        {
            InetAddress IP = InetAddress.getByName(str);
            PortScanner P = new PortScanner(IP);
        }
        catch (Exception e){}
    }
    else
    {
        JOptionPane.showMessageDialog(null, "Not an IP address");
    }

}
}

上面的代码是 GUI 部分的代码，下面是实际扫描端口的代码。

import java.io.IOException;

import java.net.InetAddress;

import java.net.Socket;

public class PortScanner
{
public PortScanner(InetAddress iA)
{
    for (int port = 1; port <= 65535; port++)
    {
        try
        {
            Socket s = new Socket(iA, port);
            System.out.println("Port " + port + " is open");
            s.close();
        }
        catch(IOException e)
        {

        }
    }
}
}
4

1 回答 1

5

这是在 Swing 事件线程（EDT 或 Event Dispatch 线程）中执行长时间运行的进程并因此冻结该线程使您的 GUI 无法运行的经典示例。我们一次又一次地看到这样的事情发布，解决方案是相同的，并且相对简单：在后台线程中执行长时间运行的进程，例如可以由 SwingWorker 提供。有关这方面的更多信息，请阅读Swing 中的并发，或在此站点上搜索类似的问题，因为再次讨论了很多。

关于您的代码的另一点，永远不要这样做：

    catch(IOException e)
    {

    }

如果您忽略异常，您实际上是在盲目飞行。至少，打印一个堆栈跟踪，即：

    catch(IOException e)
    {
       e.printStackTrace();
    }

这里也一样：

     try {
        InetAddress IP = InetAddress.getByName(str);
        PortScanner P = new PortScanner(IP);
     } catch (Exception e) {
        e.printStackTrace();  // **** add this!
     }

我自己，我在 PortScanner 构造函数中设置了 PortScanner 对象，但是会在一个单独的公共方法中运行扫描，比如调用scan(). 这将允许我在事件线程上设置 PortScanner 对象，然后通过调用其scan()方法在后台线程中对其进行扫描。有时这更方便。

编辑
我还将赋予 PortScanner 类添加 PropertyChangeListener 的能力，以便它可以允许其他类监听它的更改，包括端口信息。这样您就可以在 GUI 中发布 PortScanner 端口信息。最简单的方法是让您的 PortScanner 对象扩展 SwingWorker，因为 SwingWorkers 带有内置的 PropertyChangeSupport。然后您可以使用发布/处理方法对将端口信息发布到 GUI，而不是在 println 语句中打印出来。我上面链接的教程描述了如何执行此操作，如果您遇到困难，我们可以帮助您提供详细信息。

于 2012-08-04T14:43:52.163 回答