House buying and selling...

OMGOMGOMG

UMBWGMG (Unidentified Human Being.)
OK, you all should know that I am creating a map, from all the posts I've done, and need help with this odd thing.

I buy the house and it buys and then sell the house it sells and try to buy the house again and it still doesn't work
TRIGGERS:
Trigger:
  • House Sell
    • Events
      • Unit - A unit Sells an item (from shop)
    • Conditions
    • Actions
      • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
        • If - Conditions
          • (Item-type of (Sold Item)) Equal to Sell Medium House
        • Then - Actions
          • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
            • If - Conditions
              • (Selling unit) Equal to HouseOptions[1]
            • Then - Actions
              • Set HouseSellPoint = (Position of (Selling unit))
              • Unit - Remove (Selling unit) from the game
              • Unit - Create 1 House Seller (Medium) for Player 1 (Red) at HouseSellPoint facing Default building facing degrees
              • Unit - Change ownership of (Last created unit) to Neutral Passive and Change color
              • Player - Add 500 to Player 1 (Red) Current gold
              • Custom script: call RemoveLocation (udg_HouseSellPoint)
            • Else - Actions
          • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
            • If - Conditions
              • (Selling unit) Equal to HouseOptions[2]
            • Then - Actions
              • Set HouseSellPoint = (Position of (Selling unit))
              • Unit - Remove (Selling unit) from the game
              • Unit - Create 1 House Seller (Medium) for Player 1 (Red) at HouseSellPoint facing Default building facing degrees
              • Unit - Change ownership of (Last created unit) to Neutral Passive and Change color
              • Player - Add 500 to Player 1 (Red) Current gold
              • Custom script: call RemoveLocation (udg_HouseSellPoint)
            • Else - Actions
          • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
            • If - Conditions
              • (Selling unit) Equal to HouseOptions[3]
            • Then - Actions
              • Set HouseSellPoint = (Position of (Selling unit))
              • Unit - Remove (Selling unit) from the game
              • Unit - Create 1 House Seller (Medium) for Player 1 (Red) at HouseSellPoint facing Default building facing degrees
              • Unit - Change ownership of (Last created unit) to Neutral Passive and Change color
              • Player - Add 500 to Player 1 (Red) Current gold
              • Custom script: call RemoveLocation (udg_HouseSellPoint)
            • Else - Actions
        • Else - Actions
      • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
        • If - Conditions
          • (Item-type of (Sold Item)) Equal to Sell Small House
        • Then - Actions
          • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
            • If - Conditions
              • (Selling unit) Equal to HouseOptions[4]
            • Then - Actions
              • Set HouseSellPoint = (Position of (Selling unit))
              • Unit - Remove (Selling unit) from the game
              • Unit - Create 1 House Seller (Small) for Player 1 (Red) at HouseSellPoint facing Default building facing degrees
              • Unit - Change ownership of (Last created unit) to Neutral Passive and Change color
              • Player - Add 250 to Player 1 (Red) Current gold
              • Custom script: call RemoveLocation (udg_HouseSellPoint)
            • Else - Actions
          • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
            • If - Conditions
              • (Selling unit) Equal to HouseOptions[5]
            • Then - Actions
              • Set HouseSellPoint = (Position of (Selling unit))
              • Unit - Remove (Selling unit) from the game
              • Unit - Create 1 House Seller (Small) for Player 1 (Red) at HouseSellPoint facing Default building facing degrees
              • Unit - Change ownership of (Last created unit) to Neutral Passive and Change color
              • Player - Add 250 to Player 1 (Red) Current gold
              • Custom script: call RemoveLocation (udg_HouseSellPoint)
            • Else - Actions
        • Else - Actions
      • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
        • If - Conditions
          • (Item-type of (Item being manipulated)) Equal to Sell Large House
        • Then - Actions
          • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
            • If - Conditions
              • (Selling unit) Equal to HouseOptions[6]
            • Then - Actions
              • Set HouseSellPoint = (Position of (Selling unit))
              • Unit - Remove (Selling unit) from the game
              • Unit - Create 1 House Seller (Lage) for Player 1 (Red) at (Center of (Playable map area)) facing Default building facing degrees
              • Unit - Change ownership of (Last created unit) to Neutral Passive and Change color
              • Player - Add 750 to Player 1 (Red) Current gold
              • Custom script: call RemoveLocation (udg_HouseSellPoint)
            • Else - Actions
          • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
            • If - Conditions
              • (Selling unit) Equal to HouseOptions[7]
            • Then - Actions
              • Set HouseSellPoint = (Position of (Selling unit))
              • Unit - Remove (Selling unit) from the game
              • Unit - Create 1 House Seller (Lage) for Player 1 (Red) at HouseSellPoint facing Default building facing degrees
              • Unit - Change ownership of (Last created unit) to Neutral Passive and Change color
              • Player - Add 750 to Player 1 (Red) Current gold
              • Custom script: call RemoveLocation (udg_HouseSellPoint)
            • Else - Actions
        • Else - Actions
      • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
        • If - Conditions
          • (Item-type of (Sold Item)) Equal to Sell Mansion
        • Then - Actions
          • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
            • If - Conditions
              • (Selling unit) Equal to HouseOptions[8]
            • Then - Actions
              • Set HouseSellPoint = (Position of (Selling unit))
              • Unit - Remove (Selling unit) from the game
              • Unit - Create 1 House Seller (Mansion) for Player 1 (Red) at HouseSellPoint facing Default building facing degrees
              • Unit - Change ownership of (Last created unit) to Neutral Passive and Change color
              • Player - Add 1500 to Player 1 (Red) Current gold
              • Custom script: call RemoveLocation (udg_HouseSellPoint)
            • Else - Actions
        • Else - Actions
 

Ghan

Administrator - Servers are fun
Staff member
I have no idea what exactly you're trying to do.

(Selling unit) Equal to HouseOptions[1]

What is HouseOptions[1]? Are these different shops that are selling items?
What exactly happens that "doesn't work" as you said above?
 

OMGOMGOMG

UMBWGMG (Unidentified Human Being.)
HouseOptions[1] is a variable... unit variable with 50 (or something) things(forgot what they are called as I haven't been on warcraft 3 or world editor much) so yea, pretty much.

The unit buys an item which gets used and the owner of the buying unit buys the house. when the unit sells the house, the owner of the selling house revers back to the normal unit and anyone is able to buy the house again... what happens instead of that is it reverts back to the normal unit, anyone is able to buy the house and when they do buy the item, it doesn't buy the item... the trigger just simply doesn't work.
 

KJP

New Member
I'm not entirely sure about this, but it looks like the problem has to do with HouseOptions variable referencing a removed unit the 2nd time round. I suggest instead of removing the seller, instanly move the seller to somewhere that isn't used in your map when sold, then when the house is rebought move it back, or give the house seller some form of metamorphasis ability and when purchased or sold switch between house/seller form.

As a side note you can cut down on a bit of code using the conditions

Or - Any or the
(condition 1) or (contion 2) equals true

in your small/medium house item sold if statements meaning only one block to change
 
General chit-chat
Help Users
  • No one is chatting at the moment.
  • jonas jonas:
    I'm a bit worried about the future of data science, there's an influx of incompetent people hired by incompetent managers, that can't last. But I'm sure smart data scientists will always be useful and hireable. Same as smart security people.
  • jonas jonas:
    My sister in law worked in a vegan food truck, the owners were also making a killing, added several trucks and opened two restaurants over 5 years
  • Ghan Ghan:
    The CompTIA stuff is pretty much crap unfortunately. The places that ask for those low-level certs aren't likely places you want to work. For IT you really want a degree, but in the meantime you need to figure out some way to learn the skills. Cybersecurity is really hot right now so competition is fierce.
  • tom_mai78101 tom_mai78101:
    I realized I don't have anything much to say, other than "Good luck!". Compared to other places, I'm just very lucky we didn't have a lot of surges of cases coming in.
  • The Helper The Helper:
    My daughter just got back from school at Georgetown TX, outside of Austin and said there were 100 cases in the last week but she also said nobody was wearing a mask anywhere so people mask up and stay safe!
    +2
  • thewrongvine thewrongvine:
    happy thanksgiving folks!
    +1
  • Ralle Ralle:
    PLOOB
  • Varine Varine:
    @Gyah Yeah I figured they aren't particularly valuable, but they'll at the least act as a baseline for some entry level work somewhere.
  • Varine Varine:
    I now see how horribly I spelled @Ghan
  • Varine Varine:
    I had a rough morning
  • The Helper The Helper:
    that field is going to explode and though there is competition there is going to be room for all levels in the coming years imho.
  • tom_mai78101 tom_mai78101:
    Why are we competing for rough mornings? :(
  • Varine Varine:
    I don't want to compete for that, yall can have it.
  • Varine Varine:
    @The Helper I'm trying. Once I am more prepared to be figuring out school I'll probably ask for your advice
  • Varine Varine:
    Right now I'm just trying to get through the pandemic and figure out how I'm going to pay for it, and figure out what I need to get a job at Best Buy instead of a kitchen somewhere.
  • thewrongvine thewrongvine:
    oy just catching up on the discussion above. @Varine yeah I usually alternate between claritin & zyrtec, depending on whatever my body decides to reject at the time lol
  • thewrongvine thewrongvine:
    the more I use it the less effective it gets over time as my body accustoms to it so I try to not use it every day
  • S songbird:
    Glad to see some interest in the IS3 NUON reprint. :)
    +1
  • The Helper The Helper:
    Thank you Carl for making that happen that just might be the spark we need.
  • Varine Varine:
    I'm confused and amazed that NUON and TWV's news postings are the things still active.
  • Varine Varine:
    Or Tom Mai, not TWV
  • Varine Varine:
    Whatever happened to Cheshire? Or Firecat, whatever his real name was?
  • Varine Varine:
    I've been using both claritin and fexofenadine or something like that recently alternately and that's been kinda working, I don't feel like my face is going to explode at least. Eyes are still itchy as fuck though
  • tom_mai78101 tom_mai78101:
    I dunno, haven't seen Firecat in a long while now.
  • tom_mai78101 tom_mai78101:
    By the way, we have new file attachment size limits now.

    Staff online

    Members online

    Affiliates

    Hive Workshop
    Top