java - Spring中Service层的作用是什么，在这种情况下如何拆分我的逻辑？

Question

我正在尝试编写一个简单的网络应用程序，它将电子邮件 ID 作为参数并为具有该 ID 的用户生成一个 token 。我认为我的代码是不言自明的，所以我将它粘贴在这里，这样我就不必详细解释了。

这是我的 Controller /servlet 代码

    User user = userManager.getUserByEmailId("xyz@gmail.com");
    if (user == null) {
        //TODO handle this
    }
    if (user.getIssuedTokens() == user.getMaxTokens()) {
        // TODO handle this
    }
    String token = tokenService.createToken();
    user.setToken(token);
    user.setIssuedTokens(user.getIssuedTokens() + 1);
    userManager.updateUser(user);

userManager 和 tokenService 是服务层的实现。

@Service("tokenService")
public class TokenizationServiceImpl implements TokenizationService {

    @Autowired
    private TokenDAO tokenDAO;

    @Transactional
    public String createToken() {
        String uuid = UUID.randomUUID().toString();
        tokenDAO.createToken(uuid);
        return uuid;
    }
}


@Service("usermanager")
public class UserInterfaceImpl implements UserInterface {

    @Autowired
    private UserDAO userDAO;

    @Transactional
    public void createUser() {
        userDAO.createUser();
    }

    public User getUserByEmailId(String emailID) {
        return userDAO.getUserByEmailId(emailID);
    }

    @Transactional
    public void updateUser(User user) {
        userDAO.updateUser(user);

    }

}

我的spring配置是这样的

<tx:annotation-driven />

    <context:component-scan base-package="com.myapp.dao" />
    <context:component-scan base-package="com.myapp.service" />

    <bean id="dataSource"
        class="org.springframework.jdbc.datasource.DriverManagerDataSource">

        <property name="driverClassName" value="com.mysql.jdbc.Driver" />
        <property name="url" value="jdbc:mysql://localhost:3306/mydb" />
        <property name="username" value="root" />
        <property name="password" value="root" />
    </bean>

    <!-- dataSource TransactionManager -->
    <bean id="transactionManager"
        class="org.springframework.jdbc.datasource.DataSourceTransactionManager">
        <property name="dataSource" ref="dataSource" />
    </bean>

这是我的问题:

1. 将两个服务(usermanager 和 tokenService)注入(inject)到我的 Controller /servlet 中然后一个接一个地调用它们是否有意义？或者我应该直接在 TokenServiceImpl 中编写一个方法，直接使用 usermanager 服务或 UserDAO 吗？
2. 交易属性似乎对我不起作用。当 updateUser 方法失败时，将通过 createToken() 方法在数据库中创建一个 token 。我在这里做错了什么？为什么事务没有回滚？
3. 我通常如何决定我的服务应该使用多个 DAO 还是其他服务？

Answer 1

嗯，我对你第二个问题的看法也可能回答你的第一个问题。查看您的代码片段，我注意到您正在为 tokenService.createToken() 和 userManager.updateUser(user) 创建两个不同的事务，因为您是从外部调用它们的@Transaction 方法。为了解决此问题，您需要执行以下操作:

public class UserService {

    ...

    @Transactional
    public void assignToken() {
        User user = userManager.getUserByEmailId("xyz@gmail.com");
        if (user == null) {
        //TODO handle this
        }
        if (user.getIssuedTokens() == user.getMaxTokens()) {
        // TODO handle this
        }
        String token = tokenService.createToken();
        user.setToken(token);
        user.setIssuedTokens(user.getIssuedTokens() + 1);
        userManager.updateUser(user);
    }
}

您会注意到，为了考虑到这种新的事务行为，我创建了一个名为 UserService 的新业务组件。尽管我对您的应用程序的了解还不足以说明什么是最好的方法，但我绝对不会让它出现在您的 Controller 中。在我看来，您应该将此行为隔离在:

• 一个新的业务组件(就像我在这个例子中所做的那样)
• 或者将它封装在你的 UserManager 中，例如

现在，由您决定是否值得将 UserService 耦合到 TokenizationService 或为此创建一个新的业务类。阅读您提供的代码，在我看来 UserService 可能有一个 TokenizationService 因为 token 不会在不同的上下文中使用。

请让我知道您的意见。