if-statement with conditions.

Write a function called picker that takes three input arguments called condition, in1 and in2 in this order. The argument condition is a logical. If it is true, the function assigns the value of in1 to the output argument out, otherwise, it assigns the value of in2 to out.

This is a problem from a course that I am taking on Coursera. I get a correct output for "true" but the output for the "false" condition is always wrong. Please, any kind of help will highly appreciated.

Here is my code:

function out = picker(condition,in1,in2)

if  true
    out = in1;
else
    out = in2;
end
    
end

[Mukti Awad]

This code gives error

Assessment result: incorrectFalse

Variable out has an incorrect value.

[Sneham Shrikant Vaidya]

function out = picker(condition,in1,in2)
if condition == 1;
    out = in1;
else
    out = in2;
end




\\try this code

[Purushottam Shrestha]

right.

[Jaideep Singh]

function out=picker(condition,in1,in2)
if condition==true
    out=in1;
else
    out=in2;
end

it is a simplest code that can execute your program.

[Walter Roberson]

No, that is not the simplest ;-)

[Aniket Kumar]

Jaideep has written the right code

[Harish C S]

this is wrong because everytime matlab thinks it is true

[Rik]

@Harish C S That is why it is a question. If it were correct, why bother posting the question? See the comments and answers in this thread for working code.

Answer 1

[Star Strider]

You can safely delete this assignment:

condintion = in1 < in2;

because it overwrites the ‘condition’ argument.

You can also just use:

if conditon

You don’t have to test it again.

Also, check your spelling!

[Mukti Awad]

Correct code is:

function out = picker(condition,in1,in2)
if  condition
    out = in1;
else
    out = in2;
end
    
end

[Miguel Viñeza]

thankyouu!!

[Priya Narayan]

This code is not running.

[Rik]

Which code? And as Star Strider suggests: check your spelling, including in the code he posted.

Answer 2

[Sneham Shrikant Vaidya]

function out = picker(condition,in1,in2)
if condition == 1;
    out = in1;
else
    out = in2;
end



\\try this above code

Answer 3

[Ahmed Salmi]

function out=picker(condition,in1,in2)        
if  condition==true
    out = in1;
elseif condition==false
    out = in2;
end
  end

[Walter Roberson]

Under what circumstances can it be the case that the condition is not true and also is not false? When you use elseif you imply that there are cases where nothing that has been tested before came out true and also that the current test might not come out true either.

Answer 4

[suat karabocek]

you may use if and else and two conditions including 1 and 0. such as;

your function.........

if condition == 1;

....

....

else condition == 0;

....

....

end

[KETAN PATEL]

Thanks!!!

[Selvakumar T]

function out = picker(a,b)
if(a<b)
    fprintf("%d",a);
else
    fprintf("%d",b);
end

what the problem on this code

[Ahmed J. Abougarair]

The picker function required three input argument

[DGM]

You may use two if conditions with numeric comparison, but why should you?

if condition == 1
    out = in1;
elseif condition == 0
    out = in2;
% else
    % all other cases are unhandled and will cause the function 
    % to exit without assigning a value to the output
end

The assignment asserts that condition is a variable of class 'logical'. While the comparisons using numeric values will implicitly work with logical inputs, you're designing your code around handling logical variables as numeric variables. Your code now accepts any numeric input and will fail if it is neither 0 nor 1.

If your inputs are supposed to be logical, make sure they're logical or converted to logical.

Also, an else statement doesn't accept a condition; an elseif does.

Answer 5

[amjad khan]

function out(condition,in1,in2)
if condition>0 % it is logical
    out=in1;
else
    out=2;
end

[DGM]

The only significant difference between this code and the other examples is that this one doesn't actually work. It may be a minor and obvious error, but the fact that it's obvious indicates that this wasn't tested.

Why post untested code that's not meaningfully different than other examples?

You do have one comment in the code, so I'll give you credit for that.