search

eloipuertas / ES2013Dijous

Videojoc del Grup de Dijous d'Enginyeria del Software del Grau en Enginyeria Informàtica de la Universitat de Barcelona
http://eloipuertas.github.io/ES2013Dijous/
Other
6 stars 0 forks source link

Refactor Actor/NPC/Player #451

Closed LynosSorien closed 10 years ago

LynosSorien commented 10 years ago

US REFACTOR Refactor Actor/NPC/Player

@LynosSorien - Programmer in devel-issue-451-v2

@Neoares - Programmer in devel-issue-451-v2

@armi7 - Programmer of Scenario and make/put effects to Player/NPC (Actor.cs). @Sc-pyK - NPC actions.

Estimated Time: 15 Hours

joang commented 10 years ago

@armi7 i @LynosSorien El merge s'ha fet correctament amb #452 . Únicament hi ha hagut conflictes en:

both modified: Assets/Resources/ChickenPrefabs/robo_droid.prefab both modified: Assets/Scripts/PlayerController.cs

Com un d'ells és un binari he mantingut els que hi havien a devel-characters (compte que en @Neoares ha canviat .cs). O sigui que @armi7 pots si vols incorporar lo teu d'aquests conflictes

LynosSorien commented 10 years ago

Perfecte, moltes gràcies. Aquesta issue la utilitzarem per fer el model experimental de l'actor (tant per NPCs com per Jugador). Si tot va be i al final hi ha el vistiplau dels testers s'integrarà a devel.

LynosSorien commented 10 years ago

@Sc-pyK Una pregunta, els NPCs tenen implementat el mètode per a utilitzar les granades?, i una altre, les punxes del terreny no fan mal als NPCs oi? (tal com està ara el codi)

LynosSorien commented 10 years ago

@Sc-pyK Crec que tampoc pot rebre danys de cap tipus del terreny, i no pot agafar objectes com la vida i l'escut oi? (si m'equivoco si em puguessis dir on ho fa t'ho agrairia).

ghost commented 10 years ago

Gràcies @joang @LynosSorien No ho fan els NPCs.. jo m'encarregaré de que puguin agafar les municions automàticament com em va comentar @qfornaguera. Pel tema de punxes, guillotines i foc vam pensar que no rebessin dany per tal de que no es poguessin morir si no fos per efecte de lluita directa, pero es pot canviar si vols. Pel tema de l'escut i la vida caldria col·locar les corresponent col·lisions

LynosSorien commented 10 years ago

@armi7 Ho dic perque com que estic refactoritzant, si pujo totes aquestes coses de les col·lisions com agafar bales, vida, escut i fer-se mal amb les punxes a Actor crec que ja ho tindrem solucionat.

LynosSorien commented 10 years ago

En l'anterior commit també he afegit al lógica per a recargar bales dels cartutxos d'escopeta. 10 bales et donen, tot i que estava pensant de fer-ho random (per exemple, d'1 a 30). Què penseu?

LynosSorien commented 10 years ago

@Sc-pyK Els NPCs (la IA) ara pot rebre dany de terreny a més de poder utilitzar l'arma secundaria.

LynosSorien commented 10 years ago

Done, ready for test!

joang commented 10 years ago

Refactorització satisfactòria i funcionament correcte des de devel :+1: