with(npcs[npcscount-1]) Priority ?

[TB3]

Ok I'm having a few problems it's a bit hard to understand / explain.

I have a spawner npc i will dumb it down as the whole code isn't needed. Here is an example..

Spawner NPC:

PHP Code:

function spawn() {
  setstring this.type,Undead;
  putnpc2 x,y,{
  join temporary;
  join baddy;
  };
  with(npcs[npcscount-1]) {
    setstring this.type,#s(thiso.type);
    setstring this.owner,Baddy;
  }
} 


Class baddy

PHP Code:

//Baddies script 


Class temporary

PHP Code:

if (created) sendtonc Type: #s(this.type); 


The problem is about 1/20 times the this.type string is blank (and other strings set in with() ). I'm guessing that maybe the if (created) either loads before the with(npcs[npcscount-1]) sets the string or that with(npcs[npcscount-1]) gets an incorrect npc?

Any help , Ideas?

[Sildae]

I am very sure that Stefan uses this pattern himself a lot, so I do not see how it could not work.

[TB3]

Originally Posted by Sildae

I am very sure that Stefan uses this pattern himself a lot, so I do not see how it could not work.

Yeah, thats also why its not making sense to me O_o

[ZeroTrack]

dont use function , to do that in must be serverside, instead of calling the function triggeraction to serverside and put everything in that function in serverside

PHP Code:

if(actionserverside){
  if(strequals(#p(0),baddy)){
    putnpc2 xy,y,{
        join temporary;
        join baddy
    };
    with(npcs[npcscount-1]){
    setstring this.type,#s(thiso.type);
    setstring this.owner,Baddy; 
    }
  }
}

//#CLIENTSIDE
 // 0,0 was used not know weather it was a weapon or local npc
 if(created) triggeraction 0,0,serverside,Baddy Placer,baddy; 


[TB3]

Originally Posted by ZeroTrack

dont use function , to do that in must be serverside, instead of calling the function triggeraction to serverside and put everything in that function in serverside

PHP Code:

if(actionserverside){
  if(strequals(#p(0),baddy)){
    putnpc2 xy,y,{
        join temporary;
        join baddy
    };
    with(npcs[npcscount-1]){
    setstring this.type,#s(thiso.type);
    setstring this.owner,Baddy; 
    }
  }
}

//#CLIENTSIDE
 // 0,0 was used not know weather it was a weapon or local npc
 if(created) triggeraction 0,0,serverside,Baddy Placer,baddy; 


It's not clientside it is all serverside.
There is no client activity involved other than...
The only flags used are (created) (playerenters) and (timeout) and vars used , playercount in the level, timeout.

It isn't in a function inside the actual NPC, the script is inside of a timeout , and doesn't use any functions or triggeractions.

The function above was an example rather than showing
if (playerenters||timeout) {} and going through the trouble of adding timeout = ect.... because the problem was not / is not located there it is rather the fact that with(npcs[npcscount-1]){ is not working ahead of the (created) of the newly placed npc.... (everything else i said above)

However I don't see how it being a function has anything to do with it ??

[ZeroTrack]

Originally Posted by TB3

Its not clientside it is all serverside, there is no client activity involved other than the playercount in the level. It isn't in a function inside the actual NPC it is inside of a timeout , and doesn't use any functions or triggeractions.

However I don't see how it being a function has anything to do with it ??

uhh it doesn't the way you posted it led me to believe that because functions are clientside and you used a function, next time post correctly

[TB3]

Originally Posted by ZeroTrack

uhh it doesn't the way you posted it led me to believe that because functions are clientside and you used a function, next time post correctly

The way I posted it does not contain //#CLIENTSIDE anywhere.
Functions are not clientside they are serversided and clientsided

[Evil_Trunks]

How did you think he was using putnpc2 and with() clientside?

Not only is that not possible in general, if you had read his post (key step) you would see that it's completely impossible that this script is clientside.

Come on, I thought you knew how to script.

[Python523]

Originally Posted by ZeroTrack

because functions are clientside

You call yourself a scripter?

Edit:
I would do this

PHP Code:

function spawn() { 
  setstring this.type,Undead; 
  putnpc2 x,y,{ 
  join temporary; 
  join baddy; 
  }; 
  with(npcs[npcscount-1]) { 
    setstring this.type,#s(thiso.type); 
    setstring this.owner,Baddy; 
    callnpc -1,typeset;
  } 
} 

//class
if(typeset) sendtonc Type: #s(this.type); 


[TB3]

Originally Posted by Python523

You call yourself a scripter?

Edit:
I would do this

PHP Code:

function spawn() { 
  setstring this.type,Undead; 
  putnpc2 x,y,{ 
  join temporary; 
  join baddy; 
  }; 
  with(npcs[npcscount-1]) { 
    setstring this.type,#s(thiso.type); 
    setstring this.owner,Baddy; 
    callnpc -1,typeset;
  } 
} 

//class
if(typeset) sendtonc Type: #s(this.type); 


K i may try that out , however with the way the baddy is scripted together i would have to say put a this.active=1; inside of that typeset and also a if (this.active==1) {}
around everything else in the script , because if its created priority messes up might its playerenters also since it would be called at the same time (after created) ?

[edit]
I will try this l8r if no one else posts a solution, however if the problem is that it is getting the wrong npc (IE.. 2 npcs being spawned at same time) then it still wouldnt fix : (
The problem runs a bit deeper as the baddies are logged into a database and are removed after death / or on invalidity , being they are invalid when being placed they do not remove the correct string set by the placing npc. Thus the spawn npc gets stuck thinking there are more baddies than there realy is in the room and new ones dont spawn.
[/edit]

[Lance]

Originally Posted by ZeroTrack

uhh it doesn't the way you posted it led me to believe that because functions are clientside and you used a function, next time post correctly

Uh. Functions work both serverside and clientside, man. Also, how does one 'post correctly'? I'm not totally sure what you mean by that directive, but I think you just violated it.

[zokemon]

Originally Posted by ZeroTrack

because functions are clientside

oh my...

[Tolnaftate2004]

Are you sure you have a string nameed thiso.type?
From what you posted, it looks like you misspelled it.

[TB3]

Originally Posted by Tolnaftate2004

Are you sure you have a string nameed thiso.type?
From what you posted, it looks like you misspelled it.

o_O thiso.type is getting the flag from the original npc that is using with().
So im setting this.type in the new npc from the this.type of the old npc that way.
Excerpt From Dach Commands.rtf:
this.flag flags that only exist on server-side and belong to the npc, and are stored when the npc is a database npc
thiso.flag refer to the this. flags of the executing npc when you use the with () command: with (getnpc(npc2)) setstring thiso.temp,this.temp; will copy 'this.temp' from npc2 to the current npc

[TB3]

Im doing some extensive testing with the bug now that there are not many people on. Im using timevar2 and ids to show me the exact time that created is called and the exact time that int is called. So now If only one of them is called then I know for sure that its not getting the correct npc in the with()

[edit]
I have found the problem. I set it to send to nc; everytime both created and call npc was called ; the id of the npc.
If both were not called and only one was called it auto removed showing the this.type ect....
This is what i got after about 50 npcs lolz XD.

NPC Code:

---

Double Int - ID : 100098

Double Int - ID : 100653

[Invalid Baddy]: Name: from spawner  | Debug:  | Actual: 211/565 mal.gmap | ID: 100654

Double Int - ID : 100940

Double Int - ID : 100062

---

What this means is that in baddy 100654 only created was called and the callnpc -1 wasn't nor the strings set.
Meaning that with(npcs[npcscount-1]) was not getting the correct npc!
[/edit]

[edit2]
Ran another test adding to a variable on each call created & callnpc and every so many baddies one screws up and with() gets the wrong npc apparently as no strings are set and the call isnt made ....
[/edit2]
Thus..... WTF DO I DO???? HELP? Stefan ? Eh someone ?

[Projectshifter]

I'm with Jagen. Don't use if (created) because it is created before strings are set using the with() statement. It would only work because it's run really quickly but in theory it shouldn't. The callnpcs then it needed. When I do this I do not use the if (created) I use like if (started) and callnpc -1,started type thing so it will call a new statement after it sets the variables.
Also please note that is the pattern "Stefan uses" I believe. I do not think he would be unable to code his own engine properly

[TB3]

Originally Posted by Projectshifter

I'm with Jagen. Don't use if (created) because it is created before strings are set using the with() statement. It would only work because it's run really quickly but in theory it shouldn't. The callnpcs then it needed. When I do this I do not use the if (created) I use like if (started) and callnpc -1,started type thing so it will call a new statement after it sets the variables.
Also please note that is the pattern "Stefan uses" I believe. I do not think he would be unable to code his own engine properly

I did use that but it was not calling the npc, thats why I'm so confused. if (created) was the only thing being called .

The callnpc was being called most of the time but after so many npcs it misses one or 2 randomly. 1 or 2 npcs every few minutes = alot after a few hours : (

[Projectshifter]

Originally Posted by TB3

I did use that but it was not calling the npc, thats why I'm so confused. if (created) was the only thing being called .

The callnpc was being called most of the time but after so many npcs it misses one or 2 randomly. 1 or 2 npcs every few minutes = alot after a few hours : (

Perhaps you have a bug in another part of your script? I've found small errors in things I've made that just make it miss a few cycles because it stalls at some points then when the action happens the next time it puts it back in the loop. Perhaps you should have it do a sendtonc message and have it make like 50 and just see if it does loop it each time or if it just won't hit the process instead due to another error?

[Admins]

Global staff please stop posting BAD script!

The original script is perfectly fine, but shouldn't be bugged.
The current script is not interrupted, so of course the if (created) should see all variables set. A "callnpc" is completely nonsense.
The only reason for the spawn function not working would be if you call it from a bad place, e.g. "with (anothernpc) { spawn(); }" because then "setstring this.type,Undead;" would not set thiso.type. Eventually replace it with
"setstring thiso.type,Undead;". Check in the script that is putting the npc if the string is actually set.

[TB3]

Originally Posted by Stefan

Global staff please stop posting BAD script!

The original script is perfectly fine, but shouldn't be bugged.
The current script is not interrupted, so of course the if (created) should see all variables set. A "callnpc" is completely nonsense.
The only reason for the spawn function not working would be if you call it from a bad place, e.g. "with (anothernpc) { spawn(); }" because then "setstring this.type,Undead;" would not set thiso.type. Eventually replace it with
"setstring thiso.type,Undead;". Check in the script that is putting the npc if the string is actually set.

Yes the string is actualy set in the previous npc, it also sets a debug string with its x , y , level, and params used in the spawns calcualtion that is also missing in those npcs that get bugged.

If the string is blank in the "Spawning" npc whether it is the type or the spawnid (which is a string also set (that is missing in those few bugged npcs) with xylevelname) an error message is sent to RC. Which luckily the only one to have that problem was the one I intentionaly made to check if the error check worked.

Example Taken from Baddy that errored / removed itself because it was missing its strings : [Invalid Baddy]: Name: from spawner | Debug: | Actual: 214/161 mal_dungeon5.gmap |ID:100278|Active:1
The name is the type which doesnt get set same as the debug message which is blank, the other stuff is the npcs base attributes.
Its strings do not get set and it automaticaly removes itself reporting its error.

with() seems to be getting an incorrect npc and setting its strings thats the only explanation that seems fit. No clue why or how. Stefan if you wish to see the spawning npc it is class : system_spawn
But it isn't that much more to the npc other than a few randomish timers based on player count and some calculations for baddy spawn. ect....

[Projectshifter]

Originally Posted by Stefan

Global staff please stop posting BAD script!

The original script is perfectly fine, but shouldn't be bugged.
The current script is not interrupted, so of course the if (created) should see all variables set. A "callnpc" is completely nonsense.
The only reason for the spawn function not working would be if you call it from a bad place, e.g. "with (anothernpc) { spawn(); }" because then "setstring this.type,Undead;" would not set thiso.type. Eventually replace it with
"setstring thiso.type,Undead;". Check in the script that is putting the npc if the string is actually set.

I realize this is your engine, so you know how it should work, we're just speaking from experience of what we have used and what has worked best in the past. Generally there are some errors in doing functions and transfering variables if you try to do it all with the putnpc2 command. For some reason it just doesn't like setting variables without going back and setting them, and then it doesn't make plausible sense if you create the object (it SHOULD run if (created) at that point) and then add strings and expect it to be effected and read by the if (created)? Without knowing the logistics behind the engine I can say that it makes little sense even if it runs at an incredible speed, it just logically doesn't add up.

[Sern]

it would be nice if gobals didnt try to bash everyone who didnt know the excat syntax for every little thing, i never bothered trying functions serverside cause i just figured that it wouldnt be , oh god sue me , hence why i asked him to post correctly , gobals are supposed to be elitests there supposed to be professional and help people, no offence to ProjectShifter cause hes the only gst i ever see going around helping people on forums and on servers, thanks for being true PS

[Kaimetsu]

Originally Posted by Sern

it would be nice if gobals didnt try to bash everyone who didnt know the excat syntax for every little thing, i never bothered trying functions serverside cause i just figured that it wouldnt be

So if you've never done something, you automatically assume that it is impossible?

Being wrong (however spectacularly) wasn't your biggest sin here. If people are criticising you then it is mostly because:

1. You pushed your false beliefs on other people
2. You scolded another person for correcting you

Nobody expects you to be perfect. Nobody even expects you to be good. It is your belligerence and overconfidence that makes you a target, not your lack of skill.

[Slash-P2P]

Originally Posted by Kaimetsu

Nobody expects you to be perfect. Nobody even expects you to be good. It is your belligerence and overconfidence that makes you a target, not your lack of skill.

I agree (kind of) but I still don't see anything about bashing in "Show off your best NPCs or ask the experts". Do you?

[Kaimetsu]

Originally Posted by Slash-P2P

I agree (kind of) but I still don't see anything about bashing in "Show off your best NPCs or ask the experts"

Nor do I see anything about answering questions or helping to solve problems. Should we be banned from doing that also?

You really should think about your arguments before you use them.

[Slash-P2P]

Originally Posted by Kaimetsu

Nor do I see anything about answering questions or helping to solve problems.

Then continue not to help people