istream_iterator尝试解析无效数据

Question

嗨，我希望有人能帮助理解下面代码的这种行为。

#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表达式。为什么不是呢？是不是因为流不好()，所以它不会费心去传递它？

此外，我如何才能让它甚至不费心尝试并解析final结构(每节以换行符(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

Answer 1

如果流处于无效状态，或者如果尝试从operator>>内部的流读取失败并将流设置为无效状态，则istream_iterator会将其自身设置为流的末尾位置。因此，迭代结束时甚至没有查看部分解析的对象。

Answer 2

您的代码应如下所示：

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;
}

请阅读我上面修改中的评论！

因为这个问题已经讨论过很多次了，我之前已经回答过很多次了，所以我不想再重复了。请阅读我的另一个答案，它解释了这个问题：

• std::getline, eof, failure flag and blah blah