DEFINE Macro работает на всех платформах?

Question

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

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

THECLASS::member 

Я думал об использовании макроса определения, чтобы сделать его короче.

#define(THECLASS::member, shortMember); 

Так что мой вопрос: будет ли это вредно для перекрестных платформ или перекрестных компиляторов.

Answer 1

Ваше определение не будет работать нигде, на самом деле. Оно должно быть:

#define shortMember THECLASS::member 

И хорошая практика будет не пытаться сохранить нажатия клавиш – читаемость является более важным, а кто-то читает код хочет видеть THECLASS :: член, не shortMember, потому что тогда он знает, где искать определение.

Answer 2

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

using namespace [your-namespace]; 

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

Answer 3

Скорее всего, но вы открываете конфликты пространства имен, которые, как предполагалось, должны были избегать в первую очередь, поэтому, вероятно, это не очень хорошая идея.

Answer 4

Вы должны использовать правильный синтаксис для определения макроса:

#define shortMember THECLASS::member 

Однако не делают этого.

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

• Это вредит читаемости и понятности кода, потому что это делает его гораздо более трудным, где имена пришли и к чему они относятся

Хорошая практика в данном случае это прописать имя везде, где вы используете его.

Answer 5

Определить будет работать, но не определить, что вы написали в своем вопросе. Что-то вдоль линий:

class SingletonImpl { 
... 
SingletonImpl *getInstance() { return m_instance; } 
} 

#define SINGIMPL SingletonImpl::getInstance() 

, которые затем могут быть использованы как это:

SINGIMPL->whatever(); 

Однако имейте в виду, что макросы просто сделать замену текста и, следовательно, игнорировать такие вещи, как пространство имен.

Answer 6

Чтобы ответить на вопрос в своем названии, да, правильно написанная DEFINE будет работать на всех платформах. Однако, как утверждают другие, это не является хорошей идеей для вашей цели по причинам, указанным.