读取包含小数的文件时出现问题

时间:2011-05-01 20:14:23

标签: c++ arrays 2d ifstream

我有一个问题,试图弄清楚为什么我的文件返回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

希望有人可以帮助我,因为我研究了一些有关这方面的信息并且没有真正找到解决方案。

3 个答案:

答案 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?