What else could I use instead of this IF statement?

[dsured]

I am just curious if there is any other way I could write this code, I see 3 if statements in a row and am wondering if there is a more efficient way.

if (number1<0)
number1*=-1;
if (number2<0)
number2*=-1;
if (number2>=number1)
c=b;

[cyberfish]

That looks fine.

It's clean and readable.

Don't try to use weird obfuscated constructs to try to make it "faster", because, 1) it probably won't be, and 2) it will just make your code unreadable and unmaintainable.

[CommonTater]

3 in a row is nothing... some programs have dozens.

Unless there's something in common, such as differing values of the same variable. What you got is about as good as it gets.

[dsured]

I should have included this,

s=values[j]+values[l];
number1=r-t;
number2=r-s;
if (number1<0)
number1*=-1;
if (number2<0)
number2*=-1;
if (number2>=number1)
c=b;

I tried changing it up to:

s=values[j]+values[l];
number1 = abs(r-t);
number2 = abs(r-s);
if (number2>=number1)
c=b;

However it doesnt work, it says: implicit declaration of function 'abs'
What is happening?

[Adak]

You didn't include the header file that abs() needed. Not sure, but probably that is <math.h> that you need.

Compile and link that, and see how it works then.

[dsured]

I did include math.h in the header.

[tabstop]

abs is in <stdlib.h>

[kona49er]

I did include math.h in the header.

your mileage may vary but, if your using an unix environment try the following cli command:

$ man abs
ABS(3) NEWLIB

NAME
2.4 `abs'--integer absolute value (magnitude)

SYNOPSIS
#include <stdlib.h>
int abs(int I);
...