Блог

В моем приложении iOS objective-C есть несколько типов перечислений, которые используются в разных классах, для них, я думаю, неплохо поместить их в файл constants.h, но как насчет других, которые не обязательно используются в нескольких классах? будет ли это считаться плохой практикой?

Хотя ответ sapi верен, вот что я склонен делать…

Группа констант, которые используются в нескольких файлах, перейдет в файл. Допустим, все мои Foo константы входят FooConstants.h .

Теперь другая группа, скажем, Bar константы, они все войдут BarConstants.h .

В этих файлах будут содержаться константы, перечисления и определения протоколов.

Я импортирую Foo только те файлы, которым нужны FooConstants.h константы.

Я импортирую Bar только те файлы, которым нужны BarConstants.h константы.

И в зависимости от проекта, у меня может быть только 1 из этих файлов, или у меня может быть 10 или больше. Обычно у меня будет файл с именем SegueNames.h , где все идентификаторы перехода в моей раскадровке создаются как константы и помещаются в этот файл, чтобы я никогда не ошибался в написании имени перехода. У меня также обычно есть DefaultsKeys.h , где я храню ключи ко всему, что я добавляю NSUserDefaults .

И затем я начал время от времени осознавать, что у меня может быть файл, который использует 6 из этих файлов констант, поэтому я начал создавать Constants.h .

Constants.h в нем нет ничего, кроме импорта всех других файлов констант. Это очищает верхнюю часть некоторых моих файлов.

Но, в конце концов, я все еще сохраняю константы организованными в их собственных файлах с какой-то группировкой, объединяющей общие константы. И, как указывает sapi, любая константа, которая используется только в одном файле, должна быть определена в этом файле.

Да, это плохая практика.

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

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

Это зависит от контекста. Насколько хорошо организованы ваши классы? Если это немного запутанно, не помешает начать с файла Errors.h / m, где вы определяете свои коды ошибок как перечисления в файле .h, а свои домены ошибок как NSStrings в файле .m (с соответствующими extern NSString * const в вашем файле .h).

Если ваша организация немного лучше, то вы разделили свои классы на модули, и у каждого модуля есть точка входа, где вы должны определять эти вещи. Однако результат не меняется: заголовок ошибки для значений enum и объявлений extern, ошибка реализации для назначений extern.

Все мои файлы объявлений об ошибках выглядят следующим образом:

 // ErrorFile.h
typedef enum {
    ModuleErrorOne = 1,
    ModuleErrorTwo,
    ModuleErrorThree
} ModuleError;

extern NSString * const ModuleErrorDomain;

// ErrorFile.m
NSString * const ModuleErrorDomain = @"ModuleErrorDomain";
  

Вы можете вставить это в свой предварительно скомпилированный заголовок для повышения скорости компиляции.

РЕДАКТИРОВАТЬ: Спасибо за комментарии nhgrif и GraniteRobert, они улучшили мой ответ.