[sucharixx]

Mam taki kod w #kotlin da się to jakoś ładniej/mądrzej zrobić?

tables.forEach { table ->
 table.roomName = rooms.filter { table.roomId == it.id }[0].name
 }

#programowanie #androiddev

[Saly]

@sucharixx: tables.map {it.copy(roomName = rooms.filter { table.roomId == it.id }[0].name)}

[Saly]

@sucharixx: ale moze nie dzialac na 100% bo dawno nie uzywalem kotlina

[sucharixx]

@Saly: działa. Dzięki

[iiipablo]

@sucharixx: first zamiast filter i branie pierwszego elementu i utworzył bym metodę rozszerzającą na roomach zwracającą name po id

[Krolik]

@Saly: ale przecież to robi co innego. Kopiuje całą kolekcję, natomiast kod OP modyfikuje elementy w kolekcji. Przez takie programowanie klienci muszą mieć telefon z 16 GB RAMu aby pogodę wyświetlić.

Niemniej oba są słabe, bo jak nie będzie elementu o wskazanym id to program poleci w kosmos.