Funktion gibt keinen Wert Ich will

Question

Ich möchte in der Lage sein, die Benutzereingabe zu nehmen und den gegebenen Buchstaben einen Wert zuzuweisen. Ich denke, ich habe diesen Teil runter, jetzt ist das Problem der Rückgabewert.

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

, wenn ich die Gesamtzahl auf 0 zu erklären, ist die Rückkehr immer 0, aber wenn ich es nicht erklären, ich bekomme einen Rückgabewert von 229 .... etc Zufallszahl.

Answer 1

Wenn Sie es nicht initialisiert werden (dessen Wert gesetzt), ist es nicht definiertes Verhalten mit und kann einen beliebigen Wert zurück - einschließlich 0

Typen ohne Konstruktor, wie int wird nur Speicherplatz zuweisen und haben einen Wert undefinedd im Allgemeinen basierend auf dem, was sich an diesem Ort von der vorherigen Verwendung befindet.

word.find nicht tut, was Sie denken, es tut, ist es für i in word

Sie wollen die Suche nur word[] verwenden: Auch

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

, könnte man std::endl am Ende setzen wollen von deinen cout Zeilen

Answer 2

Ich denke word.find(i) ist wahrscheinlich nicht das, was du da anrufen willst. Um auf ein bestimmtes Zeichen innerhalb der Zeichenfolge zuzugreifen, verwenden Sie eckige Klammern, z. B .: word[i] anstelle von word.find(i).

Answer 3

Der Grund, dass Ihr Ergebnis zufällig ist, wenn Sie es nicht als 0 deklarieren, liegt daran, dass C++ und c in diesem Fall Daten nicht initialisieren. Wenn Sie eine Variable wie total deklarieren, ist der Anfangswert, was auch immer sich an diesem Ort im Speicher befindet. Es könnte wirklich alles sein. Initialisiere immer den Wert einer Variablen.

Ich denke, der Grund, warum Sie nichts Sinnvolles zurückgeben, ist, dass Sie falsch finden. std :: string :: find gibt keinen booleschen Wert zurück, wenn eine Position zurückgegeben wird. Daher möchten Sie überprüfen, ob gegen eine Zeichenfolgenposition, die besagt "das Zeichen existiert nicht in dieser Zeichenfolge". Das ist std :: string :: npos. Also, werden Sie wollen:

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