0

我正在复习我的 C++,并试图弄清楚为什么我的 toString 函数没有输出我定义的格式化字符串。

我指的功能是:friend std::ostream& operator<<(std::ostream&, const Employee&);

员工.cpp

#include <iostream>
#include <stdio.h>
using namespace std;

class Employee {
  private:
    string name;
    double rate;
    double hours;
    double getPay() const;
    friend std::ostream& operator<<(std::ostream&, const Employee&);
  public:
    Employee(string, double);
    void setHours(double);
};

Employee::Employee(string name, double rate) {
  this->name = name;
  this->rate = rate;
  this->hours = 0;
}

void Employee::setHours(double hours) {
  this->hours = hours;
}

double Employee::getPay() const {
  double gross = this->hours * this->rate;
  double overtime = this->hours > 40 ? 
      (this->hours - 40) * (this->rate * 1.5) : 0;
    return gross + overtime;
}

// toString
std::ostream& operator<<(std::ostream &strm, const Employee &e) {
  char buff[64];
  return strm << sprintf(buff, "Name: %s, Salary: $%.2f\n",
    e.name.c_str(), e.getPay());
}

int main (int* argc, char** argv) {
  Employee emp1("Bob", 28);
  Employee emp2("Joe", 32);
  emp1.setHours(44);
  emp2.setHours(25);
  cout << emp1 << endl;
  cout << emp2 << endl;
  return 0;
}
4

3 回答 3

6

sprintf返回:

  • 成功时,返回写入的字符总数。此计数不包括在字符串末尾自动附加的额外空字符。
  • 失败时,返回一个负数。

在任何情况下它都不会返回一个字符串,它总是返回一个int,这就是您要求打印的内容。大概你想要这个:

char buff[64];
sprintf(buff, "Name: %s, Salary: $%.2f\n",
  e.name.c_str(), e.getPay());
return strm << buff;

虽然如果你坚持使用而不是混合CC++标准库会更好:

return strm << "Name: " << e.name << ", Salary: $" << std::setprecision(2) << e.getPay() << "\n";
于 2013-01-12T00:30:21.580 回答
2

这真的不是 ostreams 的工作方式。事实上,如果您查看 sprintf,您会发现您实际上并不想将其返回值打印到 strm。相反,您应该打印 buf。就像是:

std::ostream& operator<<(std::ostream &strm, const Employee &e) {
  char buff[64];
  sprintf(buff, "Name: %s, Salary: $%.2f\n",
    e.name.c_str(), e.getPay());
  return strm << buff;
}

混合 sprintf 和 ostream 不是一个好主意,但至少可以让你的代码工作。

于 2013-01-12T00:31:49.130 回答
1

将 C/C++ 代码混合在一起是不好的做法,只需编写纯 C++ 代码,

std::ostream& operator<<(std::ostream &strm, const Employee &e) 
{  
  strm << "Name: " << e.name << " Salary: $" << std::setprecision(2) << e.getPay();
  return strm;
}
于 2013-01-12T00:34:43.940 回答