初始化结构时的 EXC_BAD_ACCESS

Question

我正在使用 Objective-C/OSX 开发一个应用程序（第一次尝试）。我有一个结构（NSDevice 自定义类型），需要作为公共/外部变量访问，以允许类的不同接口访问它。

这是标题中的声明。

struct NSDevice{
    LIBMTP_raw_device_t * usbrawdevice;
    int numusbrawdevice;
    uint32_t rawdeviceID;

    LIBMTP_mtpdevice_t *device;
};

extern struct NSDevice *Device;

@interface DeviceManager : NSObject

- (void) openDevice;
- (void) closeDevice;

@end

以及我如何在源文件中实现它

#import "DeviceManager.h"

struct NSDevice *Device = NULL;


@implementation DeviceManager



- (id)init{
    self = [super init];
    if(self){
        NSLog(@"Init");
        LIBMTP_Init();

        Device->device = NULL;
        Device->numusbrawdevice = 0;
        Device->rawdeviceID = 0;
        Device->usbrawdevice = NULL;
    }
    return self;
}

- (NSMtp_error) openDevice {
    LIBMTP_error_number_t error = LIBMTP_ERROR_GENERAL;

    NSLog(@"Opening Device");

    error = LIBMTP_Detect_Raw_Devices(&Device->usbrawdevice, &Device->numusbrawdevice);

当尝试在 init 接口中初始化结构时，我得到了一个 EXC_BAD_ACCESS。有什么想法吗？

我可以使用“结构”还是在 Objective-C 中有更好的方法？

谢谢

Answer 1

Seb，你需要为Device分配空间。

尝试这样做：

- (id)init{
    self = [super init];
    if(self){
        NSLog(@"Init");
        LIBMTP_Init();

        Device = malloc(sizeof(NSDevice));
        Device->device = NULL;
        Device->numusbrawdevice = 0;
        Device->rawdeviceID = 0;
        Device->usbrawdevice = NULL;
    }
    return self;
}

并像这样声明你的结构：

typedef struct {
    LIBMTP_raw_device_t * usbrawdevice;
    int numusbrawdevice;
    uint32_t rawdeviceID;

    LIBMTP_mtpdevice_t *device;
} NSDevice;

另外，我强烈建议将 NSDevice 的名称更改为 SebDevice 或不以 NS 开头的名称，因为这些前缀字符通常表示MacOS SDK 中内置了一些东西，它会让其他在您离开项目后必须查看您的代码的人感到困惑。

最后一件事，像这样的全局变量不应该以大写字母开头。在 Objective-C 中，最佳实践是变量以小写字母或g（全局）或下划线开头。将Device 更改为gDevice。