error in arrays line 7

1 view (last 30 days)
Ronald Aono
Ronald Aono on 29 Sep 2019
Commented: Adam Danz on 30 Sep 2019
function V = fuelvol2(h)
global r H L
validInput = true; %to test each value in the vector h is valid
for i = 1:length(h)
Nh = h(i);
if (Nh < 0 && (Nh > h + 2*r))
validInput = false;
break;
end
end
if validInput == true % if it is valid vector the calculate volume using vectorized operations
d = h-H;
V = 2*r*L*H + (r^2 * acos((r-d)/r) - (r-d).*sqrt(2*r.*d -d.^2))*L;
else
disp("one of more values in vector his/are valid ")
end
I keep getting tis error each time i try to run the code
Not enough input arguments.
Error in fuelvol2 (line 7)
for i = 1:length(h)
  6 Comments
Adam Danz
Adam Danz on 29 Sep 2019
Function calls are case sensitive. There is no matlab function named Length() but there is a function with the lower case length(). However, it's recommended to use numel() instead of length().
Ronald Aono
Ronald Aono on 29 Sep 2019
Edited: Adam Danz on 29 Sep 2019
Thank you for the length error, i appriciate it . how do i clear this error to, it keeps creeping up
Operands to the || and && operators must be convertible to logical scalar values.
Error in fuelvol1 (line 7)
if (h>=0) && (h <= (H + 2*r))
Error in main_fuelvol12 (line 7)
V = fuelvol1(h);

Sign in to comment.

Answers (2)

Adam Danz
Adam Danz on 29 Sep 2019
It's getting hard to follow the discussion because we're jumping around between different similarly named functions. fuelvol1, fuelvol2, main_fuelvol12.
This error appears to be in fuelvol1 but we don't have access to that function. However, I think the error can be fixed by replacing && with &.

Walter Roberson
Walter Roberson on 30 Sep 2019
Edited: Walter Roberson on 30 Sep 2019
for i = 1:length(h)
So you are expecting h to be a non-scalar.
if (Nh < 0 && (Nh > h + 2*r))
You add the non-scalar h to something and compare it to the scalar Nh, producing a non-scalar result on the right hand side of the && . However, the && operator can only be used when both sides are scalars.
Changing to & is not the correct solution. The correct solution is to stop using single-letter variable names that differ only in upper versus lower case, because it is too easy to get the wrong variable name. You do not want h there, you want H .
In your other question, H and 2*r were both positive, and Nh > some_positive_value cannot be simultaneously true with Nh < zero.
  1 Comment
Adam Danz
Adam Danz on 30 Sep 2019
Just curious, from where did you get "for i = 1:length(h)"? I see it in fuelvol2() but the error is in fuelvol1() which we don't have access to, unless I'm missing something.

Sign in to comment.

Categories

Find more on Matrices and Arrays in Help Center and File Exchange

Community Treasure Hunt

Find the treasures in MATLAB Central and discover how the community can help you!

Start Hunting!