whats wrong with my code?

1 次查看（过去 30 天）
daniel
daniel 2023-1-1
评论： Image Analyst 2023-1-1
function v=velocity(x,y,z,t)
if and (length(x)==length(y), length(y)==length(z))
if length(z)==length(t)
v=sqrt(diff(x).^2+diff(y).^2+diff(z).^2)./diff(t);
disp(velocity,sqrt)
else
disp('Vectors must have the same length for calculation');
end
end
  1 个评论
Voss
Voss 2023-1-1
What is your intent with this line?
disp(velocity,sqrt)
That calls the function velocity() again, with no arguments (and the function sqrt() with no arguments), which causes an error.
Maybe you mean this instead?
disp(v)

请先登录，再进行评论。

请先登录，再回答此问题。

回答（1 个）

Image Analyst
Image Analyst 2023-1-1
编辑：Image Analyst 2023-1-1
Well #1 is the lack of comments. All programmers should comment their code, and you should too.
Secondly, this
if and (length(x)==length(y), length(y)==length(z))
should be
if (length(x)==length(y)) && (length(y)==length(z))
And the very first line inside the function should be
v = [];
so that you at least return something if the length test fails.
  3 个评论
显示 1更早的评论
Torsten
Torsten 2023-1-1
disp(velocity,sqrt)
This statement does not make sense since "velocity" is the name of the function, but not of a variable that could be displayed.
Further, if such a variable existed, it had to be
disp(sqrt(velocity))
instead of
disp(velocity,sqrt)

请先登录，再进行评论。

类别

在 Help Center 和 File Exchange 中查找有关 Loops and Conditional Statements 的更多信息

标签

产品


版本

R2022b

Community Treasure Hunt

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

Start Hunting!

Translated by