[NSArray arrayWithObjects: nil, nil]

Question

У меня есть свойство self.shareURL, которое может быть или не быть nil, и я хотел бы обернуть его в массив. Очевидно, если это nil, я не могу этого сделать, поэтому я хотел бы иметь пустой массив в этом случае. Так что я могу написать:

NSArray *items = [self shareURL] ? @[[self shareURL]] : @[]; 

Однако, я могу построить его в одном вызове shareURL, как это:

NSArray *items = [NSArray arrayWithObjects:[self shareURL], nil]; 

Это работает, потому что arrayWithObjects: остановится в любом случае, когда он видит первый nil и стек не поврежден, потому что Objective-C ABI не требует, чтобы он очищал varargs в стеке.

Можно ли использовать вторую форму? Или первый, более четкий?

Answer 1

Оба варианта кажутся хорошо, но если вы спрашиваете, для которых один является более ясным, я бы пойти на это:

NSArray *items = [self shareURL] ? @[[self shareURL]] : @[]; 

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

Если я разработчик, и я вижу этот код, с 1-м подходом я пойму поведение, без каких-либо объяснений.

Answer 2

Оба пути правильны, и первое из них вы можете написать так:

NSArray *items = [self shareURL] ? : @[];

Это правильно.

NSArray *items = [self shareURL] ? @[[self shareURL]] : @[];

Answer 3

Вместо того, краткость, я выбрал бы для читаемости:

NSArray *items = nil; 
if ([self shareURL]) { 
    items = @[[self shareURL]]; 
} else { 
    items = @[]; 
} 

Answer 4

Ok, для записи, я на самом деле пошел с

NSMutableArray *items = [NSMutableArray new]; 
if (self.shareURL) { 
    [items addObject: self.shareURL]; 
} 

как это делает более очевидным, что является «стандартным» состоянием пустого массива, и мы пытаемся добавить один объект.