Есть ли причина, по которой я не должен просто определять #define NSFM [NSFileManager defaultManager] вместо того, чтобы писать каждый раз?

Существует встроенный ярлык NSApp, есть ли причина, по которой я не должен добавлять один для NSFileManager?Есть ли причина, по которой я не должен просто определять #define NSFM [NSFileManager defaultManager] вместо того, чтобы писать каждый раз?

#define NSFM [NSFileManager defaultManager]

Я думаю, что опущено это сделает мой код понятнее, и я не вижу никакой пользы для поддержания его. (Я планирую делать это во всех моих проектах с этого момента, так что не будет скрывать .)

NSFileManager *fm = [NSFileManager defaultManager]

источник

2010-01-21 zekel

NSFM - небезопасно для Microsoft? –

В свете немного большего опыта, определение похоже на плохой стиль. Но в моей защите тонны моих методов использовали NSFileManager, поэтому это казалось хорошей идеей. – zekel

Почему бы вам просто не использовать локальную переменную?

NSFileManager *fm = [NSFileManager defaultManager]; 
// use fm...

или еще лучше, впрыснуть файловый менеджер, как аргумент метода:

- (void)myMethod { 
    //using [NSFileManager defaultManager] 
}

становится

- (void)myMethodWithFileManager:(NSFileManager*)fm { 
//usin fm 
}

Поскольку defaultManager одноэлементно (фактически глобальный), что делает тестирование действительно трудно. Инъекция зависимостей экономит ваше печатание (как вам угодно) внутри метода и упрощает модульное тестирование - вы можете вводить тестовый двойной вместо стандартногоManager.

И, наконец, Cocoa и Objective-C обычно опускаются на привилегированный явный код по короткому коду. Философия в основном заключается в том, что использование более подробных имен делает код более легким для чтения (и, следовательно, для поддержания). Эта философия полностью подходит к селекторному стилю Objective-C с чередующимися именованными аргументами. Если вы не действительно не может обрабатывать дополнительную типизацию (и завершение кода Xcode не будет работать для вас), придерживайтесь существующих имен. Когда в Риме и все это ...

источник

2010-01-21 21:19:42

+1 Угол измерения единицы измерения является очень важным моментом.Позор мне не упоминать. –

Если это делает ваш код чище, я все для него. Просто имейте в виду, что любые другие разработчики, которые должны прочитать ваш код, не будут немедленно знают, что представляют NSFM или fm.

Я бы предположил несколько более описательное название: NSFileMgr. Для большинства разработчиков Cocoa это значительно облегчило бы назначение переменной без необходимости искать ее.

UPDATE: Смотрите Barry Wark's answer для некоторых очень хороших точек в отношении тестирования.

источник

2010-01-21 17:13:59

Мех. Сокращения. Тьфу. Если вы собираетесь это сделать, по крайней мере используйте что-то вроде * DefaultFileManager * или * DEFAULT_FILE_MANAGER *. Однако, если вы действительно используете файловый менеджер *, *, я бы предложил кэшировать возвращаемое значение в локальной или переменной экземпляра. – bbum

@bbum локальная переменная на самом деле звучит как гораздо лучшая идея: 'NSFileManager * fileManager = [NSFileManager defaultManager];' хотя я не против использования глобальной ссылки для чего-то, что для всех целей и целей является глобальной переменной , –

Я буду использовать локальную переменную каждый раз, потому что использование defaultManager не является потокобезопасным. Если в какой-то момент вы начнете использовать потоки в своем приложении, вы можете столкнуться с трудностями поиска ошибок и не знать, почему, пока вы не запустите this bit of documentation.

источник

2010-07-23 22:31:56 cocteau

Есть ли причина, по которой я не должен просто определять #define NSFM [NSFileManager defaultManager] вместо того, чтобы писать каждый раз?

ответ

Смежные вопросы