テキストの解析時の状態を維持する

Question

構文を検索し、状態ごとに変化するプログラムを作成しようとしています その状態を示す必要があります。私は持ってはいけないべき別の出力を得ている。

using namespace cppfsm; 
#include <vector> 
#include <iostream> 

using std::cin; 
using std::cout; 
using std::endl; 
using std::vector; 

int cppfsm::updateState(int& state, char c) { 
    const int state1 = 1; 
    const int state2 = 2; 

    switch (state) { 
    case state1: 
     if (c == '/') 
      cout << "1" << endl; 
      // do stuff; update state 
     else if (c == '"') 
      cout << "1" << endl; 
      // do something else; update state 

    case state2: 
     if (c == '/') 
      cout << "1" << endl; 
      // do stuff; update state 
     else if (c == '"') 
      cout << "1" << endl; 
      // do something else; update state 

    } 
    return 0; 
} 

void testFSM(string s) { 
    vector<int> stlist; // list of states. 
    int cstate = start; 
    for (unsigned long i = 0; i < s.length(); i++) { 
     stlist.push_back(updateState(cstate,s[i])); 
    } 
    // push the last state: 
    stlist.push_back(cstate); 
    cout << s << endl; 
    for (unsigned long i = 0; i < stlist.size(); i++) { 
     cout << stlist[i]; 
    } 
    cout << endl; 
} 

int main() { 
    // the finite state machine: 
    string input; 
    while(getline(cin,input)) { 
     cout << " "; 
     testFSM(input); 
    } 
    return 0; 
} 

出力は次のようになります。 1から別の

$ echo "int x; // holds stuff" | ./fsm 
int x; // holds stuff 
0111010042222222222222 
$ echo 'cout << "some string";' | ./fsm 
cout << "some string"; 
01111000033333333333300 
$ echo 'cout << "\"escape\" chars are fun";' | ./fsm 
cout << "\"escape\" chars are fun"; 
011110000353333333533333333333333300 

に行くとき 数字は状態です。しかし、私の出力は、すべて0000 ......のように出てきます。この問題を解決するにはどうすればよいですか？

Answer 1

あなたはstlistがすべて0である理由、updateStateためのreturn文を見て迷っている場合：

 return 0; 
} 

はstlistを取り込むためのあなたのコードでこれを比較します

stlist.push_back(updateState(cstate,s[i])); 

限り私が知ることができるように、すべて0がこのコードの正しい動作です。明らかに、これは予想されるか、論理的な動作ではありませんので、私はupdateStateを変更する提案：あなたは、コードを実行したときに意図したとおり

int cppfsm::updateState(int& state, char c) { 
    // ... 
    return state; 
} 

今stlistは、各状態変化が含まれている必要があります。

Answer 2

あなたは同じ値のupdateStateを呼び出しているようです。その値はスイッチでは処理されないので、関数はゼロを返します。つまり、stlistベクターにゼロを追加し続けるだけです。

スイッチの開始状態を処理し、updateState関数の戻り値をcstate変数に割り当てる必要があります。

Answer 3

あなたのコードあなたが開始状態をチェックしません：

switch (state) { 
    case state1: /* ... */ 
    case state2: /* ... */ 
    case start : /* ... */ 
}