将逗号分隔文件读入整数数组

Question

我试图将包含整数的文本文件读入整数数组。 如果输入为：1 3 4 5 6（中间有空格）它工作正常。

但如果输入是：1,3,4,5,6（逗号分隔）。它只是打印1.（第一个数字）。如果程序发现1,3,4,5,6作为单个实体，那么它应该打印1,3,4,5,6作为第一个指标ryt？ 而且File >> x，这个表达式通过检测两者之间的空间来逐个取值？

#include<iostream>
#include<fstream>
#include<string>
using namespace std;

int main()
{    
    int n = 0; //n is the number of the integers in the file ==> 12
    int num;
    int arr[100];
    int x;
    int sum = 0;
    ifstream File;
    File.open("integer.txt");
    if(!File.is_open())
    {
        cout<<"It failed"<<endl;
        return 0;
    }

    while(File>>x)
    {
        arr[n] = x; 
        n++;
    }

    File.close();
    cout<<"n : "<<n<<endl;
    for(int i=0;i<n;i++)
    {
        cout << arr[i] << " ";
    }
    return 0;
}

Answer 1

这里发生的是在提取第一个字母后，您的代码尝试将逗号提取为整数。 因为它无法做到这一点，它将返回false并结束循环。

有一个非常类似的问题在这里 。

你的while循环应该如下所示：

while(File>>x)
{
    arr[n] = x; 
    n++;
    if (ss.peek() == ',')
        ss.ignore();
}

Answer 2

当你用逗号输入文件时，它最有可能是将它作为整个字符串读取。 确保这也是逗号的分隔符，我不知道你将逗号放在这个代码中的哪个位置，但是你需要一个分隔符。

//example function you can use.
getline( ss, s, ',' )

Answer 3

但如果输入是：1,3,4,5,6（逗号分隔）。它只是打印1.（第一个数字）。 如果程序找到1,3,4,5,6作为单个实体，那么它应该打印1,3,4,5,6作为第一个索引ryt？

它只打印“1”，因为您尝试将“1,3,4,5,6”读入int对象。 但是， int不能是“1,3,4,5,6”。 简单地说，一旦达到第一个“坏”字符（即逗号），解析就会停止，并且最终得到到目前为止已建立的整数，即“1”。

其余的输入被丢弃。 这就像你的行是“1abcdef”或“1abcdef2345”。

而且File >> x，这个表达式通过检测两者之间的空间来逐个取值？

是的，这使得它非常不灵活。

我建议而不是摆弄operator>>使用std::getline ，传递','作为分隔符。 虽然函数的名称不再有意义，因为它不再读取行 （就像使用默认分隔符'\\n' ），它将正常工作。

最后将介绍使用std::stoi轻松转换为int的单个std::string对象。

当你在它的时候，摆脱raw int arr[100]并使它成为std::vector<int> ，这样你就不仅限于100个元素。 无论如何，100是一个丑陋的魔法（任意）数字。

这是一个例子：

#include <iostream>
#include <sstream>
#include <string>
#include <vector>

int main()
{
    // faking some test file input; behaves like an `std::ifstream`:
    std::istringstream is("1,2,3,4,5");

    std::vector<int> numbers;

    std::string number_as_string;
    while (std::getline(is, number_as_string, ','))
    {
        numbers.push_back(std::stoi(number_as_string));
    }

    std::cout << "n : " << numbers.size() << "\n";
    for(auto&& number : numbers)
    {
        std::cout << number << "\n";
    }
}

正如您所看到的，我还借此机会提出了一些其他优秀的C ++实践，例如避免using namespace std和std::endl 。

Answer 4

您是否尝试过使用ifstream的sscanf？ 一个简单的例子如下

    #include<iostream>
    #include<fstream>
    #include<string>
    using namespace std;

    int main()
    {
        int n = 0; //n is the number of the integers in the file ==> 12
        int num;
        int arr[100];
        char line[200];
        char *ptr=line;
        int x=0,count=0;
        int sum = 0;
        ifstream File;
        File.open("integer.txt");
        if(!File.is_open())
        {
            cout<<"It failed"<<endl;
            return 0;
        }
        File.getline(line,200);

       while((count=sscanf(ptr,"%d,",&x))>0)
        {
            arr[n] = x;
            n++;
            ptr+=count+1;
        }
        File.close();
        cout<<"n : "<<n<<endl;
        for(int i=0;i<n;i++)
        {
            cout << arr[i] << " ";
        }
        cout<<endl;
        return 0;
    }