Difference between revisions of "Debugging Tips"

From Second Life Wiki
Jump to navigation Jump to search
m (→‎Error Reporting and Debugging: I like what the moles have been up to. Keep up the good work.)
m (fixed a bug and improved readability)
Line 6: Line 6:


<lsl>
<lsl>
notify(string msg)
notify(string message)
{
{
     llOwnerSay(msg);
     llOwnerSay(message);
}
}
</lsl>
</lsl>
Line 15: Line 15:


<lsl>
<lsl>
notify("your debug message here");
//  default
//  {
//      touch_start(integer num_detected)
//      {
            notify("your debug message here");
//      }
//  }
</lsl>
</lsl>


Line 21: Line 27:


<lsl>
<lsl>
integer a = 3;
//  default
integer b = 4;
//  {
integer result = a * b;
//      touch_start(integer num_detected)
notify((string)result);
//      {
            integer a = 3;
            integer b = 4;
            integer result = a * b;
            notify((string)result);
//      }
//  }
</lsl>
</lsl>


Line 30: Line 42:


<lsl>
<lsl>
notify(string msg)
notify(string message)
{
{
     //llOwnerSay(msg);
     //llOwnerSay(message);
}
}
</lsl>
</lsl>
Line 38: Line 50:
if you're working with list, you could use something along the lines of
if you're working with list, you could use something along the lines of


<lsl>notify(llList2String(listName, integerIndex)); // notification contains one list element</lsl>
<lsl>
//  notification contains one list element
    notify(llList2String(listOfNames, integerIndex));
</lsl>
or
or
<lsl>notify(llDumpList2String(listName, ",")); // notification contains entire list as a string</lsl>
<lsl>
//  notification contains entire list as a string
    notify(llDumpList2String(listOfNames, ","));
 
// or alternatively a CSV (comma-seperated-value string):
//  notify(llList2CSV(listOfNames));
</lsl>


However, the lsl compiler is not an optimizing compiler. This means that even if you disable notify() as shown above (by commenting out its body), you will still suffer a performance hit from converting the list to a string. A better plan would be to create a separate notifyList function which takes a list parameter and dumps it:
However, the lsl compiler is not an optimizing compiler. This means that even if you disable notify() as shown above (by commenting out its body), you will still suffer a performance hit from converting the list to a string. A better plan would be to create a separate notifyList function which takes a list parameter and dumps it:


<lsl>
<lsl>
notifyList(list msg)
notifyList(list listOfMessages)
{
{
     llOwnerSay(llDumpList2String(msg, ","));
     llOwnerSay(llDumpList2String(listOfMessages, ","));
 
//  or:
//  llOwnerSay(llList2CSV(listOfMessages));
}
}
</lsl>
</lsl>
Line 55: Line 79:


<lsl>
<lsl>
notify(string genus, string msg)
notify(string genus, string message)
{
{
     key creator = "a822ff2b-ff02-461d-b45d-dcd10a2de0c2";
     key creator = "a822ff2b-ff02-461d-b45d-dcd10a2de0c2";
    //genus = "off" //uncoment this line to turn notifications off
 
// uncoment next line to turn notifications off
//  genus = "off"
     if (genus == "dbg")
     if (genus == "dbg")
     {
     {
         llInstantMessage(creator, "DEBUG: " + msg); //coment out this line to turn off debugging
    //  coment out next line to turn off debugging
         llInstantMessage(creator, "DEBUG: " + message);
     }
     }
     else if (genus == "err")
     else if (genus == "err")
     {
     {
         llOwnerSay("ERROR: " + msg);
         llOwnerSay("ERROR: " + message);
         llInstantMessage(creator, "ERROR: " + msg + " in object " + llGetObjectName() + " that belongs to " + llKey2Name(llGetOwner()) + " in region " + llGetRegionName());
         llInstantMessage(creator, "ERROR: " + message + " in object " + llGetObjectName() + " that belongs to "  
                                + llKey2Name(llGetOwner()) + " in region " + llGetRegionName());
     }
     }
     else if (genus == "info")
     else if (genus == "info")
     {
     {
        llOwnerSay(msg); //no prepend because it's a normal message
    // no prepend because it's a normal message
        llOwnerSay(message);
     }
     }
     else
     else
     {
     {
         //do nothing, notifications were turned off
         ;//do nothing, notifications were turned off
     }
     }
}
}
Line 91: Line 120:


<lsl>
<lsl>
notify(string genus, string msg)
notify(string genus, string message)
{
{
     key creator = "a822ff2b-ff02-461d-b45d-dcd10a2de0c2";
     key creator = "a822ff2b-ff02-461d-b45d-dcd10a2de0c2";
    //genus = "off" //uncoment this line to turn notifications off
 
// uncoment this line to turn notifications off
//  genus = "off"
     if (genus == "dbg")
     if (genus == "dbg")
     {
     {
         llInstantMessage(creator, "DEBUG: " + msg); //coment out this line to turn off debugging
    //  coment out this line to turn off debugging
         llInstantMessage(creator, "DEBUG: " + message);
     }
     }
     else if (genus == "err")
     else if (genus == "err")
     {
     {
         llOwnerSay("ERROR: " + msg);
         llOwnerSay("ERROR: " + message);
         llInstantMessage(creator, "ERROR: " + msg + " in object " + llGetObjectName() + " that belongs to " + llKey2Name(llGetOwner()) + " in region " + llGetRegionName());
         llInstantMessage(creator, "ERROR: " + message + " in object " + llGetObjectName() + " that belongs to "
                                + llKey2Name(llGetOwner()) + " in region " + llGetRegionName());
     }
     }
     else if (genus == "info")
     else if (genus == "info")
     {
     {
        llOwnerSay(msg);  //no prepend because it's a normal message
    // no prepend because it's a normal message
        llOwnerSay(message);
     }
     }
     else
     else
     {
     {
         //do nothing, notifications were turned off
         ;//do nothing, notifications were turned off
     }
     }
}
}
Line 120: Line 154:
         notify("info", "notification system ready");
         notify("info", "notification system ready");
     }
     }
     link_message(integer source, integer num, string str, key id)
 
     link_message(integer sender_num, integer num, string str, key id)
     {
     {
        notify(str, key); //we're using the key as a second string parameter
    // we're using the key as a second string parameter
        notify(str, id);
     }
     }
}
}
Line 132: Line 168:
default
default
{
{
     touch_start(integer c)
     touch_start(integer num_detected)
     {
     {
         string msg = llKey2Name(llDetectedKey(0)) + " attempted to steal Mjölnir!";
         key id = llDetectedKey(0);
        llMessageLinked(LINK_SET, 0, "info", msg);
        string message = llKey2Name(id) + " attempted to steal Mjölnir!";
        //this may seem odd but it's valid to use the key paramter of llMessageLinked as a second string passing facility if not actually used for a key
 
    // this may seem odd but it's valid to use the key paramter of llMessageLinked
    //  as a second string passing facility if not actually used for a key
        llMessageLinked(LINK_SET, 0, "info", message);
     }
     }
}
}

Revision as of 13:23, 31 October 2012

One of the hardest things with scripting anything non trivial in LSL, especially if you come from an software development environment where you're used to combined editor and debugger, is the seeming inability to debug LSL code.

Simple Example

The solution is to define a simple debugging utility function as follows.

<lsl> notify(string message) {

   llOwnerSay(message);

} </lsl>

then whenever we need to check the value of a variable simple call

<lsl> // default // { // touch_start(integer num_detected) // {

           notify("your debug message here");

// } // } </lsl>

don't forget to typecast as necessary, for example:

<lsl> // default // { // touch_start(integer num_detected) // {

           integer a = 3;
           integer b = 4;
           integer result = a * b;
           notify((string)result);

// } // } </lsl>

This allows us to reuse the same function for reporting anything to the owner of the object, rather than using public chat. The advantage to doing this is we can comment out a single line in the code function as follows to disable all debugging, or we can search and replace and instance of "notify" with "//notify".

<lsl> notify(string message) {

   //llOwnerSay(message);

} </lsl>

if you're working with list, you could use something along the lines of

<lsl> // notification contains one list element

   notify(llList2String(listOfNames, integerIndex));

</lsl> or <lsl> // notification contains entire list as a string

   notify(llDumpList2String(listOfNames, ","));

// or alternatively a CSV (comma-seperated-value string): // notify(llList2CSV(listOfNames)); </lsl>

However, the lsl compiler is not an optimizing compiler. This means that even if you disable notify() as shown above (by commenting out its body), you will still suffer a performance hit from converting the list to a string. A better plan would be to create a separate notifyList function which takes a list parameter and dumps it:

<lsl> notifyList(list listOfMessages) {

   llOwnerSay(llDumpList2String(listOfMessages, ","));

// or: // llOwnerSay(llList2CSV(listOfMessages)); } </lsl>

Error Reporting and Debugging

We can use the same concept for tracking issues after the code is complete as follows

<lsl> notify(string genus, string message) {

   key creator = "a822ff2b-ff02-461d-b45d-dcd10a2de0c2";

// uncoment next line to turn notifications off // genus = "off"

   if (genus == "dbg")
   {
   //  coment out next line to turn off debugging
       llInstantMessage(creator, "DEBUG: " + message);
   }
   else if (genus == "err")
   {
       llOwnerSay("ERROR: " + message);
       llInstantMessage(creator, "ERROR: " + message + " in object " + llGetObjectName() + " that belongs to " 
                               + llKey2Name(llGetOwner()) + " in region " + llGetRegionName());
   }
   else if (genus == "info")
   {
   //  no prepend because it's a normal message
       llOwnerSay(message);
   }
   else
   {
       ;//do nothing, notifications were turned off
   }

} </lsl>

We're doing a number of things here: handling different kids of messages, informational for normal running, error reporting to the owner and creator and debug messages. To turn off debug simply comment out the debug line as shown in the code, and uncomment the genus="off" line to disable all notifications.

We're using llOwnerSay for the current owner and llInstantMessage for contacting the script creator that will continue to work after ownership has passed - you just have insert your own key at the top.

Other mechanisms that could be used include llEmail or llHTTPRequest to post a message to an external logging system perhaps.

You define your own custom message types and communicate using the most appropriate method.

More Reuse

We can extend the reuse even further by making this a discreet script on it's own within the object as follows:

<lsl> notify(string genus, string message) {

   key creator = "a822ff2b-ff02-461d-b45d-dcd10a2de0c2";

// uncoment this line to turn notifications off // genus = "off"

   if (genus == "dbg")
   {
   //  coment out this line to turn off debugging
       llInstantMessage(creator, "DEBUG: " + message);
   }
   else if (genus == "err")
   {
       llOwnerSay("ERROR: " + message);
       llInstantMessage(creator, "ERROR: " + message + " in object " + llGetObjectName() + " that belongs to "
                               + llKey2Name(llGetOwner()) + " in region " + llGetRegionName());
   }
   else if (genus == "info")
   {
   //  no prepend because it's a normal message
       llOwnerSay(message);
   }
   else
   {
       ;//do nothing, notifications were turned off
   }

}

default {

   state_entry() 
   {
       notify("info", "notification system ready");
   }
   link_message(integer sender_num, integer num, string str, key id)
   {
   //  we're using the key as a second string parameter
       notify(str, id);
   }

} </lsl>

To send a notification you simply use in your implementation:

<lsl> default {

   touch_start(integer num_detected)
   {
       key id = llDetectedKey(0);
       string message = llKey2Name(id) + " attempted to steal Mjölnir!";
   //  this may seem odd but it's valid to use the key paramter of llMessageLinked
   //  as a second string passing facility if not actually used for a key
       llMessageLinked(LINK_SET, 0, "info", message);
   }

} </lsl>


Return to Good_Building_Practices