Чистый код: как правильно называть члены структур

[YvenTitan]

Здравстуйте,
Есть мнение, что члены классов должны начинаться с m_
Как считаете, к членам структур это правило должно относиться?

[ViTech]

Сначала нужно определиться чем структура отличается от класса, в понимании чистого кода .

Коротко моё мнение: если структура не инвариант, все поля публичные, нет всяких геттеров/сеттеров, то префикс/суффикс не нужен.

[YvenTitan]

Я не до конца понимаю, зачем писать префикс в членах класса. Я делаю по 2 причинам: во-первых, принято, признак хорошего тона; во-вторых, если вижу префикс, то сразу понимаю, что член класса и немного удобнее разбирать код.

Не знаю, чем отличаются в понимании чистого кода, может смысл закрытости членов класса...

У меня "геттеров/сеттеров" нет, не инвариант

[ViTech]

Я не до конца понимаю, зачем писать префикс в членах класса. Я делаю по 2 причинам: во-первых, принято, признак хорошего тона; во-вторых, если вижу префикс, то сразу понимаю, что член класса и немного удобнее разбирать код.

Пара причин для примера:
1. С префиксами нет конфликтов имён с геттерами.
2. В IDE в автодополнении они сгруппированы, по "m_" можно увидеть список членов класса.

[Racheengel]

Я не пишу, т.к. у структуры все мемберы публичны.
Если нужны геттеры - делайте класс тогда.

[deMax]

Пара причин для примера:
1. С префиксами нет конфликтов имён с геттерами.
2. В IDE в автодополнении они сгруппированы, по "m_" можно увидеть список членов класса.

имхо, геттеры лучше начинать с get.
автодополнение актуально только внутри методов класса(а там и так не так много методов), т.к. публичные члены для класса лучше не делать.

[Авварон]

Здравстуйте,
Есть мнение, что члены классов должны начинаться с m_
Как считаете, к членам структур это правило должно относиться?

Уже написали в принципе - префикс добавляется к мемберам, чтобы избежать коллизии с геттером:
a.value(); // геттер
a.setValue(); // сеттер
a.m_value; // мембер

Я лично предпочитаю писать не m_value (как в венгерской нотации), а _value. Еще есть вариант с постфиксом value_, но мне он кажется безобразным (value_->foo()) - _value->foo()->bar() встречается гораздо чаще, чем a->_foo->_bar (тоже ужасно, но внутрь привата лезешь редко).

Еще есть вариант с d-pointer'ом: T value() const { return d->value; }

[deMax]

Уже написали в принципе - префикс добавляется к мемберам, чтобы избежать коллизии с геттером:
a.value(); // геттер
a.setValue(); // сеттер
a.m_value; // мембер

имхо:
a.value; // мембер
a.setValue(); // сеттер
a.getValue(); // геттер
геттер это метод, действие, глагол, а не существительное. И если в классе слишком много member, setMember, getMember - то напрашивается сделать класс структурой, или разделить класс на структуру и класс.

getValue() - сразу видно что это геттер.
value() - это некая функция возвращающая значение, непонятно что делающая(просто взять значение или вычислить).
toValue() - преобразование

m_ подкупает автодополнением, но в принципе IDE сама могла бы подсказывать и фильтровать дополнения(например при автодополнении ctrl+M только методы...).

[ssoft]

Последнее время (лет 5) взял за правило обязательно использовать префикс для членов класса m_ и ни капли не жалею. Читабельность кода вырастает, саппорт и дальнейший разбор облегчается.
Кроме коллизий с геттерами могут быть и коллизии и с параметрами методов, и с локальными переменными. Когда требуется анализ кода без подсветки синтаксиса префикс сильно помогает.

[__Heaven__]

Предпочитаю classMember_ и structMember.

[YvenTitan]

Что можно сделать, чтобы избежать коллизии с параметрами конструктора? Нормально в конец имени члена добавить "_" или правильнее произвольно поменять имя параметра конструктора?
Например,

class Example
{
  int param1_;
  int param2_;

  Example(int param1, int param2)
  {
     param1_ = param1;
     param2_ = param2;
  }
}

[Авварон]

param1(param1)

[kambala]

приватные переменные обычно имеют префикс _ или m_. Для публичных можно в параметры конструктора в конец добавить _

[Пантер]

У меня все мемберы приватные, именуются вот так someName_, коллизии с параметрами не возникает, ибо параметры именуются someName.

[YvenTitan]

Извините, опечатался. Структура и все параметры публичные

struct Example
{
  int param1_;
  int param2_;

  Example(int param1, int param2)
  {
     param1_ = param1;
     param2_ = param2;
  }
}