My favorites | Sign in
Project Home Downloads Wiki Issues Source
READ-ONLY: This project has been archived. For more information see this post.
Search
for
  Advanced search   Search tips   Subscriptions
Issue 23: Přesunutí metod z UnitManager do Map
  Back to list
Status:  Fixed
Owner:  kaat...@gmail.com
Closed:  Feb 2007


 
Project Member Reported by jakub.ko...@gmail.com, Feb 13, 2007
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
Project Member #1 klingo...@gmail.com
(No comment was entered for this change.)
Status: Accepted
Owner: klingonek
Feb 16, 2007
Project Member #2 klingo...@gmail.com
BlackMapPresunuta, findFieldForUnit at rozhodne Katka.
Owner: kaatyaa
Feb 16, 2007
Project Member #3 jakub.ko...@gmail.com
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
Project Member #4 kaat...@gmail.com
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
Project Member #5 kaat...@gmail.com
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
Project Member #6 jakub.ko...@gmail.com
Vypadá to dobře.
S tou optimalizací bych to nepřeháněl.
Radeji nech jednodussi pomalejsi kod :)
Feb 18, 2007
Project Member #7 jakub.ko...@gmail.com
(No comment was entered for this change.)
Status: Fixed

Powered by Google Project Hosting