Objective-C NSUserDefaults: HighScoreManager, почему это не работает? - PullRequest
1 голос
/ 28 июля 2010

Мое приложение не сохраняет данные и не загружает их, оно просто падает, мне интересно, что я делаю неправильно в своем классе High Score Manager.

Я собираюсь опубликовать свой класс.h и .m файл:

.h

#import <Foundation/Foundation.h>

@interface Score : NSObject
{
    NSString *name;
    NSNumber *score;
}

@property (assign, nonatomic) NSString *name;
@property (assign, nonatomic) NSNumber *score;

+ (Score *) score: (NSNumber *)score_ name: (NSString *)name_;
- (id) initWithScore: (NSNumber *)score_ name: (NSString *)name_;

@end

@interface HighScoreManager : NSObject
{
    NSArray *an_;
    NSArray *as_;
    BOOL loaded;
}

- (void) load;
- (BOOL) noScores;
- (NSArray *) sortedScores;
- (void) add: (NSString *)name score: (NSNumber *)score;

@end

.m

#import "HighScoreManager.h"

@implementation Score
@synthesize name, score;

+ (Score *) score: (NSNumber *)score_ name: (NSString *)name_
{
    return [[[Score alloc] initWithScore: score_ name: name_] autorelease];
}

- (id) initWithScore: (NSNumber *)score_ name: (NSString *)name_
{
    if((self = [super init]))
    {
        score = score_;
        name = name_;
    }

    return self;
}

@end

@implementation HighScoreManager

- (id) init
{
    if((self = [super init]))
    {
        loaded = NO;
    }

    return self;
}

- (void) load
{
    if(!loaded) {
        NSUserDefaults *NSUD = [NSUserDefaults standardUserDefaults];
        [NSUD synchronize];
        an_ = [NSUD arrayForKey: @"game.score.names"];
        as_ = [NSUD arrayForKey: @"game.score.scores"];
        [NSUD release];

        loaded = YES;
    }
}

- (BOOL) noScores
{
    [self load];
    return [an_ count] < 1;
}

- (NSArray *) sortedScores
{
    [self load];
    NSMutableArray *scoresObj = [NSMutableArray array];

    for(NSUInteger i = 0; i < [an_ count]; i++)
    {
        [scoresObj addObject: [Score score: [as_ objectAtIndex: i] name: [an_ objectAtIndex: i]]];
    }

    NSSortDescriptor *sortDescriptor = [[[NSSortDescriptor alloc] initWithKey:@"score" ascending: NO] autorelease];
    NSArray *sortDescriptors = [NSArray arrayWithObject:sortDescriptor];

    return [scoresObj sortedArrayUsingDescriptors: sortDescriptors];
}

- (void) add: (NSString *)name score: (NSNumber *)score
{
    NSUserDefaults *NSUD2 = [NSUserDefaults standardUserDefaults];
    [NSUD2 setObject: [[NSUD2 arrayForKey: @"game.score.names"] arrayByAddingObject: name] forKey: @"game.score.names"];
    [NSUD2 setObject: [[NSUD2 arrayForKey: @"game.score.scores"] arrayByAddingObject: score] forKey: @"game.score.scores"];
    [NSUD2 synchronize];
}

- (void) dealloc
{
    [an_ release];
    [as_ release];
    [super dealloc];
}

@end

Как я пытался использовать класс

    HighScoreManager *HSM1 = [HighScoreManager new];
    [HSM1 add: @"Johannes" score: [NSNumber numberWithInt: 1298]];
    [HSM1 add: @"Johannes" score: [NSNumber numberWithInt: 8723]];
    [HSM1 add: @"Johannes" score: [NSNumber numberWithInt: 3283]];
    [HSM1 add: @"Johannes" score: [NSNumber numberWithInt: 1763]];
    [HSM1 add: @"Johannes" score: [NSNumber numberWithInt: 9931]];

    HighScoreManager *HSM = [HighScoreManager new];

    // Check if there even are any high scores.
    if([HSM noScores])
    {
        CCLabel *noScoresYet = [CCLabel labelWithString: @"There are no scores yet." fontName: @"Helvetica" fontSize: 36];
        noScoresYet.position = ccp(win.width / 2, win.height / 2);
        [noScoresYet setColor: ccc3(0, 0, 0)];
        [self addChild: noScoresYet];
    } else
    {
        // Oh look, there were some high scores!

    }

    [HSM release];

Несмотря на то, что я «сохраняю» результаты, когда я ухожу со страницы «Счета» и возвращаюсь назад, именно там происходит сбой приложения.Я не совсем уверен, что я делаю неправильно или почему он не сохраняет


Я получаю это, когда происходит сбой "_class_getMethodNoSuper_nolock" и иногда objC_send или что-то в этом роде

Ответы [ 3 ]

1 голос
/ 31 июля 2010
  • Не выпускать NSUD.Вы не создали это, и вы не сохранили это, таким образом, у вас нет этого.Это может быть причиной сбоя.

  • Вам также следует сохранить переменные вашего экземпляра.

  • [NSUD2 arrayForKey: @"game.score.names"] вернет nil в начале, поэтому[[NSUD2 arrayForKey: @"game.score.names"] arrayByAddingObject: name] также будет ноль.Вы всегда устанавливаете значения словаря в nil.

  • (И избегайте имен переменных, таких как "an_". Это затрудняет чтение вашего кода.)

0 голосов
/ 28 июля 2010

Не тестировал ваш код, но первое, что я заметил, это ваши статические фабричные методы, которые пытаются распределить себя вместо класса Score

Я почти уверен, что вам нужно:

+ (Score *) score: (NSNumber *)score_ name: (NSString *)name_
{
     return [[[Score alloc] initWithScore: score_ name: name_] autorelease];
}

Еще одна вещь. Ваши свойства в классе Score объявляются как назначить, что означает, что вы не сохраняете полученную оценку и имя, если они выходят за рамки, это также приводит к сбою вашего приложения.

0 голосов
/ 28 июля 2010

Опишите точно, где происходит сбой, и, возможно, я могу помочь вам. Я заметил по крайней мере одну проблему:

return [[[self alloc] initWithScore: score_ name: name_] autorelease];

Должно быть

return [[[Score alloc] initWithScore: score_ name: name_] autorelease];
...