• 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

Aruba Cloud - Virtual Private Server VPS
+1 głos
162 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 546 wizyt
+1 głos
4 odpowiedzi 445 wizyt
pytanie zadane 28 października 2017 w C i C++ przez Dudekdude Początkujący (320 p.)
0 głosów
2 odpowiedzi 255 wizyt
pytanie zadane 13 lipca 2018 w C i C++ przez mar123cel Nowicjusz (140 p.)

93,332 zapytań

142,326 odpowiedzi

322,405 komentarzy

62,669 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

Wprowadzenie do ITsec, tom 1 Wprowadzenie do ITsec, tom 2

Można już zamawiać dwa tomy książek o ITsec pt. "Wprowadzenie do bezpieczeństwa IT" - mamy dla Was kod: pasja (użyjcie go w koszyku), dzięki któremu uzyskamy aż 15% zniżki! Dziękujemy ekipie Sekuraka za fajny rabat dla naszej Społeczności!

...