文字列を文字配列に変換する

Question

C++が新しくなったので、ここでは私が取り組んでいるプロジェクトの一部です。文字列を使用して、最も頻繁に使用される数値と何回使用されたかを印刷します。私はこれが正しいと思ったが、何らかの理由で私のchar配列は読み込まれません。どのように修正するためのヒントや提案？

#include <string> 
#include <iostream> 

using namespace std; 

char getMostFreqLetter(string ss); 

int main() { 
    string s; //initilizing a variable for string s 
    s = ("What is the most common letter in this string "); // giving s a string 

    getMostFreqLetter(s); // caling the function to print out the most freq Letter 

    return 0; 
} 

char getMostFreqLetter(string ss) { 
    int max, index, i = 0; 
    int array[255] = {0}; 
    char letters[]; 

    // convert all letters to lowercase to make counting letters non case sensative 
    for (int i = 0; i < ss.length(); i ++){ 
     ss[i] = tolower(ss[i]); 
    } 

    //read each letter into 
    for (int i = 0; i < ss.length(); i ++){ 
     ++array[letters[i]]; 
    } 
    // 

    max = array[0]; 
    index = 0; 
    for (int i = 0; i < ss.length(); i ++){ 
     if(array[i] > max) 
     { 
      max = array[i]; 
      index = i; 
     } 
    } 

    return 0; 
} 

Answer 1

コードにいくつかの誤りがあります。

最初に、文字配列lettersは完全に使用されません。それを無視し、代わりに文字列ssを反復する必要があります。これはあなたが意図したものです。

これは、2番目のforループを++array[letters[i]];から++array[ss[i]];に変更します。

第2に、あなたの最後のforループはバグです。あなたは配列の頻度を探すためにインデックスとしてiを使いますが、文字のascii値（ss [i]）を使う必要があります。あなたは手紙のようにホワイトスペースを考慮していない場合

Most freq character: t 

Answer 2

：あなたは、文字列上で実行するときに、次の出力を得る上で変更を加えたら

index = ss[0]; 
    max = array[index]; 
    for (int i = 0; i < ss.length(); i ++){ 
     if(!isspace(ss[i]) && array[ss[i]] > max) 
     { 
      max = array[ss[i]]; // you intended to use the ascii values of the characters in s to mark their place in array. In you code, you use i which is the just the index of the character in s as opposed to the ascii value of that character. Hence you need to use array[ss[i]]. 
      index = ss[i]; 

     } 
    } 
    return index; 

：ここでコメントして固定されたバージョンです。

そして、より効率的な方法は、使用string::c_str()

vector<int> count(26,0); 
for (int i = 0; i < s.length(); i++) { 
    int range = to_lower(s[i])-'a'; 
    if (range >= 0 && range < 26) 
     count[range]++; 
} 

// Now you can do fix the max while iterating over count; 

Answer 3

だったかもしれません。 文字列を文字配列に変換します。