Thread: Selection statements are changing the values of my variables!!

It shouldn't be initialized. "oranges[3]" is not a valid array index, and cannot be used other than for a comparison against that location.


Quzah.

Remember, if you declare an array as

Code:

int array[5];

the valid indexes are from 0 to 4, not from 1 to 5.

Re-read Salem's post.

Originally Posted by Salem

> for(i=1;i<=3;i++)
All arrays start at 0, not 1
Your code has already gone wrong when it tried to use oranges[3]

for ( i = 0 ; i < 3 ; i++ )
Would be the correct loop here.

one thing that nobody else picked up on so i'll answer it - there
is a way to get the modulus of a number: there is a function in
stdlib.h called abs () (absolute value) which takes an
integer argument and returns is modulus, or for a more far
reaching version there is fabs () which is defined in math.h
and handles floats. Just so you know.

You're not supposed to post in threads older than 2 weeks.

there
is a way to get the modulus of a number: there is a function in
stdlib.h called abs () (absolute value) which takes an
integer argument and returns is modulus

No, *abs() return a positive number. That is, if the number is positive, it is returned unchanged, and if it is negative it is negated. It's the same as this (or the macro below):

Code:

int sameabs(int x) {
    return x > 0 ? x : -x;
}

Modulus doesn't come into it.

or for a more far
reaching version there is fabs () which is defined in math.h
and handles floats.

Actually, it takes a double and returns a double, but floats work with it too. Or you can write a macro:

Code:

#define ABS(x) ((x) > 0 ? (x) : -(x))