如何定义此方法的结果类型?

时间:2011-06-27 00:24:44

标签: scala playframework siena

如何在以下情况中定义方法返回类型:

工作代码

def deleteInstance(model: String, uid: Long) =  model match {
    case "menu" => Model.all(classOf[Menu]).filter("uid", uid).get().delete()
    case "articles" => Model.all(classOf[Articles]).filter("uid", uid).get().delete()
    case "news" => Model.all(classOf[News]).filter("uid", uid).get().delete()
    case "image" =>Model.all(classOf[Image]).filter("uid", uid).get().delete()
    case "files" =>Model.all(classOf[Files]).filter("uid", uid).get().delete()
    case _ => false
  }

非工作代码:

class ModelManager{
  def getModel(model: String) = {
    model match{
      case "menu" => classOf[Menu]
      case "articles" => classOf[Articles]
      case _ => false
    }

  def deleteInstance(model:String, uid: Long) = {
    Model.all(getModel(model)).filter("uid", uid).get().delete()
  }    
 }
} 

引发的错误是:

  

递归方法getModel需要结果   型

3 个答案:

答案 0 :(得分:7)

看起来你需要一个Option

class ModelManager{
   def getModel(model: String) = model match {
      case "menu" => Some(classOf[Menu])
      case "articles" => Some(classOf[Articles])
      case _ => None
   }

   def deleteInstance(model:String, uid: Long) = 
      getModel(model) map { m => 
         Model.all(m).filter("uid", uid).get().delete()
      } getOrElse false
}

您可以将Option视为最多可容纳一个元素的容器。包含元素x的选项为Some(x)。空选项为None。 Option有几种有用的方法,包括上面使用的mapgetOrElse方法。

map方法将函数应用于“容器”的每个元素。当然,如果容器是None,它什么都不做(除了可能更改Option的静态类型)。在您的情况下(假设delete返回布尔值),map方法会将Option [Class]更改为Option [Boolean]。

getOrElse方法返回选项的元素(如果有),否则返回默认值(在这种情况下为false)。

请注意,您还可以使用PartialFunction中定义的condOpt方法简化您的实施:

class ModelManager{
   def getModel(model: String) = condOpt(model) {
      case "menu" => classOf[Menu]
      case "articles" => classOf[Articles]
   }

   def deleteInstance(model:String, uid: Long) = 
      getModel(model) map { m => 
         Model.all(m).filter("uid", uid).get().delete()
      } getOrElse false
}

答案 1 :(得分:2)

看起来getModel有时会返回一个Class,而另一个则返回布尔值。在Scala中,通常使用Either类建模:

def getModel(model: String) = {
    model match{
      case "menu" => Left(classOf[Menu])
      case "articles" => Left(classOf[Articles])
      case _ => Right(false)
    }

左和右代表Either的两种可能选择。此方法的调用者需要检查返回值(可能也通过使用模式匹配)来确定方法是返回Class还是Boolean。

答案 2 :(得分:0)

看来你没有在正确的地方与parens关闭。你是说这个吗?

class ModelManager{
  def getModel(model: String) = {
    model match{
      // snip
    }
  } // end method here

  def deleteInstance(model:String, uid: Long) = {
    Model.all(getModel(model)).filter("uid", uid).get().delete()
  }    
} 

看起来你似乎没有尝试定义一个递归方法...然后你可能还有其他问题需要解决,因为你需要一个返回Class [_]而不是{{1}的组合的方法}和Boolean(这将是Class[_])。那么这可能会更好吗?

Any