Using 'for each' to modify a container
wjm at wjm.org
Wed Oct 11 18:05:54 EDT 2006
> I think the reason that 'for each' is so fast is that it scans the
> "vContainer" memory space, returns those characters, and is not meant to
> modified. My experience is that it is unpredictable to rely on the
> value of 'thisLine'
Wow, this is very interesting to me. I just checked the documentation
(2.6.1) on the "for each chunkType labelVariable" for of repeat, and was
astonished to find:
Important! You cannot change the labelVariable in a statement inside the
loop. Doing so will cause a script error. You can change the content of the
container, but doing so will probably produce unexpected results.
Ya learn something new every day! Sounds kind of cheesy to me, though. Why
wouldn't it be identical to
repeat with i = 1 to the number of chunkType in theContainer
put chunkType i of theContainer into labelVariable
-- do stuff
Your explanation is plausible, but if it is so then why permit ("You can
change...") changing the contents of the container itself within the loop?
That would seem to cause an even bigger problem.
BTW, I set up a button with the following script:
put empty into newContainer
repeat for each line thisLine in fld 1
put thisline & "***" into thisLine
put thisLine & return after newContainer
put char 1 to -2 of newContainer into fld 1
Which not only didn't generate a script error but worked perfectly as
"expected." Is the bug in the documentation, or that it doesn't generate a
script error, or that for each doesn't let you modify the labelVariable?
And I'm *really* concerned about the second part of your report
> Further, I don't reuse the name 'thisLine' in the next instance of a
> for each loop in the *same handler*, but change the name to avoid
> such as 'thisline', 'thisline1', 'thisLine2'. Again, unpredictable
Surely THAT is a definite bug?! Have other people experienced this? Are you
able to reproduce it? Is there a bug on it already filed? I do this all the
time and I'd have to update a lot of code to fix it.
This would suggest to me that maybe I shouldn't use "i" more than once in a
handler as the control variable for a repeat as well (something I always do,
unless nesting the loops of course).
I won't even get into the idea of declaring the labelVariable as a global!
> --my preference is
> put empty into newContainer
> repeat for each line thisLine in vContainer
> put doSomeFunction(item -2 of thisLine)&cr after newContainer
> end repeat
> put char 1 to -2 of newContainer into vContainer
More information about the use-livecode