run 方法有效，但线程不会调用它

Question

我必须实现一个类来存储有关学校项目的机场信息。该课程应该从我们的讲师提供给我们的以 HTML 编写的网站中读取有关航班和乘客信息的数据，为了做到这一点，我们正在使用线程。我的问题是当我在我的线程上调用start()方法时，它不会调用我实现runnable的类上的run方法，这很奇怪，因为当我第一次测试程序并直接调用run()方法时，它工作得很好。这是我的工作类的 run 方法中的 while 循环，它执行实际的数据处理。

while (1 == 1) {
    temp = scan.next();
    if ((temp.equals("</body>")))
        break;
    else {
        name = temp;
        flightNum = scan.nextInt();
        city = scan.next();
        indexOfBR = city.indexOf("<");
        city = city.substring(0, indexOfBR);

        synchronized (airplanes) {
            addFlight(flightNum, city);
            addPassengerReservation(flightNum, name);
        }
    }
}
scan.close();

该循环应该读取乘客姓名，然后是航班号，然后是目的地城市，在 html 文件中存储为：

"passenger name" "flight Number" "destination city"<br>

创建和运行线程的方法是

public void readFlightData(String[] urls) {
    for (int x = 0; x < urls.length; x++) {
        if (!(urls[x].equals(""))) {
            try {
                Worker w = new Worker(new URL(urls[x]));
                Thread work = new Thread(w);
                work.start();
            }
            catch (MalformedURLException e) {
                //do nothing
            }
        }
    }
}

当我在工作线程上调用 start 时，此方法不会调用 run 方法，我不知道为什么。任何帮助将不胜感激

我发现问题出在 while 循环中的同步块中。传递给同步块飞机的对象是一个 ArrayList，用于保存 Airport 类中的航班和乘客信息。 addFlight() 方法旨在添加具有 flightNum 和目的地城市的航班。 addPassengerReservation 将一名乘客添加到指定的航班号，名为“name”

Answer 1

代码非常脆弱。我建议进行以下更改。我认为有一个 MalformedURLException 并且您正在隐藏它。

public void readFlightData(String[] urls) {
    for (int x = 0; x < urls.length; x++) {
        if (!(urls[x].equals(""))) {
            try {
                Worker w = new Worker(new URL(urls[x]));
                Thread work = new Thread(w);
                work.start();
            }
            catch (MalformedURLException e) {
                System.err.print(e);
            }
        }
    }
}

Answer 2

我想通了。我的问题是，在我的 Airport 类完成运行我创建的线程并下载必要的信息之前，我正在调用它的方法。我不得不在所有线程上调用 join() 来解决这个问题。对于那些好奇的人，这里是工作代码：

ArrayList<Thread> threads = new ArrayList<Thread>();
int x;

for (x = 0; x < urls.length; x++) {
    if (!(urls[x].equals(""))) {
        try {
            Worker w = new Worker(new URL(urls[x]));
            Thread work = new Thread(w);
            work.start();
            threads.add(work);
        }
        catch (MalformedURLException e) {
            System.err.print(e);
        }
    }
}

for (x = 0; x < threads.size(); x++)
    try {
        threads.get(x).join();
    } catch (InterruptedException e) {
        e.printStackTrace();
}