[Guide] make Effects viewable by players

[kamote]

Originally Posted by ramix

show only the ppl is in the area... i tested and works good for me...

hmmm... how about for flowers and Fireworks effects?

[ramix]

works too... because when u use the fireworks is only for ppl see in the area...

[kamote]

Originally Posted by ramix

works too... because when u use the fireworks is only for ppl see in the area...

i don't think so.. and im quite sure you don't even test your code for flower effect and firework! if you did you will knew that all players in your server would see it...

[ramix]

nope i tested the fireworks and works 100%...

that is the code i use for fireworks

World.Action(this, Packets.ItemPacket(EntityID, 255, 26).Get);

[~Yuki~]

Originally Posted by walmartboi

Very nice. Wouldn't of thought of this by myself, so you deserve my special thanks. :P

You coulda just have asked me. =)

[-NewDawn-]

@pro4never, thx =P lol

/on topic:
If you ask me, the effect code we have already is just fine.
It works right?
Anyways, sometimes you need the statement before the effect - like checking for ping and if the character's ping is high then it won't show for them (how do you do that btw- I just came up with that on the spot).
Why change it?

[-impulse-]

Originally Posted by kamote

I feel so darn` to found out this late... in Character.cs there is a pre-made void/method called SendScreen() which has a code that would send packets to any nearby players from the executor player.

With this we may able now to use SendScreen instead of using AddSend method to any Effect code that we want... example in my previous CoolEffect UPDATE I used to have this code:

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;

namespace NewestCOServer.PacketHandling
{
   class CoolEffect
    {
        public static void ActiveCool(Main.GameClient MyClient)
        {
            byte counter = 0;

            for (byte i = 1; i < 9; i++)
            {
                if (i == 7) i++;
                Game.Item I = MyClient.MyChar.Equips.Get(i);
                if (I.ID != 0)
                {
                    Game.ItemIDManipulation Q = new CoPinoy.Game.ItemIDManipulation(I.ID);
                    if (Q.Quality == Game.Item.ItemQuality.Super)
                        counter += 1;
                }
            }

            if (MyClient.MyChar.Job >= 100)
                if (counter == 6)
                    counter = 7;
            if (MyClient.MyChar.Job >= 40 && MyClient.MyChar.Job <= 45)
                if (counter == 6)
                {
                    Game.Item I = MyClient.MyChar.Equips.Get(5);
                    I.ID = MyClient.MyChar.Equips.LeftHand.ID;
                    if (I.ID == 0)
                        counter = 7;
                }
            [B]new Thread(delegate()
            {
                ...
            }).Start();   [/B]
        }
    }
}

I can't understand why would you need a new thread to do this thing....

[Arcо]

Originally Posted by -impulse-

I can't understand why would you need a new thread to do this thing....

Exactly what I was thinking.

[kamote]

Originally Posted by -impulse-

I can't understand why would you need a new thread to do this thing....

this thread is not for CoolEffect ... i only added it as an example...

Why change it?

with this thread I'm not suggesting nor forcing anyone to change there existing code. I only intend to inform you guys that there is an existing code for it.