compare 2 values to 1 value in c#

Question

Any easier way to write this if statement

if (value1 <= 0 || value2 <= 0)

For example some think like if ((value1 || value2) <= 0

Just.. why? `value1 <= 0 || value2 <= 0` seems pretty clear. — Soner Gönül, Sep 12 '14 at 07:47
How much easier do you want it? even your "improvement" is 3 comparisons (at most) — Sayse, Sep 12 '14 at 07:48
is just if i have 10 differents values and its the same number i want to compare it to and i change the value i need to do it 10 times — Rene Nielsen, Sep 12 '14 at 07:51
A quick look at the ridiculous solutions in the answers to the duplicate question should reassure you that the way you have is the best way. — Cody Gray - on strike, Sep 12 '14 at 07:51

score 3 · Accepted Answer · answered Sep 12 '14 at 07:49

3

No, your way is correct. If you have more than two values and you want to know if any of them is less than or equal to zero use an array

if(new [] { value1, value2, value3, ... }.Any(x => x <= 0))

answered Sep 12 '14 at 07:49

Selman Genç

score 0 · Answer 2 · answered Sep 12 '14 at 07:48

0

in this case :

if (value1 <= 0 || value2 <= 0)

if value1 <=0 is true then it will short circut the evaluation and directly go inside the if loop as

1 || 0 = 1
1 || 1 = 1

answered Sep 12 '14 at 07:48

Sagar Pilkhwal

2 Answers2