Filter löschen
Filter löschen

error in arrays line 7

1 Ansicht (letzte 30 Tage)
Ronald Aono
Ronald Aono am 29 Sep. 2019
Kommentiert: Adam Danz am 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 Kommentare
Adam Danz
Adam Danz am 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 am 29 Sep. 2019
Bearbeitet: Adam Danz am 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);

Melden Sie sich an, um zu kommentieren.

Antworten (2)

Adam Danz
Adam Danz am 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 am 30 Sep. 2019
Bearbeitet: Walter Roberson am 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 Kommentar
Adam Danz
Adam Danz am 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.

Melden Sie sich an, um zu kommentieren.

Kategorien

Mehr zu Matrices and Arrays finden Sie in Help Center und File Exchange

Community Treasure Hunt

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

Start Hunting!

Translated by