What is wrong with my code in writing a for loop function

function mysums(v)
sum_even=0;
sum_odd=0;
i=1;
for k = v
if mod(i,2)==0 & i<= size(v)
sum_even = sum_even + i;
else if i <= size(v)
sum_odd = sum_odd + i;
end
i = i+1;
end
disp([sum_odd , sum_even])
end
The loop is to sum odd nums and even nums from a vector input

 Accepted Answer

Jan
Jan on 12 Feb 2017
Edited: Jan on 12 Feb 2017
size(v) replies a vector, such that i <= size(v) will not do, what you expect.
USe the automatic code indentation to find the problem of using "else if" instead of "elseif" immediately.
Prefer to write a loop like:
for i = 1:numel(v)
if mod(i, 2) == 1
...
else
...
Use v(i) to get the current element, not sum_odd = sum_odd + i, because you do not want to collect the indices - when I assume that your other question contains the text of this homework.

More Answers (0)

Categories

Find more on Programming in Help Center and File Exchange

Tags

Asked:

on 12 Feb 2017

Edited:

Jan
on 12 Feb 2017

Community Treasure Hunt

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

Start Hunting!