[Mechu2115]

Chcę zrobić losowanie drużyn lecz nie wiem jak zrobić żeby wylosowane osoby dodało do różnych drużyn, za każdym razem dodaje mi do ostatniej stworzonej drużyny. Kod pewnie pełen błędów więc proszę o sugestie co poprawić.
kod

#javascript #naukaprogramowania #programowanie

[Zavis]

kod

@Mechu2115: div do którego appendujesz osoby ma referencję do ostatniej grupy

[Zavis]

@Mechu2115: to byłoby czytelniejsze jakbyś najpierw zbudował sobie listę grup z wylosowanymi graczami, a dopiero potem to renderował

[jablkawmasle]

@Zavis: e sorry - ale skad sie nauczyles programowac? :x

[Zavis]

@jablkawmasle: z internetu

[jablkawmasle]

@Zavis: no jusz dobrze, dobrze - wiadomo, ze stackoverflow jest super, ale ja sie pytam o logike - skad wiesz, ze lepiej cos bedzie wygladac tak a nie inaczej? (Nie odpisuj jak Cie męczę :))

[Zavis]

@jablkawmasle: Bo to się po prostu źle czyta kiedy na zmianę coś wyliczasz i jednocześnie próbujesz to wyświetlić, a poza tym taki kod jest niezbyt "reużywalny" - np. możesz zechcieć sprawdzić jedynie aktualny stan grup bez ponownego renderowania.

Ogólnie dobrą i powszechnie przyjętą praktyką jest oddzielanie warstwy logicznej od warstwy interfejsu co pokrywa się z np. jednym z założeń SOLID (https://pl.wikipedia.org/wiki/SOLID_(programowanie_obiektowe)). Funkcja nie powinna być jednocześnie odpowiedzialna za wylosowanie graczy

[jablkawmasle]

@Zavis: wow! Bardzo, bardzo, bardzo Ci dziękuję za tak obszerną odpowiedź (づ•﹏•)づ