Идентификатор класса, используемый внутри декларации класса. Это хорошая практика?

Question

При просмотре одного из наших классов C++ с помощью покрытия он обнаружил сообщение об ошибке в определенном классе. Класс выглядит следующим образом:

class InputRecord 
{ 
    /* Construtor */ 
    ... 
    InputRecord::RejectRecord(); 
    ... 
    /* Destructor */ 
} 

Каково значение использования идентификатора внутри класса? Правильно ли это следовать этому?

Спасибо, Mathew Liju

Answer 1

В gcc-компиляторах (v4.1 и выше) это не скомпилировалось с ошибкой: дополнительная квалификация. Поэтому хорошей практикой не поместить его!

См. here, в котором рассматривается дополнительная квалификация как неконкурентная C++.

Answer 2

G'day,

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

Объявление класса

class InputRecord; 

Что у вас есть определение класса.

class InputRecord 
{ 
    /* Construtor */ 
    ... 
    RejectRecord(); 
    ... 
    /* Destructor */ 
} 

затем в файле .cpp у вас есть реализация

InputRecord::RejectRecord() 
{ 
    ... 
} 

Answer 3

В общем, нет никаких случаев, когда неоднозначности при условии явной InputRecord:: в вашем образце кода, вероятно, будет что-нибудь кроме избыточного.

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

Это немного похоже на использование this-> (или this. в C#/Java).

Лично я удалю любые такие избыточные спецификаторы и найду другой способ получить точку.

Answer 4

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