Npc to Npc

[kingcj]

Originally Posted by WhiteDragon

That last post is definitely going in the wrong direction.


One thing I can think of is that the onCreated() of the door or switch is getting called before the database's.

For debugging, echoing an object doesn't really make much sense, try echoing temp.door.type() instead to see if anything is actually there.

Yea Kinda figured that. I tried the temp.door.type() and got nothing in RC. Maybe this should be done a different way? Maybe place everything in one function that connects objects and all with params? I dunno.

[fowlplay4]

Originally Posted by kingcj

A trigger won't work clientside from the DB I don't think, but I don't really know. I don't understand why it won't pick this up as a function though? A trigger would be great if I could work it server to client and client to server without it being a weapon, unfortunately I have tried this.

Leads me to believe your door script looks like..

PHP Code:

//#CLIENTSIDE
function onCreated() { 
  SwitchDB = findnpc("SwitchDB"); 
  SwitchDB.addDoor("door_a", this); 
} 


public function doorOpen() { 
  this.hide(); 
  sleep(3); 
  this.show(); 
} 


which won't work because you can't call directly from server-side to client-side like that.

[kingcj]

Yea I know but I am trying to get it to work severside first. If it works severside I can put all the public functions clientside and get the same result clientside only, right? Plus my door script is at the top of this page with no //#CLIENTSIDE, and that's how it is in the script as well.

[fowlplay4]

Well for the most part because it is server-side you don't have to do the..

SwitchDB = findnpc("SwitchDB");

because the NPC is accessible via it's name without having to declare a global variable.

Following KISS theory...

SwitchDB

PHP Code:

// Empty Script
// Just using it to hold variables for now. 


Switch

PHP Code:

function onCreated() {
  this.doorid = "door_1";
  this.image = "block.png";
  echo(this.doorid @ "'s switch has been created!");
}

function onPlayerTouchsMe() {
  echo(player.account @ " touched the switch for door " @ this.doorid);
  SwitchDB.door.(@this.switchid).trigger("DoorOpened", player.account);
} 


Door

PHP Code:

function onCreated() {
  this.doorid = "door_1";
  this.image = "door.png";
  SwitchDB.door.(@this.doorid) = this;
  echo(this.doorid @ "'s door has been created!");
}

function onDoorOpened(acct) {
  echo(this.doorid @ " has been opened by " @ acct @ "!");
  hide();
  sleep(3);
  show();
} 


From there we can work off the base and create the proper SwitchDB and classes that we require.

In my own "switch" system I store the main controller object (i.e: the door) and it's respective array of switches, and send a trigger to the main controller when a switch is updated (pressed on and off for instance) and the controller loops through the switches and performs the necessary logic (all switches are pressed down) and acts accordingly.

[kingcj]

That seems easy enough. Thanks for the help, but will it work clientside. I was trying to avoid the trigger because i didn't think i could call one clientside to clientside? I will try it and see! Thanks everyone!

I tried fp4's script and it works just gotta change some of it around. Works clientside as well...

[fowlplay4]

Originally Posted by kingcj

I tried fp4's script and it works just gotta change some of it around. Works clientside as well...

To get it to work client-side you would just have to use a weapon script or a level variable.

I.e:

PHP Code:

//#CLIENTSIDE
function onCreated() {
  temp.SwitchDB = findweapon("-System/DB");
} 


The only issue is that you're trapped to the same level.

Also about your trigger worries, because you're staying on the same side it's really not that big of deal at all and has about the same impact as calling a public function. It's when you're going from client<->server that it becomes important to minimize the amount of triggerserver/clients/actions.

[kingcj]

Originally Posted by fowlplay4

To get it to work client-side you would just have to use a weapon script or a level variable.

I.e:

PHP Code:

//#CLIENTSIDE
function onCreated() {
  temp.SwitchDB = findweapon("-System/DB");
} 


The only issue is that you're trapped to the same level.

Also about your trigger worries, because you're staying on the same side it's really not that big of deal at all and has about the same impact as calling a public function. It's when you're going from client<->server that it becomes important to minimize the amount of triggerserver/clients/actions.

It worked Clientside without a weapon script or level var... I just put //#CLIENTSIDE at the top and it worked for me and my buddy on clientside only. I don't know if it is suppose to work that way or not, but it does I assure you, even without this added scripting. I haven't checked to see yet, but I don't know if I would be trapped to the same level with the trigger or not. Again thanks for the help!

[kingcj]

Ok so the main reason I was trying to use a public function was to make sure that the script could be done clientside. Here is what I am working with now. It works, once but not twice, and is very very slow. Any pointers here would be great

The Switch:

PHP Code:

//#CLIENTSIDE
function onCreated() { 
  this.switchid = "door_2"; 
  this.image = "block.png"; 
} 

function onPlayerenters() {
  onTimeOut();
}

function onTimeOut() {
  if (players.size() > 0) 
    setTimer(0.1);
    else 
    setTimer(0);
  for (temp.i : findareanpcs(this.x-2, this.y-2, 2, 2)) {
    if (temp.i.isinclass("bomb")) 
      SwitchDB.door.(@this.switchid).trigger("DoorOpened", null);
      sleep(3.2);
      onTimeOut();
     }
} 


And the Door:

PHP Code:

//#CLIENTSIDE
function onCreated() {
  this.doorid = "door_2";
  this.image = "door.png";
  SwitchDB.door.(@this.doorid) = this;
}

function onDoorOpened() {
  hide();
  sleep(3.15);
  show();
} 


[fowlplay4]

If you style your script properly you'll see the following..

PHP Code:

//#CLIENTSIDE
// other code..

function onTimeOut() { 
  if (players.size() > 0)  // Useless on client-side since script won't run
    setTimer(0.1);         // if player is in another level.
    else  
    setTimer(0);
  // Loops through all npcs near the switch
  for (temp.i : findareanpcs(this.x-2, this.y-2, 2, 2)) {
    // Checks if npc is a bomb
    if (temp.i.isinclass("bomb")) { // Added the { for clarity
      // Triggers switch
      SwitchDB.door.(@this.switchid).trigger("DoorOpened", null);
    }
    // Sleeps for 3.2 seconds
    sleep(3.2);
    // Recursively calls onTimeout again, leading to an infinite loop.
    // Without a sleep the script would break quite quickly after hitting
    // the stack/maxloop limit.
    onTimeOut(); 
  } 
} 


Notice how you're sleeping for every NPC around the switch and then calling onTimeout which causes that nasty loop.

Also something about calling onTimeout directly like that is just unsettling to me, the only time I do that is when I'm hacking around with code.

Recursion crash example:

PHP Code:

//#CLIENTSIDE
function onCreated() {
  recurse(0);
}

function recurse(a) {
  echo(a);
  recurse(a+1);
} 


That's as big as hint as I'll give you, I'm sure you can figure out what you need to change.

[kingcj]

Originally Posted by fowlplay4

That's as big as hint as I'll give you, I'm sure you can figure out what you need to change.

Thanks for the help again! I was trying to use sleep to wait untill the bomb was off the switch because it triggers the function on every timeout, so while the bomb is on the switch, it continuously triggers the door.

I don't know if that will cause any problems with lag and so forth, but it works fine when sleep and the player.size were removed. Guess i could slow the Timeout down and then speed it up again..lol.. that just came to me as I was typing....

Thanks Again.

[kingcj]

Ok on the same subject still... I place the custom bomb on a switch which opens a door, when the bomb explodes the door closes. This part works great except the door opens serverside. The bomb has an owner, and I transfer that info to the switch which has the acct in the trigger that sends it to the door. When it gets to the door the door still opens for everyone serverside. I want it to work clientside so everyone will have to open the door using their own bomb. The owner of the bomb is stored with a var (this.owner), and I don't know if it needs to be a trigger.

[fowlplay4]

Store the owner's account in attr[i]bute and check it like how you're doing it up there..

I.e: (temp.i.isinclass("bomb") && temp.i.attr[1] == player.account)