1

曾几何时，有一位年轻的绅士会阅读一些教程、一些文档、一些 StackOverflow 问题，甚至可能会问一两个问题。在他这样做之后，他认为他对事情的运作方式有了一个公平的把握，并着手构建他想要的功能，并且它奏效了！直到邪恶的女巫出现并尝试使用没有魔法 cookie 的不同浏览器时，他才意识到 cookieUserIdentity一直存在于用户之间……这并不好。

从戏剧！文档（Guice 文档说类似）

每次需要组件时都会创建新实例。如果一个组件被多次使用，那么默认情况下，将创建该组件的多个实例。如果您只想要一个组件的单个实例，那么您需要将其标记为单例。

我IndexController班的佼佼者（我的假设是每个请求都调用一次索引操作，因此请求一个新的UserIdentity ——这是错误的吗？）

@Singleton
class IndexController @Inject() (contentModel: ContentModel, site: Site, injector: Injector) extends Controller {

  def index(url: String, page: Int = 1, sort: String, dir: Int) = Action.async { implicit request =>
    implicit val queryString : QueryString = request.queryString
    val userIdentityClass = injector.getInstance(classOf[UserIdentity])
    implicit val userIdentity : Future[UserIdentity] = userIdentityClass.getUserIdentity
    doContent(url, page, sort, dir) fallbackTo doAlias(url) fallbackTo do404(url)
  }

userIdentity然后隐含地传递到机器的内部。

该getUserIdentity方法调用进行getUser身份验证的方法，并设置user : UserModel属性。这一切都在未来传递，因此当我们userIdentity稍后在应用程序中使用时，我们可以映射它，以便我们知道身份验证已完成。

这是 get user 方法，并且println是为未经身份验证的用户调用的方法。

private def getUser(implicit request:Request[AnyContent]) : Future[UserModel] = {
  if (user != null) {
    println("User already set")
    Future { user }
  } else {
    //Go and find the user from cookie/some other auth stuff

注意var user : UserModel = null在UserIdentity类上，所以它应该null在一个新实例上。

4

3 回答 3

2

要按需而不是在实例化类时获取实例，您将使用如下提供程序：

class MyController @Inject() (userIdProv: Provider[UserIdentity])

有关您的代码的更多说明：

  • 如果可以避免，请不要注入注入器，而是注入您需要的实际类（或其提供者）。一方面，您正在创建对 guice 本身的依赖。其次，你会在单元测试中注入什么？注射器的模拟？然而，最重要的是，直接注入您所依赖的组件只会使您的代码更清晰。从建模的角度考虑：你有一个依赖于另一个东西来完成任务的东西——而不是：我有一个依赖于注入器的东西（非常技术性，不具体）。

  • 为什么你的控制器是单例的？首先，播放控制器无论如何都是单例的（文档中没有说明，所以你可能不能永远依赖这种情况）其次，你为什么希望它们成为单例？请注意，拥有单例总是会引入某种状态——除非有非常特殊的需要，否则您会希望避免这种情况。

  • 不要做Future { user }。这将创造并执行一个未来。你真正想要的是一个Future.successful(user)基本上只是用正确的类型包装用户。
于 2016-05-11T05:39:58.967 回答
1

你是对的，它每次都请求一个新实例，但我认为 guice 在内部重用同一个实例。

我会很高兴发布您的模块定义以确保，但我认为您可能能够摆脱这样的事情来确保每次都创建一个新实例：

bind(classOf[UserIdentity]).toProvider(new Provider[UserIdentity]{
  override def get(): UserIdentity = new UserIdentity()
})
于 2016-05-10T07:53:42.350 回答
0

感谢汤佩的回答，我找对了地方。

这个故事的寓意是检查你的 Guice Module。我将其绑定UserIdentity为 Eager Singleton。因此，通过删除以下行，我能够解决我的问题。

bind(classOf[UserIdentity]).asEagerSingleton()

显然是邪恶的女巫把它放在那里的。

于 2016-05-10T09:46:01.877 回答