从字符串中检索特定文本行的内存管理问题

Question

我对 iOS 开发有点陌生。

我想从我拥有的 text(.rtf) 文件中检索字符串。该文件位于我的应用程序主包中。它的内容是：

#start word1 First word end word2 Second word end //lots of things to be added later

代码：

 path = [[NSBundle mainBundle]pathForResource:@"words" ofType:@"rtf"];
    if(path)
    {
        NSLog(@"path exists");
    }
    NSError *error = nil;

    NSString *file = [[NSString alloc]initWithContentsOfFile:path encoding:NSUTF8StringEncoding error:&error];
    if(error)
    {
        NSLog(@"error");
    }
    NSString *finalword= [[NSString alloc]init ];
    NSString *startfrom = [[NSString alloc] initWithFormat:@"word%i",i+1];
    i++;
    NSLog(@"%@",startfrom);
    NSString *wordoftheday = [[NSString alloc]init ];
    NSScanner *scanner = [NSScanner scannerWithString:file];
    [scanner scanUpToString:startfrom intoString:nil];
    [scanner scanUpToString:@"end" intoString:&wordoftheday];
    finalword = [wordoftheday substringFromIndex:[startfrom length]];
    NSLog(@"%@",finalword);
    Word.text = final word;  //change label text
//[final word release];
//[wordoftheday release];
//[file release];

代码运行良好，但它给我留下了内存管理问题。如果我释放最后注释代码中的变量，应用程序会崩溃。

这个方法也在我的viewdidload中。我希望标签在用户单击按钮时更改文本。我将不得不以这种方法再次编写相同的代码，这给我留下了更多的内存问题。

Answer 1

所以一一看这些，关注内存问题而不是这里的整体策略：

NSString *finalword= [[NSString alloc]init ];

在这里你分配/初始化一个新的不可变和空的 NSString，然后你最终会覆盖指向它的指针。你应该删除这一行。然后你需要将声明向下移动几行：

NSString *finalword = [wordoftheday substringFromIndex:[startfrom length]];

那么你有：

NSString *startfrom = [[NSString alloc] initWithFormat:@"word%i",i+1];

这个你需要稍后发布。或者只是将其更改为：

NSString *startfrom = [NSString stringWithFormat:@"word%i",i+1];

那么你有：

NSString *wordoftheday = [[NSString alloc]init ];

与结局相同的故事。除了您确实需要定义此变量以便稍后将其传递给扫描仪之外。所以改成：

NSString *wordoftheday = nil;

最后，您可以释放“文件”。没事儿。但是您不想发布“wordoftheday”或“finalword”，因为您不拥有这些字符串。不是您自己创建的。

还有一个注意事项：

if(error)

这不是在加载“文件”时检查错误的正确方法。您应该检查该方法的返回值，然后当且仅当返回值为 nil 时才查找错误。所以将该行更改为：

if(!file)

（好吧，这并不是真正的内存问题，而是我注意到的一个错误。）

我认为至少就内存问题而言，这就是全部。我希望这会有所帮助。

Answer 2

将这些变量作为成员变量并在dealloc中释放