Diskusia – 6. diel - Hra JellyBox v MonoGame - Sugar a Menu

Chlape, ta kontrola kolize je nějaká divná. Určitě nikdy nepoužívej for cykly místo foreache, když nepotřebuješ index. GetType lze v tomto případě nahradit operátorem is. Co mi není jasné je ten if.. else uvnitř. V else je ta samá podmínka jako v původní if, jen bez pravé části. Kód:

Vymazání se tedy provede vždy, pokud platí:

(boxes[i].GetType() != typeof(ToxicBox)

Trochu jsem to refaktoroval, ale protože nevím co to má přesně dělat, je dost možné, že jsem to udělal blbě, zkus na to ještě kouknout:

foreach (Box box in boxes)
{
        foreach (Box okolni in boxes)
        {
                if ((box != okolni) &&
                        (box.GetRectangle().Intersects(okolni.GetRectangle())) &&
                        (!(box is SugarBox || okolni is SugarBox)))
                {
                        okolni.Delete = (!okolni is ToxicBox);
                }
        }
}

Dobrá, díky za radu, je pravda že na zrovna na tento kus zasahování želé do sebe jsem od doby vytvoření vůbec nekoukl, takže na to mrknu.

Ono by bylo nejlepší dát to do metody kolize pro každý objekt. Dávat tolik cyků do sebe a ifovat typy vždycky signalizuje, že v návrhu je něco špatně.

Tohle by vypadalo mnohem lépe:

foreach (Box box in boxes)
   box.kontrolaKolize();

Tak jsem to přepsal prakticky skoro přesně dle kódu co jsi tu poznamenal a funguje to, for jsem občas používal místo foreache, protože jsem u vícekrát zažil, že když jsem přes to dělal kolizi a mazal, tak mě jiný foreach vyhodil error, že došlo ke změně kolekce.

Ano to máš naprostou pravdu

Kolekci nesmíš při iteraci nikdy modifikovat. Foreach si to hlídá, for tě to nechá udělat, ale zaděláváš si tím na problémy, jelikož se změní indexy. Další důvod ho nepoužívat.

Kdy by tím pádem bylo ideální provádět reálné mazání, abych se vyhnul tomuto erroru?

Mazání máš ve své aplikaci udělané přes Vlastnost Delete, předpokládal jsem, že je to kvůli tomu. Trik je v tom smazat položky až po iteraci. To uděláš samozřejmě tak, že položku označíš jako ke smazání. Buď nějkou její vlastností enbo možná čistěji tak, že je dáš do nějakého listu keSmazani. Ten poté proiteruješ a z prvního listu vymažeš tyto položky.

Aha mazané Ano ta vlastnost Delete je tam přesně kvůli tomu, jen jsem takto přesně nechápal ten postup s iterací.

Ahoj, bude tady někdy zbytek JellyBoxu. Nebo máš to někde hotové k nahlédnutí?