Scala中如何向Seq追加元素？代码执行不符合预期求助

你的代码核心问题在于合并已有列表和新名字的逻辑可以更严谨，当前写法不仅存在冗余变量，还容易因判断分支的意外问题（比如误写条件）导致覆盖而非追加的情况——根据你描述的仅保留最后一个名字的现象，大概率是判断grades isDefinedAt grade的分支没有被正确执行，导致每次都用新名字的单元素列表覆盖了原有值。

修正方案1：使用getOrElse简化逻辑（推荐） #

Scala的Map提供了getOrElse方法，可以直接获取对应键的值（如果存在）或者返回默认值，省去显式判断的同时，代码更简洁不易出错：

type DB = Map[Int, Seq[String]]
var grades: DB = Map.empty
def add(name: String, grade: Int): Unit = {
  // 获取对应grade的已有列表，不存在则返回空列表，将新名字追加到末尾
  val updatedNames = grades.getOrElse(grade, Seq.empty) :+ name
  grades += (grade -> updatedNames)
}

如果希望新名字被添加到列表开头，只需把:+换成+:即可：

val updatedNames = name +: grades.getOrElse(grade, Seq.empty)

修正方案2：修正原有代码的逻辑 #

如果你想保留原有代码的结构，需要确保合并逻辑正确，同时显式声明方法返回类型（原方法未指定返回类型，虽默认返回Unit但显式声明更清晰）：

type DB = Map[Int, Seq[String]]
var grades: DB = Map.empty
def add(name: String, grade: Int): Unit = {
  var names: Seq[String] = if (grades.isDefinedAt(grade)) {
    // 已有该分数的列表时，将新名字追加到已有列表末尾
    grades(grade) :+ name
  } else {
    // 无对应分数时，创建包含新名字的单元素列表
    Seq(name)
  }
  grades += (grade -> names)
}

验证结果 #

调用add("Mike", 2)、add("Michelle", 2)、add("John", 2)后，grades会变为：

Map(2 -> Seq("Mike", "Michelle", "John"))

完全符合你的预期。

内容的提问来源于stack exchange，提问作者pfari