Where is the logic failure in my code ?

Hi All

I have a matrix n x 1 called X , in which I want to move it up or down along vertical axis when I plot it vs time , in a way to make the mean value = 0 if it's not so I created a true false parameter as :

useMean 

than can be

True

or

False

and then in the code I do :

if useMean == 'True'
    if meanX <0
        X= X+abs(meanX);
    else
        X= X-abs(meanX);
    end
elseif useMean == 'False'
    X=X;
end

but I noticed the code only can perform when useMean == 'True' , not the 'False' !

this way my rest of the parameters can not be calculated

[Walter Roberson]

Unless you know that useMean is a string() object, you should use strcmp() instead of ==

Also, what value can useMean have other than True or False ? If all of the values are certain to be True or False then you do not need "elseif" and can just use "else"

[farzad]

I needed useMean only as a boolean And I also tried else, but also that one doesn't work

[farzad]

I got it , I didn't have to write 'True' but simply true, cause true gives logical value 1

[Walter Roberson]

Your code can be shortened to

if useMean
    X = X - meanX;
end

with no abs() needed.

Answer 1

[Image Analyst]

Use true or false, not 'True' or 'False'. No quotes or capitalization required.

Or don't even use them at all. Your code could be:

if useMean
    if meanX <0
        X= X+abs(meanX);
    else
        X= X-abs(meanX);
    end
else
    X=X; % Even this is not needed.
end

but best would be to use Walter's code in the last comment above.