関数が値を返さないようにする

Question

私はユーザー入力を受け取り、指定された文字に値を割り当てることができるようにしたいと考えています。私はその部分を持っていると思う、今問題は値を返す。

#include <iostream> 
#include <string> 
using namespace std; 

int ch2n(string word); 

int main() 
{ 
    string inputWord; 

    cout << "Type Word: "; 
    cin >> inputWord; 
    cout << ch2n(inputWord); 
} 

int ch2n(string word) 
{ 
    int total = 0; 
    for(int i = 0;i != word.size(); i++) 
    { 
     if(word.find(i) == 'a' || word.find(i) == 'A') 
     { 
      total += 1; 
     } 
    } 
    return total; 
} 

私は0に合計を宣言するとき、リターンは常に0ですが、私はそれを宣言していないとき、私は....などの乱数を229の戻り値を取得します。

Answer 1

あなたはそれが未定義の動作であり、任意のランダムな値を返すことができます使用して、（その値を設定し）、それを初期化しない場合 - intだけのスペースを割り当て、undefinedd値を持つことになりますように、コンストラクタなし0

タイプなどを一般的には、以前の使用からその場所にあったことに基づいています。

word.findあなたはそれがないと思う何をしない、それがiを探しているあなただけword[]を使用したい

word中：

if(word[i] == 'a' || word[i] == 'A') 

また、あなたが最後にstd::endlを置きたいかもしれませんあなたのcout行の

Answer 2

私はword.find(i)と思うかもしれません。文字列内の特定の文字にアクセスするには、角括弧（word.find(i)の代わりにword[i]）を使用します。

Answer 3

0と宣言しないと結果がランダムになるのは、そのためのC++とcがデータを初期化しないためです。変数を宣言した場合（totalなど）、初期値はメモリ内のその場所にあるものであれば何でも構いません。それは本当に何でもできる。常に変数の値を初期化します。

あなたが意味のあるものを返さない理由は、間違った使い方を使用しているためです。 std :: string :: findはブール値を返しません。したがって、「文字列がこの文字列に存在しません」という文字列の位置に対して、チェックしたいと考えています。これはstd :: string :: nposです。だから、あなたがしたいでしょう：交互

if(word.find('a') != string::npos || word.find('A') != string::npos){ 
     total += 1; 
    }