类中的字符串memebres重载operator >>时出错

时间:2018-11-04 09:40:15

标签: c++ class operator-overloading inputstream

当我输入 string 类型的值时,例如:

  

_ho我输入了Peter
   _hoten我输入了Peter Parker
   _ten我输入了Marry

我在屏幕上的输出是:

  

Peter Peter Marry

这是我的代码:

class SinhVien
{
private:
    string _ho;
    string _tenlot;
    string _ten;
public:
    static int InstanceCount;
    SinhVien();
    string ToString() const;
    friend istream& operator>>(istream& in, SinhVien* p);
    friend ostream& operator<<(ostream& out, const SinhVien* p);
    ~SinhVien();
};
istream& operator>>(istream& in, SinhVien *p)
{
    cout << "Nhap ho: \n";
    in >> p->_ho;
    rewind(stdin);
    cout << "Nhap ten lot: \n";
    in >>  p->_tenlot;
    rewind(stdin);
    cout << "Nhap ten: \n";
    in >> p->_ten;
    return in;
}
string SinhVien::ToString() const
{
    stringstream writer;
    writer << _ho << " " << _tenlot << " " << _ten << "\n";
    return writer.str();
}
ostream& operator<<(ostream &out, const SinhVien* p)
{
    out << p->ToString();
    return out;
}

void main()
{
    SinhVien *a;
    a = new SinhVien();
    cin >> a;
    cout << a;
    cout << "\nTo string:\n";
    cout << a->ToString();
    delete a;
    system("pause");
}

1 个答案:

答案 0 :(得分:2)

在您的std::basic_istream::operator>>超载中,您需要使用std::geline()而不是std::cin >>,以便您可以使用空格获取完整的输入名称 < / em>。

第二,您应该传递对象指针的引用,以便将更改应用于传递的对象,而不是其副本。

修复:

std::istream& operator>>(std::istream& in, SinhVien* &p)
{                                                // ^^ take ref of object you pass, so that the change will be applied to the object, not to the copy of it.
    std::cout << "Nhap ho: \n";
    std::getline(in, p->_ho);   // change
    std::rewind(stdin);
    std::cout << "Nhap ten lot: \n";
    std::getline(in, p->_tenlot);   // change
    std::rewind(stdin);
    std::cout << "Nhap ten: \n";
    std::getline(in, p->_ten);   // change
    return in;
}

以上将对一个输入起作用。但是,如果有多个输入并在std::cin >>中使用main(),可能会再次导致某些输入跳过问题。感谢@Yksisarvinen指出这一点。 您可以在以下SO帖子中了解更多信息:Why does std::getline() skip input after a formatted extraction?


附带说明:在现代C ++中,您不再需要管理原始指针。因为您有 smart pointers ,它将在对象超出范围时管理对象的生命周期。因此,请尽可能使用它。

这意味着您可以执行以下操作: SEE LIVE

#include <memory>

class SinhVien
{
private: // memebrs
public:
    // other member functions
    friend istream& operator>>(istream& in, const std::unique_ptr<SinhVien> &p);
    friend ostream& operator<<(ostream& out, const std::unique_ptr<SinhVien> &p);
};

std::istream& operator>>(std::istream& in, const std::unique_ptr<SinhVien> &p)
{                                                                          
    std::cout << "Nhap ho: \n";
    std::getline(in, p->_ho);   // change
    std::cout << "Nhap ten lot: \n";
    std::getline(in, p->_tenlot);   // change
    std::rewind(stdin);
    std::cout << "Nhap ten: \n";
    std::getline(in, p->_ten);   // change
    return in;
}

std::ostream& operator<<(std::ostream &out, const std::unique_ptr<SinhVien> &p)
{
    return out << p->ToString();
}

int main()
{
    auto a = std::make_unique<SinhVien>();
    std::cin >> a;
    std::cout << a;
    std::cout << "\nTo string:\n";
    std::cout << a->ToString();
    return 0;
}