Указатели, недопустимые операнды на двоичные и нубы - PullRequest
Новая
       11

Указатели, недопустимые операнды на двоичные и нубы

1 голос
/ 28 мая 2011

[Правка: определение прямоугольника добавлено внизу.] [Edit2: интерфейс XYPoint добавлен внизу.]

Я работаю над методом, который проверяет, перекрываются ли два прямоугольника. (Да, я нахожусь в программировании Кочана на Objective-C , выполняю упражнения, и я совершенно незнаком с этим.) Когда я компилирую это, появляется сообщение об ошибке: «Недопустимые операнды для двоичного +» , Я получаю его в первом операторе if и в последующем if-else .

Я думаю, что у меня есть проблема с указателями, но Кочан не говорит об этом много.

И, если я уберу эти строки, остальная часть метода будет работать нормально. И соответствующие переменные имеют плавающий тип.

Помощь?

Кроме того, любые другие мысли о методе будут полностью приветствоваться. (Например, как я могу сделать так, чтобы строки кода не выходили так долго. Как я уже сказал, мучительно плохо знакомы с этим.) 

-(void) overlap: (Rectangle *)r2
{

overlapRectangle = [[Rectangle alloc] init];
leftRectangle = [[Rectangle alloc] init];
rightRectangle = [[Rectangle alloc] init];
lowerRectangle = [[Rectangle alloc] init];
upperRectangle = [[Rectangle alloc] init];

BOOL xIntersect = NO;
BOOL yIntersect = NO;


//  Test to see if the Rectangle contains, or is equal to, Rectangle b

if (origin.x <= r2.origin.x && origin.y <= r2.origin.y && (origin.x + width) >= (r2.origin + r2.width) && (origin.y + height) >= (r2.origin.y + r2.height) ) 
{
    overlapRectangle = r2;
}

//  Test to see if Retangle b contains, or is equal to, the Rectangle

else if (origin.x >= r2.origin.x && origin.y >= r2.origin.y && origin.x + width <= r2.origin + r2.width && origin.y + height <= r2.origin.y + r2.height ) 
{
    overlapRectangle = self;
}

//  I should add tests for triangles overlapping on three 
//  sides or overlapping  on two sides, but I'm not going 
//  to right now. Just learning objects and methods.


//  Test to see if rectangles overlap on the x-axis 
//  Current is an if, because I wanted to run the code below
//  to see if it worked, and it did. 

if (origin.x <= r2.origin.x)
{
    leftRectangle = self;
    rightRectangle = r2;
}
else 
{
    rightRectangle = self;
    leftRectangle = r2;
}

if (rightRectangle.origin.x + rightRectangle.width > leftRectangle.origin.x) 
{
    xIntersect = YES;
}



//  Test to see if rectangles overlap on the y-axis

if (origin.y <= r2.origin.y)
{
    lowerRectangle = self;
    upperRectangle = r2;
}
else 
{
    lowerRectangle = self;
    upperRectangle = r2;
}

if (lowerRectangle.origin.y + lowerRectangle.height > upperRectangle.origin.y) 
{
    yIntersect = YES;
}



//  If retangles overlap on both the x-axis and y-axis, 
//  determination of overlapping rectangle's origin, height, and width
//  and display same.

if (xIntersect == YES && yIntersect == YES) 
{
    overlapRectangle.origin.y = upperRectangle.origin.y;
    overlapRectangle.origin.x = rightRectangle.origin.x;
    overlapRectangle.height = lowerRectangle.height - (upperRectangle.origin.y - lowerRectangle.origin.y);
    overlapRectangle.width = leftRectangle.width - (rightRectangle.origin.x - leftRectangle.origin.x);


    NSLog (@"Your rectangles overlap.");
    NSLog (@"Rectangle: w = %g, h = %g", overlapRectangle.width, overlapRectangle.height);
    NSLog (@"Area = %g, Perimeter = %g", [overlapRectangle area], [overlapRectangle perimeter]);
    NSLog (@"Origin at (%g, %g)", overlapRectangle.origin.x, overlapRectangle.origin.y);
}

else 
{
    NSLog (@"Your rectangles do not overlap.");
}



[overlapRectangle autorelease];
[leftRectangle autorelease];
[rightRectangle autorelease];
[lowerRectangle autorelease];
[upperRectangle autorelease];

}

Определение прямоугольника:

// Интерфейс, класс прямоугольника 

@interface Rectangle : NSObject

{
    float width;
    float height;
    XYPoint *origin;

    // For overlapping calculations

    Rectangle *overlapRectangle;
    Rectangle *leftRectangle; 
    Rectangle *rightRectangle; 
    Rectangle *lowerRectangle; 
    Rectangle *upperRectangle; 
}

@property float width, height;

-(XYPoint *) origin;
-(void) setOrigin: (XYPoint *) pt;
-(void) setWidth: (float) w andHeight: (float) h;
-(float) area;        
-(float) perimeter;   
-(void) print;
-(void) translate;
-(void) overlap: (Rectangle *)r2;
-(void) draw;


@end

Интерфейс XYPoint: 

#import <Foundation/Foundation.h>

@interface XYPoint : NSObject
{
    float x;
    float y;
}

@property float x, y;

-(void) setX: (float) xVal andY: (float) yVal;

@end

1 Ответ

3 голосов
/ 28 мая 2011

Вы только что получили, что, вероятно, опечатка: 

// Test to see if the Rectangle contains, or is equal to, 
// Rectangle b

if (origin.x <= r2.origin.x && origin.y <= r2.origin.y && 
   (origin.x + width) >= (r2.origin + r2.width) && 
                         //^^^This is trying to add an XYPoint,
                         // which is an object, to a float.
   (origin.y + height) >= (r2.origin.y + r2.height) ) 
{
   overlapRectangle = r2;
}

// Test to see if Rectangle b contains, or is equal to, 
// the Rectangle
else if (origin.x >= r2.origin.x && origin.y >= r2.origin.y && 
         origin.x + width <= r2.origin + r2.width && 
                            //^^^Same thing.
         origin.y + height <= r2.origin.y + r2.height ) 
{
...

Компилятор должен был сказать вам какие типы были, которые вы просили добавить:

ошибка: недопустимые операнды в двоичном + (имеют «struct XYPoint *» и «float»)

это ключ.Вам просто нужно изменить r2.origin на r2.origin.x, чтобы добавить два числа с плавающей запятой.

Что касается длины строк, вы можете сделать две вещи.Вы можете переместить каждый сегмент условий в разные строки, как я сделал, но, вероятно, было бы лучше создать пару методов для Rectangle, которые будут выполнять тесты для вас.Это сделает код более читабельным, поэтому, когда вы вернетесь к нему через шесть месяцев и увидите строку: 

if( [self containsRectangle:r2] || [self isEqualToRectangle:r2] ){

, вы сразу поймете, что происходит.Вот несколько советов для этого: 

- (BOOL)containsRectangle:(Rectangle *)otherRect {
    BOOL originBelow = ((origin.x <= otherRect.origin.x) && 
                        (origin.y <= otherRect.origin.y));
    float maxX = origin.x + width;
    float otherMaxX = otherRect.origin.x + otherRect.width;
    BOOL maxXGreater = maxX >= otherMaxX;
    Bfloat maxY = origin.y + height;
    float otherMaxY = otherRect.origin.y + otherRect.height;
    BOOL maxYGreater = maxY >= otherMaxY;
    return originBelow && maxXGreater && maxYGreater;
}

- (BOOL)isEqualToRectangle:(Rectangle *)otherRect {
    BOOL sizeEqual = ((width == otherRect.width) && 
                      (height == otherRect.height));
    return sizeEqual && [origin isEqualToXYPoint:otherRect.origin];

}

Примечание: я не проверял их, просто вставил их вместе с условиями ваших if с, поэтому дважды проверьте их, прежде чем использовать их,Тем не менее, я исправил опечатку.

Обратите внимание, что я также придумал метод для XYPoint здесь, isEqualToXYPoint:;Вы также можете реализовать это, чтобы вернуть BOOL, если x и y обоих XYPoint s равны.

...