Exercise 8.7 (Comments welcome)

[apexmutt]

Just looking for complaints, comments, criticism, etc...

Code: (Objective-C)

-(Rectangle *) intersect:(Rectangle *)aRect
{
    double nonOriginX, nonOriginY, nonOriginWidth, nonOriginHeight;
    Rectangle *interRect = [[Rectangle alloc] init];
    XYpoint *initXY =[[XYpoint alloc] init];
    
    [interRect setOrigin: initXY];
    
    if(origin.x > aRect.origin.x)
    {
        interRect.origin.x = origin.x;
        nonOriginX = aRect.origin.x;
        nonOriginWidth = aRect.width;
    }
    else
    {
        interRect.origin.x = aRect.origin.x;
        nonOriginX = origin.x;
        nonOriginWidth = width;
    }
    
    if(origin.y > aRect.origin.y)
    {
        interRect.origin.y = origin.y;
        nonOriginY = aRect.origin.y;
        nonOriginHeight = aRect.height;
    }
    else
    {
        interRect.origin.y = aRect.origin.y;
        nonOriginY = origin.y;
        nonOriginHeight = height;
    }
    
    
    //Test to see if an intersect exists and if so, calculate width/height and return Rectangle object
    
    if (nonOriginX + nonOriginWidth > interRect.origin.x && nonOriginY + nonOriginHeight > interRect.origin.y) 
    {
        interRect.width = (nonOriginX + nonOriginWidth) - interRect.origin.x;
        interRect.height = (nonOriginY + nonOriginHeight) - interRect.origin.y;
        return interRect;
    }
    else
    {
        interRect.height = 0;
        interRect.width = 0;
        interRect.origin.x = 0;
        interRect.origin.y = 0;
        
        return interRect;
    }}
@end