Market ability

[Vendro the Wind Mage Bann]

Cool!

 

[Cokemonkey11]

Can you please include the code in the hive description?

 

[Devalut]

Can you please include the code in the hive description?

Is this feature not sufficient for spell submissions?
(Genuine question as I regularly post the triggers/code and find it tedious.)

 

[LordHatchet95]

Can you please include the code in the hive description?

I didn't know that had to be done... anyways, changes made within the post.

Is this feature not sufficient for spell submissions?
(Genuine question as I regularly post the triggers/code and find it tedious.)
View attachment 392869

Didn't see it as well.

Cool!

Thanks

 

[xorkatoss]

PRICE CHECK:
The following repeats per player... if there's a better way to make it, feel free to tell me.

GUI

• MarketPricesCheck1
  • Events
    • Player - Player 1 (Red) types a chat message containing -market as An exact match
    • Player - Player 2 (Blue) types a chat message containing -market as An exact match
    • Player - Player 3 (Teal) types a chat message containing -market as An exact match
    • Player - Player 4 (Purple) types a chat message containing -market as An exact match
  • Conditions
  • Actions
    • Game - Display to (Player group((Triggering player))) for 5.00 seconds the text: ((String(lumber_const)) + ( lumber cost on purchase: + (String(lumber_cost_buy))))
    • Game - Display to (Player group((Triggering player))) for 5.00 seconds the text: ((String(lumber_const)) + ( lumber cost on sale: + (String(lumber_cost_sell))))

OR the Events can be added at map init faster/easier with a loop if you don't want to do it manually for 24 players

• -------- LOOP ALL PLAYERS --------
• For each (Integer i) from 1 to 24, do (Actions)
  • Loop - Actions
    • Trigger - Add to MarketPricesCheck1 <gen> the event (Player - (Player(i)) types a chat message containing -market as An exact match)

Other than that seems like a good system to have in a map.

 

[Wrda]

I know why you made the comment "NEVER make fluctuations bigger than minimum cost". The solution for that is to use Maths - Min and Max.
For MarketSellLumber trigger:

• Set VariableSet lumber_cost_sell = (lumber_cost_sell - lumber_cost_fluctuation)
• Set VariableSet lumber_cost_buy = (lumber_cost_buy - lumber_cost_fluctuation)

->

• Set VariableSet lumber_cost_sell = (Max(lumber_cost_min, (lumber_cost_sell - lumber_cost_fluctuation)))
• Set VariableSet lumber_cost_buy = (Max(lumber_cost_min, (lumber_cost_buy - lumber_cost_fluctuation)))

In MarketBuyLumber trigger, it's the same logic, but you use Min instead.


If lumber_cost_sell reaches the lumber_cost_min value or lumber_cost_buy reaches the lumber_cost_buy value, the player's resources are still depleted and earned. Moving the condition to the first if condition solves this, e.g.:

• MarketSellLumber
  • Events
    • Unit - A unit Begins casting an ability
  • Conditions
    • (Ability being cast) Equal to Sell Lumber
  • Actions
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • lumber_cost_sell Less than or equal to ((Owner of (Casting unit)) Current lumber)
        • lumber_cost_sell Greater than lumber_cost_min
      • Then - Actions
        • Player - Set (Owner of (Casting unit)).Current lumber to (((Owner of (Casting unit)) Current lumber) - lumber_const)
        • Player - Add lumber_cost_sell to (Owner of (Casting unit)).Current gold
        • Game - Display to (Player group((Owner of (Casting unit)))) for 4.00 seconds the text: (You sold + ((String(lumber_const)) + ( lumber by + ((String(lumber_cost_sell)) + gold.))))
        • Set VariableSet lumber_cost_sell = (lumber_cost_sell - lumber_cost_fluctuation)
        • Set VariableSet lumber_cost_buy = (lumber_cost_buy - lumber_cost_fluctuation)
      • Else - Actions
        • Game - Display to (Player group((Owner of (Casting unit)))) for 4.00 seconds the text: Not enough lumber t...

Same thing applies to MarketBuyLumber.
For the "not enough lumber" or "gold" messages, you'll need an additional condition on the else block checking player's specific resource.

Player - Add 100 to (Owner of (Casting unit)).Current lumber is being used on MarketBuyLumber instead of using lumber_const.
Player group (Owner of casting unit) leaks a player group, need to clear the leaks.
Is this command supposed to be for all players in the map? I'd suggest picking all players and adding the chat event to a new trigger (can be either an existing one like MarketPricesCheck1) which will contain the display messages with values for the triggering player. Also making the string command not hardcoded to "-market" would be nice. Note that Player Group - Player 1 (Red) in market prices check doesn't leak at all.

It's a modest system for GUI, which is nice. Needs some fixes still.

 

[Uncle]

Using the "new" natives you may be able to adjust the actual Gold/Lumber costs on the Charge Gold/Lumber ability via triggers. You could also potentially adjust the Tooltip of the ability to reflect the current market prices. This would rid the need of the chat command as everything can be handled on the Ability itself.

 

[LordHatchet95]

I may have to fix it after the second half of december... kinda ran out of time for some homeworks and an exam due to wasting it like a fing idiot ;_;

It's a modest system for GUI, which is nice. Needs some fixes still.

Kinda basic, yes... which makes it very easy to implement, actually. Also with how easy it was to make this small market system, I'm surprised there was none of this kind in the spells section

 

[Wrda]

Also with how easy it was to make this small market system, I'm surprised there was none of this kind in the spells section

Usually when something is too simple then it isn't even worth to upload it. Example: a "system" that kills units when they enter x region.
Well, I assure you that it's not your case.