我有一个问题,试图弄清楚为什么我的文件返回0而不是文件中的数字,这是我在C ++中读取文件时所做的代码:
#include <cstdlib>
#include <iostream>
#include <fstream>
#include <string>
using namespace std;
string cfile;
int cnum1,cnum2,cnum3,cnum4;
bool fired = false;
/*
*
*/
void printMatrix(double **x, int n)
{
int size = n;
for(int i=0; i<size; i++)
{
for(int j=0; j<size; j++)
{
std:: cout << x[i][j] << " " ;
}
std:: cout << std::endl;
}
}
void readFile(string file,double **x, int n)
{
std::ifstream myfile(file.c_str());
int size = n;
for(int i=0; i<size; i++)
{
for(int j=0; j<size; j++)
{
myfile >> x[i][j];
}
}
}
void GetCommandLineArguments(int argc, char **argv,string &file, int &n, int &k, int &m, int &i)
{
if( argc == 6 )
{
cfile = argv[1];
cnum1 = atoi(argv[2]);
cnum2 = atoi(argv[3]);
cnum3 = atoi(argv[4]);
cnum4 = atoi(argv[5]);
}
file = cfile;
n = cnum1;
k = cnum2;
m = cnum3;
i = cnum4;
}
int main(int argc, char** argv) {
int k; //Factor of n
int m; //Innner matrix size
int i; //Iteration
int n; //Matrix Size
string file;
GetCommandLineArguments(argc, argv, file, n, k, m, i);
double **matrix;
matrix = new double*[n];
for(int i = 0; i<n; i++)
matrix[i] = new double[n];
for(int j=0; j<n; j++)
for(int i=0; i<n;i++)
matrix[i][j] = 0;
readFile(file, matrix, n);
printMatrix(matrix, n);
return 0;
}
以下是我的文件示例,其中包含我要从中提取的值:
20.0
20.0
20.0
20.0
20.0
200.0
20.0
200.0
希望有人可以帮助我,因为我研究了一些有关这方面的信息并且没有真正找到解决方案。
答案 0 :(得分:1)
您的阅读和打印代码似乎有效,但您的命令行阅读代码可能存在一些问题。
我运行你的代码而没有得到命令行参数。以下代码几乎是从主减去命令行args中复制粘贴的。
int main()
{
double **matrix;
std::string file = "test.dat";
int n = 5;
matrix = new double*[n];
for(int i = 0; i<n; i++)
matrix[i] = new double[n];
for(int j=0; j<n; j++)
for(int i=0; i<n;i++)
matrix[i][j] = 0;
readFile(file, matrix, n);
printMatrix(matrix, n);
return 0;
}
根据您提供的输入,我得到输出:
20 20 20 20 20
200 20 200 0 0
0 0 0 0 0
0 0 0 0 0 0 0 0 0 0
然而查看命令行arg阅读代码,我可以看到一些潜在的问题。首先你使用atoi()。当atoi失败时,它返回0.你知道这段代码是否正常工作?一切都正确初始化了吗?或者atoi在输入上失败,导致n为0,因此无法读入任何内容? (你可能希望调查stringstreams做这种事情。)
此外,当argc不是6时,你会默默地失败并从未初始化的全局内存中读取。这个记忆是垃圾。你知道这不会发生吗?如果您正在做:
your.exe test.dat 5
然后5将不会从命令行读取,因为argc不是6.你是否总是在测试时传递6个参数?也许不是,因为在当前状态下你的代码实际上只需要文件名和大小。
最重要的是,看看你是否从命令行得到了你想要的东西。
PS这是g ++:
$ g ++ --version g ++(Ubuntu / Linaro 4.4.4-14ubuntu5) 4.4.5版权所有(C)2010 Free Software Foundation,Inc。这是免费的 软件;看到复制的来源 条件。没有保修;不 即使是适销性或健身 特别针对目的。
答案 1 :(得分:0)
我建议像这样修复,让事情更健壮:
#include <fstream>
#include <iostream>
#include <string>
#include <stdexcept>
template <size_t size>
void readFile(std::string file, double (&x)[size][size])
{
std::ifstream myfile(file.c_str());
for(int i=0; i<size; i++)
{
for(int j=0; j<size; j++)
{
if (!(myfile >> x[i][j]))
throw std::runtime_error("Couldn't read double from " + file);
}
}
}
int main(int argc, const char *const args[])
{
try
{
double data[10][10];
readFile("input.txt", data);
} catch(const std::exception& e)
{
std::cerr << "Whoops: " << e.what() << std::endl;
return 255;
}
return 0;
}
答案 2 :(得分:0)
您的输入操作员将读取数字,但在文件中留下换行符。
请参阅此答案,了解如何在阅读您的值后ignore()
结束行
Why would we call cin.clear() and cin.ignore() after reading input?