scala - 使用可为空而不是选项？

Question

我有单例:

class MySing private (var1: SomeType) { ... }
object MySing {

  @volatile
  private var instance: Option[MySing] = None

  def apply(var1: SomeType): Option[MySing] = {
    if (instance.isEmpty) {
      this.synchronized {
        if (instance.isEmpty) instance = Some(new MySing(var1))
      }
    }

    instance
  }
}

所以每当我想获取它的实例时，我必须使用 get 或模式匹配

MySing(myVar1) match {
  case Some(x) => ...
  case None =>
}

或

MySing(myVar1).get

即使 Scala 也鼓励使用 Option，但在我的情况下，不使用它来摆脱上面所示的无聊、冗余的操作不是合理的吗？

object MySing {

  @volatile
  private var instance: MySing = null

  def apply(var1: SomeType): MySing = {
    if (instance == null) {
      this.synchronized {
        if (instance == null) instance = new MySing(var1)
      }
    }

    instance
  }
}

Answer 1

返回Option[MySing]意味着您可能不会从apply返回值(即返回None)，但我不这样做看看这有什么意义。您总是从 apply 返回一个实例，因此您可以使用返回类型中的 Option ，如下所示

class MySing private (val var1: String)
object MySing {

  @volatile
  private var instance: Option[MySing] = None

  def apply(var1: String): MySing = {
    instance getOrElse {
      this.synchronized {
        instance.getOrElse {
         instance = Some(new MySing(var1))
         instance.get
        }
      }
    }
  }
}

使用示例

scala> val x = MySing("hello")
x: MySing = MySing@232b0a52

scala> x.var1
res0: String = hello

scala> val y = MySing("hola")
y: MySing = MySing@232b0a52

scala> y.var1
res1: String = hello // are you sure this is sensible?

但是，它看起来仍然有点难看:您正在创建一个带有参数的单例，因此忽略了第一次调用后传递给 apply 方法的每个参数。呃!

您可能想要重新考虑您的设计(并忘记 null)