C++の範囲ベースforループに関する混乱

Question

このコードでは、文字列を受け取り、文字列の偶数および奇数の文字をスペースで区切られた2つの別々の文字列として書き出します。私は標準forループを使用して問題を解決しました。しかし、私は、Bjarneの2017 CPPCON基調演説を受けてから、通常のforループの代わりに、範囲ベースのforループを使用しようとしています。通常のforループは正常に動作し、次のコードブロックでコメントしています。

問題：コードはg ++ -std = c + 11コマンドでコンパイルされますが、偶数および奇数の文字列が文字化けして出力され、バイナリファイルのように読み込まれます。私が間違ってやっている事や、ここで何が起こっているのかを説明してください。明確な説明は非常に高く評価されます。ありがとうございました。

string S,even,odd; 
    cout << "Enter a string:\n"; 
    cin.ignore(); // So that getline does not catch 
    //the eol character 
    getline(cin,S); 
    // for (int j=0; j<S.length(); j++){ 
    //  if(j==0 || j%2==0){even.push_back(S[j]);} 
    //  else {odd.push_back(S[j]);} 
    // } 
    for (auto j : S){ 
     if(j==0 || j%2==0){even.push_back(S[j]);} 
     else {odd.push_back(S[j]);} 
    } 
    cout << "You wrote: " << S <<'\n'; 
    cout << "Even(including 0) positioned character(s) 
    of " << S << " is(are) " << even <<'\n'; 
    cout << "Odd positioned character(s) of " << S << 
    " is(are) " << odd <<'\n'; 

Answer 1

範囲ベースのforループは、コンテナの要素を反復処理します。あなたのコード中の 'j'は、インデックスではなく文字列内の文字です。これを試してください：

for (auto character : S) 
{ 
    if (even.length() > odd.length()) 
    { 
    odd.push_back(character); 
    } 
    else 
    { 
    even.push_back(character); 
    } 
}