Home World Forum: Stars! Nova - Development

Home » Stars! Clones, Extensions, Modding » Stars! Nova - Development » FleetIntel question

Show: Today's Messages :: Show Polls :: Message Navigator
E-mail to friend

Re: FleetIntel question

Thu, 14 July 2011 05:11

Aeglos

Chief Warrant Officer 1

Stars! Nova developer

Messages: 142
Registered: May 2011
Location: Chile

I just commited r713 with these changes (and some others related as well).

It's just on Stars & Star Reports, so we can still do something else with fleets or change how it works.

I've done the following changes:

Item now implements IComparable instead of having a separate SortableItem inside StarMap (was a fixme pri. 5).
StarIntel now inherits from Item, but besides that is unrelated to Star OO wise.
Star has a method GenerateReport() to return a StarIntel object according to ScanLevel (renamed IntelLevel).
StarMap uses only StarReports now.
Planet Summary uses StarReports and pulls a couple of things from OwnedStars when needed (Minerals on surface bars mostly).
Planet Detail uses OwnedStars only.
Moved Star Hab calculations to Race, so that the AI can use them with either stars or reports for colonization. Kinda makes more sense that a race checks hab on a star, than a star for it's race anyway.

Again, I found that the 'has a' approach was equally cumbersome as the 'is a' one, since we would have had to extract a duplicated and modified StarIntel from Star to select proper fields according to ScanLevel, so it was kinda redundant. Having them as separate things feels a bit cleaner and simpler after all the redesign turmoil.

EDIT: Fixed some crashes by making Fleet.InOrbit accept an Item instead of a Star; interested parties can see if it's an ItemType Star or StarIntel.

[Updated on: Thu, 14 July 2011 07:25]

Report message to a moderator

[Message index]

		FleetIntel question By: Musmuris on Tue, 05 July 2011 17:45
		Re: FleetIntel question By: Aeglos on Tue, 05 July 2011 22:00
		Re: FleetIntel question By: Daniel on Wed, 06 July 2011 18:52
		Re: FleetIntel question By: Aeglos on Wed, 06 July 2011 22:06
		Re: FleetIntel question By: Daniel on Thu, 07 July 2011 05:03
		Re: FleetIntel question By: Musmuris on Wed, 06 July 2011 14:57
		Re: FleetIntel question By: Aeglos on Wed, 06 July 2011 17:01
		Re: FleetIntel question By: Musmuris on Wed, 06 July 2011 17:25
		Re: FleetIntel question By: Aeglos on Wed, 06 July 2011 21:05
		Re: FleetIntel question By: Aeglos on Thu, 07 July 2011 05:40
		Re: FleetIntel question By: Musmuris on Thu, 07 July 2011 05:58
		Re: FleetIntel question By: Daniel on Sat, 09 July 2011 20:22
		Re: FleetIntel question By: Aeglos on Sun, 10 July 2011 16:53
		Re: FleetIntel question By: Daniel on Mon, 11 July 2011 03:15
		Re: FleetIntel question By: Aeglos on Sat, 09 July 2011 20:38
		Re: FleetIntel question By: Daniel on Sat, 09 July 2011 23:50
		Re: FleetIntel question By: Aeglos on Mon, 11 July 2011 04:11
		Re: FleetIntel question By: Musmuris on Mon, 11 July 2011 04:49
		Re: FleetIntel question By: Musmuris on Mon, 11 July 2011 04:55
		Re: FleetIntel question By: Daniel on Mon, 11 July 2011 07:19
		Re: FleetIntel question By: Aeglos on Mon, 11 July 2011 14:44
		Re: FleetIntel question By: Aeglos on Wed, 13 July 2011 21:35
		Re: FleetIntel question By: Daniel on Thu, 14 July 2011 04:53
		Re: FleetIntel question By: Aeglos on Thu, 14 July 2011 05:11
		Re: FleetIntel question By: Musmuris on Thu, 14 July 2011 17:12
		Re: FleetIntel question By: Aeglos on Thu, 14 July 2011 17:46
		Re: FleetIntel question By: Aeglos on Thu, 21 July 2011 23:59
		Re: FleetIntel question By: Musmuris on Fri, 22 July 2011 13:49
		Re: FleetIntel question By: Musmuris on Fri, 22 July 2011 14:51
		Re: FleetIntel question By: Aeglos on Fri, 22 July 2011 16:45
		Re: FleetIntel question By: Musmuris on Fri, 22 July 2011 16:54
		Re: FleetIntel question By: Daniel on Fri, 22 July 2011 17:52

Previous Topic:	Bugs/Other Trackers
Next Topic:	Anyone still around?

Goto Forum:

-=] Back to Top [=-

Current Time: Sat May 04 03:34:48 EDT 2024