Этот код выглядит хорошо. Единственное, что вы можете иметь в виду, это то, что вы можете использовать авто-релиз, чтобы избежать явных выпусков, когда вы закончите. Например:
for(int counter=1; counter<=5; counter++) {
NSString *dataValue = [[[NSString alloc]
initWithFormat:@"DATA%d", counter] autorelease];
[fgStore addObject:dataValue];
}
Я не верю, что существует какой-либо метод NSMutableArray, который возвращает сохраненный объект. Помните, что методы, которые возвращают сохраненные значения, имеют имена, которые начинаются с alloc, new или copy.