If Problem
 

ok
its like this

NPC Code:

---

chat=thiso.newtarget;

if(thiso.newtarget!=-1)this.target=thiso.newtarget;

chat=this.target;

---

now when i make it

NPC Code:

---

chat=thiso.newtarget;

if(thiso.newtarget!=-1)this.target=thiso.newtarget;

//chat=this.target;

---

it says a acc, like projectigi
when i make it

NPC Code:

---

chat=thiso.newtarget;

if(thiso.newtarget!=-1)this.target=thiso.newtarget;

chat=this.target;

---

it says -1(this.target is set to -1 before this)

I don't see any difference between the scripts other than the fact that the second one has the chat= commented out. What are you trying to say? Mind providing some more detail other than "its like this" and "i make it"?

i say that thiso.newtarget is not -1
but it dont sets this.target to thiso.newtarget

When describing a problem you should also describe the input and output, then we can help checking if the actualy program is correct.
Input = values of this.target and thiso.target before the program runs
Output = value of 'chat' after the program is ran

ok maybe i should post the whole script XD
maybe theres something wrong with with() or so

NPC Code:

---

function onCreated(){

  this.cur_target=-1;

  this.walkgani="hex-metalspider-walk";

  this.idlegani="hex-metalspider-idle";

  this.hitgani="hex-metalspider-attack";

  showcharacter();

  setCharAni(this.idlegani,"");

  setTimer(0.05);

}

function onTimeout(){

//chat=playerscount;

  if(playerscount>0){

    if(this.cur_target==-1){

      this.pindexes=getnearestplayers(x,y);

      thiso.newdistance=10;

      thiso.newtarget=-1;

      thiso.baddy_x=x;

      thiso.baddy_y=y;

      for(i: this.pindexes){

        with(players[i]){

          this.ddb_dx=player.x-thiso.baddy_x;

          this.ddb_dy=player.y-thiso.baddy_y;

          this.dist=(this.ddb_dx*this.ddb_dx+this.ddb_dy*thi  s.ddb_dy)^.5;

          if(this.dist<thiso.newdistance){

            thiso.newtarget=player.account;

            thiso.newdistance=this.dist;

            player.chat="targeted";

          }

        }

      }

      chat=thiso.newtarget;

      if(thiso.newtarget!=-1)this.cur_target=thiso.newtarget;

      chat=this.cur_target;

    }else{

     this.tmp_plr=findplayer(this.target);

      if(this.tmp_plr.level==level){

      }else{

        

        this.target=-1;

        this.tmp_plr=null;

      }

    }

    setTimer(0.05);

  }

}

function onPlayerEnters(){

  setTimer(0.05);

}

function onPlayerChats(){

  if(player.chat="/dbb")

    destroy();

}

//#CLIENTSIDE

function onCreated(){

//  chat="test";

}

---

well as i said, this three lines

NPC Code:

---

      chat=thiso.newtarget;

      if(thiso.newtarget!=-1)this.cur_target=thiso.newtarget;

      chat=this.cur_target;

---

it says -1 (means this.cur_target is till -1)

NPC Code:

---

      chat=thiso.newtarget;

      if(thiso.newtarget!=-1)this.cur_target=thiso.newtarget;

//      chat=this.cur_target;

---

there it says an acc (means thiso.newtarget must be a acc);

.account is the default parameter to the player object, I believe. If you just do player.chat = player; I believe it shows their account. Try doing: player.chat = this.cur_target.getdynamicvarnames();

chat=this.cur_target.getdynamicvarnames();
now it says nothing XD
at
chat=thiso.newtarget.getdynamicvarnames();
it dont say anything either

What about

it says
"Hmm, something is in it."
but why this check? i mean it cant be null
because its set to -1

Okay, the lack of commenting in that script makes me want to run and hide (just being honest).

well im not good at explaining but here:

NPC Code:

---

function onCreated(){



  this.cur_target=-1; // reset current target



  this.walkgani="hex-metalspider-walk";  // set the walk



  this.idlegani="hex-metalspider-idle";  //idle



  this.hitgani="hex-metalspider-attack";  //and hit gani



  showcharacter();  //well lol



  setCharAni(this.idlegani,""); // set the current gani to the idlegani



  setTimer(0.05); // start the timeout



}



function onTimeout(){



//chat=playerscount;



  if(playerscount>0){  //when there still are players in the room

 

    if(this.cur_target==-1){  //if no target is set



      this.pindexes=getnearestplayers(x,y);  // get the indexes of the players



      thiso.newdistance=10;  // set the newdistance to 10 so everything under a distance of 10 gets targeted



      thiso.newtarget=-1;  // set the new target to -1



      thiso.baddy_x=x;  // store baddy



      thiso.baddy_y=y;  // & y



      for(i: this.pindexes){ // loop through the indexes



        with(players[i]){  //get the player



          this.ddb_dx=player.x-thiso.baddy_x;  //calculate dx



          this.ddb_dy=player.y-thiso.baddy_y; // & dy



          this.dist=(this.ddb_dx*this.ddb_dx+this.ddb_dy*thi  s.ddb_dy)^.5; //calculate distance



          if(this.dist<thiso.newdistance){  //when the distance is smaller than the newdistance



            thiso.newtarget=player.account; //set new target to the players account



            thiso.newdistance=this.dist; //set newdistance to the current distance(to target the players, whos nearest at the baddie)



            player.chat="targeted"; //debug help



          }



        }



      }



      chat=thiso.newtarget; // tel the newtarget



      if(thiso.newtarget!=-1)this.cur_target=thiso.newtarget; //when newtarget is not -1 then set current target to the newtarget



      chat=this.cur_target; // tell the current target



    }else{



     this.tmp_plr=findplayer(this.target); //get targeted player

      if(this.tmp_plr.level==level){ // if player is in baddie level

         //to be done later

      }else{  // if not



        



        this.cur_target=-1;  //reset target



        this.tmp_plr=null;  // delete the temp player object



      }



    }



    setTimer(0.05); // start timeout again



  }



}



function onPlayerEnters(){



  setTimer(0.05); (restart the timeout if an player enters)



}



function onPlayerChats(){



  if(player.chat="/dbb")



    destroy(); //to destroy them (only temp until its finished)



}



//#CLIENTSIDE



function onCreated(){



//  chat="test";



}

---

I would have set the newtarget value to 0 on created and done:
if (!this.newtarget) instead :P

Might be the same thing though, not sure.

i always used -1
xD

0 technically equates to false, which is more useful.

You're doing a .05 timeout on serverside?

yeah I do lol
give me another way to make a baddy smooth =D
also this thread was supposed to talk about why the if doesnt work and not to talk about .05 serverside timeouts
its not even done xD

I haven't scripting movement in a long time, but I remember that the 'move' command had ways to cache movement so that serverside NPCs didn't look so "laggy."

Perhaps look into that? I'm sure there are threads in the forums about it.

EDIT: Found this, might help (for move command's last param):

as i said this thread is NOT supposed to talk about timeouts/lags
it supposed to talk about why this if doesnt works
and as i said it isnt done

Hmm...

Didn't you say that? Was just trying to help.

I think someone needs to understand that the minimum timeout on serverside is .1.

Actually, you can have any timeout serverside now, it's just not wise to have a small timeout, not even 0.1.

However, there's still a bit of functionality in small timeouts. Let's say that you had an object you laid, you trigger it to activate, and then there's a time until deactivation required in the parameters of the trigger. Perhaps you'd want that to be 0.05, or perhaps 0.1. In most cases it probably wouldn't make sense, but there's probably a scenario that exists in which it does.

Now, looping timeouts serverside is a different story. I'd say 0.6 is the highest I'd go for them, and then only if the level has players in it. Such the case for Baddies.

A looping 0.05 serverside timeout is certainly enough to seriously risk crashing the NPC server.