0

Это из книги Аарона Хиллегаса "Программирование на Objective-C". Упражнение к Главе 19.

Используя класс StockHolding из предыдущего упражнения, напишите программу, которая создает экземпляр класса Portfolio (портфель акций) И заполняет его объектами StockHolding из упражнения предыдущей главы. Объект Portfolio должен уметь вычислять свою текущую стоимость.

Если я просто помещаю этот блок в конец man, то получаю нужную сумму пакетов акций (33772.80):

CGFloat sum = 0;
for (StockHolding *n in portfolio) {
    sum += n.valueInDollars;
}
NSLog(@"%.2f", sum);

Но при переносе данной калькуляции в метод класса Portfolio - при вызове данного метода в main у меня появляется ошибка.

Imgur

Ссылка на проект

Код:

*Portfolio.h*

    #import <Foundation/Foundation.h>
    #import "StockHolding.h"

    @interface Portfolio : NSMutableArray

    - (CGFloat)totalPortfolio;

    @end

*Portfolio.m*

    #import "Portfolio.h"
    #import "StockHolding.h"

    @implementation Portfolio

    - (CGFloat)totalPortfolio
    {
        CGFloat sum = 0;
        for (StockHolding *n in self) {
            sum += n.valueInDollars;
        }
        return sum;
    }

    @end

*main.m*

    #import <Foundation/Foundation.h>
    #import "StockHolding.h"
    #import "Portfolio.h"

    int main(int argc, const char * argv[])
    {

        @autoreleasepool {


            StockHolding *DHL = [[StockHolding alloc] init];
            [DHL setNameOfShare:@"DHL"];
            [DHL setPurchaseSharePrice:345.67];
            [DHL setNumberOfShares:23];
            [DHL setCurrentSharePrice:389.23];

            StockHolding *Sony = [[StockHolding alloc] init];
            [Sony setNameOfShare:@"Sony"];
            [Sony setPurchaseSharePrice:587.12];
            [Sony setNumberOfShares:14];
            [Sony setCurrentSharePrice:603.81];

            StockHolding *EPAM = [[StockHolding alloc] init];
            [EPAM setNameOfShare:@"EPAM"];
            [EPAM setPurchaseSharePrice:178.45];
            [EPAM setNumberOfShares:35];
            [EPAM setCurrentSharePrice:190.64];

            StockHolding *BELAZ =[[StockHolding alloc] init];
            [BELAZ setNameOfShare:@"BELAZ"];
            [BELAZ setPurchaseSharePrice:24.43];
            [BELAZ setNumberOfShares:89];
            [BELAZ setCurrentSharePrice:108.93];

            Portfolio *portfolio = [[NSMutableArray alloc] initWithObjects:DHL, Sony, EPAM, BELAZ, nil];


            NSLog(@"%.2f", [portfolio totalPortfolio]);

        }
        return 0;
    }

1 ответ 1

1

А вы прочитайте сообщение об ошибке в консоли:

*** Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: '*** -[NSMutableArray initWithCapacity:]: method only defined for abstract class.  Define -[Portfolio initWithCapacity:]!

Суть в том, что, вызывая -initWithObjects:, вы на деле вызываете и метод -initWithCapacity:, который, как видно из лога, должен быть написан отдельно для каждого дочернего класса NSArray.

Вообще, наследование от NSArray — дело не такое уж и простое, потому что это не обычный класс, а так называемый Class Cluster: класс-обёртка над кучей других классов. О правильном наследовани от абстрактных классов можно прочитать в этой замечательной статье.


Что касается решения вашей проблемы: я бы не стал мудрить и определил бы класс Portfolio следующим образом:

@interface Portfolio : NSObject
{
    NSArray *_stocks;
}

// У нас может быть различное количество аргументов; главное, чтобы
// последним шёл `nil`.
- (id)initWithStocks: (id)first, ... NS_REQUIRES_NIL_TERMINATION;
- (CGFloat)totalValue;

@end

/* ------------ CUT HERE --------------- */

@implementation Portfolio

- (id)initWithStocks: (id)first, ...
{
    if ((self = [super init])) {
        if (!first) {
            return self;
        }

        NSMutableArray *tmp = [NSMutableArray array];

        // Работаем со списком аргументов
        va_list args;
        va_start(args, first);
        StockHolding *stock = first;

        while (stock != nil) {
            [tmp addObject: stock];
            stock = va_arg(args, StockHolding*);
        }
        va_end(args);

        _stocks = [tmp copy];
    }

    return self;
}

- (CGFloat)totalValue
{
    CGFloat sum = 0.0;
    for (StockHolding *stock in _stocks) {
        sum += stock.valueInDollars;
    }

    return sum;
}

@end

Ну и в main.m прийдётся изменить вызов на

    Portfolio *portfolio = [[Portfolio alloc] initWithStocks: DHL, Sony, EPAM, BELAZ, nil];

    NSLog(@"%.2f", [portfolio totalValue]);

P.S. разумеется, можно было пойти ещё дальше и упростить всё до ужаса, реализовав протокол NSFastEnumeration для нашего класса, чтобы можно было писать прямо:

    for (StockHolding *stock in self) {
        sum += stock.valueInDollars;
    }

и на самом деле это совсем не сложно, но не так интересно (^^,)


Ссылки по теме:

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.