单例未正确初始化

Question

我有使用此语句调用的以下代码： SQLiteDB *db = [[[SQLiteDB alloc] init] autorelease];

问题是“sharedSQLiteDB”没有被调用，而是“allocWithZone”被调用，因此“checkIfDatabaseExists”没有被调用，这是创建数据库的地方。

我不明白为什么......（即我做错了什么？）

#import "SQLiteDB.h"

static SQLiteDB *sharedSQLiteDB = nil;  //  makes this a singleton class

@implementation SQLiteDB

@synthesize searchPaths, documentPath, databasePath, cDatabasePath;

#pragma mark Singleton Methods

+ (SQLiteDB *) sharedSQLiteDB  {

    if(!sharedSQLiteDB)  {
        sharedSQLiteDB = [[SQLiteDB alloc] init];
        [sharedSQLiteDB checkIfDatabaseExists];  //  check to see if d/b exists
    }
    return sharedSQLiteDB;
}   

+(id)allocWithZone:(NSZone *)zone  {  //  makes sure another instance is not allocated
    if(!sharedSQLiteDB)  {
        sharedSQLiteDB = [super allocWithZone:zone];
        return  sharedSQLiteDB;
    }
    else {
        return nil;
    }
}   

-(id)copyWithZone:(NSZone *)zone  {
    return self;
}

-(void) release  {
    //  no-op
}

Answer 1

在单例模式中，您的使用模式应该是：

SQLiteDB* db = [SQLiteDB sharedSQLiteDB];

您调用它的方式不适合单例模式。所有访问都应该通过您的共享SQLiteDB消息进行。

换句话说，您不应该在类的范围之外通过典型的 Cocoa 模式进行初始化（SQLiteDB *db = [[[SQLiteDB alloc] init] autorelease]; 是不正确且充满问题的）。

在使用语言默认初始化模式（ObjC 的 alloc/init 或 C++ 的默认构造函数）的单例中，应该生成编译时错误消息，因为构造函数/init 方法应该受到保护。

请参阅维基百科条目。请参阅设计模式 C++ 圣经。甚至还有一个 Cocoa 的版本

祝你好运。

Answer 2

它不会执行您的 + (SQLiteDB *)sharedSQLiteDB 方法，因为您实际上并未在任何地方调用该方法。

正如您所看到的，当您调用[[SQLiteDB alloc] init]时，会调用allocWithZone方法。

将您的调用更改为 SQLiteDB *db = [SQLiteDB sharedSQLiteDB]，在本例中它将调用您的 checkIfDatabaseExists 方法。但是，如果在其他地方调用[[SQLiteDB alloc] init]，则仍会跳过checkIfDatabaseExists方法调用。

也许可以考虑将 checkIfDatabaseExists 方法移动到 init 方法中，以便为您的单例方法和 allocWithZone 调用它。

Answer 3

老实说我没有看到任何错误......
不过，我发布了用于创建单例的代码。它来自一个来源，现在我不记得链接了......这不是我的代码。

static DataManager *_instance;

@implementation DataManager

+ (DataManager*)sharedInstance
{
    @synchronized(self) {

        if (_instance == nil) {

            _instance = [[super allocWithZone:NULL] init];

            // Allocate/initialize any member variables of the singleton class her
            // example
    //_instance.member = @"";
        }
    }
    return _instance;
}

#pragma mark Singleton Methods

+ (id)allocWithZone:(NSZone *)zone
{   
    return [[self sharedInstance]retain];   
}


- (id)copyWithZone:(NSZone *)zone
{
    return self;    
}

- (id)retain
{   
    return self;    
}

- (unsigned)retainCount
{
    return NSUIntegerMax;  //denotes an object that cannot be released
}

- (void)release
{
    //do nothing
}

- (id)autorelease
{
    return self;    
}

我希望它有帮助

Answer 4

我强烈建议使用 Matt Gallagher 首先创建的 SyntesizeSingleton 头文件。

在这里找到最新版本（我知道的）：

https://github.com/cjhanson /Objective-C-Optimized-Singleton

它使创建单例变得非常简单。

下面是一个示例标头：

#import <Foundation/Foundation.h>
@interface Example : NSObject {}
+(Example*)sharedExample;
@end

相应的 .m:

#import "FMUser.h"
#import "SynthesizeSingleton.h"
@implementation Example
SYNTHESIZE_SINGLETON_FOR_CLASS(Example);
@end

[Example sharedExample] 已为您创建。非常甜蜜。