You really want to use an ExecutorService instead of individual threads, it’s much cleaner, likely more performant and will enable you to change things more easily later on (thread counts, thread names, etc.):
ExecutorService pool = Executors.newFixedThreadPool(10);
for (String name : fileNames) {
pool.submit(new DownloadTask(name, toPath));
}
pool.shutdown();
pool.awaitTermination(Long.MAX_VALUE, TimeUnit.MILLISECONDS);
// all tasks have now finished (unless an exception is thrown above)
And somewhere else in your class define the actual work horse DownloadTask:
private static class DownloadTask implements Runnable {
private String name;
private final String toPath;
public DownloadTask(String name, String toPath) {
this.name = name;
this.toPath = toPath;
}
@Override
public void run() {
// surround with try-catch if downloadFile() throws something
downloadFile(name, toPath);
}
}
The shutdown() method has a very confusing name, because it “will allow previously submitted tasks to execute before terminating”. awaitTermination() declares an InterruptedException you need to handle.