为什么变量的隐式初始化在 iPhone 上不一致？

Question

所以这是我的场景 - 在类的头文件中我这样做：

@interface MyClass : NSObject
{
NSString *string1;
NSString *string2;
}

- (void) methodOne: (NSString *) passedString;
- (void) methodTwo: (NSString *) passedString;

@end

在实现文件中我这样做：

 #import MyClass.h

    @implementation MyClass

    - (void) methodOne: (NSString *) passedString
    {
       NSLog(@"%@", passedString);
       string1 = passedString;
    }

    - (void) methodTwo: (NSString *) passedString
    {
       NSLog(@"%@", passedString);
       string2 = passedString;
    }

我发现这样做与 [NSString alloc] initWithString:] 之间存在某种不一致。

正如您所看到的，string1 和 string2 的处理方式完全相同，但发生的情况是 string1 已设置，但 string2 仍为空。当我稍后引用它时，我的访问权限很差。

我想也许我将一个空字符串传递给 methodTwo: 所以我添加了 NSLog 来证明它不为空，但具有预期的字符串。

因为我在决定更改为：

        - (void) methodOne: (NSString *) passedString
        {
           NSLog(@"%@", passedString);
           string1 = passedString;
        }

        - (void) methodTwo: (NSString *) passedString
        {
           NSLog(@"%@", passedString);
           string2 = [[NSString alloc] initWithString: passedString];
        }

现在两个字符串都按预期工作之前就注意到了这种不一致。我的问题是为什么会出现这种不一致？

这不是我唯一一次遇到这种情况。各种物体都发生过这种情况。唯一似乎每次都起作用的是 alloc init。像 stringWithString: 这样的方法在大多数情况下都有效，但并非总是如此。

Answer 1

这是因为在第一个示例中，您没有保留或复制字符串。 string2 在使用之前的某个时刻会被释放。 string1 没问题实际上纯属运气。您应该将代码更改为如下所示：

- (void) methodOne: (NSString *) passedString
{
   NSLog(@"%@", passedString);
   NSString* oldString = string1;
   string1 = [passedString copy];
   [oldString release];
}

- (void) methodTwo: (NSString *) passedString
{
   NSLog(@"%@", passedString);
   NSString* oldString = string2;
   string2 = [passedString copy];
   [oldString release];
}

并在 dealloc 中释放

-(void) dealloc
{
    [string1 release];
    [string2 release];
    // other stuff
    [super dealloc];
}

我强烈建议您为 string1 和 string2 创建属性来处理所有引用计数内容：

@interface MyClass : NSObject
{
NSString *string1;
NSString *string2;
}

- (void) methodOne: (NSString *) passedString;
- (void) methodTwo: (NSString *) passedString;

@property (copy) NSString* string1;
@property (copy) NSString* string2;  

@end

@imlementation MyClasss
@synthesize string1, string2;

- (void) methodOne: (NSString *) passedString
{
   NSLog(@"%@", passedString);
   [self setString1: passedString];
}

- (void) methodTwo: (NSString *) passedString
{
   NSLog(@"%@", passedString);
   [self setString2: passedString];
}

// dealloc as before

@end

Answer 2

您正在犯内存管理错误。当您将字符串分配给 ivars 时，您必须保留或复制字符串（并稍后释放它们）。

即使对象已被释放，但其占用的内存尚未被覆盖时，您仍然可以访问该对象。但你不能依赖它。

Answer 3

如果传递的字符串是自动释放的，则在分配它们时不会保留。常量字符串 (@str") 本质上永远不会被释放，需要保留诸如 stringWithFormat 等创建的字符串。

请向调用者展示。

将 @properties 与保留一起使用将消除许多保留问题。或者考虑使用 ARC，这消除了保留的需要/释放/自动释放。

Answer 4

正如其他人在这个线程中所说的那样，您在这里遇到了一些内存管理问题，也许您不太了解 NSObject 分配和保留的方式，您应该阅读 Objective-C 内存管理。同时，您可以采取两种方法来解决上述问题。

您可以将 NSString 成员变量（string1 和 string2）保留为类的属性，除了将它们声明为属性的一些其他功能之外，还可以为它们提供 setter 和 getter 访问器，您可以调用它们而不是 method1 和 method2。因此，这会将您的代码更改为在头文件中如下所示

@interface MyClass : NSObject
{
     NSString *string1;
     NSString *string2;
}

@property( nonatomic, retain)NSString* string1;
@property( nonatomic, retain)NSString* string1;

然后记住将以下内容添加到源文件中（通常在 @implementation MyClass 行之后的文件顶部）

@implementation MyClass
@synthesize string1;
@synthesize string2;

然后在调用 method1 的类中method2 你可以将代码更改为如下所示

    //Lets assume somewhere you've called an init Method for your MyClass Object, something like
    MyClass* myClassObject = [[MyClass alloc] init];

//you can then call the setters to set the string like so
[myClassObject setString1:@"some string"]; //or myClassObject.string1 = @"some string";
[myClassObject setString2:@"some string"]; //or myClassObject.string2 = @some other string";
//Calling these member variables either of the above ways is valid, I prefer the former as it's a bit easier on the eye for me

//and to get the values back out of the strings you could call
NSString* output = [myClassObject string1];
//or
NSString* output2 = myClassObject.string2;

现在，由于某种原因，你可能不想对 NSString 成员变量使用 @property，因此你可以将原始源（.m）文件修改为如下所示

@implementation MyClass

- (void) methodOne: (NSString *) passedString
{
   NSLog(@"%@", passedString);
   if( string1 != nil )
   {
      [string1 release];
   }
   string1 = [[NSString alloc] initWithString:passedString];
}

- (void) methodTwo: (NSString *) passedString
{
   NSLog(@"%@", passedString);
   if( string2 != nil )
   {
      [string2 release];
   }
   string2 = [[NSString alloc] initWithString:passedString];
}

这应该可以解决以下问题为什么你的字符串无效，因为您不会覆盖内存并尝试以这种方式读回垃圾。如果这些 NSString 不为零，你必须记住在你的 dealloc 中释放它们，否则你也会在那里遇到内存泄漏。

希望这有帮助。