0008128: Conditionals do not work within a for loop

(0031594)
Lotek (reporter)
2017-02-08 14:09

llGetPrimitiveParams returns a list, while you are comparing it to a string. This may perhaps work in SL but OpenSim is more strict.

Try
if(llList2String(llGetLinkPrimitiveParams(i,[PRIM_NAME]),0) == "glass"){

(0031595)
Gezebu MindBlue (reporter)
2017-02-08 17:27

Thx for hint Lotek, solve a problem. But anyway here have an issue, because in any case the IF it should not return any output if not find what you check, and it returns one for each primitives in the linkset. Something is not working well.

(0031596)
BillBlight (developer)
2017-02-08 19:19
edited on: 2017-02-08 19:22

Actually the conditions seem to work inside the for statement, just not with that particular statement, if you put it outside the if statement but inside the for it works ...

ie.

default
{
    state_entry()
    {
        llSay(0, "I'm going crazy?");
    }

    touch_start(integer num)
    {
        integer i;

        for(i=0;i<=llGetNumberOfPrims();i++)
        {
            string primname =(string)llGetLinkPrimitiveParams(i,[PRIM_NAME]);
            if(primname == "glass")
            {
               llOwnerSay(primname);
            }
        }
    }
}

Seems like , like lotek said, it is having an issue validating the string against the list. Maybe it returns everything because it is a list in your script so it is not sure where to stop and start evaluating , but something is screwy for sure.

(0031597)
BillBlight (developer)
2017-02-08 19:26
edited on: 2017-02-08 19:37

BTW it also works like this ..

default
{
    state_entry()
    {
        llSay(0, "I'm going crazy?");
    }

    touch_start(integer num)
    {
        integer i;

        for(i=0;i<=llGetNumberOfPrims();i++)
        {

            if((string)osGetLinkPrimitiveParams(i,[PRIM_NAME]) == "glass")
            {
               llOwnerSay((string)osGetLinkPrimitiveParams(i,[PRIM_NAME]));
            }
        }
    }
}

or with LL functions.

default
{
    state_entry()
    {
        llSay(0, "I'm going crazy?");
    }

    touch_start(integer num)
    {
        integer i;

        for(i=0;i<=llGetNumberOfPrims();i++)
        {

            if((string)llGetLinkPrimitiveParams(i,[PRIM_NAME]) == "glass")
            {
               llOwnerSay((string)llGetLinkPrimitiveParams(i,[PRIM_NAME]));
            }
        }
    }
}

(0031599)
Total Sorbet (reporter)
2017-02-11 05:31

if(llList2String(llGetLinkPrimitiveParams(i,[PRIM_NAME]),0)=="glass"){llOwnerSay("I promise not to attempt typecasting an entire list again (even if it does consist of just 1 element) :p");}

Issue History
Date Modified	Username	Field	Change
2017-02-08 09:32	Gezebu MindBlue	New Issue
2017-02-08 14:09	Lotek	Note Added: 0031594
2017-02-08 17:27	Gezebu MindBlue	Note Added: 0031595
2017-02-08 19:19	BillBlight	Note Added: 0031596
2017-02-08 19:22	BillBlight	Note Edited: 0031596	View Revisions
2017-02-08 19:26	BillBlight	Note Added: 0031597
2017-02-08 19:29	BillBlight	Note Edited: 0031597	View Revisions
2017-02-08 19:37	BillBlight	Note Edited: 0031597	View Revisions
2017-02-11 05:31	Total Sorbet	Note Added: 0031599

Notes
(0031594) Lotek (reporter) 2017-02-08 14:09	llGetPrimitiveParams returns a list, while you are comparing it to a string. This may perhaps work in SL but OpenSim is more strict. Try if(llList2String(llGetLinkPrimitiveParams(i,[PRIM_NAME]),0) == "glass"){

(0031595) Gezebu MindBlue (reporter) 2017-02-08 17:27	Thx for hint Lotek, solve a problem. But anyway here have an issue, because in any case the IF it should not return any output if not find what you check, and it returns one for each primitives in the linkset. Something is not working well.

(0031596) BillBlight (developer) 2017-02-08 19:19 edited on: 2017-02-08 19:22	Actually the conditions seem to work inside the for statement, just not with that particular statement, if you put it outside the if statement but inside the for it works ... ie. default { state_entry() { llSay(0, "I'm going crazy?"); } touch_start(integer num) { integer i; for(i=0;i<=llGetNumberOfPrims();i++) { string primname =(string)llGetLinkPrimitiveParams(i,[PRIM_NAME]); if(primname == "glass") { llOwnerSay(primname); } } } } Seems like , like lotek said, it is having an issue validating the string against the list. Maybe it returns everything because it is a list in your script so it is not sure where to stop and start evaluating , but something is screwy for sure.

(0031597) BillBlight (developer) 2017-02-08 19:26 edited on: 2017-02-08 19:37	BTW it also works like this .. default { state_entry() { llSay(0, "I'm going crazy?"); } touch_start(integer num) { integer i; for(i=0;i<=llGetNumberOfPrims();i++) { if((string)osGetLinkPrimitiveParams(i,[PRIM_NAME]) == "glass") { llOwnerSay((string)osGetLinkPrimitiveParams(i,[PRIM_NAME])); } } } } or with LL functions. default { state_entry() { llSay(0, "I'm going crazy?"); } touch_start(integer num) { integer i; for(i=0;i<=llGetNumberOfPrims();i++) { if((string)llGetLinkPrimitiveParams(i,[PRIM_NAME]) == "glass") { llOwnerSay((string)llGetLinkPrimitiveParams(i,[PRIM_NAME])); } } } }

(0031599) Total Sorbet (reporter) 2017-02-11 05:31	if(llList2String(llGetLinkPrimitiveParams(i,[PRIM_NAME]),0)=="glass"){llOwnerSay("I promise not to attempt typecasting an entire list again (even if it does consist of just 1 element) :p");}

Relationships