2011-08-23 12 views
0

Dr. Martin Odersky mentioned over at Skillmatters님께 서 전화 번호부 연상 기호에 대한 코드 리뷰를하고 있습니다. 여기 withDefaultValue가 작동하지 않는 것 같습니다.

그가 무슨의 조각이다 : 나는 행동 withDefaultValue 방식을 볼 수 기능과 변수로 모두를 선언하려고했습니다

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() 

그리고 이것은 내가 가진 것입니다 :

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을 사용하고 있습니다.

감사합니다.

편집 :

나는 내가 발을 생성 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에 정의하려고했습니다.

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 것 같다 : - 작은 코드 조각

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가 :

defined class Coder 

scala> val words4 = List("hi", "Hello world", "[email protected]") 
words4: List[java.lang.String] = List(hi, Hello world, [email protected]) 

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 = [email protected] 

여기 내가 사용 전체 코드입니다 지도에없는 문자는 가져 가지 않습니다. 이 경우에 TheDefaultValue가 쓸모 없다는 것을 의미합니까? 그렇다면 제시된 코드에 약간의 결함이 있다고 생각하십니까?

+1

"전체 코드"를 REPL에': paste' 명령으로 붙여 넣으면 오류없이 컴파일됩니다. 2.9.1.RC3을 사용하고 있습니다. 예기치 않게 작동하는 작은 예를 생각해 낼 수 있습니까? –

+0

작은 스 니펫을 추가했습니다. 감사합니다. – mythicalprogrammer

+1

그 코드에는 버그가있었습니다. OSCON 2011에서 발표 한 코드가 수정되었습니다. –

답변

3

마지막주의가 정확합니다. 문자가지도에 없으면 wordCode 부분의 map charCode 부분이 실패하고 예외가 발생합니다. 따라서 wihDefaultPart에 도달하지 않습니다.

결함이있는 경우 문자가지도에 없을 때 예상되는 동작에 따라 다릅니다.

관련 문제