How do I simplify an IF statement that returns true or false?

Question

    public bool CheckStuck(Paddle PaddleA)
    {
        if (PaddleA.Bounds.IntersectsWith(this.Bounds))
            return true;
        else
            return false;
    }

I feel like the above code, within the procedure, is a bit redundant and was wondering if there was a way to shorten it into one expression. Sorry if I'm missing something obvious.

At the moment if the statement is true, it returns true and the same for false.

So, is there a way to shorten it?

how about just return PaddleA.Bounds.IntersectsWith(this.Bounds); — Inv3r53, Jun 26 '13 at 13:11
Can anyone explain the downvotes? Although a basic question, I don't think it lacks of quality. — Uooo, Jun 26 '13 at 13:16
What's wrong with `return !!PaddleA.Bounds.IntersectsWith(this.Bounds) ? !!(parseInt("14644", 10) === 14644) : !!(parseFloat("567.44", 10) === 436362346);`?? — David Sherret, Jun 26 '13 at 13:19

score 15 · Accepted Answer · answered Jun 26 '13 at 13:11

15

public bool CheckStuck(Paddle PaddleA)
{
    return PaddleA.Bounds.IntersectsWith(this.Bounds)
}

The condition after return evaluates to either True or False, so there's no need for the if/else.

answered Jun 26 '13 at 13:11

Bill the Lizard

369,957
201
546
842

Thanks you! I can't believe I didn't think of that. – Shivam Malhotra Jun 26 '13 at 13:33

score 8 · Answer 2 · answered Jun 26 '13 at 13:11

8

You can always shorten an if-else of the form

if (condition)
  return true;
else
  return false;

to

return condition;

answered Jun 26 '13 at 13:11

Sebastian Redl

61,331
8
105
140

score 1 · Answer 3 · answered Jun 26 '13 at 13:11

1

Try this:

public bool CheckStuck(Paddle PaddleA)
{
    return PaddleA.Bounds.IntersectsWith(this.Bounds);
}

answered Jun 26 '13 at 13:11

rspencer

2,231
2
17
28

score 0 · Answer 4 · answered Jun 26 '13 at 13:12

0

public bool CheckStuck(Paddle PaddleA)
{
    return PaddleA.Bounds.IntersectsWith(this.Bounds);
}

answered Jun 26 '13 at 13:12

Yu Hao

111,229
40
211
267

score 0 · Answer 5 · answered Jun 26 '13 at 13:13

0

public bool CheckStuck(Paddle PaddleA)
    {
        return (PaddleA.Bounds.IntersectsWith(this.Bounds));
    }

Or were you looking for something else?

answered Jun 26 '13 at 13:13

user1676075

2,958
1
17
25

score 0 · Answer 6 · answered Jun 26 '13 at 13:13

0

the below code should work:

public bool CheckStuck(Paddle PaddleA) {
    // will return true or false
   return PaddleA.Bounds.IntersectsWith(this.Bounds); 
 }

answered Jun 26 '13 at 13:13

Apollo SOFTWARE

11,420
4
41
61

How do I simplify an IF statement that returns true or false?

6 Answers6

Linked