代码输出废话到输出文件而不是字符串?
Code outputs nonsense to output file instead of string?
以下代码适用于我必须执行的项目,我收到一个文本文件,其中包含学生的名字和姓氏,后跟他的成绩。然后我必须将其转换成一个输出文件,其中包含他的名字和他的平均分。我收到的文件中有多个学生逐行分开。输出应该看起来像
Rzam, Look = 0.00
Bambi, Lambi = 40.47
Coop, Jason = 27.31
但我的只是打印垃圾,例如
0x7fffb08e8698= 0.000x7fffb08e8698= 0.000x7fffb08e8698= 0.000x7fffb08e8698= 0.000x7fffb08e8698= 0.000x7fffb08e8698= 0.000x7fffb08e8698= 0.000x7fffb08e8698= 0.000x7fffb08e8698= 0.000x7fffb08e8698= 0.00
这是我目前的情况:
#include <iostream>
#include <fstream>
#include <sstream>
#include <iomanip>
using namespace std;
struct Student
{
string fname;
string lname;
double average;
};
int read(ifstream &fin, Student s[]);
void print(ofstream &fout, Student s[], int amount);
int main()
{
const int size = 10;
ifstream fin;
ofstream fout;
string inputFile;
string outputFile;
Student s[size];
cout << "Enter input filename: ";
cin >> inputFile;
cout << "Enter output filename: ";
cin >> outputFile;
cout << endl;
fin.open(inputFile.c_str());
fout.open(outputFile.c_str());
read(fin , s);
print(fout, s, size);
fin.close();
fout.close();
}
int read(ifstream &fin, Student s[])
{
string line;
string firstName;
string lastName;
double score;
double total;
int i=0;
int totalStudents=0;
Student stu;
while(getline(fin, line)){
istringstream sin;
sin.str(line);
while(sin >> firstName >> lastName){
stu.fname = firstName;
stu.lname = lastName;
while(sin >> score){
total *= score;
i++;
}
stu.average = (total/i);
}
s[totalStudents]=stu;
totalStudents++;
}
return totalStudents;
}
void print(ofstream &fout, Student s[], int amount)
{
ostringstream sout;
for(int i = 0; i<amount; i++)
{
sout << left << setw(20) << s[i].lname << ", " << s[i].fname;
fout << sout << setprecision(2) << fixed << "= " << s[i].average;
}
}
您有一些错误,这些错误加起来导致了您的问题:
在您的 print
函数中,您写入 ostringstream
然后尝试将其写入文件流。这很好,但它正在打印 ostringstream
缓冲区的地址。因此进行此更改将导致它打印内容:
fout << sout.str() << setprecision(2) << fixed << "= " << s[i].average;
注意.str()
的用法。虽然你根本不需要临时流...
- 您没有在输出中放置换行符,因此所有内容都以一行结尾,难以阅读:
所以再做一个改变,让它看起来像这样:
fout << sout.str() << setprecision(2) << fixed << "= " << s[i].average << '\n';
您需要将 ostringstream sout;
放在循环中,因此每次都会重置它。否则你会得到奇怪的复合输出。
您没有使用您的阅读功能计算的学生人数!所以它总是尝试打印 10!做这样的事情:
int count = read(fin , s);
print(fout, s, count);
如果没有读取分数,我想你会被零除。所以你应该加一张支票。
您应该确保阅读的学生不超过 size
人。或者更好的是,只需将它们放在函数中的 std::vector
和 return 中。它更简单,更不容易出错。
每次开始读一个学生都需要重新设置i
,否则后面的学生会分的太多。每个都需要有一个独立的计数。
我不知道这些是否是唯一的问题,但肯定会让您走上正确的轨道:-)
以下代码适用于我必须执行的项目,我收到一个文本文件,其中包含学生的名字和姓氏,后跟他的成绩。然后我必须将其转换成一个输出文件,其中包含他的名字和他的平均分。我收到的文件中有多个学生逐行分开。输出应该看起来像
Rzam, Look = 0.00
Bambi, Lambi = 40.47
Coop, Jason = 27.31
但我的只是打印垃圾,例如
0x7fffb08e8698= 0.000x7fffb08e8698= 0.000x7fffb08e8698= 0.000x7fffb08e8698= 0.000x7fffb08e8698= 0.000x7fffb08e8698= 0.000x7fffb08e8698= 0.000x7fffb08e8698= 0.000x7fffb08e8698= 0.000x7fffb08e8698= 0.00
这是我目前的情况:
#include <iostream>
#include <fstream>
#include <sstream>
#include <iomanip>
using namespace std;
struct Student
{
string fname;
string lname;
double average;
};
int read(ifstream &fin, Student s[]);
void print(ofstream &fout, Student s[], int amount);
int main()
{
const int size = 10;
ifstream fin;
ofstream fout;
string inputFile;
string outputFile;
Student s[size];
cout << "Enter input filename: ";
cin >> inputFile;
cout << "Enter output filename: ";
cin >> outputFile;
cout << endl;
fin.open(inputFile.c_str());
fout.open(outputFile.c_str());
read(fin , s);
print(fout, s, size);
fin.close();
fout.close();
}
int read(ifstream &fin, Student s[])
{
string line;
string firstName;
string lastName;
double score;
double total;
int i=0;
int totalStudents=0;
Student stu;
while(getline(fin, line)){
istringstream sin;
sin.str(line);
while(sin >> firstName >> lastName){
stu.fname = firstName;
stu.lname = lastName;
while(sin >> score){
total *= score;
i++;
}
stu.average = (total/i);
}
s[totalStudents]=stu;
totalStudents++;
}
return totalStudents;
}
void print(ofstream &fout, Student s[], int amount)
{
ostringstream sout;
for(int i = 0; i<amount; i++)
{
sout << left << setw(20) << s[i].lname << ", " << s[i].fname;
fout << sout << setprecision(2) << fixed << "= " << s[i].average;
}
}
您有一些错误,这些错误加起来导致了您的问题:
在您的
print
函数中,您写入ostringstream
然后尝试将其写入文件流。这很好,但它正在打印ostringstream
缓冲区的地址。因此进行此更改将导致它打印内容:fout << sout.str() << setprecision(2) << fixed << "= " << s[i].average;
注意.str()
的用法。虽然你根本不需要临时流...
- 您没有在输出中放置换行符,因此所有内容都以一行结尾,难以阅读:
所以再做一个改变,让它看起来像这样:
fout << sout.str() << setprecision(2) << fixed << "= " << s[i].average << '\n';
您需要将
ostringstream sout;
放在循环中,因此每次都会重置它。否则你会得到奇怪的复合输出。您没有使用您的阅读功能计算的学生人数!所以它总是尝试打印 10!做这样的事情:
int count = read(fin , s); print(fout, s, count);
如果没有读取分数,我想你会被零除。所以你应该加一张支票。
您应该确保阅读的学生不超过
size
人。或者更好的是,只需将它们放在函数中的std::vector
和 return 中。它更简单,更不容易出错。每次开始读一个学生都需要重新设置
i
,否则后面的学生会分的太多。每个都需要有一个独立的计数。
我不知道这些是否是唯一的问题,但肯定会让您走上正确的轨道:-)