背景

我有一个在内部使用 vector 的容器类。我已经为这个包装类提供了一个方法 AddChar(std::string)，它对内部 vector 执行 push_back()。在我的代码中，有时我必须向容器中添加多个项目。为此，我必须使用

container.AddChar("First");
container.AddChar("Second");

这会使代码变大。所以为了让它更容易，我打算重载运算符<>

container << "First" << "Second"

两个项目将被添加到基础 vector 中。

这是我使用的代码

class ExtendedVector
{
private:
    vector<string> container;

public:
    friend ExtendedVector& operator<<(ExtendedVector& cont,const std::string str){
        cont.AddChar(str);
        return cont;
    }

    void AddChar(const std::string str)
    {
        container.push_back(str);
    }

    string ToString()
    {
        string output;
        vector<string>::iterator it = container.begin();
        while(it != container.end())
        {
            output += *it;
            ++it;
        }
        return output;
    }
};

它按预期工作。

问题

1. 运算符重载写得正确吗？
2. 在这种情况下重载运算符是一种好习惯吗？
3. 此代码会不会有任何性能问题或任何其他问题？

有什么想法吗？

编辑

在听到优秀的评论后，我决定不重载 <>

class ExtendedVector
{
private:
    vector<string> container;

public:

    ExtendedVector& AddChar(const std::string str)
    {
        container.push_back(str);
        return *this;
    }

         .. other methods
}

这允许我添加

container.AddChar("First").AddChar("Second")

在 C# 中，我可以使用 params 关键字更轻松地完成此操作。代码会像

void AddChar(params string[] str)
{
    foreach(string s in str)
       // add to the underlying collection
}

我知道在 C++ 中，我们可以使用 ... 来指定参数的可变长度。但是据我所知，它不是类型安全的。那么这样做是推荐的做法吗？这样我就可以写

container.AddChar("First","Second")

感谢您的回复。

最佳答案

Is operator overload written correctly?

是的，但可以做得更好。就像其他人提到的那样，您的功能可以完全从现有的公共(public)功能中定义。为什么不让它只使用那些？现在，它是一个 friend ，这意味着它属于实现细节。如果将 operator<><>非成员、非好友函数。

class ExtendedVector {
    ...
};

// note, now it is *entirely decoupled* from any private members!
ExtendedVector& operator<<(ExtendedVector& cont, const std::string& str){
    cont.AddChar(str);
    return cont;
}

如果你改变了你的类，你将不确定你的操作符<是否仍然有效。但是，如果您的><>

Is it a good practice to overload operators in situations like this?

正如另一个人所说，这是有争议的。在许多情况下，运算符重载乍一看看起来很“整洁”，但明年看起来会很糟糕，因为您不再知道在给某些符号特别喜欢时您的想法是什么。对于 operator<，我认为这是一个不错的用法。它用作流的插入运算符是众所周知的。我知道 qt="" 和="" kde="">

QStringList items; 
items << "item1" << "item2";

类似的情况是 boost.format，它也重用 operator% 来为其字符串中的占位符传递参数:

format("hello %1%, i'm %2% y'old") % "benny" % 21

在那里使用它当然也有争议。但是它对 printf 格式指定的使用是众所周知的，所以它的使用也可以，恕我直言。但一如既往，风格也是主观的，所以要持保留态度 :)

How can i accept variable length arguments in a typesafe way?

好吧，如果您正在寻找齐次参数，可以采用接受 vector 的方法:

void AddChars(std::vector<std::string> const& v) {
    std::vector<std::string>::const_iterator cit =
        v.begin();
    for(;cit != v.begin(); ++cit) {
        AddChar(*cit);
    }
}

虽然通过它并不是很舒服。你必须手动构建你的 vector 然后通过......我看到你已经对可变参数样式函数有了正确的感觉。不应将它们用于此类代码，并且仅在与 C 代码交互或调试函数时才使用。处理这种情况的另一种方法是应用预处理器编程。这是一个高级主题，非常 hacky。这个想法是自动生成达到某个上限的重载，大致如下所示:

#define GEN_OVERLOAD(X) \
void AddChars(GEN_ARGS(X, std::string arg)) { \
    /* now access arg0 ... arg(X-1) */ \
    /* AddChar(arg0); ... AddChar(arg(N-1)); */ \
    GEN_PRINT_ARG1(X, AddChar, arg) \
}

/* call macro with 0, 1, ..., 9 as argument
GEN_PRINT(10, GEN_OVERLOAD)

那是伪代码。您可以查看升压预处理器库 here .

下一个 C++ 版本将提供更好的可能性。可以使用初始化列表:

void AddChars(initializer_list<std::string> ilist) {
    // range based for loop
    for(std::string const& s : ilist) {
        AddChar(s);
    }
}

...
AddChars({"hello", "you", "this is fun"});

在下一个 C++ 中也可以使用 variadic templates 支持任意多个(混合类型)参数. GCC4.4 将支持它们。 GCC 4.3 已经部分支持它们。

关于c++ - 重载运算符 << - C++，我们在Stack Overflow上找到一个类似的问题： https://stackoverflow.com/questions/601219/