This is sound logic but notice how there seems to be a lot of logic on this line. Next time you see a bunch of logic, try to outsource pieces of it by writing well-named functions. Then in this line, call those functions.

For example:

```
(@points - @opponent.points) == 1
```
can be the only line in a ```def player_ahead?``` function that returns a true/false boolean. 

Then in this line, your code looks more like english ```if player_ahead? && blah_blah?```