Spring Batch示例中直接调用@Bean方法是否为错误写法？

这种写法完全不是错误，是Spring框架针对@Configuration类设计的正常用法，下面给你拆解清楚：

• @Configuration的代理机制是核心
  Spring会给标注了@Configuration的类生成CGLIB代理类。当你在同一个配置类里调用另一个标注@Bean的方法时，代理会拦截这个调用：不会直接执行方法创建新对象，而是返回Spring容器中已经初始化好的对应Bean实例。所以itemReader()里直接调用lineMapper()，拿到的和通过参数注入的LineMapper Bean是同一个，效果完全等价。

• 两种写法的适用场景
  你习惯的「参数注入LineMapper」写法当然没问题，和直接调用@Bean方法都是合法的配置方式：

  • 如果LineMapper Bean定义在同一个配置类里，直接调用写法更简洁；
  • 如果LineMapper Bean来自其他配置类，那必须用参数注入或者@Autowired的方式获取。

• lineMapper()是否需要声明为@Bean？
  分两种情况判断：

  • 如果这个LineMapper只给当前的itemReader用，不需要被其他组件复用，也没有需要Spring注入的内部依赖，那确实可以不用加@Bean，直接在itemReader()里new出来就行；
  • 但如果这个LineMapper需要被其他Bean复用，或者内部有依赖Spring管理的组件，就必须加@Bean让Spring把它纳入容器管理。Spring Batch示例里标注@Bean，通常是因为这个LineMapper可能会被其他Reader复用，或者它本身依赖Spring管理的对象。

内容的提问来源于stack exchange，提问作者al.truisme