如果要调用多个成员函数，对象编辑器是一个好方法吗？

我经常对这样的类成员函数的顺序调用感到恼火（忽略新用法，它是针对 Qt 的，但它与 Qt 并不严格相关）

A a = new A();
a->fun1("one");
a->fun2(1, 2);
...
a->fun10("end");

我总觉得这样的代码应该写成简单的指令，而不是项目主线。

来自 Qt 的简单示例：

auto* spinBox = new QSpinBox();
spinBox->setRange(-100, 100);
spinBox->setValue(50);
spinBox->setSingleStep(5);
newLayout->addWidget(spinBox);

但我更愿意用简单的单行来完成此操作而不是那样。所以我写了这样的东西：

class {
public:
    template<class X>
    auto& operator()(X* ptr) {
        this->ptr = ptr;
        return *this;
    }

    template<class X, class R, class... Args>
    auto& operator()(R (X::* fun)(Args...), Args... args) {
        if(ptr == nullptr) {
            std::cerr << "Editor can't edit nullptr" << std::endl;
            return *this;
        }
        auto call = std::mem_fn(fun);
        call(*static_cast<X*>(ptr), args...);
        return *this;
    }

    template <class X>
    operator X*() {
        auto* result = static_cast<X*>(ptr);
        ptr = nullptr;
        return result;
    }
private:
    void *ptr = nullptr;
} EDITOR;

现在的用法：

newLayout->addWidget(EDITOR(new QSpinBox)(&QSpinBox::setRange,-100, 100)(&QSpinBox::setValue, 50)(&QSpinBox::setSingleStep, 5));

除了类型不安全之外，这是个好方法吗？ （我可以忍受）

--- EDIT ---

另一种类型安全的方法是：

template<class X>
class EDITOR2 {
public:
    EDITOR2(X* ptr) {
        this->ptr = ptr;
    }

    template<class R, class... Args>
    auto& operator()(R (X::* fun)(Args...), Args&&... args) {
        if(ptr == nullptr) {
            std::cerr << "Editor can't edit nullptr";
            return *this;
        }
        auto call = std::mem_fn(fun);
        call(*ptr, args...);
        return *this;
    }

    operator X*() {
        return ptr;
    }

    X *ptr;
};

与用法：

newLayout->addWidget(EDITOR2<QSpinBox>(new QSpinBox)(&QSpinBox::setRange, -100, 100)(&QSpinBox::setValue, 50)(&QSpinBox::setSingleStep, 5));

但这需要每次重新创建编辑器对象，并添加额外的使用代码。

我们来战斗吧，

第一个对手，普通方法：

auto* spinBox = new QSpinBox();
spinBox->setRange(-100, 100);
spinBox->setValue(50);
spinBox->setSingleStep(5);
newLayout->addWidget(spinBox);

• Pros:

  • 这是所有人都这样做的方式
  • 大约 30 列宽，可读性很好

• Cons:

  • 需要5行
  • 动作的主题spinBox每次都会重复

第二个对手，奇特的方法：

newLayout->addWidget(EDITOR2<QSpinBox>(new QSpinBox)(&QSpinBox::setRange, -100, 100)(&QSpinBox::setValue, 50)(&QSpinBox::setSingleStep, 5));

• Pros:

  • 需要一根线
  • 这是一种更函数式的编程方法

• Cons:

  • 这是一个很长的行，很难阅读
  • 默认情况下，人们的反应会是以下的一种变体wtf因为他们不习惯
  • 实例名称spinBox不重复而是类名QSpinBox now is

最终的选择取决于你对每个点的重视程度，我确实比较了一行中的第二种方法，就好像你使用换行符一样，你基本上回到了你试图修复的原始问题

以我的拙见，增加这么少的类开销是不值得的，而且我是那种对原始方法的缺点感到恼火的人，致命点是可读性损失，这很可能会迫使你使用换行符反过来，这几乎意味着你所做的这一切都是徒劳的。