New optionnal feature: storage in hangars

[Juju Dredd]

Hi,

Being used with the original game, taking in account items on Skyrangers in the base storage capacity somewhat hurt me. I believe to compensate, we should add some storage in the hangar, at least as an optional feature. A value such as 10 would be nice, because 20 soldiers each with a riffle, a rifle clip, an electro-flare and a grenade would need this storage. It sounds to me logical some equipment can be stored in the hangar, or some Stingray or Avalanche missiles.

But I'm not sure what is the better way to do that. I have tried to modify the RuleBaseFacility::load() function adding the following:

Code: [Select]

_storage += _crafts * Options::getInt("storageInHangar");

This seems to work perfectly, but am I doing it the good way?

[Align]

Much easier to just change it with ruleset editing.

Code: [Select]

facilities:
  - type: STR_HANGAR
    spriteShape: 9
    spriteFacility: 9
    size: 2
    buildCost: 200000
    buildTime: 25
    monthlyCost: 25000
    crafts: 1
    mapName: XBASE_16
    storage: 10

[Juju Dredd]

Much easier to just change it with ruleset editing.

I understand, but this way it will never be an option the user could change in the UI.

[Juju Dredd]

I have thought of another way to do this which seem more logical to me, modifying Base::getAvailableStores() function:

Code: [Select]

total += (*i)->getRules()->getStorage() + Options::getInt("storageInHangar") * (*i)->getRules()->getCrafts();


[Amunak]

Oh, and I thought that you want to be able to have hangar as a huge storage when there's no aircraft assigned. That'd be a good idea too I think.

[Align]

I understand, but this way it will never be an option the user could change in the UI.

Ah, as one of the Advanced options?

[Juju Dredd]

Oh, and I thought that you want to be able to have hangar as a huge storage when there's no aircraft assigned. That'd be a good idea too I think.

This would be also a good idea, but way more complex to implement. And if you go this way you could tell any unused building could be used for storage, and implementing that would be really painful.

Anyway, a hangar is a sort of big square, right, and none of the ships can occupy the whole place, so even an occupied hangar should be able to store some boxes with various supplies, such as missiles for interceptors or laser weapons in case a skyranger has to take off in an emergency.

Ah, as one of the Advanced options?

Voila, this is exactly what I want to do.

I'm going to try to make a pool request with this very simple development, this will be a good exercise for me since as I have never used Git.

[SupSuper]

Feel free to use it as an exercise, but don't expect to see it merged, we really don't need more one-man options, specially if it can be done much more simply with a ruleset.

[Amunak]

This would be also a good idea, but way more complex to implement. And if you go this way you could tell any unused building could be used for storage, and implementing that would be really painful.

Well...But hangar is, just as you said, an empty space. Labs, prison, workshop, soldier's quarters... Those aren't really empty. Also there are often times when you build a hangar and then don't have enough money for a craft (or you lose one). All other places are usually occupied, and even if they aren't that's just for few days.

Anyway, a hangar is a sort of big square, right, and none of the ships can occupy the whole place, so even an occupied hangar should be able to store some boxes with various supplies, such as missiles for interceptors or laser weapons in case a skyranger has to take off in an emergency.

That's very true. Maybe by the same logic there could be like 5 storage unit in every other building (you know workshops, labs, ... contain their items near) and like 10 for living quarters. I think it'd unbalance the game too much, it'd be just some extra realism.

[Juju Dredd]

@SupSuperMaster

I have tried to send a pull request, I am not sure I have done it the good way.
I have also created a serial patch, please tell me if I have done it the good way.

[Juju Dredd]

I don't see my push request in the list, I'm pretty sure I was not able to create it correctly...

[SupSuper]

Seems to be working now.
Anyways, rulesets will eventually get a GUI, I didn't hide them in the "options.cfg" out of spite, they're just an incomplete implementation that the community ran away with.

[Juju Dredd]

Anyways, rulesets will eventually get a GUI, I didn't hide them in the "options.cfg" out of spite, they're just an incomplete implementation that the community ran away with.

If you do this, I hope the rulesets will be duplicated in user directory. Data in the installation directory should not be writable for lambda users. I hope you agree with me.