Не могу заставить работать с DefaultValue - PullRequest
0 голосов
/ 23 августа 2011

Я делаю обзор кода для мнемоники телефонной книги, о котором доктор Мартин Одерский упомянул на Skillmatters .

Вот фрагмент того, что у него есть:

class Coder(words: List[String]) {

  private val mnemonics = Map(
    '2' -> "ABC", '3' -> "DEF", '4' -> "GHI", '5' -> "JKL",
    '6' -> "MNO", '7' -> "PQRS", '8' -> "TUV", '9' -> "WXYZ")

  /** Invert the mnemonics map to give a map from chars 'A' ... 'Z' to '2' ... '9'*/
  private val charCode: Map[Char, Char] =
    for ((digit, str) <- mnemonics; letter <- str) yield (letter -> digit)


  /** Maps a word to the digit string it can represent */
  private def wordCode(word: String): String = word.toUpperCase map charCode

  /** A map from digit strings to the words that represent them, 
    * e,g. 5282 -> Set(Java, Kata, Lava, ...) */

  private val wordsForNum: Map[String, List[String]] = 
    (words groupBy wordCode) withDefaultValue List()

Я попытался объявить все как функцию и переменную, чтобы увидеть, как ведет себя withDefaultValue, и вот что я получил:

scala> val words3 = List("moo", "1111")
words3: List[java.lang.String] = List(moo, 1111)

scala> (words3 groupBy wordCode) withDefaultValue List()
java.util.NoSuchElementException: key not found: $
    at scala.collection.MapLike$class.default(MapLike.scala:224)

В видео он говорил о том, что если слово не отображается, то мы должны получить список элементов (это начинается с отметки 17 минут на видео).Я получаю ошибку?Я использую Scala REPL 2.9.0.1.

Спасибо за ваше время.

Редактировать:

Я почти уверен, что правильно сгенерировал val и функцию def в REPL.

scala> :paste
// Entering paste mode (ctrl-D to finish)

val mnemonics = Map(
    '2' -> "ABC", '3' -> "DEF", '4' -> "GHI", '5' -> "JKL",
    '6' -> "MNO", '7' -> "PQRS", '8' -> "TUV", '9' -> "WXYZ")

// Exiting paste mode, now interpreting.

mnemonics: scala.collection.immutable.Map[Char,java.lang.String] = Map(8 -> TUV, 4 -> GHI, 9 -> WXYZ, 5 -> JKL, 6 -> MNO, 2 -> ABC, 7 -> PQRS, 3 -> DEF)

scala> :paste
// Entering paste mode (ctrl-D to finish)

val charCode: Map[Char, Char] =
    for ((digit, str) <- mnemonics; letter <- str) yield (letter -> digit)

// Exiting paste mode, now interpreting.

charCode: Map[Char,Char] = Map(E -> 3, X -> 9, N -> 6, T -> 8, Y -> 9, J -> 5, U -> 8, F -> 3, A -> 2, M -> 6, I -> 4, G -> 4, V -> 8, Q -> 7, L -> 5, B -> 2, P -> 7, C -> 2, H -> 4, W -> 9, K -> 5, R -> 7, O -> 6, D -> 3, Z -> 9, S -> 7)

def wordCode(word: String): String = word.toUpperCase map charCode

wordCode: (word: String)String

Я также попытался определить весь класс в REPL.

defined class Coder

scala> val words4 = List("hi", "Hello world", "$t@r")
words4: List[java.lang.String] = List(hi, Hello world, $t@r)

scala> var listPhoneNumber = new Coder(words4)
java.util.NoSuchElementException: key not found:  
    at scala.collection.MapLike$class.default(MapLike.scala:224)

scala> val words5 = List("hi","hello","ciao")
words5: List[java.lang.String] = List(hi, hello, ciao)

scala> var listPhoneNumber = new Coder(words5)
listPhoneNumber: Coder = Coder@119c2af

Вот весь код, который я использую:

class Coder(words: List[String]) {



  private val mnemonics = Map(

    '2' -> "ABC", '3' -> "DEF", '4' -> "GHI", '5' -> "JKL",

    '6' -> "MNO", '7' -> "PQRS", '8' -> "TUV", '9' -> "WXYZ")



  /** Invert the mnemonics map to give a map from chars 'A' ... 'Z' to '2' ... '9'*/

  private val charCode: Map[Char, Char] =

    for ((digit, str) <- mnemonics; letter <- str) yield (letter -> digit)



  /** Maps a word to the digit string it can represent */

  private def wordCode(word: String): String = word.toUpperCase map charCode



  /** A map from digit strings to the words that represent them, 

    * e,g. 5282 -> Set(Java, Kata, Lava, ...) */

  private val wordsForNum: Map[String, List[String]] = 

    (words groupBy wordCode) withDefaultValue List()



  /** Return all ways to encode a number as a list of words */

  def encode(number: String): Set[List[String]] =

    if (number.isEmpty)

      Set(List())

    else {

      for {

        splitPoint <- 1 to number.length

        word <- wordsForNum(number take splitPoint)

        rest <- encode(number drop splitPoint)

      } yield word :: rest

    }.toSet



  /** Maps a number to a list of all word phrases that can represent it */

  def translate(number: String): Set[String] = encode(number) map (_ mkString " ")

}

Edit2 - меньший фрагмент:

  val mnemonics = Map(
    '2' -> "ABC", '3' -> "DEF", '4' -> "GHI", '5' -> "JKL",
    '6' -> "MNO", '7' -> "PQRS", '8' -> "TUV", '9' -> "WXYZ")

  val charCode: Map[Char, Char] =
    for ((digit, str) <- mnemonics; letter <- str) yield (letter -> digit)

  def wordCode(word: String): String = word.toUpperCase map charCode

  val words = List("Hello", "1111") // doesn't work

  (words groupBy wordCode) withDefaultValue List()

  val words2 = List("Hello", "Odersky")

  (words2 groupBy wordCode) withDefaultValue List() //works

Я только что заметил, что метод / функция wordCode не будет принимать никаких символов, которых нет на карте.Если это так, значит ли это, что withDefaultValue бесполезен?Если так, то я думаю, что представленный код имеет небольшой недостаток?

1 Ответ

3 голосов
/ 23 августа 2011

Ваше последнее уведомление верно: как только персонажа не окажется на карте, часть map charCode функции wordCode завершится ошибкой, что приведет к исключению.Таким образом, вы не достигнете wihDefaultPart.

. Если есть изъян или нет, зависит от поведения, которого вы ожидаете, когда персонажа нет на карте.

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...