objective-cコードの潜在的な問題

Question

ここには小さなコードがあります。彼らのインタビューの一環としてロシアの会社Yandexによって掲示される。潜在的な問題は何ですか？それは私が見ることができない隠された問題でなければならない、非常に簡単に見えます。

まずヘッダ

//Foo.h 
#import <Cocoa/Cocoa.h> 
@interface Foo : NSObject 
{ 
    NSString* str; 
    static int i = 0; 
} 

- (NSString*) str; 
@end 

別のファイル

//Foo.m 
#import "Foo.h" 
@implementation 
- (id) init 
{ 
    return [self initWithStr:"number:" someInt:6]; 
} 

- (id) initWithStr:(NSString*)theStr someInt:(int)value 
{ 
    self = [super init]; 
    str = [NSString stringWithFormat:@"%@%d", theStr, value]; 
    return self; 
} 

- (NSString*) str 
{ 
    return str; 
} 

- (void) setStr:(NSString*)theStr 
{ 
    str = theStr; 
} 
@end 

、別のファイルの最後のファイル

//main.m 
#import <Cocoa/Cocoa.h> 
#import "Foo.h" 
int main(int argc, char *argv[]) 
{ 
    Foo objA; 
    NSLog([objA str]); 
    [objA setStr:@"hello world!"]; 
    NSLog([objA str]); 

    Foo* objB = [[Foo alloc] init]; 
    Foo* objC = [[Foo alloc] initWithStr:@"My magic number:" value:265]; 
    objB = objC; 

    NSLog([objB str]); 

    [objA release]; 
    [objB release]; 
    [objC release]; 

    return 0; 
} 

Answer 1

：

@implementation 

何の実装？指定する必要があります。最後のファイルで

：これはクラッシュします

Foo objA; 
    NSLog([objA str]); 
    [objA setStr:@"hello world!"]; 
    NSLog([objA str]); 

、ローカル変数foo OBJAが初期化されていない、nilにメッセージを客観CでOKですが、それはあるので、それは、nilに設定された罰金だろうない。ここで

：その方法は、インタフェースで宣言されていないので、この方法は、コンパイルの警告を与えるだろうが、それはまだメソッドを呼び出します

[objA setStr:@"hello world!"]; 

。ここで

：

- (id) init 
{ 
    return [self initWithStr:"number:" someInt:6]; 
} 

文字列のため、@を欠落@ "番号：" ここ

：

objB = objC; 

今解放するために有効な参照がないのであなただけの、objBを漏洩しました以前の割り当て。

[objA release]; 

これは決して割り当てられませんでした。

[objB release]; 

[objC release]; 

両者が同じオブジェクトを参照するので、二つ目がクラッシュし、保持カウントが1だけ

最初のファイルはまた、のように見える方法を宣言するなど、いくつかの潜在的な問題を持っています設定ツールと同じ、ivarのプロパティを宣言していないゲッターは、単にプロパティを宣言する方が良いでしょう。

Answer 2

@interface Foo : NSObject 
{ 
    NSString* str; 
    static int i = 0; 
} 

static int i = 0;をここで定義することはできません。タイプ名はストレージクラスを指定できませんFoo.h

Answer 3

また、設定者は前の文字列を解放して新しい文字列を保持する必要があります。

- (void) setStr:(NSString*)theStr 
{ 
    if(str) { 
    [str release]; 
    } 
    str = [theStr retain]; 
}