我在这里结束了我的智慧。我正在用C ++编写一个程序，它将读取一个包含以下内容的文本文件作为一个简短的例子：
+ 23 34
- 9 8
+ 100 1
* 8 7
^ 2 5
/ 45 8
读者会将第一个操作数存储在char类型中，并根据它检索的char，调用一个函数来对两个数字执行操作，这里是一个示例函数，可以执行此操作。
void doDivision(ifstream &inFile) {
char ch;
int num1, num2;
inFile >> ch >> num1 >> num2;
cout << "Division " << num1 << " " << num2 << " " << "Quotient " << " " << num1/num2 << " Remain " << num1%num2 << endl;
}
有一件事是我不确定为什么这个论点是＆amp; inFile这个函数原型不是由我制作的，但它来自一本书，也许这就是为什么我不能让它工作。
这是我的主要功能：
int main()
{
ifstream inFile;
char ch;
int num1, num2;
inFile.open("math.txt");
if (inFile.fail())
{
cout << ch;
cout << "The math.txt input file failed to open";
return -1;
}
while(inFile)
{
switch (ch) {
case '+':
doAddition(inFile);
break;
case '-':
doSubtraction(inFile);
break;
case '*':
doMultiplication(inFile);
cout << "debug " << ch;
break;
case '/':
doDivision(inFile);
break;
case '!':
doFactorial(inFile);
break;
default:
cout << "Invalid Operation" << endl;
}
inFile >> ch;
}
inFile.close();
return 0;
}
所有这些一起产生以下意外结果
Invalid Operation
Addition 3 34 sum 37 (wrong data in text file is 23 and 34)
subtraction 0 8 difference 8 (data in textfile is 9 and 8 respectively)
我如何实现这一点，因为我之前从未使用过文件，所以我感到不知所措。
答案 0 :(得分：4)
一个简单的错误是在while（）循环内部，直到第一次循环之后才调用inFile >> ch。尝试修复它，看看它是否有帮助。
此外，Aniket said in their answer是您需要注意的另一个问题。
简而言之，你的循环应该大致如下：
inFile >> ch;
while(inFile) {
switch(ch) {
case '+':
...
}
inFile >> ch;
}
你的功能应该类似于：
void doDivision(ifstream &inFile) {
int num1, num2;
inFile >> num1 >> num2;
...
}
答案 1 :(得分：2)
由于您已在inFile >> ch中执行main()，因此在doAddition()和其他方法中再次阅读它会是错误的（逻辑上）。
由于您已在doDivision()字符中发布了ch方法，因此我假设您在doAddition()中也是这样做的。
还有你的输出：
增加3 34总和37（文本文件中的错误数据是23和34）
告诉我们所有人 - 您已阅读1个字符（找到+），然后阅读doAddition()方法中的另一个字符， - 将2读为ch doAddition() 1}}然后阅读3和34 ..
这是错误的实际位置。
解决方案：将您的doAddition()和所有其他功能更改为以下内容。
void doAddition(ifstream &inFile) {
char ch;
int num1, num2;
inFile >> num1 >> num2;
cout << "Addition of " << num1 << " and " << num2 << " = "<< (num1+num2) << '\n';
}
另外，在main()函数中：
while循环应如下所示：
while(inFile)
{
inFile >> ch;
switch (ch) {
case '+':
doAddition(inFile);
break;
case '-':
doSubtraction(inFile);
break;
case '*':
doMultiplication(inFile);
cout << "debug " << ch;
break;
case '/':
doDivision(inFile);
break;
case '!':
doFactorial(inFile);
break;
default:
cout << "Invalid Operation" << endl;
}
}
答案 2 :(得分：2)
第一次迭代时，您没有读入ch。实际上，当您检查文件是否打开没有问题时，尽管cout << ch;未初始化，但您仍然ch。您只需将inFile >> ch;移至while循环的顶部即可。
下一个问题是，在每个doSomething函数中，您再次执行inFile >> ch，这将尝试再次读取操作字符。您的doSomething函数不需要知道ch，因为函数本身是根据ch的值选择的。
以下是我写这个的方法：
ifstream inFile("math.txt"); // You can specify the file name here
char op;
int left_operand, right_operand;
// Use extraction has while condition
while (inFile >> op >> left_operand >> right_operand) {
switch (op) {
// Pass operands to the relevant function
case '+': doAddition(left_operand, right_operand); break;
// ...
}
}
// You do not need to close inFile, it will be closed when it goes out of scope
return 0;
答案 3 :(得分：1)
理想情况下，有更好的方式来做你的建议，但是如果你必须使你的代码看起来像这样，那么适当的解决方案应该是某种东西这类：
#include <vector>
#include <fstream>
#include <iostream>
#include <string>
#include <cstdlib>
using namespace std;
void doAddition(ifstream &inFile) {
int num1, num2;
inFile >> num1 >> num2;
cout << "Addition of " << num1 << " and " << num2 << " = "<< (num1+num2) << '\n';
}
void doSubtraction(ifstream &inFile) {
int num1, num2;
inFile >> num1 >> num2;
cout << "Subtraction of " << num1 << " and " << num2 << " = "<< (num1-num2) << '\n';
}
void doMultiplication(ifstream &inFile) {
int num1, num2;
inFile >> num1 >> num2;
cout << "Multiplication of " << num1 << " and " << num2 << " = "<< (num1*num2) << '\n';
}
void doDivision(ifstream &inFile) {
float num1, num2;
inFile >> num1 >> num2;
cout << "Division of " << num1 << " and " << num2 << " = "<< (num1/num2) << '\n';
}
void doFactorial(ifstream &inFile) {
int t1, t2;
inFile >> t1 >> t2;
//perform factorial here
}
void readToNextLine(ifstream& inFile) {
string t1, t2;
inFile >> t1 >> t2;
}
int main()
{
ifstream inFile;
char ch;
int num1, num2;
inFile.open("math.txt");
if (inFile.is_open()){
inFile >> ch;
while (!inFile.eof())
{
switch (ch)
{
case '+':
doAddition(inFile);
break;
case '-':
doSubtraction(inFile);
break;
case '*':
doMultiplication(inFile);
break;
case '/':
doDivision(inFile);
break;
case '!':
doFactorial(inFile);
break;
default:
readToNextLine(inFile);
cout << "Invalid Operation" << endl;
}
inFile >> ch;
}
inFile.close();
}
else
{
cout << "The math.txt input file failed to open";
return -1;
}
inFile.close();
return 0;
}
这里有几点需要注意：
其他人建议的一些解决方案，不考虑默认情况 - 这只是放弃阅读剩下的行，导致逻辑错误 - 其中没有一个人想要。
现在，对于“更好”，更通用的解决方案，首先将所有内容存储为字符串，然后进行标记化，并尝试将相应的标记转换为所需的类型输出将更为理想。
但是，正如我之前所说，如果你希望遵守以前的代码，那么这是合适的。
答案 4 :(得分：0)
除非你感觉自虐，否则几乎肯定最容易同时阅读操作员和两个操作数，然后执行操作：
char operation;
int operand1, operand2;
while (infile >> op >> operand1 >> operand2)
switch(operation) {
case '+': add(operand1, operand2); break;
case '-': sub(operand1, operand2); break;
case '*': mul(operand1, operand2); break;
case '/': div(operand1, operand2); break;
}
对于只有四个运营商而言，这种方式运作良好。如果你还有更多的东西，你可能最好使用指针函数表来代替：
typedef int (*op)(int, int);
op operators[UCHAR_MAX];
for (int i=0; i<UCHAR_MAX; i++)
operators[i] = report_bad_operator;
operators['+'] = add;
operators['-'] = sub;
operators['/'] = div;
operators['*'] = mul;
// more operators here
while (infile >> operation >> operand1 >> operand2)
operators[operation](operand1, operand2);
其他明显的原因/时间是处理非二元运算符（即那些不一定只需要两个操作数的运算符）。
答案 5 :(得分：0)
首先，在为其分配值之前，您正在读取chr。
像这样更改你的while循环：
while(inFile)
{
inFile >> ch; // assign before reading
switch (ch) {
case '+':
doAddition(inFile);
break;
case '-':
doSubtraction(inFile);
break;
case '*':
doMultiplication(inFile);
cout << "debug " << ch;
break;
case '/':
doDivision(inFile);
break;
case '!':
doFactorial(inFile);
break;
default:
cout << "Invalid Operation" << endl;
}
}
并改变你的doDivision功能：
void doDivision(ifstream &inFile)
{
int num1, num2;
inFile >> num1 >> num2; // your already read the arithmetic operator
cout << "Division " << num1 << " " << num2 << " " << "Quotient " << "
}
答案 6 :(得分：0)
首先，初始传递，您的ch变量未初始化。 在循环的底部初始化。
尝试添加：
inFile >> ch;
。