Spring-如何正确使用@Autowired？

Question

我在Spring应用程序中发现了一个奇怪的行为。

实体的@autowired并非一直都在工作。

我使用的Elid项目，以建立一个JSONAPI和一些定制的控制器。

在一个控制器中，一个@Autowrited实体保持为空，但是当从Elide调用时它使用正确。

控制器：

@RestController
public class UploadController {
    @Autowired
    private ProjectRepository projectRepository;

    @PostMapping(value = "/api/projects/{projectId}/upload")
    public String uploadItem(@PathVariable long projectId, @RequestParam("file") MultipartFile file,
                              @RequestParam("projectName") String projectName,
                              RedirectAttributes redirectAttributes) throws IOException {
        Project project = projectRepository.findOneByProjectIdAndName(projectId, projectName);
        Integer result = project.getNumberOfItems();

        return "";
    }
}

实体

@Setter
@NoArgsConstructor
@Table(name = "projects")
@Entity
@Include(rootLevel = true, type = "projects")
public class Project extends DiffShelfBase {

    @Autowired
    @Transient
    private ItemRepository itemRepository;

    @Transient
    @ComputedAttribute
    public Integer getNumberOfItems() {
        return itemRepository.countByProjectId(this.getId());
    }
}

知识库

@Repository
@Transactional
public interface ItemRepository extends JpaRepository<Item, Long> {
    Integer countByProjectId(long projectId);
}

组态

@Configuration
@EnableSpringConfigured
public class MyConfiguration {
    @Bean
    public WebMvcConfigurer corsConfigurer() {
        return new WebMvcConfigurerAdapter() {
            @Override
            public void addCorsMappings(CorsRegistry registry) {
                registry.addMapping("/**")
                        .allowedMethods("*")
                        .allowedOrigins("http://localhost:4200");
            }
        };
    }
}

我不明白为什么，但是Project.getNumberOfItems的itemRepository为null。

Answer 1

这不是您应该如何做的方法。 不要在您的实体中使用@Autowired 。 这个注释是由Spring管理的，但是您的实体是由Hibernate管理和创建的，它并不关心Spring注释。 您可以找到另一种方法将此属性添加到您的实体（例如，使其成为数据库中的字段）。

Answer 2

您遇到的问题是，您的Project没有在春季之前进行管理。 您必须在类中添加@Configurable批注。 然后，您的自动Autowired注释将起作用。

但是我真的会重构您的代码。 您的实体中不应包含弹簧对象。

Answer 3

一个Entity应该是一个美化的POJO ，永远不要使它复杂地嵌入repos里面。

因此，可以自动上ItemRepository ， UploadController其UploadController本身。

@RestController
public class UploadController {

    @Autowired
    private ProjectRepository projectRepo;

    @Autowired
    private ItemRepository itemRepo;

    @PostMapping(value = "/api/projects/{projectId}/upload")
    public String uploadItem(@PathVariable long projectId, @RequestParam("file") MultipartFile file,
                              @RequestParam("projectName") String projectName,
                              RedirectAttributes redirectAttributes) throws IOException {

        Project project = projectRepo.findOneByProjectIdAndName(projectId, projectName);
        Integer result = itemRepo.countByProjectId(project.getId());

        return "";
    }
}

作为一种好习惯，您总是可以进行构造函数注入，从而易于测试

例如代替

@Autowired
private ProjectRepository projectRepo;

@Autowired
private ItemRepository itemRepo;

做一个构造函数注入

private final ProjectRepository projectRepo;
private final ItemRepository itemRepo;

@Autowired
public UploadController(final ProjectRepository projectRepo, final ItemRepository itemRepo) {
    this.projectRepo = projectRepo;
    this.itemRepo = itemRepo;
}

这条路，

1. 通过传递Mockito实例很容易进行测试。
2. 您可以将它们@Autowire （直接使用@Autowire对变量进行@Autowire是不可能的）