istream_iterator尝试解析无效数据

时间:2011-08-17 19:31:52

标签: c++ stream c++11 iterator stl-algorithm

您好,我希望有人能帮助理解以下代码的这种行为。

#include <iostream>
#include <algorithm>
#include <string>
#include <limits>
#include <fstream>
#include <iterator>
#include <stdexcept>


struct asound_stanza
{
    unsigned index;
    std::string name;

    friend std::istream &operator>>(std::istream &is, asound_stanza &stan)
    {
        is >> stan.index;
        if(!is.good())
            return is;
        std::getline(is, stan.name);
        std::string::const_iterator
            eol   = stan.name.cend(),
            start = std::find(stan.name.cbegin(), eol, '['),
            end   = std::find(start, eol, ' ');
        stan.name = std::string(++start, end);
        is.ignore(std::numeric_limits<std::streamsize>::max(), '\n');
        // std::istream_iterator<char> it;
        // while(*it++!=0x0a);
        return is;
    }
};

int main()
{
    std::ifstream is("/proc/asound/cards", std::ifstream::in); 
    std::istream_iterator<asound_stanza> it(is), end;
    unsigned devid = 0;
    std::for_each(it, end, [&](const asound_stanza &stan)->void{
        if(stan.name!="CODEC")
            return;
        devid = stan.index;
    });
    std::cout << devid;
    return 0;
}

这有效,但我有几个问题。在完成所有有效迭代之后,它会转向头部并尝试解析另一个不可避免地失败的因素(因此if(!is.good())... _)。它试图解析它但它从未将最终格式错误的结构传递给lambda表达式。为什么不呢?是因为流不好()所以它不会试图传递它吗?

另外,我怎么能让它甚至不打扰尝试解析最终结构,(每个节以换行符结束(0x0a),所以我会认为忽略流直到换行将导致EOF最终的有效迭代,但它没有。

感谢您的指导。

也可以随意传递其他编码正确性评论。

PS:我的文件看起来像这样

0 [Intel          ]: HDA-Intel - HDA Intel
                     HDA Intel at 0xfc500000 irq 46
1 [HDMI           ]: HDA-Intel - HDA ATI HDMI
                     HDA ATI HDMI at 0xfc010000 irq 47
2 [CODEC          ]: USB-Audio - USB Audio CODEC
                     Burr-Brown from TI USB Audio CODEC at usb-0000:00:1d.7-3.1, full speed

2 个答案:

答案 0 :(得分:1)

您的代码应如下所示:

friend std::istream &operator>>(std::istream &is, asound_stanza &stan)
{
    if ( !(is >> stan.index) )
        return is;  //return if couldn't read unsigned int

    if(std::getline(is, stan.name))//go inside ONLY IF read is successful
    {
       std::string::const_iterator 
       eol  =stan.name.cend(),
       start=std::find(stan.name.cbegin(), eol, '['),
       end  =std::find(start, eol, ' ');
       stan.name=std::string(++start, end);    
       is.ignore(std::numeric_limits<std::streamsize>::max(), '\n');
    }
    return is;
}

阅读上述修改中的评论!

由于这已经讨论了很多次,而且我之前已多次回答过,所以我不想再重复一次。请阅读我的另一个解释这个问题的答案:

答案 1 :(得分:1)

如果流处于无效状态或者尝试从运营商内部的流中读取&gt;&gt;如果失败并将流设置为无效状态,则istream_iterator将自身设置为流位置的结尾。因此,迭代结束时甚至没有查看部分解析的对象。