Как правильно создать статические NSArrays, полные NSNumbers в Objective-C (Какао)?

Question

Почему в следующем коде я не могу просто сделать статический массив из NSNumbers? Я бы просто использовал массивы C и int, но их нельзя скопировать, и, как вы можете видеть в init (), я должен скопировать массив в другой. Я получаю сообщение об ошибке: «Элемент инициализатора не является постоянным». Это очень запутанно; Я даже не уверен, что это значит, учитывая, что у меня нет ни одного ключевого слова const.

Кроме того, в качестве идентификатора, метод getNextIngredient выдает ошибку «невозможно использовать объект в качестве параметра для метода» и «несовместимые типы в ответ», но я не уверен почему.

Вот код:

// 1 = TOMATO
// 2 = LETTUCE
// 3 = CHEESE
// 4 = HAM

#import "Recipe.h"




@implementation Recipe

// List of hardcoded recipes
static NSArray *basicHam = [[NSArray alloc] initWithObjects:[[NSNumber alloc] numberwithInt:1], [[NSNumber alloc] numberwithInt:2], [[NSNumber alloc] numberWithInt:3], [[NSNumber alloc] numberwithInt:4]];

// Upon creation, check the name parameter that was passed in and set the current recipe to that particular array.
// Then, set nextIngredient to be the first ingredient of that recipe, so that Game can check it.
-(id) initWithName: (NSString*)name {
    self = [super init];

    indexOfNext = 0;

    if (self) {
        if ([name isEqualToString: @"Basic Ham"]) {
            currRecipe = [NSArray arrayWithArray: basicHam]; 
        }                                
    }
}

-(NSNumber) getNextIngredient {
    return [currRecipe  objectAtIndex:indexOfNext];
}

Answer 1

В наше время вы бы использовали dispatch_once() для одноразовой инициализации. Xcode имеет удобный шаблон, встроенный для выполнения именно этого.

<Ч />

NSArray никогда не является статически размещенным объектом и, следовательно, не может быть инициализатором статической переменной.

Сделайте что-то вроде:

@implementation Recipe

+ (NSArray *) basicHam {
    static NSArray *hams;
    if (!hams) 
        hams = [[NSArray alloc] initWithObjects:[NSNumber numberwithInt:1], [NSNumber numberwithInt:2], [NSNumber numberWithInt:3], [NSNumber numberwithInt:4], nil];
    return hams;
}

<Ч />

Однако обратите внимание на пару вещей:

• Я немного изменил ваш код. Вы не выделяете, тогда numberWithInt: и NSNumber. Это не сработает.

• Я добавил nil в конце списка аргументов. Это необходимо.

И, тем не менее, следует заметить, что массив, который эффективно содержит небольшой набор натуральных счетных чисел в порядке без пробелов, довольно явно странен. Каждый раз, когда x = foo[x] является выражением идентичности, оно обычно указывает, что в используемых шаблонах есть что-то явно странное.

Answer 2

Классический способ сделать это с помощью метода + initialize:

static NSArray *basicHam;

@implementation Recipe

+ (void)initialize {
    if (self == [Recipe class]) {
        basicHam = [[NSArray alloc] initWithObjects:[NSNumber numberWithInt:1], [NSNumber numberWithInt:2],
                                                    [NSNumber numberWithInt:3], [NSNumber numberWithInt:4, nil]];
    }
}

Альтернатива, которая работает, если вам нужно это в C, а не присоединяется к классу Obj-C, выглядит примерно так:

static NSArray *basicHam;

static void initBasicHam() __attribute__((constructor)) {
    basicHam = [[NSArray alloc] initWithObjects:[NSNumber numberWithInt:1], [NSNumber numberWithInt:2],
                                                [NSNumber numberWithInt:3], [NSNumber numberWithInt:4, nil]];
}

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

Answer 3

Это можно сделать потокобезопасным способом с помощью dispatch_once. Например:

- (NSArray *)staticSpeeds {
    static dispatch_once_t onceToken;
    static NSArray *speeds;
    dispatch_once(&onceToken, ^{
        speeds = [NSArray
                  arrayWithObjects:[NSNumber numberWithFloat:1.0], nil];
    });
    return speeds;
}

Answer 4

вот более подробный пример (который также использует описанную идиому bbum какао).он указывает на несколько других ошибок и обращается к вашей sidenote:

/* Recipe.h */

@interface Recipe : NSObject
{
    NSUInteger indexOfNext;
    NSArray * currentRecipe;
}

- (id)initWithName:(NSString *)name;
- (id)initWithBasicHam;

- (NSNumber *)getNextIngredient;

@end

extern NSString * const Recipe_DefaultRecipeName_BasicHam;

/* Recipe.m */

NSString * const Recipe_DefaultRecipeName_BasicHam = @"Basic Ham";

@implementation Recipe

/* @return list of hardcoded recipes */
+ (NSArray *)basicHam
{
    // there may be a better place to declare these
    enum { TOMATO = 1, LETTUCE = 2, CHEESE = 3, HAM = 4 };
    static NSArray * result = 0;
    if (0 == result) {
        result = [[NSArray alloc] initWithObjects:[NSNumber numberWithInt:TOMATO], [NSNumber numberWithInt:LETTUCE], [NSNumber numberWithInt:CHEESE], [NSNumber numberWithInt:HAM], nil];
    }
    return result;
}

/* Upon creation, check the name parameter that was passed in and set the current recipe to that particular array. */
/* Then, set nextIngredient to be the first ingredient of that recipe, so that Game can check it. */
- (id)initWithName:(NSString *)name
{
    self = [super init];
    if (0 != self) {
    /* note: set your ivar here (after checking 0 != self) */
        indexOfNext = 0;

        if ([name isEqualToString:Recipe_DefaultRecipeName_BasicHam]) {
            currentRecipe = [[Recipe basicHam] retain];
        }
    }
    return self;
}

- (id)initWithBasicHam
{
    self = [super init];
    if (0 != self) {
        indexOfNext = 0;
        currentRecipe = [[Recipe basicHam] retain];
    }
    return self;
}

- (NSNumber *)getNextIngredient
{
    assert(currentRecipe);
    return [currentRecipe objectAtIndex:indexOfNext];
}

@end

вместо строковых литералов, может быть лучше создать строковые ключи, а также использовать вспомогательные конструкторы, такие как - (id)initWithBasicHam (какпродемонстрировано).

Кроме того, вы обычно звоните [[Recipe basicHam] copy], а не Recipe basicHam] retain] - но это не обязательно в данном конкретном примере.