2012-10-24 19 views
11

Chcę napisać funkcję, która łączy listę.Scala spłaszczyć Listę

object Flat { 
    def flatten[T](list: List[T]): List[T] = list match { 
    case Nil => Nil 
    case head :: Nil => List(head) 
    case head :: tail => (head match { 
     case l: List[T] => flatten(l) 
     case i => List(i) 
    }) ::: flatten(tail) 
    } 
} 

object Main { 
    def main(args: Array[String]) = { 
    println(Flat.flatten(List(List(1, 1), 2, List(3, List(5, 8))))) 
    } 
} 

Nie wiem, dlaczego to nie działa, zwraca List(1, 1, 2, List(3, List(5, 8))) ale powinno być List(1, 1, 2, 3, 5, 8).

Czy możesz dać mi wskazówkę?

+0

Jest to zabawa jako ćwiczenie. Dla prawdziwego kodu istnieje oczywiście metoda 'spłaszczania' na liście' List'. – AshleyF

+0

To nie zadziałałoby w tym przypadku. Oto lista 'List [Any]', więc będziesz musiał zdefiniować niejawną konwersję z Any => TraversableOnce [_], aby wywołać spłaszczanie. Musi być możliwe, ale wątpię, by było prostsze niż ta funkcja. – rjsvaljean

+0

Spójrz na błędy i ostrzeżenia kompilatora: podadzą one pewne ważne wskazówki –

Odpowiedz

9

Przez kasowania linii 4

case head :: Nil => List(head) 

Dostaniesz właściwą odpowiedź.

Pomyśl o przypadku testowego

List(List(List(1))) 

z linią 4 ostatni element na liście nie będą przetwarzane

26

Nie trzeba do gniazda swoje wypowiedzi meczu. Zamiast tego dopasuj na miejscu tak:

def flatten(xs: List[Any]): List[Any] = xs match { 
    case Nil => Nil 
    case (head: List[_]) :: tail => flatten(head) ++ flatten(tail) 
    case head :: tail => head :: flatten(tail) 
    } 
+0

bardzo eleganckie rozwiązanie, dziękuję za udostępnienie! – Ashalynd

15

Mój, odpowiednik rozwiązania SDJMcHattie's.

def flatten(xs: List[Any]): List[Any] = xs match { 
    case List() => List() 
    case (y :: ys) :: yss => flatten(y :: ys) ::: flatten(yss) 
    case y :: ys => y :: flatten(ys) 
    } 
+3

Najbardziej eleganckie rozwiązanie moim zdaniem – Vic

+0

Jak "(x :: ys) ::: yss" oznacza "jeśli jest to lista elementów, po której następuje inna lista"? (ponieważ '()' oznacza krotkę AFAIK nie na liście) –

1
def flatten(ls: List[Any]): List[Any] = ls flatMap { 
    case ms: List[_] => flatten(ms) 
    case e => List(e) 
    }