我正在尝试编写一个简单的webapp,它将电子邮件ID作为参数并为具有该ID的用户生成令牌。我认为我的代码是不言自明的,因此我将其粘贴在这里,这样我就不必详细解释了。
这是我的控制器/ servlet代码
User user = userManager.getUserByEmailId("[email protected]");
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);
}
}
我的春季配置是这样的
<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>
这是我的问题:
好吧,我对第二个问题的想法可能还会回答第一个问题。在您的代码片段中,我注意到您正在为创建两个不同的事务tokenService.createToken()
,userManager.updateUser(user)
因为您是从@Transaction
方法外部调用它们的。为了解决此问题,您需要执行以下操作:
public class UserService {
...
@Transactional
public void assignToken() {
User user = userManager.getUserByEmailId("[email protected]");
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
。即使我不太了解您的应用程序并不能说什么是最好的方法,但我绝对不会将其放入您的控制器中。我认为,您应该在以下情况中隔离此行为:
UserManager
的内部现在,由您决定是否值得与之耦合或UserService
为此TokenizationService
创建新的业务类。阅读您提供的代码,在我看来UserService
可能会有一个TokenizationService
原因,因为令牌不会在不同的上下文中使用。
请让我知道您的意见。
本文收集自互联网,转载请注明来源。
如有侵权,请联系[email protected] 删除。
我来说两句