Attacker

[Mmmmm 7]

Hi everybody

I have this attacker, but i think is poor, somebody know how to better?

        private void Timer_Ataque_Tick(object sender, EventArgs e)

        {

            Player p = client.GetPlayer();

            if (p.TargetId == 0)

            {

                foreach (Creature c in client.BattleList.GetCreatures())

                {

                    foreach (String monLista in listBox1.Items)

                    {

                        if (c.Name == monLista)

                        {

                            if (c.IsReachable())

                            {

                                c.Attack();

                            }

                        }

                    }

                }

            }

        }

Sorry for write in english, but i am mx, and i am able to understand br words but i cant write it.

[Renato 659]

Try this: "adapted to your variables"

      private void Timer_Ataque_Tick(object sender, EventArgs e)

        {

	foreach (Creature c in c.BattleList.GetCreatures) {

		foreach (string monLista in listBox1.Items) {

			if (target.Name == monLista) {

				if (c.IsReachable & c.Type == CreatureType.NPC) {

					if (p.TargetId == 0) {

						c.Attack();

					}

				}

			}

		}

        }             

} 

[/code]