So I need help with another project for my comp sci class, so we have to find the min and max of two numbers
seems easy but we have to use a reporter function
so my code says:
<When green flag is clicked
ask “what is the first number”
set first number to answer
ask “what is the second number”
say maximum for 2 seconds>

then I made a reporter function for maximum which says;

<Maximum
script variable “return value”
if value 1 < value 2
set value 2 to return value
else
if value 1 > value 2
set value 1 to return value>

report “return value”
what did I do wrong?

mnh78614 wrote:

if value 1 < value 2
set value 2 to return value
else
if value 1 > value 2
set value 1 to return value>

report “return value”
what did I do wrong?

Isn't the “set <variable> <newValue>” mixed up here?

michael42 wrote:

mnh78614 wrote:

if value 1 < value 2
set value 2 to return value
else
if value 1 > value 2
set value 1 to return value>

report “return value”
what did I do wrong?

Isn't the “set <variable> <newValue>” mixed up here?

Yep, you are correct. You need to set the return value to either value 1 or 2, not the other way around. So:

set [return value v] to [Whatever you return.]

With reassurances,

ErnieParke

so I made the changes, but for a response I'm getting zero! I need help this is due tonight!!

mnh78614 wrote:

so I made the changes, but for a response I'm getting zero! I need help this is due tonight!!

Okay, did you do your tests for min/max after you got your values?

With questions,

ErnieParke

Can you share your project?

Here's the picture, I'm using a program called BYOB it's the same exact thing as stractch, the only difference is the sprite.

mnh78614 wrote:

Here's the picture, I'm using a program called BYOB it's the same exact thing as stractch, the only difference is the sprite.

Wait, aren't number1 and number2 supposed to be drop-down menus? I feel as if you're assigning the variable 0 what's really supposed to go into the two number vars, therefore ruining your project.

With questions,

ErnieParke

well i made that change, now variables “number 1” and “number 2” are drop downs. Still getting zero.

mnh78614 wrote:

Here's the picture, I'm using a program called BYOB it's the same exact thing as stractch, the only difference is the sprite.

It is the exact same thing as stratch except where it is different (which appears to be a lot of places). From a syntax,

set (number 1) to (answer)  //scratch won't let you do this

set [number 1 v] to (answer) //it makes you do this

And I notice that that number 1 is a different color than the ones used in the IF. Is it really the same number 1?

We need a new picture

Yes I did that, still getting zero as an answer

Need a new picture

Also, can you

 
say (return value) for [2] secs //at the end of the maximum function

if that doesn't tell you anything useful, say number 1 and number 2 at the start of the maximum function.

http://imageshack.us/f/14/pxd0.jpg/ new pic

poof

You are setting number 1 twice

changed that, still getting zero as my return value!!!

Need picture of both scripts and can you do the SAYs I mentioned earlier?

Nvm fixed it somehow, now I need to do max of 3 numbers.

OK - I will check back occasionally to see if you need any help.