Find関数をC++で

Question

char aまたはchar 1を文字列で検索しようとしています。 見つからない場合、、私のプログラムにエラーメッセージが表示され、終了します。これは私の機能ですが、動作していないようです。私を助けてください、ありがとう。

*更新、文字列がa2の場合、無効なメッセージが表示されますが、文字列がa1の場合のみ続行されます。

bool hasAor1(const std::string& string) 
    { 
     return string.find_first_of("a1") != std::string::npos; 
    } 

    //find valid key, key must have numbers like 1234 instead of 134 
    void validstring(string key) 
    { 
     if(key.length() == 2) 
     { 
     hasAor1(key) == false; 
     cerr << "invalid key"; 
     }  
    } 

Answer 1

std::findここで使用する方がはるかに簡単です。

auto iter = find_if(begin(key), end(key), 
     [](char v){ return v=='a' || v=='1'; }); 

if (iter == key.end()) cerr << "invalid key"; 

Answer 2

機能hasAor1はOKです。

「hasAor1（key）== false;」という行を変更します。 （！hasAor1（キー））CERR < < "無効なキー" であれば

のようなものに;。

ので

//find valid key, key must have numbers like 1234 instead of 134 
    void validstring(string key) 
    { 
     if(key.length() == 2) 
     { 
     if (hasAor1(key) == false) 
      cerr << "invalid key"; 
     }  
    } 

それが何をした場合に行う必要が行うことになっているものvalidstringは明らかではありませんキーの長さは2ではありませんか？これはブールか何かを返すべきではありませんか？それは、メッセージを印刷することですが、キーの長さが2の場合のみです。キーが "543 "または" 145 "または" 1a3 "？

コード "//有効なキーを見つける、キーは134の代わりに1234のような数字を持たなければなりません"有効なキーのすべての例にhasAor1が無効とマークする文字 '1'が含まれているので意味がありません。

考えられる入力のバリエーションとそれぞれの場合に起こりたいことについて考えることができます。