• Najnowsze pytania
  • Bez odpowiedzi
  • Zadaj pytanie
  • Kategorie
  • Tagi
  • Zdobyte punkty
  • Ekipa ninja
  • IRC
  • FAQ
  • Regulamin
  • Książki warte uwagi

Problem z prostą funkcją w Kotlinie. Nie wiem gdzie popełniam błąd

Object Storage Arubacloud
+1 głos
121 wizyt
pytanie zadane 21 sierpnia 2020 w Inne języki przez matir85 Bywalec (2,410 p.)

Witam.

Aż jest mi wstyd ale nie mogę zdefiniować co robię źle, że nie mogę uzyskać pożądanego efektu. Zadaniem funkcji jest pobranie prostej Listy a następnie wyświetlenie jej jako ładnego menu. Moja funkcja to:

fun menu(menuList: List<String>){
    menuList.forEach { menu ->
        val (type, name, price) = menu.split(',')
        println("$name${repeat(40 - name.length){ print("*")}}$price")
    }

}

Problem polega na tym że zamiast wyświetlania:

Nazwa********************Cena

Wyświetla:

********************Nazwakotlin.UnitCena

1 odpowiedź

+2 głosów
odpowiedź 21 sierpnia 2020 przez Milesq Nałogowiec (32,020 p.)
wybrane 21 sierpnia 2020 przez matir85
 
Najlepsza

Linię 4 mógłbyś rozbić na kilka lini, tak, żeby wyjąć logikę generowania gwiazdek z print'a:

fun menu(menuList: List<String>){
    menuList.forEach { menu ->
        val (type, name, price) = menu.split(',')
        val stars = repeat(40 - name.length) { print("*") }
        println("$name$stars$price")
    }
 
}

w tym momencie nadal będzie się wyświetlać to samo. Dlaczego tak się dzieje?

W momencie wykonywania się lini z `println` najpierw ustalana jest wartość argument(ów) funkcji `println`, dopiero potem jest ona wywoływana i wyświetla się to co zostanie do niej przekazane.

Czyli funkcja repeat wywołuje się najpierw. Dlatego też gwiazdki wyświetlają się jako pierwsze

jeśli spojrzysz w dokumentację funkcji repeat

https://kotlinlang.org/api/latest/jvm/stdlib/kotlin/repeat.html

to zobaczysz, że ta funkcja wywołuje posłany do niej callback n razy i zwraca Unit, więc kiedy wywołujesz funkcję repeat, to funkcja wyświetlająca gwiazki wykonuje się te n, razy a potem funkcja repeat zwraca Unit

wynik więc jest taki:

    menuList.forEach { menu ->
        val (type, name, price) = menu.split(',')
        println("$name

      // kotlin zna wartość zmiennej price i name, więc wykonuje się następna lini
      ${repeat(40 - name.length){ print("*")}} // o ta linia, i w tym momencie, przed wykonaniem się println'a
       // repeat wywołuje ten callback { print("*") }  40 - name.length razy
       
      // następnie funkcja repeat zwraca kotlin.Unit

       $price
        // teraz kotlin łączy stringa w całość, i  go przekazuje do println
       ")
    }

 

komentarz 21 sierpnia 2020 przez Milesq Nałogowiec (32,020 p.)

Tak musiałaby wyglądać funkcja, żeby uzyskać efekt o który ci chodzi

fun menu(menuList: List<String>){
    menuList.forEach { menu ->
        val (type, name, price) = menu.split(',')
        val stars = Array(40 - name.length) { "*" }.joinToString("")
        // Tworzymy nową tablicę, o wielkości 40 - name.length, którą wypełniamy gwiazdkami i robimy join
        println("$name$stars$price")
    }
}

 

1
komentarz 21 sierpnia 2020 przez matir85 Bywalec (2,410 p.)
100 krotne dzięki za wyjaśnienie problemu.

Podobne pytania

+1 głos
2 odpowiedzi 366 wizyt
+1 głos
4 odpowiedzi 295 wizyt
pytanie zadane 28 października 2017 w C i C++ przez Dudekdude Początkujący (320 p.)
0 głosów
2 odpowiedzi 203 wizyt
pytanie zadane 13 lipca 2018 w C i C++ przez mar123cel Nowicjusz (140 p.)

92,757 zapytań

141,679 odpowiedzi

320,437 komentarzy

62,101 pasjonatów

Motyw:

Akcja Pajacyk

Pajacyk od wielu lat dożywia dzieci. Pomóż klikając w zielony brzuszek na stronie. Dziękujemy! ♡

Oto polecana książka warta uwagi.
Pełną listę książek znajdziesz tutaj.

Akademia Sekuraka

Niedawno wystartował dodruk tej świetnej, rozchwytywanej książki (około 940 stron). Mamy dla Was kod: pasja (wpiszcie go w koszyku), dzięki któremu otrzymujemy 10% zniżki - dziękujemy zaprzyjaźnionej ekipie Sekuraka za taki bonus dla Pasjonatów! Książka to pierwszy tom z serii o ITsec, który łagodnie wprowadzi w świat bezpieczeństwa IT każdą osobę - warto, polecamy!

...