float changes to int?

[WesHowe]

When you declare a variable within a block (a pair of curly braces) the scope is limited to that block, and any blocks nested within that. If you look at your code, you do not declare the float $maxYJoint until after the first opening curly brace of your for() loop.

Your second print statement is after the matching closing curly brace, meaning that the float declaration is gone, it is out-of-scope, and you are getting a newly created int variable as a default. The fix should be to move the declaration above the first curly brace.

<* Wes *>

[Nyro]

I'm surprised the first print prints anything at all, since you just declare an empty float variable $maxYJoint and then proceed to compare some float to it ($joinXFormY > $maxYJoint)

Bu that aside, there's something else in your script that you should take alook at:

If at all possible, never never use the select command to get lists of objects. I did it too in my early days because I hadn't realised that you can pass a list of objects to virtually any command; they don't just operate on the active list.
Selecting is a very slow process in MEL (sometimes slow even in realtime terms, not just floating point calculation terms), so it unnecessarily slows down your script once you have lots and lots of joints.

Its funny anyway, if I may say so: You select all joints using the `ls` command, but then you list all the joints using the selection (that used the `ls`). It's like writing ls(`ls -type joint`);

Instead of writing

select -r `ls -type joint`;

$jointChain = `ls -sl`;

just write

$jointChain = `ls -type joint`;

[benio33]

Exactly like guys said - both WesHowe and Nyro were correct. When I ran your code I got "Error: Line 39.26: "$maxYJoint" is an undeclared variable. " error. It basically means you had the maxYJoint stored in the memory, so you got "1" all the time. This one works though.

$jointChain = `ls -type joint`; float $maxYJoint;
string $maxJoint;
for ($joints in $jointChain)
{
$jointXForm = `xform -q -ws -t $joints`;
float $jointXFormY = $jointXForm[1];
if ($jointXFormY > $maxYJoint)
{
$maxJoint = $joints;
$maxYJoint = $jointXFormY;
print ($maxJoint +"\n");
print ($maxYJoint +"\n");
}
}
print ($maxYJoint +"\n");

[SAmunategui]

heh, thanks guys

More than answered my question. I didn't realize variables within blocks get limited to that block. I'm surprised I haven't run into this before. Clearly i'm not a pro (yet), but i've written a handful of things before

Also, not sure why I keep overlooking putting selections into one line. I pretty much do that for anything else I do :)

[CGTalk Moderation]

This thread has been automatically closed as it remained inactive for 12 months. If you wish to continue the discussion, please create a new thread in the appropriate forum.