文章/答案/技术大牛

发布

社区首页 >问答首页 >Scala中的FizzBuzz

问Scala中的FizzBuzz
EN

Code Review用户

提问于 2016-11-16 22:18:40

回答 1查看 521关注 0票数 2

正如标题所示，这是我在Scala中的FizzBuzz实现。

大约两个月前，我开始学习Scala (和一般的函数式编程)，最近的一项任务是FizzBuzz任务。下面是我想出的解决这个问题的简单代码。它运行得很完美，但我想CodeReview可以很容易地指出我错过了什么。

我确实认为硬编码的魔术值(3,5,15，"Fizz"，"Buzz"，"FizzBuzz")有点问题，但我不确定这是否是普遍共识，或者如何处理它是最好的方式。

FizzBuzz.scala

package fizzbuzz

object FizzBuzz {

  def fizzbuzz(i: Int): String = i match {
    case i if i % 15 == 0 => "FizzBuzz" //because i | 15 is the same as i | 5 and i | 3
    case i if i % 5 == 0 => "Buzz"
    case i if i % 3 == 0 => "Fizz"
    case i => i toString
  }

  def fizzbuzz(start: Int, end: Int): List[String] = (start to end map fizzbuzz) toList

  //assignment requirement: return list of first 100 fizzbuzzed numbers
  def fizzbuzz100(): List[String] = fizzbuzz(1,100)
}

Main.scala ( FizzBuzz对象的示例用法)

package fizzbuzz

object main {
  def main(args: Array[String]): Unit = {
     FizzBuzz.fizzbuzz100() map println
  }
}

scala

fizzbuzz

回答 1

Code Review用户

回答已采纳

发布于 2016-11-17 01:35:03

尽管Rosetta代码并不总是为每种语言提供最地道的代码，但是请看“习语解”：

object FizzBuzz extends App {
  1 to 100 foreach { n =>
    println((n % 3, n % 5) match {
      case (0, 0) => "FizzBuzz"
      case (0, _) => "Fizz"
      case (_, 0) => "Buzz"
      case _ => n
    })
  }
}

您的代码没有很好地使用模式匹配(您使用它的方式有点单一)。我建议您使用您所拥有的解决方案，并重写它以使用模式匹配，更像Rosetta代码解决方案。

票数 2

页面原文内容由Code Review提供。腾讯云小微IT领域专用引擎提供翻译支持

原文链接：

https://codereview.stackexchange.com/questions/147249

复制

相似问题

问Scala中的FizzBuzz
EN

FizzBuzz.scala

Main.scala ( FizzBuzz对象的示例用法)

回答 1

Code Review用户

社区

活动

圈层

关于

腾讯云开发者

热门产品

热门推荐

更多推荐

问Scala中的FizzBuzzEN

FizzBuzz.scala

Main.scala ( FizzBuzz对象的示例用法)

回答 1

Code Review用户

社区

活动

圈层

关于

腾讯云开发者

热门产品

热门推荐

更多推荐

问Scala中的FizzBuzz
EN