1

こんにちは、私は誰かが以下のコードのこの振る舞いを理解するのを手伝ってくれることを望んでいました。

#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())..._）。解析を試みますが、最終的な不正な構造体をラムダ式に渡すことはありません。なんでそうじゃないの？ストリームがgood（）ではないので、わざわざそれを渡そうとしないからですか？

また、最終的な構造体をわざわざ解析しようとしないようにするにはどうすればよいですか（各スタンザは改行（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
4

2 に答える 2

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

上記の私の変更のコメントを読んでください！

これは何度も議論されてきたので、私は何度も答えたので、二度と繰り返したくありません. まさにそれを説明する私の他の答えを読んでください：

于 2011-08-17T19:36:34.360 に答える
1

ストリームが無効な状態にある場合、または operator>> 内のストリームから読み取ろうとして失敗し、ストリームを無効な状態に設定した場合、istream_iterator は自身をストリーム位置の末尾に設定します。そのため、部分的に解析されたオブジェクトを見ることさえせずに反復が終了します。

于 2011-08-17T19:47:58.130 に答える