aaronvark / PeerReview1819

Repo for peer review assignments for year 2 development class of 18/19
0 stars 0 forks source link

Stijn van Deijzen - Tool #78

Open StijnOnline opened 4 years ago

StijnOnline commented 4 years ago

Created Issue for KM2

aaronvark commented 4 years ago

Leuk idee! Dit soort systemen kunnen al snel heel complex worden, dus je zou in eerste instantie je kunnen beperken tot alleen direct damage door afgevuurde projectiles o.i.d. Modifiers klinken als een aardige manier om gedrag aan te passen, en klinkt als een Decorator pattern. Dan zou je bijv. een lijst van modifiers kunnen hebben die uitgevoerd worden als de ability wordt afgevuurd. Ligt er een beetje aan wat ze allemaal kunnen aanpassen (sowieso zou ik die ook op een could plaatsen, en eerst de basis maken zonder te veel na te denken over modifications).

Ga vanuit wat er allemaal mogelijk is een set aan variabelen verzinnen, en maak daar je eerste data class van. Wat voor info heb je allemaal nodig om op een specifiek moment de ability te kunnen weergeven? In mijn hoofd vormt zich dan een soort formule met snelheid, richting, startpos, endpos, etc. Dit kan ook een scriptable object zijn, zodat je abilities assets worden in je project folder.

aaronvark commented 4 years ago

Raamwerk van je code ziet er helder genoeg uit. De splitsing maakt het hebben van overzicht hier wel iets lastiger in, maar als ik het goed zie is er nu vooral de tooling om alles te doen, zonder dat er data is (de inhoudelijke actions), met uitzondering van de debug action. Nu zaak om een goede use-case te verzinnen, zo concreet mogelijk "ik wil dit effect implementeren" en kijken wat daarvoor nodig is.

StijnOnline commented 4 years ago

Look at #82 I messed those reference numbers up ._.

StijnOnline commented 4 years ago

Everthing uploaded

82

Full Project: https://github.com/StijnOnline/SpellCreator