0

我不完全确定为什么它在注释行返回错误“不是 Objective-C 对象”。任何帮助，将不胜感激。

此外，我对 Objective-C 非常陌生，我确实意识到这很有可能是一个非常愚蠢的错误。但是，任何建议都会有所帮助。

#import "CalculatorBrain.h"

@interface CalculatorBrain()
@property (nonatomic, strong) NSMutableArray *operandStack;
@end

@implementation CalculatorBrain

@synthesize operandStack = _operandStack;

- (NSMutableArray *)operandStack
{
    if(!_operandStack){
    _operandStack = [[NSMutableArray alloc] init];
    }// end if
    return _operandStack;
}//end operandStack

- (void)pushOperand:(double)operand
{
    NSNumber *operandObject = [NSNumber numberWithDouble:operand];
    [self.operandStack addObject:operandObject];
}//end pushOperand

- (double)popOperand
{
    NSNumber *operandObject = [self.operandStack lastObject];//error "Not an objective-c object"
    if(operandObject) [self.operandStack removeLastObject];
    return [operandObject doubleValue];
}//end popOperand

- (double)performOperation:(NSString *)operation
{
    double result = 0;

    if([operation isEqualToString:@"+"]){
    result = [self popOperand] + [self popOperand];
} else if([operation isEqualToString:@"-"]){
    double subtrahend = [self popOperand];
    result = [self popOperand] - subtrahend;
} else if([operation isEqualToString:@"*"]){
    result = [self popOperand] * [self popOperand];
} else if([operation isEqualToString:@"/"]){
    double divisor = [self popOperand];
    if(divisor) result = [self popOperand] / divisor;
}//end if
[self pushOperand:result];
return result;

}//performOperation

@end
4

1 回答 1

0

我认为这可能是由于函数 popOperand 中的操作顺序不正确造成的。看到我的评论内联，最后一行 return [operandObject doubleValue] 访问了一个已经发布的对象。多次运行此代码后，可能会导致内存问题，反过来，您可能会在注释行看到错误。

- (double)popOperand
{
    NSNumber *operandObject = [self.operandStack lastObject];
    if(operandObject) [self.operandStack removeLastObject];
    // When you remove the lastObject(operandObject) from the Array, the operandObject retainCount is zero.
    // Here, the operandObject is deallocated and should not be used.
    // But you call it after it was released.
    return [operandObject doubleValue];
}//end popOperand

当您只调用一次或多次 popOperand 时，这可能没问题。但这会导致内存错误。我猜每次调用 popOperand 时都不会发生错误。这是我的解决方案。

- (double)popOperand
{
    double result = 0.0;
    NSNumber *operandObject = [self.operandStack lastObject];
    if(operandObject) 
    {
        result = [operandObject doubleValue];
        [self.operandStack removeLastObject];
    }
    return result;
}
于 2012-08-03T02:48:07.333 回答