• Listen to a special audio message from Bill Roper to the Hive Workshop community (Bill is a former Vice President of Blizzard Entertainment, Producer, Designer, Musician, Voice Actor) 🔗Click here to hear his message!
  • Read Evilhog's interview with Gregory Alper, the original composer of the music for WarCraft: Orcs & Humans 🔗Click here to read the full interview.

Tell if a rect is overlapping with another

Status
Not open for further replies.
Level 15
Joined
Aug 7, 2013
Messages
1,338
Hi,

I wrote this method to do it, but I think it's logic is wrong, faulty, and possible missing some checks.

I know it's wrong because it returned a false positive. But I'm a bit tired of evaluating all the statements. Can someone correct it?

JASS:
    method isOverlapping takes real cminX, real cmaxX, real cminY, real cmaxY returns boolean
        if (cmaxX >= maxX or cminX <= minX) and (cminY > minY or cmaxY > minY) then
            return true
        elseif (cmaxY >= maxY or cminY <= minY) and (cminX < maxX or cmaxX < maxX) then
            return true
        else
            return false
        endif
    endmethod
 
Status
Not open for further replies.
Top