Is there a way to compress this ?

[Rain408]

	if (checkquest(60520) != -1) {
	if (checkquest(60521) != -1) {
	if (checkquest(60518) != -1) {

Is there a way to compress it so i dont have to type the same thing over and over, and risk messing up with my else.....

Because the code just looks ugly like that

 

I was thinking it could look something like this.

 

if (checkquest(60520 ll 60521 ll 60518) != -1) {

 

[Azura Skyy]

	if (checkquest(60520) != -1) {
	if (checkquest(60521) != -1) {
	if (checkquest(60518) != -1) {

Is there a way to compress it so i dont have to type the same thing over and over, and risk messing up with my else.....

Because the code just looks ugly like that

 

 

if (checkquest(60520) != -1 || (60521) != -1 || (60518) != -1)) {

Should work fine; Not entirely sure it looks better this way, but if that is what you want to do. Actually looking at my own use of a similar format, it looks nice in script editor.   

 

Important side note.. in your post, when scripting 'or' ||

You use 'll' <- two lower case L's.

You need to use '||' <- two shift+\

Regards,

~Azura Skyy

[Rain408]

 

	if (checkquest(60520) != -1) {
	if (checkquest(60521) != -1) {
	if (checkquest(60518) != -1) {

Is there a way to compress it so i dont have to type the same thing over and over, and risk messing up with my else.....

Because the code just looks ugly like that

 

 

if (checkquest(60520) != -1 || (60521) != -1 || (60518) != -1)) {

Should work fine; Not entirely sure it looks better this way, but if that is what you want to do. Actually looking at my own use of a similar format, it looks nice in script editor. 

 

Important side note.. in your post, when scripting 'or' ||

You use 'll' <- two lower case L's.

You need to use '||' <- two shift+\

Regards,

~Azura Skyy

 

 

Thank you. I had a couple question about your script.

 

Checkquest(60520)= = 2 , why you set it equal to 2?

 

Next question is if ($Questleve >0), what does that command do?

 

I'm just curious to learn these things so I can become a better scripter.

[Azura Skyy]

Thank you. I had a couple question about your script.

 

Checkquest(60520)= = 2 , why you set it equal to 2?

 

Next question is if ($Questleve >0), what does that command do?

 

I'm just curious to learn these things so I can become a better scripter.

 

 

Checking if a quest == 2 is just a check to make sure the quest is completed. The $Questlevel variable is my own variable in that script, whereby if it is set to 1, the script requires that players meet the base level requirement to do the quest (the full script is a quest skipper, so... toggles for level and item requirements). Not important to your situation I think. If you want to read more about variables, click here.

Regards,

~Azura Skyy