назначения на функцию, которая возвращает ссылку

Question

В настоящее время я рефакторинг кода и интересно ли следующее (т.е. присваивание функции, которая возвращает ссылку) на самом деле плохой стиль (по крайней мере, это выглядит странно):

#include <iostream> 

class A{ 
    public : 
     double & operator()() { 
      return this->x; 
     }; 

     double operator()() const{ 
      return this->x; 
     }; 

    protected : 
     double x = 0.0; 
}; 

int main(){ 
    A a; 
    a() = 4.0; 
    std::cout<<a()<<std::endl; //prints 4.0 as expected. 
}; 

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

A.d 

просто не вариант, в то время как:

A() 

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

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

Answer 1

a() = 4.0; 

Это на самом деле не плохая практика. Вы делаете что-то эквивалентное каждый раз, когда вы разыменования контейнер:

a[i] = 4.0; 

Здесь operator[] возвращает ссылку, и вы присваиваете его.

operator() не может быть более читаемым выбором здесь. В зависимости от контекста кода, что-то вроде operator* может быть лучшим выбором, но это дело вкуса;)

#include <iostream> 

class A{ 
    public : 
     double & operator*() { 
      return this->x; 
     } 

     double operator*() const{ 
      return this->x; 
     } 

    protected : 
     double x = 0.0; 
}; 

int main(){ 
    A a; 
    *a = 4.0; 
    std::cout<<(*a)<<std::endl; //prints 4.0 as expected. 
}; 

---

double & operator()() { 
    return this->x; 
}; 

Обратите внимание, что вам не нужно отставая ; при определении функция. У:

class A{ 
    // ... 
    double & operator()() { 
     return this->x; 
    } 
    // ... 
}; 

Или:

class A{ 
    // ... 
    double & operator()(); 
    // ... 
}; 

double & A::operator()() { 
    return this->x; 
}