PetSystem v0.2 [GUI] Multiplayer

[Moderator]

14:44, 16th Jun 2010
Hanky:
Your system got some leaks and it's just to simple to be a real system. Add more stuff to your system otherwise I can't approve it.

 

[Mainy]

=D newest

 

[Axarion]

Don't post only =D newest^^

@topic
maybe add the trigger and describe what it does.

 

[Shdow89]

Ok, this is one very bad system, badly scripted (if you can call that a script), leaks a lot, and pets can not be 'adopted', but purchased. No commands for the pets, you can't control it in any way, you can't revive it or heal it.

Leaks:

• Set PETOwnerLoc = (Position of PETOwner)

not removed:

• call RemoveLocation(udg_PETOwnerLoc)

• Unit - Create 1 Grizzle for (Owner of PETOwner) at PETOwnerLoc facing Default building facing degrees

- leaks a location. Again.

• Unit Group - Pick every unit in (Units within 200.00 of PETOwnerLoc matching (((PETDummy[(Player number of (Owner of PETOwner))] is alive) Equal to True) and (((Owner of PETDummy[(Player number of (Owner of PETOwner))]) Equal to (Owner of PETOwner)) and ((Picked unit) Not equal to PETOwner) and do (Actions)
  • Loop - Actions
    • Set PETGroup = (Last created unit group)

- leaks a group and a location.

This system has no advance usage at all, it most certainly is not easy to use, and it is highly primitive.

My suggestion:

- Develop an AI for the pet, get rid of the Locust ability. Add some documentation, more features (right now, I'm not even sure you have a feature). Remove leaks and improve your scripting a lot.

My rating for this 'system' would be 1/5. I'm sorry, but this is very bad and primitive.

 

[Mainy]

Don't post only =D newest^^

@topic
maybe add the trigger and describe what it does.

First read before complaining, put the information from the description

 

[Mainy]

I will make improvements, it is only a test version to see what's necessary to improve

 

[pippo]

Well...
I don't like the locust ability for such thing...
Make the grizzle uncontrollable but selectionable...
Make it MUI and MPI man...

First of all fix this, the idea is great anyway...

EDIT: I think you can use hastables (unit key ID) (to make it mui and mpi), then make that when the owner of the pet attacks, the pet attacks too... and then make that the pet follow the owner...

 

[jim7777]

I also want to use this system for my map..... Could you please post the changelogs of this system so I can know which things are changed so we can not ask for another which you have already fixed.

Btw. I like pippo's recommendation about how the pet is controlled or move. Like how the pet attacks.....

Hope you improve this more....

2/5

 

[Shendoo2]

Yea This system is not "Mult-Player" like you state because of you don't use arrays for Pet Owner when your setting the pet so if another player was to buy a pet yours wouldn't work.

 

[supertoinkz]

your.....
CRUSHER FROM THEHELPER?

Hai
I waz gwafu

 

[Mainy]

PETDummy[(Player number of (Owner of PETOwner))]
p1 = PETDummy [1]
p2 = PETDummy [2]
...
allright multplayer

 

[manstie]

I made my own pet system for my RPG that allows you to controll the pet, but whenever it goes more than 200 range from the owner its ordered to move back to the owner, and over 400, it is teleported to the owner.
Wouldn't that just be better?

Plus it's MPI (I didn't make it MUI because its supposed to be 1 hero only)

The purchasing mechanism can be greatly simplified.
Try something like mine:

• BuyPet
  • Events
    • Unit - A unit Sells a unit
  • Conditions
  • Actions
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • Or - Any (Conditions) are true
          • Conditions
            • (Unit-type of (Sold unit)) Equal to Pet (Cat)
            • (Unit-type of (Sold unit)) Equal to Pet (Dog)
        • PlayerBag[(Player number of (Owner of (Buying unit)))] Equal to No unit
      • Then - Actions
        • Set PlayerBag[(Player number of (Owner of (Buying unit)))] = (Sold unit)
        • Unit Group - Add (Sold unit) to CameraGroup
      • Else - Actions
        • Unit - Remove (Sold unit) from the game
        • Custom script: call DisplayTextToPlayer((GetOwningPlayer(GetSoldUnit())), 0, 0, "You already have a pet!")
        • Player - Add 100 to (Triggering player) Current gold

 

[DarthDeath]

Can you mount him to his pet?

 

[xorkatoss]

the system is very good and could proove useful !! 4/5
i gave 4/5 because i found a leak
the below action leaks

• Unit - Create 1 Grizzle for (Owner of PETOwner) at PETOwnerLoc Facing Default building facing degrees

so you should create another custom script like the one below to clean it

• Custom script : call RemoveLocation(udg_PETOwnerLoc)

im not an expert GUIer but i couldn't miss that lol
if i find any other leaks i will inform you asap ;p