並べ替え中に非常に奇妙なNullPointerExceptionが発生しました

Question

リストにnull値があるため、NULLポインタ例外がありますadPics。まれにしか起こらない。どのように可能ですか？

（並列にこのコードをダウンロード画像とローカルに保存されます。）

List<String> downloadAdImages(List<String> imagesUrls, final String itemFolder) { 
     final List adPics = new ArrayList<>(); 
     final ExecutorService executor = newFixedThreadPool(20); 
     imagesUrls.forEach(
       picUrl -> executor.submit(() -> { 
        try { 
         String imageNewFileName = imagesUrls.indexOf(picUrl) + "." + getExtension(picUrl); 
         String bigPicUrl = picUrl.replace("b.jpg", "ab.jpg"); // big version 
         copyURLToFile(new URL(bigPicUrl), new File(itemFolder, imageNewFileName), 10, 10); 
         adPics.add(imageNewFileName); 
        } catch (IOException ex) { 
         log.log(Level.WARNING, "Could not download image {0} ({1})", new Object[]{picUrl, ex.getMessage()}); 
        } 
       })); 
     executor.shutdown(); 
     try { 
      executor.awaitTermination(15L, MILLISECONDS); 
     } catch (InterruptedException ex) { 
      log.log(Level.WARNING, "Could not wait for all images downloads"); 
     } 
     Collections.sort(adPics); // null values at list lead to NPE here. How are there null values? 
     return adPics; 
    } 

時々adPicsリストはnull値を有しています。それがNPEの理由です。しかしどうですか？スレッドで実行されたコードを分析すると、nullの値を追加することはできません。イメージのダウンロードに問題があった場合、IOExceptionがスローされます。 imageNewFileNameはnullにはなりません。

このコードはJava 8で、Apache Commons IO libを使用しています。

Answer 1

メソッドawaitTerminationは実行中のスレッドを停止しません。すべてのスレッドが完了するか、またはtimeoutに達するまで待ちます。したがって、あなたのスレッドは依然としてあなたのリストに項目を追加します。

また、タイムアウトに達してもファイルシステムへのダウンロードとコピーが実行中であることを考慮する必要があります。

シンプルだが完璧ではない解決策は、タイムアウトに達したときにフラグを設定し、アイテムを追加する前にフラグをチェックすることです。

より良いアプローチは、タイムアウトに達した後にスレッドを中断させることです。これには、ダウンロードとファイルのコピーの中断も含まれます。

Answer 2

コードにはいくつかの問題があります。パウロはすでに一つの問題を指摘している。他の問題は、List adPicsに同時にアクセスしていることです。

同期リストを使用せずに問題を解決するには、リスト[CompletableFuture]を作成し、終了時にはCompletableFuture.allOfを呼び出します。 各CompletableFutureはイメージファイル名を返すはずですが、イメージをダウンロードする前に、その操作を開始するかどうかを確認する必要があります。そうでない場合は、null値でCompletableFutureを完了できます。 CompletableFuture.allOfでは、null値なしで新しいListを作成し、そのリストをソートできます。