| Issue 23: | Přesunutí metod z UnitManager do Map | |
| Back to list |
Napsal jsem do UnitManager dvě metody, které patří spíše do Map. Mám je tam přesunout sám nebo to chceš udělat ty? klidně to udělám, jen abych ti pořád nešahal do zdrojáků :) Jsou oznacene v UnitManager.h|cpp "// TO MAP:" a jsou to UnitManager::findFieldForUnit UnitManager::addBlackMinimapChange (plus 2 členské proměnné)
Feb 16, 2007
BlackMapPresunuta, findFieldForUnit at rozhodne Katka.
Owner:
kaatyaa
Feb 16, 2007
OKI, ale uplne to nefunguje a hlavně myslím, že je třeba refactoring ještě více dotáhnout. Map patří Katce, Honza do toho píše a já bych do toho potřeboval psát. Jak by to mělo dle mě nakonec fungovat s blackmapu a vykreslováním: - Map::render cachovat jak je. - jedno pole m_anBlackMapChanges pro změny viditelnosti a v metoda Map::addBlackMapChange(x,y,nChange) resp. Map::changeFieldVisibility(Field*, nChange) - tj. podobná changeAreaVisibility - aktualizaci si vyřeší Map uvnitř - jak cache tak minimap Co je ale důležitější je, kdo to má udělat. Měla by to udělat Katka, ale je to hromada kódu, který musí přečíst a napsal jsem ho já s Honzou. Přesto bych byl nejradši kdyby to udělala Katka - je to její soubor. Prosím než začnete horlivě psát, počkejte až se všichni tři domluvíme jak dál. P.S. Mám po zkouskách - mám čas. A mám ještě pár dalších změn, které bych rád s Map udělal, případně i naimplementoval tu hrubou mapu (pro lepší Unit-Explore).
Feb 16, 2007
ahoj, co se clovek vsechno nedozvi kdyz par dni nekouka na issues... presun metody findFieldForUnit do Map neni problem, presunu to. k tomu cachovani mapy a minimapy - psala jsem Honzovi na ICQ, on to uz naimplementoval, az na problem kdyz se zaroven scrolluje a pohybuje se jednotka. To zkusim opravit.
Feb 17, 2007
takze metodu findFieldForUnit jsem presunula pod Map, dale jsem pridelala Map::changeFieldVisibility(Field *, int), ktera zjednodusi Jakubovi praci v Unit::startMAmove(). Tu chybu se scrollovanim a pohybem jednotek jsem zda se opravila.. Aspon me uz se ted nepovedlo nic podezreleho scrollovanim vytvorit:-) Zbyva doresit, jak (a jestli vubec) dat dohromady ty dve pole m_anBlackMapChanges. Bohuzel moc nevim jak, jedno pole se zpracovava v Map::doBlackMapChanges a druhe se zpracovava v Panel::updateMapLayer. Spis bych nechala dve pole a zkusila kouknout jestli by to neslo nekde optimalizovat. Napr. volani renderToTarget() v changeAreaVisibility(), tady by mozna slo neprekreslovat to cele ale jen cast. Zkusim to vecer.
Feb 17, 2007
Vypadá to dobře. S tou optimalizací bych to nepřeháněl. Radeji nech jednodussi pomalejsi kod :)
Feb 18, 2007
(No comment was entered for this change.)
Status:
Fixed
|
Owner: klingonek