Help adding owner only to this script.

[TrinityReclusive]

I need a bit of help. How to add owner only to this script so only the only can activate it.

integer CountDownTime = 60;
float CountDownIncrements = 1;
integer StartTime;
integer Rounds;
 
// Script starts here ....
default
{
    touch_start(integer numbb)
    {
        llSay(0, (string)CountDownTime + " ...");
        llSetTimerEvent(CountDownIncrements);
        StartTime = llGetUnixTime();
    }
 
    timer()
    {
        if (llGetUnixTime() >= StartTime +CountDownTime)
        {
            llSay(0, "Bang");
            llSetTimerEvent(0);
        }
 
        else
        {
            Rounds++;
            llSay(0, (string)(CountDownTime -  Rounds) + " ...");
        }
    }
}

The script works fine but want it to only be run by the owner. I tried to add

{        if (llDetectedKey(0)==llGetOwner()){    

to it but then i get a syntax error on the timer.

 

 

 

Please help thanks a bit new to scripting but somewhat figured it out.

[KT Kingsley]

Mismatched braces (curly brackets). Remove the one in line 13, and put a closing brace after line 18.

You get the syntax error on the timer because that's where the compiler realises something's wrong: two lots of opening braces without their corresponding closing braces when it hits the timer event code.

[Rolig Loon]

Everything is fine.  You just have to watch that you are matching open { and closed }  brackets properly.  Use curly brackets to demarcate the start and end of a scope -- either the start and end of an event or state, or the code in a conditional step ( controlled by an if or while  condition, for example) .

[TrinityReclusive]

Great that fixed it thought it might have been something simple i still get brackets messed up some-days. But still learning Thanks alot hope you have a great day!

[Kyrah Abattoir]

If you are lazy like me you can also invert the check and drop early.

if( llDetectedKey(0) != llGetOwner() )
	return;

 

[Wulfie Reanimator]

3 hours ago, Kyrah Abattoir said:

If you are lazy like me you can also invert the check and drop early.

if( llDetectedKey(0) != llGetOwner() )
	return;

 

This is what I do, but it's not just about laziness!

I prefer having code that's not very "wide." Lots of nested scopes is messy to read, eg:

state
{
    event
    {
        if owner
        {
            if other condition
            {
                if other other condition
                {
                    // code;
                }
                // code;
            }
        }
    }
}

vs

state
{
    event
    {
        if not owner;

        if condition
        {
            // code;
        }

        if other condition
        {
            // code;
        }
    }
}

 

Edited November 11, 2019 by Wulfie Reanimator

[Kyrah Abattoir]

Yeah reducing code depth is nice for keeping things readable.

[TrinityReclusive]

Resolved