NullPointerException on Autowired object inside of abstract class

Question

我的代码有一个奇怪的问题。我已经实现了 template method pattern 以提供相同算法的不同实现。所以我创建了摘要 class:

@Service
public abstract class ExcelRdi_Iupr_Sl {
    @Autowired
    private Environment env;

    private static String PROPERTY_NAME_FILESYSTEM_EXCELPATH = "temporary.excelpath";
    private XSSFWorkbook workbook; 
    private XSSFCellStyle unlockedNumericStyle;
    private XSSFSheet sheet;

    /**
     * Create the excel file with the RDI,IUPR or SL
     * @param car
     * @param fileName
     * @return
     * @throws Exception
     */
    public final String retrieve(Car car, String fileName) throws Exception{
        writeInitialize();
        Map<Integer,Integer> defMap = firstTwoRow(car, sheet);
        elaboration(car, sheet, unlockedNumericStyle, defMap);
        return writeFile(car, fileName);
    }

    /**
     * Import the excel file with the RDI,IUPR or SL
     * @param file
     * @param car
     * @throws Exception
     */
    public final int update(MultipartFile file, Car car) throws Exception{
        String filePath = saveFile(file, car);
        readInitialize(filePath);
        return updateRdi(car, sheet);
    }

    /**
     * Save the imported file into file system inside a temporary folder
     * @param file
     * @param car
     * @return
     * @throws Exception
     */
    private String saveFile(MultipartFile multipartFile, Car car) throws Exception {
        String path = env.getRequiredProperty(PROPERTY_NAME_FILESYSTEM_EXCELPATH) + File.separator + String.valueOf(System.currentTimeMillis());
        File file = new File (path+ File.separator + multipartFile.getOriginalFilename());
        file.getParentFile().mkdirs();
        multipartFile.transferTo(file);
        return file.getAbsolutePath();
    }

    /**
     * Initialize the object for the creating procedure
     */
    protected void writeInitialize(){
        //initialize class variable
    }

    /**
     * Initialize the object for the reading procedure
     * @param filePath
     * @throws Exception
     */
    protected void readInitialize(String filePath) throws Exception{
        //read ile and initialize class variable

    }

    /**
     * Write the created file inside the file system
     * @param car
     * @param fileName
     * @return
     * @throws Exception
     */
    private String writeFile(Car car, String fileName) throws Exception{        
        //write on file
    }

    /**
     * This excel creating method has to be implemented by the concrete class
     * @param car
     * @param sheet
     * @throws Exception
     */
    protected abstract Map<Integer,Integer> firstTwoRow(Car car, XSSFSheet sheet) throws Exception;

    /**
     * This excel creating method has to be implemented by the concrete class
     * @param car
     * @param sheet
     * @param defMap
     * @throws Exception
     */
    protected abstract void elaboration(Car car, XSSFSheet sheet, XSSFCellStyle unlockedNumericStyle, Map<Integer,Integer> defMap);

    /**
     * This excel import method has to be implemented by the concrete class
     * @param car
     * @param sheet
     * @return number of elaborated row
     * @throws Exception
     */
    protected abstract int updateRdi(Car car, XSSFSheet sheet) throws Exception;
}

扩展上述 class 的 class 之一具有此实现：

@Component
public class ExcelRdi extends ExcelRdi_Iupr_Sl {
    @Autowired
    private RdiServices rdiServices;
    @Autowired
    private AcquisitionServices acquisitionServices;
    static final Logger LOG = LoggerFactory.getLogger(ExcelRdi.class);

    /**
     * Create the first two row of excel file
     */
    @Override
    protected Map<Integer,Integer> firstTwoRow(Car car, XSSFSheet sheet) throws Exception {
        //code
    }

    /**
     * Create the row with RDI for excel file
     */
    @Override
    protected void elaboration(Car car, XSSFSheet sheet, XSSFCellStyle unlockedNumericStyle, Map<Integer,Integer> defRdiMap) {
        //code
    }

    /**
     * Import the Excel file
     * @throws Exception 
     */
    @Override
    @Transactional(rollbackFor= Exception.class)
    protected int updateRdi(Car car, XSSFSheet sheet) throws Exception{
        //COde
    }
}

所以从通过控制器接收请求的服务我有一个简单的开关：

@Autowired
private ExcelRdi excelRdi;
@Autowired
private ExcelIupr excelIupr;
@Autowired
private ExcelSl excelSl;

@Override
    public String getExcel(Car car, String type) throws Exception {
        String fileName = type + "$"+car.getFleet().getFleetName().getFleetName() +"$"+ car.getFleet().getApplication()+"$"+ car.getCarType().getIdCarType()+car.getId() +"$"+car.getIdCar()+".xlsx";                   
        switch (type){
        case "RDI": return excelRdi.retrieve(car, fileName);
        case "IUPR": return excelIupr.retrieve(car, fileName);
        case "SL": return excelSl.retrieve(car, fileName);
        default: throw new FileFormatException("You can create only RDI, SL or IUPR file!");
        }
    }

    @Override
    public int importExcel(MultipartFile file, Car car, String type) throws Exception {
        //Check if the file has the correct name before import
        String fileName = type + "$"+car.getFleet().getFleetName().getFleetName() +"$"+ car.getFleet().getApplication()+"$"+ car.getCarType().getIdCarType()+car.getId() +"$"+car.getIdCar()+".xlsx";                   
        if (!file.getOriginalFilename().equals(fileName))
            throw new FileFormatException("The file is not for this car");
        switch (type){
        case "RDI": return excelRdi.update(file, car);
        case "IUPR": return excelIupr.update(file, car);
        case "SL": return excelSl.update(file, car);
        default: throw new FileFormatException("You can create only RDI, SL or IUPR file!");
        }

    }

这就是问题所在：

如果我使 protected 和 updateRdi 一切正常，但 @Transactional 注释需要 public 方法，否则它不起作用
_如果我将 public 设为 updateRdi，我会在 firstTwoRow 的 sheet 对象上收到 nullPointerException，env 变量也是如此. 您知道我为什么会有这种行为以及如何解决吗？非常感谢

Answer 1

问题在于您的 receive 和 update 方法被标记为 final。此外，如果您观察日志记录，您将看到一条警告，指出无法代理 final 方法。

发生的事情是，由于没有接口，将创建一个基于 class 的代理。该代理将向所有 public 方法添加行为（例如 @Transactional）。它通过覆盖实际方法、应用建议并在非代理实例（驻留在代理内）上调用方法来实现这一点。

然而，由于 final 的性质，这不会发生，并且将在代理而不是代理内的实际对象上调用该方法。由于代理不是 Spring bean，所有 @Autowired 字段都是 null，因此您的 NullPointerException。

如果在 update 方法中放置一个断点，您可以观察到这种行为，您会在调试器中看到所有实例字段都是 null 并且 classname 类似于 YOurClass$CgLibProxy。还将有一个字段 target（如果我没记错的话），它包含实际的 Spring bean，它注入了依赖项。

您的问题的解决方案实际上非常简单，只需从您的方法中删除 final 即可。

NullPointerException on Autowired object inside of abstract class

NullPointerException on Autowired object inside implementation of abstract class

java

spring

design-patterns

abstract

autowired