Small Script Help! Can't get Statue working - 2

Johnaudi · 07-13-2012

Okay Thanks Emera...
am currently working on an Event, but is there a way to find (in the level) the guy with the most ... lets say clientr.ec
anyway I can warp him and find the command? like is there something like player.most.clientr.ec.setlevel2?

Emera · 07-13-2012

Use for (temp.pl : players) { to scan through all of the players in the level. You can then use temp.pl.clientr.ec to check their ec clientr flag.

Johnaudi · 07-13-2012

Posted by Emera

Use for (temp.pl : players) { to scan through all of the players in the level. You can then use temp.pl.clientr.ec to check their ec clientr flag.

I know that, I just wanted to know, how can I created something to make the one with the MOST EC's be warped or uses chat for a certain thing...
something like for (winner: playerwithmostclientr.ec) {
isnt there something like this similar? like I will use winner.setlevel2() if it worked etc..

callimuc · 07-14-2012

You could do something like this:

PHP Code:


			
this.mostec = NULL;
for (temp.pl: players) {
  temp.track = findPlayer(this.mostec);
  if (temp.pl.clientr.event_ec > temp.track.clientr.event_ec) this.mostec = temp.pl;
}
findPlayer(this.mostec).setLevel2(levelname, x, y);

Emera · 07-14-2012

Posted by callimuc

You could do something like this:

PHP Code:


			
this.mostec = NULL;

for (temp.pl: players) {

  temp.track = findPlayer(this.mostec);

  if (temp.pl.clientr.event_ec > temp.track.clientr.event_ec) this.mostec = temp.pl;

}

findPlayer(this.mostec).setLevel2(levelname, x, y);

Beat me to it

Johnaudi · 07-15-2012

Posted by callimuc

You could do something like this:

PHP Code:


			
this.mostec = NULL;
for (temp.pl: players) {
  temp.track = findPlayer(this.mostec);
  if (temp.pl.clientr.event_ec > temp.track.clientr.event_ec) this.mostec = temp.pl;
}
findPlayer(this.mostec).setLevel2(levelname, x, y);

Thank you, but I still don't see the usage of temp.pl and not plys. Like I use, what's the difference?

callimuc · 07-15-2012

You can call it however you want to. You can also call it temp.stefan, temp.john, ... . On Graal it's mostly called temp.pl for one player and temp.plyrs/temp.plys for more players. It's just easier for others to see what you want to do with it. Kinda like organisation but there are no rules for that.
The 2nd "param", in this case players, is always an array. In this case players is an array, just like allplayers, serveroptions.staff, ... . You could also do something like

PHP Code:


			
temp.testarray = {"hello", "my", "name", "is", "calli"};
for (temp.whatever: temp.testarray) {
  echo(temp.whatever);
}

Basicly its the same (in this case) as

PHP Code:


			
temp.testarray = {"hello", "my", "name", "is", "calli"};
for (temp.i=0; temp.i<temp.testarray.size(); temp.i++) {
  echo(temp.testarray[temp.i]);
}

in my opinion.

I just prefer the for() loop in this case

Emera · 07-15-2012

Posted by callimuc

PHP Code:


			
temp.testarray = {"hello", "my", "name", "is", "calli"};

for (temp.i=0; temp.i<temp.testarray.size(); temp.i++) {

  echo(temp.testarray[temp.i]);

}

Sorry, but it was my understanding that coders styled their code. I must be mistaken? DO IT AGAIN!

callimuc · 07-15-2012

Posted by Emera

Sorry, but it was my understanding that coders styled their code. I must be mistaken? DO IT AGAIN!

What?

Johnaudi · 07-16-2012

Posted by callimuc

You can call it however you want to. You can also call it temp.stefan, temp.john, ... . On Graal it's mostly called temp.pl for one player and temp.plyrs/temp.plys for more players. It's just easier for others to see what you want to do with it. Kinda like organisation but there are no rules for that.
The 2nd "param", in this case players, is always an array. In this case players is an array, just like allplayers, serveroptions.staff, ... . You could also do something like

PHP Code:


			
temp.testarray = {"hello", "my", "name", "is", "calli"};

for (temp.whatever: temp.testarray) {

  echo(temp.whatever);

}

Basicly its the same (in this case) as

PHP Code:


			
temp.testarray = {"hello", "my", "name", "is", "calli"};

for (temp.i=0; temp.i<temp.testarray.size(); temp.i++) {

  echo(temp.testarray[temp.i]);

}

in my opinion.

I just prefer the for() loop in this case

I know about the type of calling, but why temp. not this.? I'm still getting confused as this point

IPAD* · 07-16-2012

Posted by callimuc

What?

What the **** cant you read he said you cant style

callimuc · 07-16-2012

Posted by IPAD*

What the **** cant you read he said you cant style

So captain obvious tell me what is not styled on that.

Posted by Johnaudi

I know about the type of calling, but why temp. not this.? I'm still getting confused as this point

temp.flags only stay in the function. Using this.flags will be usable in the whole script and can be changed external through other scripts (thats why i made it this.mostec since you might need it later on). There is no need in using this.flags in that case but its up to you.

IPAD* · 07-16-2012

Posted by callimuc

So captain obvious tell me what is not styled on that.

temp.flags only stay in the function. Using this.flags will be usable in the whole script and can be changed external through other scripts (thats why i made it this.mostec since you might need it later on). There is no need in using this.flags in that case but its up to you.

I just answered chapter 1 I didn't prepare for chapter 2 man!

Johnaudi · 07-16-2012

Posted by callimuc

So captain obvious tell me what is not styled on that.

temp.flags only stay in the function. Using this.flags will be usable in the whole script and can be changed external through other scripts (thats why i made it this.mostec since you might need it later on). There is no need in using this.flags in that case but its up to you.

Oh alright!! Got that!! Thanks.

So exactly what I'm asking is : so temp.number can't replace this.number in this case?
And second : is there any other way without using this.number?
Thanks a lot!
P.S @iPad : his script style is readable and nice to the eye.

Emera · 07-16-2012

Posted by callimuc

So captain obvious tell me what is not styled on that.

Spaces between symbols like + and =
Nothing big mango, just pointing it out ;)

Posted by Johnaudi

Oh alright!! Got that!! Thanks.

So exactly what I'm asking is : so temp.number can't replace this.number in this case?
And second : is there any other way without using this.number?
Thanks a lot!
P.S @iPad : his script style is readable and nice to the eye.

Speaking of styling, there's an absence of it in your code. You'll have to get acquanited with styling your code so others can read it. It's hard to read when there's no styling.

Here's some links to help get you started:
http://forums.graalonline.com/forums...ad.php?t=61805
http://fp4.ca/gs2beautifier/

Also, I don't get what you're trying to say about this.number and temp.number.

07-14-2012	19
callimuc Registered User Join Date: Sep 2011 Location: Germany Posts: 6,122	You could do something like this: PHP Code: `this.mostec = NULL; for (temp.pl: players) { temp.track = findPlayer(this.mostec); if (temp.pl.clientr.event_ec > temp.track.clientr.event_ec) this.mostec = temp.pl; } findPlayer(this.mostec).setLevel2(levelname, x, y);`

07-15-2012	22
callimuc Registered User Join Date: Sep 2011 Location: Germany Posts: 6,122	You can call it however you want to. You can also call it temp.stefan, temp.john, ... . On Graal it's mostly called temp.pl for one player and temp.plyrs/temp.plys for more players. It's just easier for others to see what you want to do with it. Kinda like organisation but there are no rules for that. The 2nd "param", in this case players, is always an array. In this case players is an array, just like allplayers, serveroptions.staff, ... . You could also do something like PHP Code: `temp.testarray = {"hello", "my", "name", "is", "calli"}; for (temp.whatever: temp.testarray) { echo(temp.whatever); }` Basicly its the same (in this case) as PHP Code: `temp.testarray = {"hello", "my", "name", "is", "calli"}; for (temp.i=0; temp.i<temp.testarray.size(); temp.i++) { echo(temp.testarray[temp.i]); }` in my opinion. I just prefer the for() loop in this case

07-13-2012	16
Johnaudi 299,792,458 m/s Join Date: Mar 2012 Location: Lebanon Posts: 3,581	Okay Thanks Emera... am currently working on an Event, but is there a way to find (in the level) the guy with the most ... lets say clientr.ec anyway I can warp him and find the command? like is there something like player.most.clientr.ec.setlevel2?

07-13-2012	17
Emera Nerd Join Date: Sep 2011 Location: Berlin Posts: 3,824	Use for (temp.pl : players) { to scan through all of the players in the level. You can then use temp.pl.clientr.ec to check their ec clientr flag.