功能Scala的重构/布局

import io.Source
println(
  Source.fromFile("names.txt").getLines.mkString.split(",").map{
    x => x.slice(1,x.length -1)
  }.sortBy {x => x}.zipWithIndex.map{
    t =>{ (t._2 +1)*(t._1.map{_.toChar - "A"(0).toChar + 1}.sum)}
  }.sum
)

下一步是清理一下,使用模式匹配来映射元组列表和身份而不是x => x. toChar对于字符也是不必要的,单引号可用于表示字符文字.

import io.Source
println(
  Source.fromFile("names.txt").getLines.mkString.split(",").map {
    x => x.slice(1,x.length -1)
  }.sortBy(identity).zipWithIndex.map {
    case (v,idx) =>{ (idx+1)*(v.map{_ - 'A' + 1}.sum)}
  }.sum
)

还有一些更改也有助于使代码的意图更清晰：

import io.Source
println(
  Source.fromFile("names.txt").getLines.mkString.split(",")
  .map { _.stripPrefix(""").stripSuffix(""") }
  .sortBy(identity)
  .map { _.map{_ - 'A' + 1}.sum }
  .zipWithIndex
  .map { case (v,idx) => (idx+1) * v }
  .sum
)

下一步,使它更“功能”,就是将其分解为“功能”(偷偷摸摸,是吗？)理想情况下,每个功能都将有一个清楚表达其目的的名称,它将很短(旨在使其成为一个单一表达式,因此不需要大括号)：

import io.Source

def unquote(s:String) = s.stripPrefix(""").stripSuffix(""")

def wordsFrom(fname:String) =
  Source.fromFile(fname).getLines.mkString.split(",").map(unquote)

def letterPos(c:Char) = c - 'A' + 1

println(
  wordsFrom("names.txt")
  .sortBy(identity)
  .map { _.map(letterPos).sum }
  .zipWithIndex
  .map { case (v,idx) => (idx+1) * v }
  .sum
)

wordsFrom是一个明显的1行内容,但是我将其拆分为在StackOverflow上更容易的格式化