JAVA多线程笔试题-获取支付方式

一、题目内容

二、我的答案

　　利用了线程池、考虑了超时处理、不知道这样写是否还有其他问题，或者更好更优的解决方案？

import java.util.*;
import java.util.concurrent.*;
 
public class MoreThread {
 
    public static void main(String[] args) {
        List<String> allPaymentList=Arrays.asList("余额","红包","余额宝","银行卡");
        long start=System.nanoTime();
        List<String> list=filterDisablePayment(allPaymentList);
        double seconds=(System.nanoTime()-start)/1000000000.0;
        System.out.println("总共耗时:"+seconds+"s");
        for(String paymentType:list){
            System.out.println(paymentType);
        }
    }
 
 
    public static List<String> filterDisablePayment(List<String> allPaymentList){
        List<String> results=new ArrayList<>();
        ExecutorService executorService= Executors.newFixedThreadPool(allPaymentList.size());
        List<Future<String>> futures=new ArrayList<>();
        for(String paymentType:allPaymentList) {
            futures.add(executorService.submit(new PaymentMethodCallable(paymentType)));
        }
        executorService.shutdown();
        for(Future<String> future:futures){
            try {
                //超时处理机制
                String result= future.get(4,TimeUnit.SECONDS);
                if(result!=null){
                    results.add(result);
                }
            } catch (Exception e) {
                e.printStackTrace();
            }
        }
        return results;
    }
 
 
    public static Boolean PaymentIsEnabled(String paymentType) {
        try {
            //模拟远程服务调用所用时间
            Thread.sleep(3000);
            Random random=new Random();
            boolean result=random.nextBoolean();
            System.out.println("获取到结果:"+paymentType+":"+result);
            return result;
        } catch (Exception e) {
            e.printStackTrace();
        }
        return false;
    }
 
 
    static class PaymentMethodCallable implements Callable<String> {
 
        private String paymentType;
 
        public String getPaymentType() {
            return paymentType;
        }
 
        public void setPaymentType(String paymentType) {
            this.paymentType = paymentType;
        }
 
        public PaymentMethodCallable(String paymentType) {
            this.paymentType = paymentType;
        }
 
        @Override
        public String call() {
            if(PaymentIsEnabled(paymentType)) return paymentType;
            return null;
        }
    }
}